Add support for member pointers to const_cast.

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@63055 91177308-0d34-0410-b5e6-96231b3b80d8

diff --git a/lib/Sema/SemaNamedCast.cpp b/lib/Sema/SemaNamedCast.cpp
index 06fc9d83bad05bcbf4e5dc30691ebb17a6c5652e..2aeab44db3df97ee9515778bbad2bf6ea0ca7374 100644 (file)
--- a/lib/Sema/SemaNamedCast.cpp
+++ b/lib/Sema/SemaNamedCast.cpp
@@ -133,7 +133,10 @@ CheckConstCast(Sema &Self, Expr *&SrcExpr, QualType DestType,
     SrcType = SrcExpr->getType();
   }
 
-  if (!DestType->isPointerType()) {
+  // C++ 5.2.11p5: For a const_cast involving pointers to data members [...]
+  //   the rules for const_cast are the same as those used for pointers.
+
+  if (!DestType->isPointerType() && !DestType->isMemberPointerType()) {
     // Cannot cast to non-pointer, non-reference type. Note that, if DestType
     // was a reference type, we converted it to a pointer above.
     // C++ 5.2.11p3: For two pointer types [...]
@@ -141,7 +144,8 @@ CheckConstCast(Sema &Self, Expr *&SrcExpr, QualType DestType,
       << OrigDestType << DestRange;
     return;
   }
-  if (DestType->isFunctionPointerType()) {
+  if (DestType->isFunctionPointerType() ||
+      DestType->isMemberFunctionPointerType()) {
     // Cannot cast direct function pointers.
     // C++ 5.2.11p2: [...] where T is any object type or the void type [...]
     // T is the ultimate pointee of source and target type.

diff --git a/test/SemaCXX/const-cast.cpp b/test/SemaCXX/const-cast.cpp
index 03a0e908ebbff4954e52a4b67d6ad769a7989689..0c334bf9faa51fa807e43039405cfdb5ef2bfa9e 100644 (file)
--- a/test/SemaCXX/const-cast.cpp
+++ b/test/SemaCXX/const-cast.cpp
@@ -1,5 +1,7 @@
 // RUN: clang -fsyntax-only -verify %s
 
+struct A {};
+
 // See if aliasing can confuse this baby.
 typedef char c;
 typedef c *cp;
@@ -33,6 +35,9 @@ char ***good_const_cast_test(ccvpcvpp var)
   f fp = 0;
   // Don't misidentify fn** as a function pointer.
   f *fpp = const_cast<f*>(&fp);
+  int const A::* const A::*icapcap = 0;
+  int A::* A::* iapap = const_cast<int A::* A::*>(icapcap);
+
   return var4;
 }
 
@@ -52,5 +57,7 @@ short *bad_const_cast_test(char const *volatile *const volatile *var)
   f fp1 = 0;
   // Function pointers.
   f fp2 = const_cast<f>(fp1); // expected-error {{const_cast to 'f', which is not a reference, pointer-to-object, or pointer-to-data-member}}
+  void (A::*mfn)() = 0;
+  (void)const_cast<void (A::*)()>(mfn); // expected-error {{const_cast to 'void (struct A::*)(void)', which is not a reference, pointer-to-object, or pointer-to-data-member}}
   return **var3;
 }

diff --git a/www/cxx_status.html b/www/cxx_status.html
index 664e452987e3dd71a87087ee7de2060e26e7df2c..5c57f913abac1d03d563fada57489cd22b1d443c 100644 (file)
--- a/www/cxx_status.html
+++ b/www/cxx_status.html
@@ -578,7 +578,7 @@ welcome!</p>
 </tr>\r
 <tr>\r
   <td>&nbsp;&nbsp;&nbsp;&nbsp;5.2.10 [expr.reinterpret.cast]</td>\r
-  <td class="complete" align="center">&#x2713;</td>  \r
+  <td class="complete" align="center">&#x2713;</td>\r
   <td class="advanced" align="center"></td>\r
   <td class="advanced" align="center"></td>\r
   <td></td>\r
@@ -586,11 +586,11 @@ welcome!</p>
 </tr>\r
 <tr>\r
   <td>&nbsp;&nbsp;&nbsp;&nbsp;5.2.11 [expr.const.cast]</td>\r
-  <td class="complete" align="center">&#x2713;</td>  \r
-  <td class="advanced" align="center"></td>\r
-  <td class="advanced" align="center"></td>\r
+  <td class="complete" align="center">&#x2713;</td>\r
+  <td class="complete" align="center">&#x2713;</td>\r
+  <td class="complete" align="center">&#x2713;</td>\r
+  <td></td>\r
   <td></td>\r
-  <td>Missing member pointer conversions</td>  \r
 </tr>\r
 <tr><td>&nbsp;&nbsp;5.3 [expr.unary]</td><td></td><td></td><td></td><td></td><td></td></tr>\r
 <tr><td>&nbsp;&nbsp;&nbsp;&nbsp;5.3.1 [expr.unary.op]</td><td></td><td></td><td></td><td></td><td></td></tr>\r