rjmccall added a comment.

Wow, the IR improvements here are amazing.



================
Comment at: lib/CodeGen/CGDecl.cpp:1119
+        if ((CXXRD && !CXXRD->hasTrivialDestructor()) ||
+            RD->isNonTrivialToPrimitiveCopy()) {
           // Create a flag that is used to indicate when the NRVO was applied
----------------
This should be isNonTrivialToPrimitiveDestroy(), I think.  The dynamic NRVO 
flag is specifically tied to whether we should destroy the local variable in 
its normal cleanup.


Repository:
  rC Clang

https://reviews.llvm.org/D44968



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to