[PATCH] D57904: [llvm-objdump] Allow short options to be grouped
Matthew Voss via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Feb 13 13:32:28 PST 2019
ormris marked an inline comment as done.
ormris added inline comments.
Comment at: llvm/tools/llvm-objdump/llvm-objdump.cpp:110
> ormris wrote:
> > rupprecht wrote:
> > > This conflicts with -d and -f when grouped args are allowed
> > Is renaming this option possible? It doesn't seem to fit with the naming scheme...
> Yep, it should be fine to rename it to "disassemble-functions" or something equally reasonable. You'll have to update test/tools/llvm-objdump/X86/disasm-specific-funcs.test.
Sounds good. I'll add that to the patch.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the llvm-commits