From: Chris Lattner <sabre@nondot.org>
Date: Wed, 13 May 2009 04:12:56 +0000 (+0000)
Subject: Fix rdar://6880874 - [sema] crash on array types with different index sizes
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=38aeec7299c48cb79523f7f89776fb258c84aeea;p=clang

Fix rdar://6880874 - [sema] crash on array types with different index sizes



git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@71634 91177308-0d34-0410-b5e6-96231b3b80d8
---

diff --git a/lib/AST/ASTContext.cpp b/lib/AST/ASTContext.cpp
index ac46180b94..ccd3762da0 100644
--- a/lib/AST/ASTContext.cpp
+++ b/lib/AST/ASTContext.cpp
@@ -1133,9 +1133,14 @@ QualType ASTContext::getMemberPointerType(QualType T, const Type *Cls)
 /// getConstantArrayType - Return the unique reference to the type for an 
 /// array of the specified element type.
 QualType ASTContext::getConstantArrayType(QualType EltTy, 
-                                          const llvm::APInt &ArySize,
+                                          const llvm::APInt &ArySizeIn,
                                           ArrayType::ArraySizeModifier ASM,
                                           unsigned EltTypeQuals) {
+  // Convert the array size into a canonical width matching the pointer size for
+  // the target.
+  llvm::APInt ArySize(ArySizeIn);
+  ArySize.zextOrTrunc(Target.getPointerWidth(EltTy.getAddressSpace()));
+  
   llvm::FoldingSetNodeID ID;
   ConstantArrayType::Profile(ID, EltTy, ArySize, ASM, EltTypeQuals);
       
diff --git a/test/Sema/types.c b/test/Sema/types.c
index 3ee76756ee..7ab4e0672c 100644
--- a/test/Sema/types.c
+++ b/test/Sema/types.c
@@ -19,3 +19,14 @@ int b() {
   int __int128_t;
   int __uint128_t;
 }
+
+
+// Array type merging should convert array size to whatever matches the target
+// pointer size.
+// rdar://6880874
+extern int i[1LL];
+int i[(short)1];
+
+enum e { e_1 };
+extern int j[sizeof(enum e)];  // expected-note {{previous definition}}
+int j[42];   // expected-error {{redefinition of 'j' with a different type}}