[Review]: Fix Integer Division Expansion special case of divide by 1

Aditya Nandakumar aditya_nandakumar at apple.com
Mon Jul 6 16:26:48 PDT 2015


I don’t have a test case before and after for in-tree target but it fixed test for out of tree target.
The logic is what I am pointing out is wrong - we need to check if CTLZ(divisor) == Num_bits - 1 rather than CTLZ(divisor) - CTLZ(dividend).
> On Jul 6, 2015, at 2:29 PM, Michael Ilseman <milseman at apple.com> wrote:
> 
> Do you have a test case that demonstrates the failure before-hand, and the fix after? 
> 
>> On Jul 2, 2015, at 12:26 PM, Aditya Nandakumar <aditya_nandakumar at apple.com> wrote:
>> 
>> <DiffIntUnsignedDivision.patch>
> 





More information about the llvm-commits mailing list