[PATCH] D61884: [RISCV] Support stack offset exceed 32-bit for RV64

Sam Elliott via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 12 08:58:52 PDT 2019


lenary accepted this revision.
lenary added a comment.
This revision is now accepted and ready to land.

Great! This looks good to me! I'm glad we're getting rid of all those `mov a0, a0` (which are really useless `addi a0, a0, 0`).

I'm not worried about changes from `addi` to `addiw`.

I'm happy that this means improvements to RISCVMatInt will affect these offsets as well as anywhere else we use that code.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61884





More information about the llvm-commits mailing list