<div dir="ltr">(and to be clear, reverting it would just hide the bugs pretty well, they'd fix it)<div><br></div><div>I think i have a good way of fixing this properly, patch coming.</div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 27, 2016 at 5:26 PM, Daniel Berlin <span dir="ltr"><<a href="mailto:dberlin@dberlin.org" target="_blank">dberlin@dberlin.org</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">These are all caused by a set of failures to properly mark memory users as changed in some cases, and to initialize the memory operand equivalence table properly.<div><br></div><div>I'm thinking how best to solve it.</div><div>In the meantime, reverting the store equivalence changes should fix most of it (there is one other bug in updateReachableEdge that won't fix, where it misses marking memoryphi's)</div><div><br></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 27, 2016 at 3:34 PM, Daniel Berlin <span dir="ltr"><<a href="mailto:dberlin@dberlin.org" target="_blank">dberlin@dberlin.org</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"><br><div class="gmail_extra"><br><div class="gmail_quote"><span>On Tue, Dec 27, 2016 at 3:22 PM, Nemanja Ivanovic via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</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><div><div>I assume that this passes all the lit/lnt test cases on the platform that the developers work on.</div></div></div></div></blockquote><div><br></div></span><div>Yes</div><div>I bootstrap/test it.</div><span><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div> It causes 31 lnt test case failures on PPC. I've opened a bug with the one that is easiest to reduce.<br></div>Also, I'm happy to assist by providing any further PPC specific information you require regarding this.<br></div><br></div></div></blockquote></span><div>Thank you</div><div><div class="m_2490625407010845274h5"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div></div>Nemanja<br></div><div class="m_2490625407010845274m_7798354177190470203HOEnZb"><div class="m_2490625407010845274m_7798354177190470203h5"><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Dec 26, 2016 at 9:55 PM, Mehdi Amini via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Also: does clang bootstrap and pass the tests (and builds the test-suite correctly?).<br>
<br>
—<br>
Mehdi<br>
<div class="m_2490625407010845274m_7798354177190470203m_-2055834249532895274HOEnZb"><div class="m_2490625407010845274m_7798354177190470203m_-2055834249532895274h5"><br>
> On Dec 26, 2016, at 12:54 PM, Mehdi Amini via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
><br>
> Hi,<br>
><br>
> I’m curious if the test-suite running cleanly with this flag?<br>
> (That would be a first step before wider testing).<br>
><br>
> Thanks,<br>
><br>
> —<br>
> Mehdi<br>
><br>
>> On Dec 26, 2016, at 10:45 AM, Davide Italiano via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
>><br>
>> Hi everybody.<br>
>> NewGVN was recently committed and a few minute ago I added a flag to<br>
>> enable the new experimental pass.<br>
>><br>
>> For the brave soul, passing `-mllvm -enable-newgvn` should do the<br>
>> trick. We'll be happy to receive bug reports to analyze/fix, bonus<br>
>> point if they contain a synthetic/reduced testcase.<br>
>> Open a bug linked to <a href="https://llvm.org/bugs/show_bug.cgi?id=30995" rel="noreferrer" target="_blank">https://llvm.org/bugs/show_bug<wbr>.cgi?id=30995</a> would<br>
>> be probably best so that all the people working on NewGVN will have<br>
>> visibility, but I'm aware there are issues with bugzilla registration<br>
>> disabled recently so if you have issues/don't have an account feel<br>
>> free to shoot me a mail directly and I'll open a bug on your behalf.<br>
>><br>
>> Looking forward to hearing from you,<br>
>><br>
>> --<br>
>> Davide<br>
>><br>
>> "There are no solved problems; there are only problems that are more<br>
>> or less solved" -- Henri Poincare<br>
>> ______________________________<wbr>_________________<br>
>> LLVM Developers mailing list<br>
>> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
>> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
><br>
> ______________________________<wbr>_________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
> <a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
<br>
______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
</div></div></blockquote></div><br></div>
</div></div><br>______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
<br></blockquote></div></div></div><br></div></div>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>