On Feb 5, 2014, at 10:00 AM, Howard Hinnant <[email protected]> wrote:

> On Feb 5, 2014, at 12:50 PM, Marshall Clow <[email protected]> wrote:
> 
>> Turns out you have to treat NULL special.
>> Added a couple tests.
> 
> This looks awesome!  I recommend a test throwing nullptr as well, just as a 
> sanity check.  

Good idea. Done and committed as revision 200864 .

> And I think, but am not sure, that http://llvm.org/bugs/show_bug.cgi?id=17221 
> might be a duplicate.  

Sadly, not a duplicate.
I added a test, and it fails.

> And http://llvm.org/bugs/show_bug.cgi?id=17222 is in the same neighborhood, 
> but probably not a duplicate.

Definitely not a duplicate.

— Marshall


_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to