Douglas Gregor wrote: > Author: dgregor > Date: Fri Feb 27 19:09:26 2009 > New Revision: 65669 > > URL: http://llvm.org/viewvc/llvm-project?rev=65669&view=rev > Log: > Add a FIXME for something I can't look at just yet > > 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=65669&r1=65668&r2=65669&view=diff > > ============================================================================== > --- cfe/trunk/lib/Sema/SemaTemplateInstantiate.cpp (original) > +++ cfe/trunk/lib/Sema/SemaTemplateInstantiate.cpp Fri Feb 27 19:09:26 2009 > @@ -134,6 +134,8 @@ > // 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); > Shouldn't you just use Context.getSizeType() for this?
Sebastian _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
