r247367 - [MS ABI] Remove another call to RequireCompleteType

David Majnemer via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 10 17:53:15 PDT 2015


Author: majnemer
Date: Thu Sep 10 19:53:15 2015
New Revision: 247367

URL: http://llvm.org/viewvc/llvm-project?rev=247367&view=rev
Log:
[MS ABI] Remove another call to RequireCompleteType

I cannot come up with a testcase which would rely on this call to
RequireCompleteType, I believe that it is superfluous given the current
state of clang.

Modified:
    cfe/trunk/lib/Sema/SemaCast.cpp

Modified: cfe/trunk/lib/Sema/SemaCast.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaCast.cpp?rev=247367&r1=247366&r2=247367&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaCast.cpp (original)
+++ cfe/trunk/lib/Sema/SemaCast.cpp Thu Sep 10 19:53:15 2015
@@ -1496,10 +1496,6 @@ TryStaticImplicitCast(Sema &Self, ExprRe
       msg = 0;
       return TC_Failed;
     }
-  } else if (DestType->isMemberPointerType()) {
-    if (Self.Context.getTargetInfo().getCXXABI().isMicrosoft()) {
-      Self.RequireCompleteType(OpRange.getBegin(), DestType, 0);
-    }
   }
 
   InitializedEntity Entity = InitializedEntity::InitializeTemporary(DestType);




More information about the cfe-commits mailing list