vsapsai added a comment.
In D55543#1328025 <https://reviews.llvm.org/D55543#1328025>, @rjmccall wrote:
> Nice catch. This also fixes problems where there might be cleanups entered
> by `EmitParmDecl`, e.g. in ObjC++ with a parameter of type `struct A {
> __strong id x; }`; could you please add a test for that and verify that the
> parameter is destroyed at an appropriate point in the inlined call?
Aha, I see we have at least `EHStack.pushCleanup<ConsumeARCParameter>` in
`EmitParmDecl`. Will make sure to add a test that executes this code path.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D55543/new/
https://reviews.llvm.org/D55543
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
