<div><div dir="auto">I like all of your suggestions.  Applied in SVN r323839.</div><br><div class="gmail_quote"><div>On Tue, Jan 30, 2018 at 12:49 PM Rafael Avila de Espindola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Saleem Abdulrasool via llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>><br>
writes:<br>
<br>
> +``.linker-options`` Section (linker options)<br>
> +^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^<br>
> +<br>
> +In order to support passing linker options from the frontend to the linker, a<br>
> +special section of type ``SHT_LLVM_LINKER_OPTIONS`` (usually named<br>
> +``.linker-options`` though the name is not significant as it is identified by<br>
> +the type).  The contents of this section is a simple pair-wise encoding of<br>
> +options for consideration by the linker.<br>
<br>
s/options/directives/ just to make it clear that they are *not* command<br>
line options.<br>
<br>
<br>
>  The strings are encoded as standard<br>
> +null-terminated UTF-8 strings.  They are emitted inline to avoid having the<br>
> +linker to traverse the object file for retrieving the value.<br>
<br>
s/to traverse/traverse/<br>
<br>
> The linker is<br>
> +permitted to not honour the option and instead provide a warning/error to the<br>
> +user that the requested option was not honoured.<br>
> +<br>
> +The section is marked as ``SHT_LLVM_LINKER_OPTIONS``<br>
<br>
s/is marked as/has type/<br>
<br>
> +LLVM emits the following options:<br>
> +  - lib<br>
> +<br>
> +    The parameter identifies a library to be linked against.  The library will<br>
> +    be looked up in the default and any specified library search paths<br>
> +    (specified to this point).<br>
> +<br>
> +  - path<br>
> +<br>
> +    The paramter identifies an additional library search path to be considered<br>
> +    when looking up libraries after the inclusion of this option.<br>
<br>
What is meant my "LLVM emits"? LLVM itself prints any options it sees in<br>
the metadata.<br>
<br>
Maybe just say "The following options are specified:"?<br>
<br>
Thanks,<br>
Rafael<br>
</blockquote></div></div><div dir="ltr">-- <br></div><div class="gmail_signature" data-smartmail="gmail_signature">Saleem Abdulrasool<br>compnerd (at) compnerd (dot) org</div>