[cfe-commits] [Patch 4 of 7] -verify fixes and enhancement
andyg1001 at hotmail.co.uk
Mon Jul 2 11:45:46 PDT 2012
On Monday, July 02, 2012 7:48 PM, Jordan Rose wrote:
> On Jun 30, 2012, at 2:12 PM, Andy Gibbs wrote:
>> Part 4: Removed accidentally included override on diagnostic notes.
> This one I don't like. -verify tests shouldn't also be fatal error tests
> anyway. Even
> though it doesn't actually cause any problems. I don't think this is
> (Also, it invites future code like ReallySuppressAllDiagnostics, etc.,
> even if none
> of us would ever do that.)
I take your point, but my counter argument would be that it may be very
to have the scenario where a test-case that originally didn't fatal error,
an unintentional change the the compiler end up in a fatal error and which
missed it because the diagnostics following the fatal error get swallowed
now I can't be 100% sure but I think this actually came up during testing.
It was certainly
one of the later iterations of the patch that included this change.
It may be worth thinking of another alternative, but I personally feel that
it *is* very
useful to have diagnostics from -verify in the situation (intended or not!)
fatal error occurs. (Of course, any alternative would still need to leave
space for the
future ReallySuppressAllDiagnostics option!! -- joking, of course!).
More information about the cfe-commits