<div dir="ltr">These warnings do not seem useful for all users of --symbol-ordering-file. I am aware of at least one user that uses --symbol-ordering-file as an opportunistic request for sections to appear in a particular order, rather than a mandatory request. This user has --fatal-warnings enabled, and would not want the build to fail if an older --symbol-ordering-file is being used and a symbol disappears from the final output due to a refactoring.<div><br></div><div>I imagine that one option would be to put these warnings behind a default-on flag.</div><div><br></div><div>Peter<br><div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Jan 22, 2018 at 4:28 PM, Rui Ueyama via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><span>On Mon, Jan 22, 2018 at 2:00 AM, James Henderson via llvm-dev <span dir="ltr"><<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">

<p class="MsoNormal">Hi,</p>

<p class="MsoNormal"> </p>

<p class="MsoNormal">In our proprietary linker, we have a switch which has
essentially identical functionality to the --symbol-ordering-file LLD switch.
One of the main areas that it differs in is that our linker warns in various
circumstances, if there is a problem with the contents of the order file. We
would like to propose adding these same warnings to LLD, if there is interest
in them. Following are the situations we warn in, and some of the justifications
behind them:</p>

<p class="MsoNormal"> </p>

<p class="MsoNormal">When a symbol in the order file is not found - might help
catch typos in that file.</p>

<p class="MsoNormal">Imported/undefined symbols - symbols in other modules
clearly can’t be reordered in the current link.</p>

<p class="MsoNormal">Absolute symbols - these do not have a position in the file,
so ordering them makes no sense.</p>

<p class="MsoNormal">Removed symbols (i.e. via --gc-sections) - it doesn’t make
much sense to try to order such symbols, since they have no position in the
output.</p></div></blockquote><div><br></div></span><div>All sound good to me, and these features will be pretty useful.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span><div dir="ltr"><p class="MsoNormal"> </p>

<p class="MsoNormal">Would people be interested in these warnings?</p>

<p class="MsoNormal"> </p>

<p class="MsoNormal">Regards,</p><span class="gmail-m_-5159861585938570557m_5089985175372236578m_-11763317578997189m_3741609790516036190HOEnZb"><font color="#888888">

<p class="MsoNormal"> </p>

<p class="MsoNormal">James</p>

</font></span></div>
<br></span>______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
<br></blockquote></div><br></div></div>
<br>______________________________<wbr>_________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/<wbr>mailman/listinfo/llvm-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail-m_-5159861585938570557m_5089985175372236578gmail_signature"><div dir="ltr">-- <div>Peter</div></div></div>
</div></div></div></div>