[cfe-commits] r65669 - /cfe/trunk/lib/Sema/SemaTemplateInstantiate.cpp
dgregor at apple.com
Mon Mar 9 11:49:08 CDT 2009
On Feb 28, 2009, at 1:05 AM, Sebastian Redl wrote:
> 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
>> Add a FIXME for something I can't look at just yet
>> 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
>> // 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?
More information about the cfe-commits