[cfe-commits] r56078 - /cfe/trunk/lib/Parse/ParseStmt.cpp

[Argiris Kirtzidis]

Eli Friedman wrote:
> On Wed, Sep 10, 2008 at 4:46 PM, Argiris Kirtzidis <akyrtzi at gmail.com> wrote:
>   
>> Author: akirtzidis
>> Date: Wed Sep 10 18:46:08 2008
>> New Revision: 56078
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=56078&view=rev
>> Log:
>> -getLang().C99 is true in C++ too, remove the use of the C99orCXX variable.
>>     
>
> ?!?!  This is completely, utterly wrong.  The fact that C99 is defined
> for C++ is just an bug in InitializeLanguageStandard in clang.cpp.
> C++ and C99 are distinct standards, and abusing this is just
> confusing.
>
> Would you mind backing this out, and fixing InitializeLanguageStandard?
>   

Done:
http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20080908/007512.html
http://lists.cs.uiuc.edu/pipermail/cfe-commits/Week-of-Mon-20080908/007513.html

Thanks!

-Argiris