[PATCH] D94854: [Clang] Fix SwiftCallingConv's aggregate lowering for _Atomic(_Bool).

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 15 21:27:00 PST 2021


rjmccall added a comment.

Hmm.  The way you'd test it on the Clang side would be to just write a test case that passes such an aggregate and tests that it's passed as an `i8`.  But I'm not sure it's at all unreasonable to pass this as an `i1` rather than an `i8`; seems like something that Swift should handle correctly in its call-lowering code.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94854



More information about the cfe-commits mailing list