Fix LowerSDIV24

Matt Arsenault arsenm2 at
Wed Jul 23 23:17:56 PDT 2014

On Jul 10, 2014, at 6:28 AM, Jan Vesely <jan.vesely at> 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.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: add_piglit_sdiv24_tests.diff
Type: application/octet-stream
Size: 96484 bytes
Desc: not available
URL: <>

More information about the llvm-commits mailing list