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

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 12 08:48:44 PDT 2023


nemanjai accepted this revision.
nemanjai added a comment.
This revision is now accepted and ready to land.

LGTM. Whatever the choice is wrt. 32-bit mode, please just add a comment to the function explaining it.



================
Comment at: llvm/lib/Object/XCOFFObjectFile.cpp:1223
+    return StringRef("future");
+  return std::nullopt;
+}
----------------
Why do we restrict this to 64-bit? Just because we do the same for ELF?


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