[PATCH] D78032: Default to the "apple-latest" mcpu when disassembling arm64 Mach-O binaries.

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Apr 13 10:45:14 PDT 2020


MaskRay added a comment.

Can you clarify whether the intention is:

- For MachOObjectFile::getArchTriple <https://github.com/llvm/llvm-project/blob/master/llvm/lib/Object/MachOObjectFile.cpp#L2598>, we want to use a conservative arch flag, because it may be used by both producers and consumers. We don't want to impose an arch feature restriction for binaries created by the producers.
- For llvm-objdump and may be llvm-nm/llvm-size, the read-only consumers, we want to use the latest arch flag so that all features can be decoded correctly.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78032





More information about the llvm-commits mailing list