[PATCH] D104717: [llvm-objdump] Add --no-print-imm-hex to the command guide

[James Henderson via Phabricator via llvm-commits]

jhenderson accepted this revision.
jhenderson added a comment.
This revision is now accepted and ready to land.

In D104717#2833469 <https://reviews.llvm.org/D104717#2833469>, @MaskRay wrote:

> I am on the fence. For certain boolean options, I think it is usually sufficient to document just the non-default option (e.g. `--foo`), no need for the default option (e.g. `--no-foo`).

I don't think it is necessarily obvious which options can be negated. Certainly, for some options in some tools, there is no --no-* equivalent.

LGTM.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104717/new/

https://reviews.llvm.org/D104717