<p dir="ltr">Are they still failing after the followup in r209065?<br>
</p>
<div class="gmail_quote">On May 17, 2014 5:41 AM, "Tobias Grosser" <<a href="mailto:tobias@grosser.es">tobias@grosser.es</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 17/05/2014 01:47, Louis Gerbarg wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: louis<br>
Date: Fri May 16 18:47:24 2014<br>
New Revision: 209049<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=209049&view=rev" target="_blank">http://llvm.org/viewvc/llvm-<u></u>project?rev=209049&view=rev</a><br>
Log:<br>
Add support for combining GEPs across PHI nodes<br>
<br>
Currently LLVM will generally merge GEPs. This allows backends to use more<br>
complex addressing modes. In some cases this is not happening because there<br>
is PHI inbetween the two GEPs:<br>
<br>
   GEP1--\<br>
         |-->PHI1-->GEP3<br>
   GEP2--/<br>
<br>
This patch checks to see if GEP1 and GEP2 are similiar enough that they can be<br>
cloned (GEP12) in GEP3's BB, allowing GEP->GEP merging (GEP123):<br>
<br>
   GEP1--\                     --\                           --\<br>
         |-->PHI1-->GEP3  ==>    |-->PHI2->GEP12->GEP3 == >    |-->PHI2->GEP123<br>
   GEP2--/                     --/                           --/<br>
<br>
This also breaks certain use chains that are preventing GEP->GEP merges that the<br>
the existing instcombine would merge otherwise.<br>
<br>
Tests included.<br>
<br>
rdar://15547484<br>
</blockquote>
<br>
After this commit, I see plenty of execution time failures (miscompiles?) on my LNT (clang -O3) servers:<br>
<br>
<a href="http://lab.llvm.org:8011/builders/polly-perf-O3/builds/3322" target="_blank">http://lab.llvm.org:8011/<u></u>builders/polly-perf-O3/builds/<u></u>3322</a><br>
<br>
Cheers,<br>
Tobias<br>
______________________________<u></u>_________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">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/<u></u>mailman/listinfo/llvm-commits</a><br>
</blockquote></div>