[PATCH] D128582: Move SeparateConstOffsetFromGEPPass() before LSR() and enable EnableGEPOpt by default.

Shubham Narlawar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 21 12:05:41 PDT 2022


gsocshubham added inline comments.


================
Comment at: llvm/test/CodeGen/AArch64/GlobalISel/arm64-irtranslator-gep.ll:54
   %v2 = load i32, i32* %gep2
   %res = add i32 %v2, %v2
   ret i32 %res
----------------
dmgreen wrote:
> Make this %v1, %v2, I think is better. One of the loads is otherwise unused.
Done.


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

https://reviews.llvm.org/D128582



More information about the llvm-commits mailing list