[PATCH] D59618: [llvm-objdump] Support llvm-objdump -sj.foo
Jordan Rupprecht via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 21 11:24:12 PDT 2019
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
More information about the llvm-commits
mailing list