On Nov 30, 2006, at 9:01 PM, Reid Spencer wrote:

> On Thu, 2006-11-30 at 20:51 -0800, Chris Lattner wrote:
>> On Nov 30, 2006, at 7:56 PM, Reid Spencer wrote:
>>
>>>
>>>    case Instruction::PtrToInt:
>>> -    // Cast of a global address to boolean is always true.
>>> -    if (const GlobalValue *GV = dyn_cast<GlobalValue>(V)) {
>>> -      if (DestTy == Type::BoolTy && !GV->hasExternalWeakLinkage())
>>> -        return ConstantBool::getTrue();
>>> -    }
>>>      break;
>>
>> Thanks Reid, the patch looks great.  I think you can remove this
>> empty case now though,
>
> The default asserts, trying to catch bad opcodes.

Ah, good call, thanks again!

-Chris

_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Reply via email to