[PATCH] D139553: [llvm-objdump][RISCV] Use new common method to parse ARCH RISCV attribute

Alexander Richardson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 8 00:09:05 PST 2022


arichardson added inline comments.


================
Comment at: llvm/test/tools/llvm-objdump/ELF/RISCV/unknown-arch-attr.test:11
 
+# Fail as unsupported extension is provided
+# XFAIL: *
----------------
IMO objdump should just ignore features it doesn't know about and still decode whatever it knows about. It could still print a warning though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139553



More information about the llvm-commits mailing list