
<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:19 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"><br><div class="gmail_quote">On Mon, Apr 28, 2014 at 5:12 PM, Adam Nemet <span dir="ltr"><<a href="mailto:anemet@apple.com" target="_blank">anemet@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto;"><div>OK, so as I understand this is the plan:</div><div><br></div><div>* file a PR on the original LSR problem with the C test case</div>

<div>* check in the fix again (without the test case), referencing the PR</div><div>* keep the PR open at this point until we figure out a way to have an LLVM IR test case for this</div></blockquote><div><br></div><div>I would file the PR with the C test case for "llc cannot reproduce LSR behavior observed with clang" or some such. It should have nothing to do with the particular bug fix, and the bug fix shouldn't reference the PR (we don't ever reference PRs from source code).</div>

<div><br></div><div>Another way to look at this is that the PR should essentially be "make llc able to test this, and commit the test case when that works”.</div></div></div></div></blockquote><div><br></div>OK.</div><div><br><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div>* remove clang from the substitutions in lit.cfg so that the next person won’t get confused like I got</div></blockquote></div><br>Sure, but in a separate commit.</div></div></blockquote><div><br></div>Of course.</div><div><br><blockquote type="cite"><div dir="ltr"><div class="gmail_extra">Despite all of this, I'm a little worried that this will get shelved in a PR and no one will ever look at it. This kind of deficiency in our testing infrastructure seems like a really scary and important thing, but its not even something I know how to chase down (not being an expert on LSR).</div>

</div>

</blockquote></div><br><div>I can certainly try to tackle this with some help from Andy.</div><div><br></div><div>Adam</div><div><br></div></body></html>