[lld] r331000 - [ELF] Clarify help wording for --symbol-ordering-file

Shoaib Meenai via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 26 15:25:04 PDT 2018


Author: smeenai
Date: Thu Apr 26 15:25:04 2018
New Revision: 331000

URL: http://llvm.org/viewvc/llvm-project?rev=331000&view=rev
Log:
[ELF] Clarify help wording for --symbol-ordering-file

The input file for this option should contain a list of symbols, not a
list of sections, so explicitly refer to ordering symbols (but keep the
reference to laying out sections, since that's how the option must
operate). Referring to the file itself as the "symbol ordering file" is
consistent with --warn-symbol-ordering and less ambiguous than "symbol
file" (albeit slightly redundant).

Differential Revision: https://reviews.llvm.org/D46099

Modified:
    lld/trunk/ELF/Options.td

Modified: lld/trunk/ELF/Options.td
URL: http://llvm.org/viewvc/llvm-project/lld/trunk/ELF/Options.td?rev=331000&r1=330999&r2=331000&view=diff
==============================================================================
--- lld/trunk/ELF/Options.td (original)
+++ lld/trunk/ELF/Options.td Thu Apr 26 15:25:04 2018
@@ -291,7 +291,7 @@ def strip_all: F<"strip-all">, HelpText<
 def strip_debug: F<"strip-debug">, HelpText<"Strip debugging information">;
 
 defm symbol_ordering_file: Eq<"symbol-ordering-file">,
-  HelpText<"Layout sections in the order specified by symbol file">;
+  HelpText<"Layout sections to place symbols in the order specified by symbol ordering file">;
 
 defm sysroot: Eq<"sysroot">, HelpText<"Set the system root">;
 




More information about the llvm-commits mailing list