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

Shubham Narlawar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 20 01:15:30 PDT 2022


gsocshubham added a comment.

In D128582#3657201 <https://reviews.llvm.org/D128582#3657201>, @dmgreen wrote:

> Thanks - Please reread this and see what "with full context" means: https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface. The patch should be created with -U9999999.

Understood. Thanks. I have updated patch using `-U9999999`. Is it fine now?

> I was thinking of the original longer test from the issue, but it is a bit long. This one might be OK too.




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

https://reviews.llvm.org/D128582



More information about the llvm-commits mailing list