[PATCH] D65529: [PowerPC] Use xxleqv to set all one vector IMM(-1).
Zixuan Wu (Zeson) via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 31 22:24:23 PDT 2019
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
More information about the llvm-commits
mailing list