[PATCH] D94816: [P10] [Power PC] Exploiting new load rightmost vector element instructions.

Nemanja Ivanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 9 07:39:25 PST 2021


nemanjai added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCInstrVSX.td:3972
+// Any big endian Power9 VSX subtarget
+let Predicates = [HasVSX, HasP9Vector, IsBigEndian] in {
+// Power10 VSX subtargets produce a shorter pattern for little endian targets
----------------
The patterns appear to be the same. Why do we split this out into LE/BE if the patterns are endianness neutral? If they are the same, merge them into a single block that doesn't have an endianness predicate.


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

https://reviews.llvm.org/D94816



More information about the llvm-commits mailing list