[PATCH] D123496: Add Stackmap support for RISC-V

Sacha Coppey via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 13 08:48:38 PDT 2022


Zeavee updated this revision to Diff 422529.
Zeavee added a comment.

I replaced `assert` by `report_fatal_error` to indeed stop the execution when reaching this point. I did not include any code for when the intrinsics are run on RV32 as accordingly to the LLVM documentation, Stack map is a 64 bits feature. Thus I only implemented it for RV64. Should I do it for RV32 too?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123496

Files:
  llvm/lib/Target/RISCV/RISCVAsmPrinter.cpp
  llvm/lib/Target/RISCV/RISCVISelLowering.cpp
  llvm/test/CodeGen/RISCV/rv64-patchpoint.ll
  llvm/test/CodeGen/RISCV/rv64-stackmap-nops.ll
  llvm/test/CodeGen/RISCV/rv64-stackmap.ll
  llvm/test/CodeGen/RISCV/stackmap-frame-setup.ll
  llvm/test/CodeGen/RISCV/stackmap.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D123496.422529.patch
Type: text/x-patch
Size: 38033 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220413/2ed10e9f/attachment.bin>


More information about the llvm-commits mailing list