[PATCH] D65529: [PowerPC] Use xxleqv to set all one vector IMM(-1).

[Zixuan Wu (Zeson) via Phabricator via llvm-commits]

wuzish added inline comments.


================
Comment at: llvm/lib/Target/PowerPC/PPCInstrVSX.td:1298
+  
+  def : Pat<(v4i32 (bitconvert (v16i8 immAllOnesV))),
+            (XXLEQVOnes)>;
----------------
Has immAllOnesV contained bitcast semantic? It's equal to ISD::isBuildVectorAllOnes.


================
Comment at: llvm/lib/Target/PowerPC/PPCInstrVSX.td:4079
               (v16i8 (COPY_TO_REGCLASS (XXSPLTIB imm:$A), VSRC))>;
-    def : Pat<(v16i8 immAllOnesV),
-              (v16i8 (COPY_TO_REGCLASS (XXSPLTIB 255), VSRC))>;
----------------
If we don't enum all the type, just use v4i32 version. Are other types bitcast/canonical into v4i32 automatically?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D65529