aaron.ballman added a comment.

Generally looks reasonable to me.



================
Comment at: clang/test/AST/Interp/records.cpp:52
 static_assert(!ints2.c, "");
 
 constexpr Ints ints3 = getInts();
----------------
Can you think of a reasonable way for us to test that you're handling the RVO 
pointer properly?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134175/new/

https://reviews.llvm.org/D134175

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

Reply via email to