[cfe-commits] r148642 - in /cfe/trunk: lib/Sema/SemaDecl.cpp test/SemaCXX/MicrosoftCompatibility.cpp test/SemaCXX/MicrosoftExtensions.cpp

[Francois Pichet]

Author: fpichet
Date: Sat Jan 21 17:26:50 2012
New Revision: 148642

URL: http://llvm.org/viewvc/llvm-project?rev=148642&view=rev
Log:
In Microsoft Mode, disable the C++11 strict integral conversion rules for 
enumerator that were introduced with r148439. Otherwise MSVC headers won't 
compile in C++ 11 mode.

Modified:
    cfe/trunk/lib/Sema/SemaDecl.cpp
    cfe/trunk/test/SemaCXX/MicrosoftCompatibility.cpp
    cfe/trunk/test/SemaCXX/MicrosoftExtensions.cpp

Modified: cfe/trunk/lib/Sema/SemaDecl.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaDecl.cpp?rev=148642&r1=148641&r2=148642&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaDecl.cpp (original)
+++ cfe/trunk/lib/Sema/SemaDecl.cpp Sat Jan 21 17:26:50 2012
@@ -9559,7 +9559,8 @@
       EltTy = Context.DependentTy;
     else {
       SourceLocation ExpLoc;
-      if (getLangOptions().CPlusPlus0x && Enum->isFixed()) {
+      if (getLangOptions().CPlusPlus0x && Enum->isFixed() &&
+          !getLangOptions().MicrosoftMode) {
         // C++11 [dcl.enum]p5: If the underlying type is fixed, [...] the
         // constant-expression in the enumerator-definition shall be a 
converted
         // constant expression of the underlying type.

Modified: cfe/trunk/test/SemaCXX/MicrosoftCompatibility.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/MicrosoftCompatibility.cpp?rev=148642&r1=148641&r2=148642&view=diff
==============================================================================
--- cfe/trunk/test/SemaCXX/MicrosoftCompatibility.cpp (original)
+++ cfe/trunk/test/SemaCXX/MicrosoftCompatibility.cpp Sat Jan 21 17:26:50 2012
@@ -1,4 +1,4 @@
-// RUN: %clang_cc1 %s -triple i686-pc-win32 -fsyntax-only -Wmicrosoft -verify 
-fms-compatibility -fexceptions -fcxx-exceptions
+// RUN: %clang_cc1 %s -triple i686-pc-win32 -fsyntax-only -std=c++11 
-Wmicrosoft -verify -fms-compatibility -fexceptions -fcxx-exceptions
 
 
 
@@ -135,5 +135,10 @@
 
 }
 
+enum ENUM2 {
+       ENUM2_a = (enum ENUM2) 4,
+       ENUM2_b = 0x9FFFFFFF, // expected-warning {{enumerator value is not 
representable in the underlying type 'int'}}
+       ENUM2_c = 0x100000000 // expected-warning {{enumerator value is not 
representable in the underlying type 'int'}}
+};
 
 

Modified: cfe/trunk/test/SemaCXX/MicrosoftExtensions.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/MicrosoftExtensions.cpp?rev=148642&r1=148641&r2=148642&view=diff
==============================================================================
--- cfe/trunk/test/SemaCXX/MicrosoftExtensions.cpp (original)
+++ cfe/trunk/test/SemaCXX/MicrosoftExtensions.cpp Sat Jan 21 17:26:50 2012
@@ -99,11 +99,6 @@
 enum ENUM1* var3 = 0;// expected-warning {{forward references to 'enum' types 
are a Microsoft extension}}
 
 
-enum ENUM2 {
-       ENUM2_a = (enum ENUM2) 4,
-       ENUM2_b = 0x9FFFFFFF, // expected-warning {{enumerator value is not 
representable in the underlying type 'int'}}
-       ENUM2_c = 0x100000000 // expected-warning {{enumerator value is not 
representable in the underlying type 'int'}}
-};
 
 
 void f(long long);


_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits