[PATCH] D155114: [llvm-objdump] Default to --mcpu=future for PPC32

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 12 13:41:09 PDT 2023


MaskRay added inline comments.


================
Comment at: llvm/docs/CommandGuide/llvm-objdump.rst:31
   Disassemble all executable sections found in the input files. On some
-  architectures (AArch64, PPC64, x86), all known instructions are disassembled by
+  architectures (AArch64, Power, x86), all known instructions are disassembled by
   default. On the others, :option:`--mcpu` or :option:`--mattr` is needed to
----------------
nemanjai wrote:
> Maybe `PowerPC` instead of `Power`.
Thanks for the super quick review. I was thinking of https://en.wikipedia.org/wiki/Power_ISA Power is canonical while PowerPC is the old name.
In software projects, it's true that "PowerPC" and "powerpc" are still much more common ...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155114



More information about the llvm-commits mailing list