[llvm-commits] [llvm] r64066 - /llvm/trunk/docs/LangRef.html

[Bill Wendling]

On Feb 7, 2009, at 6:37 PM, Chris Lattner wrote:

> On Feb 7, 2009, at 5:40 PM, Bill Wendling wrote:
>
>> Author: void
>> Date: Sat Feb  7 19:40:31 2009
>> New Revision: 64066
>>
>> URL: http://llvm.org/viewvc/llvm-project?rev=64066&view=rev
>> Log:
>> Document the 'llvm.OP.with.overflow' intrinsics.
>
> Thanks Bill!
>
>> +      <li><a href="#int_overflow">Arithmetic with Overflow
>> Intrinsics</a>
>> +        <ol>
>> +          <li><a
>> href="#int_sadd_ovf">'<tt>llvm.sadd.with.overflow.*</tt> Intrinsics</
>> a></li>
>> +          <li><a
>> href="#int_uadd_ovf">'<tt>llvm.uadd.with.overflow.*</tt> Intrinsics</
>> a></li>
>> +          <li><a
>> href="#int_ssub_ovf">'<tt>llvm.ssub.with.overflow.*</tt> Intrinsics</
>> a></li>
>> +          <li><a
>> href="#int_usub_ovf">'<tt>llvm.usub.with.overflow.*</tt> Intrinsics</
>> a></li>
>> +          <li><a
>> href="#int_smul_ovf">'<tt>llvm.smul.with.overflow.*</tt> Intrinsics</
>> a></li>
>
> We also support umul.  The int_overflow anchor is also missing/dead.
> Please add the section header.
>
umul is badly broken. It probably should be removed so that no one can  
use it. I didn't want to document it because of that.


>> +<!--
>> _______________________________________________________________________
>> -->
>> +<div class="doc_subsubsection">
>> +  <a name="int_sadd_ovf">'<tt>llvm.sadd.with.overflow.*</tt>'
>> Intrinsics</a>
>
> Please use the full anchor name, there are links *into* langref from
> other documents, so we want something stable and predictable.  All the
> instructions are i_* all intrinsics are int_* with the name spelled  
> out.
>
Okay.

>>
>> +</div>
>> +
>> +<div class="doc_text">
>> +
>> +<h5>Syntax:</h5>
>> +
>> +<p>This is an overloaded intrinsic. You can use
>> <tt>llvm.sadd.with.overflow</tt>
>> +on any integer bit width. However, not all targets support all bit
>> widths.</p
>
> Is this true?  I thought that legalize handled any bitwidth?
>
removed.

-bw