[PATCH] D34627: [Power9] Disable removing extra swaps on P9 since it should not be needed.

Hiroshi Inoue via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 29 06:38:49 PDT 2017


inouehrs added inline comments.


================
Comment at: lib/Target/PowerPC/PPCTargetMachine.cpp:371
   if (TM->getTargetTriple().getArch() == Triple::ppc64le &&
+      !TM->getSubtargetImpl()->isISA3_0() &&
       !DisableVSXSwapRemoval)
----------------
I feel it is better to use `Subtarget.needsSwapsForVSXMemOps()` to make the meaning of the check clearer.


https://reviews.llvm.org/D34627





More information about the llvm-commits mailing list