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

Wed Jul 20 23:31:09 PDT 2022

dmgreen accepted this revision.
dmgreen added a comment.
This revision is now accepted and ready to land.

Thanks. The buildbot failure seems unrelated.

This LGTM if you can alter the test slightly and things are passing locally.

================
Comment at: llvm/test/CodeGen/AArch64/cond-br-tuning.ll:36
   %c = add nsw i32 %a, %b
-  %d = icmp ne i32 %c, 0
+  %d = icmp ne i32 %c, 10
   br i1 %d, label %L1, label %L2
----------------
%d = icmp ne i32 %c, 10

================
Comment at: llvm/test/CodeGen/AArch64/cond-br-tuning.ll:39
 L1:
   store i32 0, i32* %ptr, align 4
   ret void
----------------
store i32 10, i32* %ptr, align 4

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

https://reviews.llvm.org/D128582