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
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
