llvm dev - May 2005 - [LLVMdev] LongTy in LowerInvoke.cpp

On Fri, 2005-05-13 at 08:06 +0200, Markus F.X.J. Oberhumer
wrote:
> Actually that's the only case I stumbled over this problem in a
somewhat
> larger C++ program, and it's clearly the wrong type in LowerInvoke.cpp
-
> it really should be IntPtrTy. But maybe we could use just IntTy here to 
> avoid target dependencies.
Wait a minute.  You want to lower a 64 bit thing to a 32 bit and "avoid
target dependencies" when the right type is a pointer?  You know the
64-bit targets won't like this one bit.

Andrew

On Fri, 13 May 2005, Andrew Lenharth wrote:
> On Fri, 2005-05-13 at 08:06 +0200, Markus F.X.J. Oberhumer wrote:
>> Actually that's the only case I stumbled over this problem in a
somewhat
>> larger C++ program, and it's clearly the wrong type in
LowerInvoke.cpp -
>> it really should be IntPtrTy. But maybe we could use just IntTy here to
>> avoid target dependencies.
>
> Wait a minute.  You want to lower a 64 bit thing to a 32 bit and
"avoid
> target dependencies" when the right type is a pointer?  You know the
> 64-bit targets won't like this one bit.
The code is fine, 64-bit targets support 32-bit gep offsets just like 
32-bit targets support 64-bit offsets.  We're talking about the constant 
zero here, so there are no bits to lose.  See LangRef.html for more info.

-Chris

-- 
http://nondot.org/sabre/
http://llvm.cs.uiuc.edu/