[llvm-commits] [llvm] r51216 - in /llvm/trunk: lib/VMCore/ConstantFold.cpp test/Transforms/InstCombine/2008-05-17-FoldIntToPtr.ll

[Nick Lewycky]

Gordon Henriksen wrote:
> On 2008-05-17, at 05:03, Nick Lewycky wrote:
> 
>> +    case Instruction::PtrToInt:
>> +    case Instruction::IntToPtr:
>> +      // inttoptr(x1) != inttoptr(x2) iff x1 != x2
> 
> Nick, one way to write an 'is 64-bit' ConstantExpr would be to as  
> (icmp ne (inttoptr i64 0x100000000), (inttoptr i64 0x0)). Likewise,  
> ptrtoint can also perform a target-dependent truncation. Are you  
> correct re these behaviors?

Aha! No, I'm not. I wasn't aware that this sort of behaviour was 
well-defined.

I've reverted the patch.

Nick