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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits