
<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Apr 28, 2014, at 5:49 PM, Andrew Trick <<a href="mailto:atrick@apple.com">atrick@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><meta http-equiv="Content-Type" content="text/html charset=windows-1252"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Apr 28, 2014, at 5:05 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr"><div class="gmail_extra">I don't object to having the fix checked in but:</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Apr 28, 2014 at 4:51 PM, Andrew Trick <span dir="ltr"><<a href="mailto:atrick@apple.com" target="_blank">atrick@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">one of the major difficulties in getting it fixed was we just have absolutely no way to force it in llc</blockquote></div>

<br>This is *exactly* why we should absolutely spend time isolating what it is about this test case that doesn't reproduce with llc. This is *critical* to the LLVM test infrastructure and project. It's not something we can or should paper over, for exactly the reason you described: it makes finding these bugs nearly impossible.</div>

</div>

</blockquote></div><br><div>I figured out exactly why this specific LSR bug would not reproduce with llc. That was one year ago now. The bug has been open that long. It is well known that we have a fundamental problem testing individual passes because serialization changes the use lists, and we want to be able to write passes that depend on the use list order. Someone else was looking at fixing that problem at the time…. it must not be easy because it still isn’t fixed.</div></div></blockquote><div><br></div><a href="http://llvm.org/PR5680">http://llvm.org/PR5680</a></div><div><br></div><div>reported in 2009.</div><div><br><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div><br></div><div>-Andy</div></div>_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits<br></blockquote></div><br></body></html>