[llvm] a5e093e - [llvm-objcopy] Remove stray periods from usage text

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 14 11:31:47 PDT 2023


Author: Thomas Köppe
Date: 2023-07-14T11:31:43-07:00
New Revision: a5e093eda4e144e94a08584b80ce8fe25bef759e

URL: https://github.com/llvm/llvm-project/commit/a5e093eda4e144e94a08584b80ce8fe25bef759e
DIFF: https://github.com/llvm/llvm-project/commit/a5e093eda4e144e94a08584b80ce8fe25bef759e.diff

LOG: [llvm-objcopy] Remove stray periods from usage text

The convention for TableGen help messages is to omit the period.

Reviewed By: MaskRay

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

Added: 
    

Modified: 
    llvm/tools/llvm-objcopy/ObjcopyOpts.td

Removed: 
    


################################################################################
diff  --git a/llvm/tools/llvm-objcopy/ObjcopyOpts.td b/llvm/tools/llvm-objcopy/ObjcopyOpts.td
index 0fddd443a4cc78..a7e4263271d356 100644
--- a/llvm/tools/llvm-objcopy/ObjcopyOpts.td
+++ b/llvm/tools/llvm-objcopy/ObjcopyOpts.td
@@ -27,7 +27,7 @@ def O : JoinedOrSeparate<["-"], "O">,
 defm new_symbol_visibility : Eq<"new-symbol-visibility", "Visibility of "
                                 "symbols generated for binary input or added"
                                 " with --add-symbol unless otherwise"
-                                " specified. The default value is 'default'.">;
+                                " specified. The default value is 'default'">;
 
 def compress_debug_sections
     : Joined<["--"], "compress-debug-sections=">,
@@ -37,7 +37,7 @@ def compress_debug_sections
 def : Flag<["--"], "compress-debug-sections">, Alias<compress_debug_sections>,
       AliasArgs<["zlib"]>;
 def decompress_debug_sections : Flag<["--"], "decompress-debug-sections">,
-                                HelpText<"Decompress DWARF debug sections.">;
+                                HelpText<"Decompress DWARF debug sections">;
 defm split_dwo
     : Eq<"split-dwo", "Equivalent to extract-dwo on the input file to "
                       "<dwo-file>, then strip-dwo on the input file">,
@@ -52,7 +52,7 @@ defm rename_section
          "Renames a section from old to new, optionally with specified flags. "
          "Flags supported for GNU compatibility: alloc, load, noload, "
          "readonly, exclude, debug, code, data, rom, share, contents, merge, "
-         "strings.">,
+         "strings">,
       MetaVarName<"old=new[,flag1,...]">;
 defm redefine_symbol
     : Eq<"redefine-sym", "Change the name of a symbol old to new">,
@@ -64,7 +64,7 @@ defm redefine_symbols
          "contains two symbols per line separated with whitespace and may "
          "contain comments beginning with '#'. Leading and trailing "
          "whitespace is stripped from each line. May be repeated to read "
-         "symbols from many files.">,         
+         "symbols from many files">,         
       MetaVarName<"filename">;
 
 defm only_section : Eq<"only-section", "Remove all but <section>">,
@@ -74,18 +74,18 @@ def j : JoinedOrSeparate<["-"], "j">,
         HelpText<"Alias for --only-section">;
 defm add_section
     : Eq<"add-section",
-         "Make a section named <section> with the contents of <file>.">,
+         "Make a section named <section> with the contents of <file>">,
       MetaVarName<"section=file">;
 
 defm set_section_alignment
-    : Eq<"set-section-alignment", "Set alignment for a given section.">,
+    : Eq<"set-section-alignment", "Set alignment for a given section">,
       MetaVarName<"section=align">;
 
 defm set_section_flags
     : Eq<"set-section-flags",
          "Set section flags for a given section. Flags supported for GNU "
          "compatibility: alloc, load, noload, readonly, exclude, debug, code, "
-         "data, rom, share, contents, merge, strings.">,
+         "data, rom, share, contents, merge, strings">,
       MetaVarName<"section=flag1[,flag2,...]">;
 
 defm set_section_type
