rjmccall added a comment.

In D123898#3455933 <https://reviews.llvm.org/D123898#3455933>, @theraven wrote:

> I'd like to.  The test case from the original bug report was very large but 
> I'm not sure how to trigger this with anything comprehensibly small.  You 
> need something where the callee is responsible for destroying the argument 
> and I'm not sure what combination of properties / ABIs results in that.  The 
> original test was `-(void)foo:(std::string)aString` on the MSVC ABI, but I 
> don't know how much of the Visual Studio `std::string` implementation is 
> necessary to trigger this behaviour.  Is it just a non-trivial destructor?

Yeah, a non-trivial destructor should be good enough.  Or a consumed ObjC 
pointer argument in ARC might also work, although I don't know if that takes 
the same path.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123898

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

Reply via email to