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

[Nemanja Ivanovic via Phabricator via llvm-commits]

nemanjai added inline comments.


================
Comment at: lib/Target/PowerPC/PPCTargetMachine.cpp:371
   if (TM->getTargetTriple().getArch() == Triple::ppc64le &&
+      !TM->getSubtargetImpl()->isISA3_0() &&
       !DisableVSXSwapRemoval)
----------------
inouehrs wrote:
> I feel it is better to use `Subtarget.needsSwapsForVSXMemOps()` to make the meaning of the check clearer.
This is actually a really good point. I forgot that we added that function. Please use that instead.


https://reviews.llvm.org/D34627