[llvm-commits] [llvm] r63481 - in /llvm/trunk: lib/Transforms/Scalar/InstructionCombining.cpp test/Transforms/InstCombine/multi-use-or.ll

[Eli Friedman]

On Sun, Feb 1, 2009 at 4:24 PM, Eli Friedman <eli.friedman at gmail.com> wrote:
> On Sat, Jan 31, 2009 at 12:40 AM, Chris Lattner <sabre at nondot.org> wrote:
>> +      if ((DemandedMask & (~RHSKnownZero) & LHSKnownOne) ==
>> +          (DemandedMask & (~RHSKnownZero)))
>> +        return I->getOperand(0);
>
> Is this doing the right thing with respect to KnownZero/KnownOne?  It
> looks suspicious.

Oops, nevermind... I wasn't looking carefully enough.
KnownZero/KnownOne don't matter if the return is non-null.

-Eli