[PATCH] D155089: [PowerPC][MC] Set 'future' as default CPU type of AIX

Qiu Chaofan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 12 08:58:20 PDT 2023


qiucf added a subscriber: MaskRay.
qiucf added inline comments.


================
Comment at: llvm/lib/Object/XCOFFObjectFile.cpp:1223
+    return StringRef("future");
+  return std::nullopt;
+}
----------------
nemanjai wrote:
> Why do we restrict this to 64-bit? Just because we do the same for ELF?
Yes. I'm also not sure is there some specific reason why PPC ELF does this only for 64-bit (since added by D127824, maybe @MaskRay ). After I tested things on 32-bit is okay, the 64-bit constraint will be removed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155089



More information about the llvm-commits mailing list