Committed as r252690.

Could I tempt you to put together a documentation patch for this, and a
change for the release notes?

On Tue, Nov 10, 2015 at 3:06 PM, Nicholas Allegra <com...@gmail.com> wrote:

> comex added a comment.
>
> I don't have SVN access.  If you want to commit it, feel free to remove
> that comment, or else I'll update the diff tomorrow.
>
> I just tried the test case in 25449 and it now gives a proper error rather
> than crashing.  (Though when trying to reduce a test failure I was
> encountering, I did find yet another pre-existing crasher, which doesn't
> seem to be related to the code being changed here.  I should definitely
> check which of the crashers I found have existing bug reports and file the
> rest.)
>
>
> http://reviews.llvm.org/D12686
>
>
>
>
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to