[llvm-commits] Adding add.with.overflow intrinsic support to CBackend.

Chris Lattner clattner at apple.com
Mon Jun 20 23:01:26 PDT 2011


On Jun 20, 2011, at 11:32 AM, Anna Zaks wrote:

> Hi,
> 
> Attached is a CBackend patch for review. It adds support for sadd.with.overflow and uadd.with.overflow intrinsics. Essentially, we emit implementation for each add.with.overflow intrinsic that occurs in the module. 
> 
> Issues are lack of support for arbitrary size integers and assumption about the two's compliment representation for integers (casts from signed to unsigned and backwards). However, it seems that these are the limitations of the current CBackend implementation as well.  
> 
> We know that the CBackend is no longer supported and might get completely rewritten, but this patch removes the known blockers for us and might be useful for someone else as well.

Sure, seems reasonable.  Please commit, thanks!

-Chris



More information about the llvm-commits mailing list