[PATCH] D130082: [RISCV][test] Update fold-addi-loadstore.ll for D129686

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 30 12:14:23 PST 2023


reames added inline comments.
Herald added a subscriber: luke.


================
Comment at: llvm/test/CodeGen/RISCV/fold-addi-loadstore.ll:242
 
+; Check for folds in accesses to multiple array offsets.
+
----------------
I think this case is interesting and should land.  I'd encourage you to add a case with a larger alignment on the global (or maybe just the load?) to show that if we have alignment info, we can optimize the codegen.  


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

https://reviews.llvm.org/D130082



More information about the llvm-commits mailing list