[PATCH] D128582: Enable SeparateConstOffsetFromGEPPass() at -O3 and -O2

[Nikita Popov via Phabricator via llvm-commits]

nikic requested changes to this revision.
nikic added a comment.
This revision now requires changes to proceed.

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

> Hello. Unfortunately, I doubt that people will be in favour of this approach, especially if it is introducing ptr2int and int2ptr's so early in the pass pipeline. It looks like a pass that needs to be run as part of the backend.

Indeed. This pass, //especially// in lowerToArithmetics() mode, is only usable in the backend.


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

https://reviews.llvm.org/D128582