aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
LGTM! ================ Comment at: clang/test/AST/Interp/records.cpp:52 static_assert(!ints2.c, ""); constexpr Ints ints3 = getInts(); ---------------- tbaeder wrote: > aaron.ballman wrote: > > Can you think of a reasonable way for us to test that you're handling the > > RVO pointer properly? > I think this is tested already by any function returning a non-primitive > value, i.e. a struct or an array, e.g. `getInts()` above. Oh derp, you're right! 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
