[PATCH] D159224: [llvm-objdump] Add flags to force enable/disable color output

[Jonas Devlieghere via Phabricator via llvm-commits]

JDevlieghere added a comment.

In D159224#4630031 <https://reviews.llvm.org/D159224#4630031>, @MaskRay wrote:

> GNU objdump from 2.39 onwards provides `--disassembler-color=on`. I wonder whether we want similar option names, but I can see that `--disassembler-color=on` is long and inconvenient..

I expect this flag to be mostly used for writing tests so I'm not too worried about being verbose. FWIW I ended up going for the two separate flags because no other `objdump` options take a predefined set of options (there's a few ones that take arbitrary strings such as `--debug-vars=` or `--arch-name=`) but if we feel that something like `--disassembler-color={on, off, auto}` is the right thing to do, then I'm happy to go that route.


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

https://reviews.llvm.org/D159224