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

[Chris Lattner]

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.


> +<!--  
> _______________________________________________________________________ 
>  -->
> +<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.

>
> +</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?

Otherwise, looks great, thanks Bill!

-Chris