<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Mon, Dec 14, 2015 at 11:05 AM Tim Northover <<a href="mailto:t.p.northover@gmail.com">t.p.northover@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">> I don't think it's going to be even vaguely that bad here and that you're<br>
> blowing it a bit out of proportion. [...] It does make the tests a little harder<br>
> to write, but having done a bunch of them it's not that bad.<br>
<br>
I've also tried to write such tests in the past, and I did find it that bad.<br>
<br>
> Also you're relying on the behavior of<br>
> backend optimizations to figure out what's going on with your clang IR<br>
> generation which also seems less than optimal.<br>
<br>
I think it's fine if it leads to simpler tests that are easier to<br>
maintain. It's not completely isolated, but none of the testing we do<br>
really is.<br><br></blockquote><div><br></div><div>There really is a pretty good separation of concerns and for a lot if not most of the tests here all the difference is checking is the arguments, keeping track of an alloca and making sure that goes is pretty simple and not really hard for a test.</div><div><br></div><div>-eric </div></div></div>