[llvm-commits] [llvm-gcc-4.2] r86371 - /llvm-gcc-4.2/trunk/gcc/llvm-convert.cpp

[Eric Christopher]

On Nov 8, 2009, at 11:29 AM, Duncan Sands wrote:

> Hi Eric,
> 
>>> Args[1] is required by the LLVM intrinsic to be an i32, but what guarantees
>>> that it is?
>> The check above on the tree I believe.
> 
> my understanding is that check checks that the value is of some integer type
> (int, long int etc) and not that it's int.

Hmm.. yeah - could be an i8 too. At any rate to make it a bit more safe I'll cast it to i32 so we're not depending on the type sent in from the tree.

thanks

-eric