[cfe-commits] r66431 - /cfe/trunk/lib/Sema/SemaTemplateInstantiate.cpp

Douglas Gregor dgregor at apple.com
Mon Mar 9 13:07:22 PDT 2009


Author: dgregor
Date: Mon Mar  9 15:07:22 2009
New Revision: 66431

URL: http://llvm.org/viewvc/llvm-project?rev=66431&view=rev
Log:
Revert r66423, which was not the right fix for this issue.

Modified:
    cfe/trunk/lib/Sema/SemaTemplateInstantiate.cpp

Modified: cfe/trunk/lib/Sema/SemaTemplateInstantiate.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaTemplateInstantiate.cpp?rev=66431&r1=66430&r2=66431&view=diff

==============================================================================
--- cfe/trunk/lib/Sema/SemaTemplateInstantiate.cpp (original)
+++ cfe/trunk/lib/Sema/SemaTemplateInstantiate.cpp Mon Mar  9 15:07:22 2009
@@ -134,7 +134,9 @@
   // BuildArrayType. Since we have already checked the size as part of
   // creating the dependent array type in the first place, we know
   // there aren't any errors.
-  IntegerLiteral ArraySize(T->getSize(), SemaRef.Context.getSizeType(), Loc);
+  // FIXME: Is IntTy big enough? Maybe not, but LongLongTy causes
+  // problems that I have yet to investigate.
+  IntegerLiteral ArraySize(T->getSize(), SemaRef.Context.IntTy, Loc);
   return SemaRef.BuildArrayType(ElementType, T->getSizeModifier(), 
                                 &ArraySize, T->getIndexTypeQualifier(), 
                                 Loc, Entity);





More information about the cfe-commits mailing list