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

[Shubham Narlawar via Phabricator via llvm-commits]

gsocshubham added inline comments.


================
Comment at: llvm/test/Transforms/SeparateConstOffsetFromGEP/split-gep.ll:2
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+; RUN: llc < %s -O3 -mtriple=aarch64-linux-gnu -aarch64-enable-gep-opt | FileCheck %s
+
----------------
dmgreen wrote:
> Remove -aarch64-enable-gep-opt - it is on by default now.
Done.


================
Comment at: llvm/test/Transforms/SeparateConstOffsetFromGEP/split-gep.ll:30
+ ret i32 %res
+
+
----------------
dmgreen wrote:
> Remove extra line.
Done.


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

https://reviews.llvm.org/D128582