NRVO translation is not NRVO eligible about cppinsights HOT 2 CLOSED

Hello @rmccampbell,

thanks for starting this conversation!

NRVO is tricky. It is an optimization performed by the compiler in the back end. C++ Insights looks at the front end. The thing that plays the major role here is guaranteed copy elison.

What C++ Insights shows is what Clang tells me. In the initial AST (https://godbolt.org/z/v8brhz16E) you see:

|   `-ReturnStmt <line:19:5, col:12> nrvo_candidate(Var 0xba30518 'c' 'C')
|     `-CXXConstructExpr <col:12> 'C' 'void (C &&)'
|       `-ImplicitCastExpr <col:12> 'C' xvalue <NoOp>
|         `-DeclRefExpr <col:12> 'C' lvalue Var 0xba30518 'c' 'C'

Notice that it says nrvo_candidate at this point. The compiler may decide differently. However, I will use that information and keep the variable in that case. A fix is on its way.

Andreas

from cppinsights.

I wonder if it would make sense to add another comment to the return statement to emphasize that NRVO is happening (in case the variable declaration is far away)?

from cppinsights.