[llvm-commits] [llvm] r126637 - /llvm/trunk/lib/Analysis/ValueTracking.cpp

[Duncan Sands]

On 28/02/11 10:20, Nick Lewycky wrote:
> Duncan Sands wrote:
>> Hi Nick, -10 srem -10 is zero, so has a sign bit of zero regardless of the
>> fact that -10 has a sign bit of one.
>
> Hrm, the LangRef states that the srem instruction returns a result that "has the
> same sign as the dividend, op1". That's not confusing at all.
>
> I'm backing out r126637 and the relevant part of r126635.

Thanks Nick!  How about correcting the LangRef too, otherwise this is sure to
happen again.

Ciao, Duncan.