[PATCH] D55543: [CodeGen] Fix assertion on throwing object with inlined inherited constructor and non-trivial destructor.
John McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Dec 17 21:22:47 PST 2018
rjmccall added inline comments.
================
Comment at: clang/test/CodeGenObjCXX/inheriting-constructor-cleanup.mm:10
+ // Use variadic args to force inlining the inherited constructor.
+ Base(const Strong &s, ...) {}
+};
----------------
To test what I'd like to see, this needs to take `Strong` by value, not by reference. There won't be a cleanup for the inlined parameter if it's taken by reference.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55543/new/
https://reviews.llvm.org/D55543
More information about the cfe-commits
mailing list