<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Nov 12, 2015 at 4:26 PM, Rui Ueyama via llvm-commits <span dir="ltr"><<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: ruiu<br>
Date: Thu Nov 12 18:26:12 2015<br>
New Revision: 252978<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=252978&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=252978&view=rev</a><br>
Log:<br>
ELF2: Add comments and reorganize option list a bit.<br>
<br>
Modified:<br>
    lld/trunk/ELF/Options.td<br>
<br>
Modified: lld/trunk/ELF/Options.td<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Options.td?rev=252978&r1=252977&r2=252978&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Options.td?rev=252978&r1=252977&r2=252978&view=diff</a><br>
==============================================================================<br>
--- lld/trunk/ELF/Options.td (original)<br>
+++ lld/trunk/ELF/Options.td Thu Nov 12 18:26:12 2015<br>
@@ -133,17 +133,21 @@ def alias_script_T : Separate<["-"], "T"<br>
 def alias_strip_all: Flag<["-"], "s">, Alias<strip_all>;<br>
 def alias_undefined_u : Separate<["-"], "u">, Alias<undefined>;<br>
<br>
-// Options listed below are silently ignored now.<br>
-def build_id : Flag<["--"], "build-id">;<br>
-def eh_frame_hdr : Flag<["--"], "eh-frame-hdr">;<br>
+// Our symbol resolution algorithm handles symbols in archive files better<br>
+// than traditional linkers, so we don't need --start-group and --end-group.<br></blockquote><div><br></div><div>Can we just say "differently"?</div><div><br></div><div>-- Sean Silva</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
+// These options are recongized but ignored for compatibility.<br>
 def end_group : Flag<["--"], "end-group">;<br>
 def end_group_paren: Flag<["-"], ")">;<br>
+def start_group : Flag<["--"], "start-group">;<br>
+def start_group_paren: Flag<["-"], "(">;<br>
+<br>
+// Options listed below are silently ignored for now for compatibility.<br>
+def build_id : Flag<["--"], "build-id">;<br>
+def eh_frame_hdr : Flag<["--"], "eh-frame-hdr">;<br>
 def fatal_warnings : Flag<["--"], "fatal-warnings">;<br>
 def no_add_needed : Flag<["--"], "no-add-needed">;<br>
 def no_fatal_warnings : Flag<["--"], "no-fatal-warnings">;<br>
 def no_warn_mismatch : Flag<["--"], "no-warn-mismatch">;<br>
-def start_group : Flag<["--"], "start-group">;<br>
-def start_group_paren: Flag<["-"], "(">;<br>
 def version_script : Separate<["--"], "version-script">;<br>
 def warn_common : Flag<["--"], "warn-common">;<br>
 def warn_shared_textrel : Flag<["--"], "warn-shared-textrel">;<br>
<br>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits</a><br>
</blockquote></div><br></div></div>