<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 12, 2015 at 5:03 PM, Francois Pichet <span dir="ltr"><<a href="mailto:pichet2000@gmail.com" target="_blank">pichet2000@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>I think it would make sense for (1) and (2). I am not sure if (3) is feasible in <span style="font-size:12.8000001907349px">instcombine</span>. (I am not too familiar with LoopInfo)<br></div><div><br></div><div>For the Octasic's Opus platform, I modified shouldMergeGEPs in our fork to:</div><div><br></div><div><span class=""><div>  if (GEP.hasAllZeroIndices() && !Src.hasAllZeroIndices() &&</div><div>      !Src.hasOneUse())</div><div>    return false;</div><div><br></div></span><div>  return Src.hasAllConstantIndices(); // was return false;</div></div></div></blockquote><div><br></div><div>Sorry I meant            // was return true; </div></div></div></div>