Extend the Trivial setter documentation to support cases where the value is moved into a field using std::move.
nit: you could skip this check if you like, the other variant isn't going to be on the RHS of an assignment :-)
I guess you want !ND->getIdentifier() || ND->getName() != "move" || ... to guard against it being a special name somehow - getName() asserts in that case.
Updated function doc
Fix potential assertion
You never know what crazy code people could concoct. It could be argued that this has a performance win by easily filtering out some bad candidates quickly and avoid needing to run the (slightly) more expensive checks on the name later down the line :-)
Can you explain the reasoning for moving the comparison to the end?