[LLVMdev] Possible bug in ExpandShiftWithUnknownAmountBit

[Javier Martinez]

Hi Duncan,

Just wondering if you had a chance to look at the expansion.

Thanks,
Javier

On Tue, 01 Dec 2009 20:20:14 +0100, Duncan Sands <baldrick at free.fr> wrote:
> Hi Javier,
> 
>> It seems that the code you pasted came from the function
>> ExpandShiftByConstant and indeed it looks correct. In my example I used
6
>> as the shift amount but forgot to mention that it's stored in a
register.
> 
> I see, sorry I didn't read your email more carefully.  It does look like
> ExpandShiftWithUnknownAmountBit is bogus - at a glance it looks like the
> Cmp condition is inverted.  I may have time to take a closer look
tomorrow.
> 
> Best wishes,
> 
> Duncan.