<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Sat, Feb 23, 2013 at 2:09 PM, Cameron Zwarich <span dir="ltr"><<a href="mailto:zwarich@apple.com" target="_blank">zwarich@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div class="im"><div>On Feb 23, 2013, at 12:26 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:</div>
<br><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Feb 23, 2013 at 12:23 PM, Cameron Zwarich <span dir="ltr"><<a href="mailto:zwarich@apple.com" target="_blank">zwarich@apple.com</a>></span> wrote:</div>
</div></div></blockquote></div><div class="im"><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="auto"><div>Running it on test-suite with -g caused assertions to fail, so I assumed that was good enough.</div>
</div></blockquote><div><br></div><div>Generally we prefer to have regression tests checked in with code changes - the test suite is used to catch things we failed to test but we take the opportunity to fix/improve the regression test coverage when the test-suite uncovers something.</div>
</div></div></div></blockquote><br></div></div><div>That is generally the case for LLVM, but for experimental codegen changes in the past we have generally used the test-suite without adding a LIT test for every issue. </div>
</div></blockquote><div><br></div><div style>That's not been my experience - I realize some experimental out-of-tree code sometimes requires in tree changes that aren't testable without a unit test (custom code to call into the APIs in a way that would emulate the out-of-tree code), but for anything that can be tested with the existing tools (even using custom flags, specific passes, etc) I usually expect & have seen test cases.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div>Maybe that should change (and it would be easier to do if we had codegen tests that could just run a single pass), but it's nothing unusual.</div>
</div></blockquote><div><br></div><div style>Certainly the LLVM code generation layer has some issues in terms of fine-grained testability, but it's still generally tested so far as I know.<br><br>- David </div></div>
<br></div></div>