<div dir="ltr">On Fri, Mar 22, 2013 at 1:16 PM, John McCall <span dir="ltr"><<a href="mailto:rjmccall@apple.com" target="_blank">rjmccall@apple.com</a>></span> wrote:<br><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 style="word-wrap:break-word"><div><div class="im"><div>On Mar 22, 2013, at 9:58 AM, Reid Kleckner <<a href="mailto:rnk@google.com" target="_blank">rnk@google.com</a>> wrote:</div>
<blockquote type="cite"><div dir="ltr">On Thu, Mar 21, 2013 at 11:14 PM, John McCall <span dir="ltr"><<a href="mailto:rjmccall@apple.com" target="_blank">rjmccall@apple.com</a>></span> wrote:<br><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><span style="color:rgb(34,34,34)">All these tests are pretty inadequate;  you could be accidentally doing</span><br>

</div>
a lot of stuff besides the stores.  You should basically be testing the entire<br>
function body, including the define line.  This will also let you effectively<br>
test that you're generating the right IR type, which will be important when<br>
you start generating different IR types for different cases.<br></blockquote><div><br></div><div>OK, now the alloca and its size is part of the test.  That should constrain the size of the member pointer to a single slot.</div>
</div></div></div></blockquote><div><br></div></div><div>Might as well use CHECK-NEXT for everything after the first alloca line.</div></div></div></blockquote><div><br></div><div style>Sure, modulo the labels I omitted around the if lowering.</div>
<div><br></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><div>With that, feel free to commit, thanks!</div></div></div></blockquote>
<div><br></div><div style>Thanks for the review!</div><div><br></div></div></div></div>