<div dir="ltr">I agree. I remembered the plan was to disable the blind GEP merging which have shown big damages to performance, reported independently by different users, and tune the  heuristic to enable beneficial merging selectively. <div><br></div><div>David<br><div><br></div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, May 19, 2015 at 3:10 PM, Wei Mi <span dir="ltr"><<a href="mailto:wmi@google.com" target="_blank">wmi@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Quentin,<br>
<br>
Yes, it is a tuning of the heuristic from <a href="http://reviews.llvm.org/D8911" target="_blank">http://reviews.llvm.org/D8911</a>.<br>
<br>
Because the regression I saw here has much smaller negative impact<br>
compared with the original negative impact of gep merging, is it<br>
possible that we leave r235455 there while we discuss a better fix?<br>
Because in this way we can check is there any other analysis affected<br>
by disabling gep merging.<br>
<br>
Thanks,<br>
Wei.<br>
<div class="HOEnZb"><div class="h5"><br>
On Tue, May 19, 2015 at 3:00 PM, Quentin Colombet <<a href="mailto:qcolombet@apple.com">qcolombet@apple.com</a>> wrote:<br>
> Hi Wei,<br>
><br>
> Correct me if I am wrong, but this is a tuning of the heuristic from <a href="http://reviews.llvm.org/D8911" target="_blank">http://reviews.llvm.org/D8911</a>, right?<br>
><br>
> Given that you actually see a regression now (and the fact that after discussing with Chandler, I am not convinced the initial change makes sense), I would suggest that you revert r235455 and that we start again the discussion on how to properly fix that.<br>
><br>
> Thanks,<br>
> -Quentin<br>
><br>
><br>
> REPOSITORY<br>
>   rL LLVM<br>
><br>
> <a href="http://reviews.llvm.org/D9865" target="_blank">http://reviews.llvm.org/D9865</a><br>
><br>
> EMAIL PREFERENCES<br>
>   <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
><br>
><br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
</div></div></blockquote></div><br></div>