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

[hfinkel at anl.gov]

hfinkel added a comment.

In http://reviews.llvm.org/D11051#204369, @atrick wrote:

> 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


Can you even do this if you don't have inbounds GEPs? Nuno, do you have an opinion about this?

> (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