[cfe-commits] r121961 - /cfe/trunk/test/SemaCXX/warn-unreachable.cpp

[Ted Kremenek]

Weird.  I checked it in *because* the buildbots were breaking.

On Dec 16, 2010, at 12:54 AM, Douglas Gregor wrote:

> This was breaking the buildbots, so I reverted it in r121963.
> 
> 	- Doug
> 
> On Dec 16, 2010, at 12:22 AM, Ted Kremenek wrote:
> 
>> Author: kremenek
>> Date: Thu Dec 16 02:22:16 2010
>> New Revision: 121961
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=121961&view=rev
>> Log:
>> Tweak location of diagnostic for -Wunreachable-code
>> test due to recent changes to the CFG.  The
>> diagnostic is somewhat in the wrong place, but
>> the -Wunreachable-code diagnostic needs to be
>> revamped anyway since most of the diagnostics
>> in this test case are redundant.
>> 
>> Modified:
>>   cfe/trunk/test/SemaCXX/warn-unreachable.cpp
>> 
>> Modified: cfe/trunk/test/SemaCXX/warn-unreachable.cpp
>> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/warn-unreachable.cpp?rev=121961&r1=121960&r2=121961&view=diff
>> ==============================================================================
>> --- cfe/trunk/test/SemaCXX/warn-unreachable.cpp (original)
>> +++ cfe/trunk/test/SemaCXX/warn-unreachable.cpp Thu Dec 16 02:22:16 2010
>> @@ -39,8 +39,10 @@
>> void test3() {
>>  halt()
>>    --;         // expected-warning {{will never be executed}}
>> -  halt()
>> -    ?           // expected-warning {{will never be executed}}
>> +  // FIXME: The unreachable part is just the '?', but really all of this
>> +  // code is unreachable and shouldn't be separately reported.
>> +  halt()        // expected-warning {{will never be executed}}
>> +    ? 
>>    dead() : dead();
>>  live(),
>>    float       // expected-warning {{will never be executed}}
>> 
>> 
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>