[PATCH] D11051: Extend LICM to hoist loop invariant GEP out

[Andrew Trick]

atrick added a comment.

I don't think GEP reassociation should be done as a canonical InstCombine for two reasons
(1) the inbounds property would need to be dropped
(2) Unlike Reassociate, InstCombine has no knowledge of the CFG

I think this should be part of SeparateConstOffsetFromGEP, which is also optimizing for register pressure and addressing modes. As part of this pass it is target configurable and controlled by EnableGEPOpt.


Repository:
  rL LLVM

http://reviews.llvm.org/D11051