On Apr 25, 2007, at 7:28 AM, Anton Korobeynikov wrote:
> --- llvm/docs/LangRef.html:1.237      Sun Apr 22 09:56:37 2007
> +++ llvm/docs/LangRef.html    Wed Apr 25 09:27:10 2007
> @@ -498,7 +498,8 @@
>  <p>It is illegal for a function <i>declaration</i>
>  to have any linkage type other than "externally visible",  
> <tt>dllimport</tt>,
>  or <tt>extern_weak</tt>.</p>
> -
> +<p>Aliases can have only <tt>external</tt>, <tt>internal</tt> and  
> <tt>weak</tt>
> +linkages.
>  </div>

What do these mean?  What semantics does the linker have?  Why isn't  
it always valid to RAUW an internal alias, then delete it?

> +<!--  
> ====================================================================== 
> = -->
> +<div class="doc_subsection">
> +  <a name="aliasstructure">Aliases</a>
> +</div>

Should this section go in the TOC at the start of the file?

> +<div class="doc_text">
> +  <p>Aliases act as "second name" for the aliasee value (which can  
> be either
> +  function or global variable). Aliases may have an
> +  optional <a href="#linkage">linkage type</a>, and an
> +  optional <a href="#visibility">visibility style</a>.</p>

I suggest mentioning that aliases are only supported on some  
targets.  Is there any way to emulate them on targets that don't  
support them?

Also, can you please add CBE support for them?

-Chris


_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Reply via email to