[PATCH] D70401: [WIP][RISCV] Implement ilp32e ABI

Sam Elliott via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Dec 13 01:49:31 PST 2021


lenary marked an inline comment as done.
lenary added a comment.

In D70401#3188138 <https://reviews.llvm.org/D70401#3188138>, @zixuan-wu wrote:

> In D70401#3175266 <https://reviews.llvm.org/D70401#3175266>, @khchen wrote:
>
>>> Is it (D70401 <https://reviews.llvm.org/D70401>) good enough to solve or complete rv32e issue?
>>
>> It need to
>>
>> 1. disallow ilp32e ABI with D ISA extension. https://github.com/riscv-non-isa/riscv-elf-psabi-doc/blob/3f81fae0412bb9ad4002a4ade508be7aa5e1599b/riscv-cc.adoc#ilp32e-calling-convention
>> 2. emit predefined marco in header (`__riscv_e`)
>> 3. markSuperRegs for X16-X31
>> 4. update tests after rebase on main.
>
> Nice. If no body objects, @pcwang-thead will take this task and re-draft a review.

Please do feel free to commandeer the current patch. I cannot continue to work on it (so, you won't see review comments from me).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D70401



More information about the cfe-commits mailing list