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

[Douglas Gregor]

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
>> 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?

Yep. Thanks!

	- Doug