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

[Sebastian Redl]

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