From: David Majnemer <david.majnemer@gmail.com>
Date: Fri, 11 Sep 2015 00:53:15 +0000 (+0000)
Subject: [MS ABI] Remove another call to RequireCompleteType
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=664720572c07caa043bb9991847acec14c49d4f1;p=clang

[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.

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

diff --git a/lib/Sema/SemaCast.cpp b/lib/Sema/SemaCast.cpp
index 446c68055a..fa41499089 100644
--- a/lib/Sema/SemaCast.cpp
+++ b/lib/Sema/SemaCast.cpp
@@ -1496,10 +1496,6 @@ TryStaticImplicitCast(Sema &Self, ExprResult &SrcExpr, QualType DestType,
       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);