Fix LowerSDIV24

Matt Arsenault arsenm2 at gmail.com
Thu Jul 24 00:10:34 PDT 2014


On Jul 23, 2014, at 11:17 PM, Matt Arsenault <arsenm2 at gmail.com> wrote:

> 
> On Jul 10, 2014, at 6:28 AM, Jan Vesely <jan.vesely at rutgers.edu> wrote:
>>> 
>> 
>> I think we need to move i24 handling to SDIVREM since i32 and i64 algs
>> for udivrem produce rem values as a side product and don't do X - Y *
>> (X / Y).
>> 
>> All llvm tests and piglits 'scalar-arithmetic-(u)int' still pass for me
>> with the attached patches applied on top of yours.
> 
> 
> 
> These look fine to me. I’ll commit my part. Here are also some more tests for sdiv24 and srem24 someone can apply to piglit.
> 
> <add_piglit_sdiv24_tests.diff>

r213843-r213844



More information about the llvm-commits mailing list