[llvm-commits] Patch: simplification of overflow operations

Chris Lattner clattner at apple.com
Sun Nov 8 23:09:03 PST 2009


On Nov 6, 2009, at 11:52 PM, Alastair Lynn wrote:

> Hello-
>
> The attached patch will simplify cases where overflow intrinsics are  
> used but the overflow bit is not to the equivalent basic  
> instructions (llvm.uadd.with.overflow => add, for instance).
>
> This is my first patch, please don't judge me too harshly!

Thanks, applied in r86524 with some minor formatting changes, I also  
added a testcase:
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20091109/090673.html

It would also be nice to simplify the case when just the overflow  
result is used (in cases where the overflow check is a single compare).

Thanks for doing this!

-Chris



More information about the llvm-commits mailing list