<div dir="ltr">Sorry, I got used to relying on phabricator to keep track of things, and forgot about this one.<div><br></div><div>Could you split the test in to a separate file and add `// XFAIL: *` at the top? That way you can commit it with the guts of the patch and we don't have to worry about forgetting to submit the test later.</div><div><br></div><div>Other than that, LGTM.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 10, 2014 at 5:12 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">Ping?<br>
<div class="HOEnZb"><div class="h5"><br>
On Thu, Aug 28, 2014 at 5:20 PM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>
> I apologize for the noise on this thread. It turns out the *original*<br>
> spelling was __cxa_bad_array_new_length, but the current spelling is<br>
> __cxa_throw_bad_array_new_length. This patches uses the most current<br>
> spelling.<br>
><br>
> ~Aaron<br>
><br>
> On Thu, Aug 28, 2014 at 2:52 PM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>
>> v2 of this patch, this one without tests. Clang does not currently<br>
>> codegen the correct call to __cxa_bad_array_new_length, so the test<br>
>> case from the previous patch would result in passing -1 to ::operator<br>
>> new[], which would then throw a std::bad_alloc, causing the test to<br>
>> fail. This patch elides the test temporarily, and once Clang codegens<br>
>> the proper call to __cxa_bad_array_new_length, the test will be<br>
>> committed for it.<br>
>><br>
>> ~Aaron<br>
>><br>
>> On Thu, Aug 28, 2014 at 2:07 PM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br>
>>> This patch adds support for __cxa_bad_array_new_length, which is<br>
>>> required for C++11 runtime support to throw std::bad_array_new_length.<br>
>>><br>
>>> Thanks!<br>
>>><br>
>>> ~Aaron<br>
</div></div></blockquote></div><br></div>