[PATCH] D82513: [CodeGen] Store the return value of the target function call to the thunk's return value slot directly when the return type is an aggregate instead of doing so via a temporary

[John McCall via Phabricator via cfe-commits]

rjmccall added a comment.

This seems fine.  I do wonder if the "real" bug is that this ought to be 
handled properly in EmitReturnFromThunk, but regardless, the fix seems 
acceptable.

Is there a similar bug with trivial_abi?  Should we have a test case for that?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82513



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