[PATCH] D52961: [RISCV] Introduce the RISCVMatInt::generateInstSeq helper
Alex Bradbury via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Oct 11 05:52:31 PDT 2018
asb added a comment.
The other motivation is that many of our codegen tests are going to include both RV32 and RV64 check lines. It's distracting to see `addiw` for materialising constants when it's not necessary, and it represents an unnecessary diff vs the equivalent RV32 code.
This patch still applies cleanly against HEAD.
@niosHD, did you have any thoughts about this approach?
https://reviews.llvm.org/D52961
More information about the llvm-commits
mailing list