diff options
author | Reid Kleckner <reid@kleckner.net> | 2014-11-03 21:56:03 +0000 |
---|---|---|
committer | Reid Kleckner <reid@kleckner.net> | 2014-11-03 21:56:03 +0000 |
commit | 8a8ccbf4fb898b7d12d9a0ac66e4c49044098118 (patch) | |
tree | 02323fe93633d8af0f1b71ac1571b4044c8bb16c /lib/Sema/SemaDecl.cpp | |
parent | 734e0ab01d133dc07f3c67b116894a4b1bb0be3c (diff) |
Move the no-prototype calling conv check after decl merging
Now we don't warn on this code:
void __stdcall f(void);
void __stdcall f();
My previous commit regressed this functionality because I didn't update
the relevant test case which used a definition.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@221188 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Sema/SemaDecl.cpp')
-rw-r--r-- | lib/Sema/SemaDecl.cpp | 27 |
1 files changed, 14 insertions, 13 deletions
diff --git a/lib/Sema/SemaDecl.cpp b/lib/Sema/SemaDecl.cpp index b9f8f637e2..109c01c889 100644 --- a/lib/Sema/SemaDecl.cpp +++ b/lib/Sema/SemaDecl.cpp @@ -7383,10 +7383,21 @@ Sema::ActOnFunctionDeclarator(Scope *S, Declarator &D, DeclContext *DC, if (!NewFD->isInvalidDecl() && NewFD->isMSVCRTEntryPoint()) CheckMSVCRTEntryPoint(NewFD); + if (!NewFD->isInvalidDecl()) + D.setRedeclaration(CheckFunctionDeclaration(S, NewFD, Previous, + isExplicitSpecialization)); + else if (!Previous.empty()) + // Make graceful recovery from an invalid redeclaration. + D.setRedeclaration(true); + assert((NewFD->isInvalidDecl() || !D.isRedeclaration() || + Previous.getResultKind() != LookupResult::FoundOverloaded) && + "previous declaration set still overloaded"); + // Diagnose no-prototype function declarations with calling conventions that - // don't support variadic calls. - const FunctionType *FT = R->castAs<FunctionType>(); - if (FT->isFunctionNoProtoType() && !D.isFunctionDefinition()) { + // don't support variadic calls. Only do this in C and do it after merging + // possibly prototyped redeclarations. + const FunctionType *FT = NewFD->getType()->castAs<FunctionType>(); + if (isa<FunctionNoProtoType>(FT) && !D.isFunctionDefinition()) { CallingConv CC = FT->getExtInfo().getCC(); if (!supportsVariadicCall(CC)) { // Windows system headers sometimes accidentally use stdcall without @@ -7397,16 +7408,6 @@ Sema::ActOnFunctionDeclarator(Scope *S, Declarator &D, DeclContext *DC, << FunctionType::getNameForCallConv(CC); } } - - if (!NewFD->isInvalidDecl()) - D.setRedeclaration(CheckFunctionDeclaration(S, NewFD, Previous, - isExplicitSpecialization)); - else if (!Previous.empty()) - // Make graceful recovery from an invalid redeclaration. - D.setRedeclaration(true); - assert((NewFD->isInvalidDecl() || !D.isRedeclaration() || - Previous.getResultKind() != LookupResult::FoundOverloaded) && - "previous declaration set still overloaded"); } else { // C++11 [replacement.functions]p3: // The program's definitions shall not be specified as inline. |