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); _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits