[PATCH] D80020: [PowerPC] Add support for -mcpu=pwr10 in both clang and llvm

Victor Huang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 25 14:29:00 PDT 2020


NeHuang marked an inline comment as done.
NeHuang added inline comments.


================
Comment at: clang/lib/Basic/Targets/PPC.cpp:339
 
   if (!(ArchDefs & ArchDefinePwr9) && (ArchDefs & ArchDefinePpcgr) &&
       llvm::find(FeaturesVec, "+float128") != FeaturesVec.end()) {
----------------
lei wrote:
> NeHuang wrote:
> > I think we also need to check for `ArchDefinePwr10` and `ArchDefineFuture` based on the comment  "// We have __float128 on PPC but not power 9 and above."
> > 
> > `!(ArchDefs & ArchDefinePwr9)` -> `!(ArchDefs & (ArchDefinePwr9 | ArchDefinePwr10 | ArchDefineFuture))`
> I would think this would be redundant since `-mcpu=pwr10` also defines `ArchDefinePwr9` ...
Yeah. That makes sense. Thanks for the explanation!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80020





More information about the cfe-commits mailing list