[llvm-commits] [llvm] r121905 - in /llvm/trunk: lib/Transforms/InstCombine/InstCombineCompares.cpp test/Transforms/InstCombine/overflow.ll

[Owen Anderson]

On Dec 15, 2010, at 3:22 PM, Benjamin Kramer wrote:

> Is llvm.sadd.with.overflow implemented on all targets? http://llvm.org/bugs/show_bug.cgi?id=4917#c5 suggests that there are problems with mul+overflow, do we legalize add+overflow intrinsics for targets that don't support them?
> 

LegalizeDAG.cpp contains a generic lowering for ISD::SADDO, which is the DAG node that sadd.with.overflow lowers to, so it ought to work on all targets.

--Owen
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20101215/e3c97a34/attachment.html>