[PATCH] D79689: [RISCV] Add tests for folds of ADDIs into load/stores

Sam Elliott via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 11 03:10:58 PDT 2020


lenary added a comment.

One nit, beyond that happy for this to land. Don't forget `[NFC]` in the title.



================
Comment at: llvm/test/CodeGen/RISCV/fold-addi-loadstore.ll:135
+
+define void @store_g_4() nounwind {
+; RV32I-LABEL: store_g_4:
----------------
I realise the code is the same, but it would be good to have tests for store to `g_0`, `g_1`, `g_2` as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79689





More information about the llvm-commits mailing list