diff options
| author | David Majnemer <david.majnemer@gmail.com> | 2015-01-14 08:08:52 +0000 |
|---|---|---|
| committer | David Majnemer <david.majnemer@gmail.com> | 2015-01-14 08:08:52 +0000 |
| commit | d1091e5e34bd9bf984edf51d252663f1c848657b (patch) | |
| tree | f32dc1e9f137d7f656608d11e5f41c983c9a8256 /clang/lib/Sema/SemaDecl.cpp | |
| parent | baaaa67819c3b933577b13ae8776de43de2debd5 (diff) | |
| download | bcm5719-llvm-d1091e5e34bd9bf984edf51d252663f1c848657b.tar.gz bcm5719-llvm-d1091e5e34bd9bf984edf51d252663f1c848657b.zip | |
Sema: It's cheaper to ask LookupResult::empty than to calculate linkage
llvm-svn: 225960
Diffstat (limited to 'clang/lib/Sema/SemaDecl.cpp')
| -rw-r--r-- | clang/lib/Sema/SemaDecl.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp index b967e17148d..007470344f1 100644 --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -8059,7 +8059,7 @@ bool Sema::CheckFunctionDeclaration(Scope *S, FunctionDecl *NewFD, // the function returns a UDT (class, struct, or union type) that is not C // compatible, and if it does, warn the user. // But, issue any diagnostic on the first declaration only. - if (NewFD->isExternC() && Previous.empty()) { + if (Previous.empty() && NewFD->isExternC()) { QualType R = NewFD->getReturnType(); if (R->isIncompleteType() && !R->isVoidType()) Diag(NewFD->getLocation(), diag::warn_return_value_udt_incomplete) |