@@ -136,7 +136,7 @@ defm localize_symbol : Eq<"localize-symbol", "Mark <symbol> as local">,
                        MetaVarName<"symbol">;
 defm localize_symbols
     : Eq<"localize-symbols",
-         "Reads a list of symbols from <filename> and marks them local.">,
+         "Reads a list of symbols from <filename> and marks them local">,
       MetaVarName<"filename">;
 
 def L : JoinedOrSeparate<["-"], "L">,
@@ -148,13 +148,13 @@ defm globalize_symbol : Eq<"globalize-symbol", "Mark <symbol> as global">,
 
 defm globalize_symbols
     : Eq<"globalize-symbols",
-         "Reads a list of symbols from <filename> and marks them global.">,
+         "Reads a list of symbols from <filename> and marks them global">,
       MetaVarName<"filename">;
 
 defm keep_global_symbol
     : Eq<"keep-global-symbol",
          "Convert all symbols except <symbol> to local. May be repeated to "
-         "convert all except a set of symbols to local.">,
+         "convert all except a set of symbols to local">,
       MetaVarName<"symbol">;
 def G : JoinedOrSeparate<["-"], "G">,
         Alias<keep_global_symbol>,
@@ -166,14 +166,14 @@ defm keep_global_symbols
          "--keep-global-symbol=<symbol> is set for each one. <filename> "
          "contains one symbol per line and may contain comments beginning with "
          "'#'. Leading and trailing whitespace is stripped from each line. May "
-         "be repeated to read symbols from many files.">,
+         "be repeated to read symbols from many files">,
       MetaVarName<"filename">;
 
 defm weaken_symbol : Eq<"weaken-symbol", "Mark <symbol> as weak">,
                      MetaVarName<"symbol">;
 defm weaken_symbols
     : Eq<"weaken-symbols",
-         "Reads a list of symbols from <filename> and marks them weak.">,
+         "Reads a list of symbols from <filename> and marks them weak">,
       MetaVarName<"filename">;
 
 def W : JoinedOrSeparate<["-"], "W">,
@@ -184,7 +184,7 @@ def weaken : Flag<["--"], "weaken">,
 
 defm strip_symbols
     : Eq<"strip-symbols",
-         "Reads a list of symbols from <filename> and removes them.">,
+         "Reads a list of symbols from <filename> and removes them">,
       MetaVarName<"filename">;
 
 defm keep_symbols
@@ -193,7 +193,7 @@ defm keep_symbols
          "--keep-symbol=<symbol> is set for each one. <filename> "
          "contains one symbol per line and may contain comments beginning with "
          "'#'. Leading and trailing whitespace is stripped from each line. May "
-         "be repeated to read symbols from many files.">,
+         "be repeated to read symbols from many files">,
       MetaVarName<"filename">;
 
 defm dump_section
@@ -209,11 +209,11 @@ defm prefix_alloc_sections
       MetaVarName<"prefix">;
 
 defm set_start : Eq<"set-start", "Set the start address to <addr>. Overrides "
-                    "any previous --change-start or --adjust-start values.">,
+                    "any previous --change-start or --adjust-start values">,
                  MetaVarName<"addr">;
 defm change_start : Eq<"change-start", "Add <incr> to the start address. Can be "                        
                        "specified multiple times, all values will be applied "
-                       "cumulatively.">,
+                       "cumulatively">,
                     MetaVarName<"incr">;
 def adjust_start : JoinedOrSeparate<["--"], "adjust-start">,
                    Alias<change_start>,
@@ -224,9 +224,9 @@ defm add_symbol
          "global, local, weak, default, hidden, protected, file, section, object, "
          "function, indirect-function. Accepted but ignored for "
          "compatibility: debug, constructor, warning, indirect, synthetic, "
-         "unique-object, before.">,
+         "unique-object, before">,
       MetaVarName<"name=[section:]value[,flags]">;
 
 defm update_section
-    : Eq<"update-section", "Replace the contents of section <name> with contents from a file <file>.">,
+    : Eq<"update-section", "Replace the contents of section <name> with contents from a file <file>">,
       MetaVarName<"name=file">;


        


More information about the llvm-commits mailing list