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

Douglas Gregor dgregor at apple.com
Mon Mar 9 09:48:55 PDT 2009


Author: dgregor
Date: Mon Mar  9 11:48:55 2009
New Revision: 66423

URL: http://llvm.org/viewvc/llvm-project?rev=66423&view=rev
Log:
Fix a little FIXME, thanks to Sebastian

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=66423&r1=66422&r2=66423&view=diff

==============================================================================
--- cfe/trunk/lib/Sema/SemaTemplateInstantiate.cpp (original)
+++ cfe/trunk/lib/Sema/SemaTemplateInstantiate.cpp Mon Mar  9 11:48:55 2009
@@ -134,9 +134,7 @@
   // 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.
-  // 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);
+  IntegerLiteral ArraySize(T->getSize(), SemaRef.Context.getSizeType(), Loc);
   return SemaRef.BuildArrayType(ElementType, T->getSizeModifier(), 
                                 &ArraySize, T->getIndexTypeQualifier(), 
                                 Loc, Entity);





More information about the cfe-commits mailing list