[PATCH] D101695: [llvm-objdump] Add -M {att,intel} as user-facing alternatives to --x86-asm-syntax={att,intel}
Hongtao Yu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 3 13:15:53 PDT 2021
hoy added a comment.
Thanks for working on this.
================
Comment at: llvm/docs/CommandGuide/llvm-objdump.rst:153
+ * ``reg-names-raw``: ARM only. Use r followed by the register number.
+ * ``no-aliases``: RISC-V only. Print raw instruction mnemonic instead of pesudo instruction mnemonic.
+ * ``numeric``: RISC-V only. Print raw register names instead of ABI mnemonic. (e.g. print x1 instead of ra)
----------------
Do these RISC-V switches just work? Wondering if testing is needed.
================
Comment at: llvm/tools/llvm-objdump/llvm-objdump.cpp:2480
+ llvm::erase_if(DisassemblerOptions, [&](StringRef V) {
+ if (V == "att") {
+ LLVMArgs.push_back("--x86-asm-syntax=att");
----------------
thakis wrote:
> equals_lower() for better objdump compat?
Can this be made an alias switch of `OBJDUMP_x86_asm_syntax_att`?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D101695/new/
https://reviews.llvm.org/D101695
More information about the llvm-commits
mailing list