[PATCH] D137309: [clang] Added Swift support for RISCV

Alsey Coleman Miller via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 14 23:26:03 PST 2022


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



More information about the cfe-commits mailing list