[PATCH] D59618: [llvm-objdump] Support llvm-objdump -sj.foo

[Jordan Rupprecht via Phabricator via llvm-commits]

rupprecht added a comment.

In D59618#1437591 <https://reviews.llvm.org/D59618#1437591>, @jhenderson wrote:

> LGTM. I imagine that there might be more options like this, both in llvm-objdump and the other tools?


The only other alias in objdump that doesn't map to opt<bool> is `--dissassembler-options` (or `-M`), which is also a list<string>, so that //should// be fine to also include, although it's also comma delimited (`-j` is not), so I'll have to double check that it still works fine.

Ack - I'll also look to see if other tools need this, in a separate patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D59618