[all-commits] [llvm/llvm-project] 5dd9f4: [LLD] Improve --strip-all help text

bd1976bris via All-commits all-commits at lists.llvm.org
Thu May 6 04:34:32 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5dd9f44c17ec0d8b6b88bb015560b3c566622fdc
      https://github.com/llvm/llvm-project/commit/5dd9f44c17ec0d8b6b88bb015560b3c566622fdc
  Author: Ben Dunbobbin <Ben.Dunbobbin at sony.com>
  Date:   2021-05-06 (Thu, 06 May 2021)

  Changed paths:
    M lld/ELF/Options.td
    M lld/docs/ld.lld.1

  Log Message:
  -----------
  [LLD] Improve --strip-all help text

This is a slight improvement to the help text, as I was slightly
surprised when strip-all did more than remove the symbol table.

Currently, we match gold's help text for strip-all and strip-debug.
I think that the GNU documentation for these options is not particularly
clear. However, I have opted to make only a minor change here and keep
the help text similar to gold's as these are mature options that are
well understood.

ld.bfd (https://sourceware.org/binutils/docs/ld/Options.html) has a
similar implication although it defines strip-debug as a subset of
strip-all. However, felt that noting that strip-all implies strip-debug
is better; because, with the ld.bfd approach you have to read both the
--strip-debug and the --strip-all help text to understand the behaviour
of --strip-all (and the --strip-all help text doesn't indicate that he
--strip-debug help text is related).

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




More information about the All-commits mailing list