[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