Author: cornedbee
Date: Tue Feb 2 20:08:48 2010
New Revision: 95188
URL: http://llvm.org/viewvc/llvm-project?rev=95188&view=rev
Log:
Don't do an expensive definition check where a cheap 'is this C?' check
suffices.
Modified:
cfe/trunk/lib/Sema/SemaDecl.cpp
Modified: cfe/trunk/lib/Sema/SemaDecl.cpp
URL:
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaDecl.cpp?rev=95188&r1=95187&r2=95188&view=diff
==============================================================================
--- cfe/trunk/lib/Sema/SemaDecl.cpp (original)
+++ cfe/trunk/lib/Sema/SemaDecl.cpp Tue Feb 2 20:08:48 2010
@@ -1300,7 +1300,8 @@
// C++ doesn't have tentative definitions, so go right ahead and check here.
const VarDecl *Def;
- if (New->isThisDeclarationADefinition() == VarDecl::Definition &&
+ if (getLangOptions().CPlusPlus &&
+ New->isThisDeclarationADefinition() == VarDecl::Definition &&
(Def = Old->getDefinition())) {
Diag(New->getLocation(), diag::err_redefinition)
<< New->getDeclName();
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
