<div dir="ltr">I've changed the test so that it calls __cxa_throw_bad_array_new_length() (I'm mad at whoever gave this function such a long name) directly rather than expecting the compiler to call it for bad input to new[]. That check can go in the compiler tests.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 11, 2014 at 6:39 AM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wed, Sep 10, 2014 at 11:52 AM, Dan Albert <<a href="mailto:danalbert@google.com">danalbert@google.com</a>> wrote:<br>
> Sorry, I got used to relying on phabricator to keep track of things, and<br>
> forgot about this one.<br>
<br>
</span>No worries!<br>
<span class=""><br>
> Could you split the test in to a separate file and add `// XFAIL: *` at the<br>
> top? That way you can commit it with the guts of the patch and we don't have<br>
> to worry about forgetting to submit the test later.<br>
><br>
> Other than that, LGTM.<br>
<br>
</span>I've split the test out into its own file, and have attached the patch here.<br>
<br>
Since I don't have a way to test this locally, and no other tests have<br>
XFAIL lines, I'm not quite comfortable committing this without someone<br>
who can run the tests confirming that it runs cleanly.<br>
<br>
Thanks!<br>
<span class="HOEnZb"><font color="#888888"><br>
~Aaron<br>
</font></span></blockquote></div><br></div>