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

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 29 00:36:12 PDT 2017


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

LGTM.



================
Comment at: lib/Target/PowerPC/PPCTargetMachine.cpp:369
   // For little endian, remove where possible the vector swap instructions
   // introduced at code generation to normalize vector element order.
   if (TM->getTargetTriple().getArch() == Triple::ppc64le &&
----------------
Please add a comment such as
```
// Power ISA 3.0 has load/store instructions that preserve
// element order so swaps aren't introduced. The pass to remove
// them is thereby not necessary.
```


https://reviews.llvm.org/D34627





More information about the llvm-commits mailing list