diff options
| author | Eli Friedman <eli.friedman@gmail.com> | 2009-01-24 23:49:55 +0000 |
|---|---|---|
| committer | Eli Friedman <eli.friedman@gmail.com> | 2009-01-24 23:49:55 +0000 |
| commit | 8b7c52658bd5966c65a74d1404fa2fa96adc9edc (patch) | |
| tree | f56f13ee5bafbeca0cfd1965e1e25a6bc59c42fb /clang/lib/Sema/SemaDecl.cpp | |
| parent | 588395702f004f14cd67dad2a62366274391cb09 (diff) | |
| download | bcm5719-llvm-8b7c52658bd5966c65a74d1404fa2fa96adc9edc.tar.gz bcm5719-llvm-8b7c52658bd5966c65a74d1404fa2fa96adc9edc.zip | |
Fix for PR2100: merge types for variables.
llvm-svn: 62947
Diffstat (limited to 'clang/lib/Sema/SemaDecl.cpp')
| -rw-r--r-- | clang/lib/Sema/SemaDecl.cpp | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp index 0fb939e23ab..9f308842d9e 100644 --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -658,15 +658,15 @@ VarDecl *Sema::MergeVarDecl(VarDecl *New, Decl *OldD) { MergeAttributes(New, Old); - // Verify the types match. - QualType OldCType = Context.getCanonicalType(Old->getType()); - QualType NewCType = Context.getCanonicalType(New->getType()); - if (OldCType != NewCType && !Context.typesAreCompatible(OldCType, NewCType)) { + // Merge the types + QualType MergedT = Context.mergeTypes(New->getType(), Old->getType()); + if (MergedT.isNull()) { Diag(New->getLocation(), diag::err_redefinition_different_type) << New->getDeclName(); Diag(Old->getLocation(), diag::note_previous_definition); return New; } + New->setType(MergedT); // C99 6.2.2p4: Check if we have a static decl followed by a non-static. if (New->getStorageClass() == VarDecl::Static && (Old->getStorageClass() == VarDecl::None || |

