[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