colemancda added a comment.

In D137309#3914550 <https://reviews.llvm.org/D137309#3914550>, @asb wrote:

> In D137309#3914494 <https://reviews.llvm.org/D137309#3914494>, @colemancda 
> wrote:
>
>> RV32 is enabled as well. I'll add unit tests.
>
> Great, thanks. Is it really the case that shouldPassIndirectlyForSwift has 
> the same result regardless of native word size?

Yes, if you check other implementations like Arm and AArch64, the 
implementation is the same.
Where is a good place to start for the unit test development for this ABI?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137309

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D137309: [cla... Alsey Coleman Miller via Phabricator via cfe-commits
    • [PATCH] D137309:... Alex Bradbury via Phabricator via cfe-commits
    • [PATCH] D137309:... Alsey Coleman Miller via Phabricator via cfe-commits

Reply via email to