<div dir="ltr"><div><div><div>Hi Chandler,<br></div>thanks for bringing attention to this. I'm not sure what happened to the email thread and why it doesn't show up on the revision now, but Nirav had responded to my comment and committed a fix within an hour or two of me adding the comment. The bots went back to green after that.<br><br></div>Thanks,<br></div>Nemanja<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Dec 29, 2017 at 3:54 PM, Chandler Carruth via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">chandlerc added subscribers: nemanjai, chandlerc.<br>
chandlerc added a comment.<br>
<br>
There was a comment by @nemanjai on the submitted revision in phab that I think got lost:<br>
<br>
In <a href="https://reviews.llvm.org/rL321089#229740" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>rL321089#229740</a>, @nemanjai wrote:<br>
<br>
> This has broken all the PowerPC buildbots. I'll pull this patch to get the bots back to green unless you have a fix available. The smallest failing benchmark from the test-suite is `sieve.c`.<br>
>  This patch removes half of the pre-inc stores that are actually needed. I imagine that the DAGCombiner is getting rid of these stores before they become pre-inc stores since there's a check for unindexed stores as part of this patch, but I haven't spent the time to really understand what this patch does.<br>
>  I'm attaching both the IR generated for `sieve.c` as well as a bugpoint-reduced version of it (the reduction was done by comparing the number of pre-inc stores emitted before the patch and after it).F5703558: sieve.ll <<a href="https://reviews.llvm.org/F5703558" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>F5703558</a>><br>
><br>
> F5703557: reduced.ll <<a href="https://reviews.llvm.org/F5703557" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>F5703557</a>><br>
<br>
<br>
Notably, all PPC bots were broken by this, and it has stayed in tree for a long time. we should get it reverted.<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D40969" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D40969</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>