[llvm] r364289 - [PowerPC] Emit XXSEL for vec_sel and code that has the same pattern

[Nemanja Ivanovic via llvm-commits]

Author: nemanjai
Date: Tue Jun 25 03:46:13 2019
New Revision: 364289

URL: http://llvm.org/viewvc/llvm-project?rev=364289&view=rev
Log:
[PowerPC] Emit XXSEL for vec_sel and code that has the same pattern

As pointed out in https://bugs.llvm.org/show_bug.cgi?id=41777
we do not emit a vector select even when the pretty much asks for one.
This patch changes that.

Differential revision: https://reviews.llvm.org/D61658

Added:
    llvm/trunk/test/CodeGen/PowerPC/vec-select.ll
Modified:
    llvm/trunk/lib/Target/PowerPC/PPCInstrVSX.td

Modified: llvm/trunk/lib/Target/PowerPC/PPCInstrVSX.td
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/PowerPC/PPCInstrVSX.td?rev=364289&r1=364288&r2=364289&view=diff
==============================================================================
--- llvm/trunk/lib/Target/PowerPC/PPCInstrVSX.td (original)
+++ llvm/trunk/lib/Target/PowerPC/PPCInstrVSX.td Tue Jun 25 03:46:13 2019
@@ -971,6 +971,10 @@ let AddedComplexity = 400 in { // Prefer
 
 def : Pat<(v4i32 (vnot_ppc v4i32:$A)),
           (v4i32 (XXLNOR $A, $A))>;
+def : Pat<(v4i32 (or (and (vnot_ppc v4i32:$C), v4i32:$A),
+                     (and v4i32:$B, v4i32:$C))),
+          (v4i32 (XXSEL $A, $B, $C))>;
+
 let Predicates = [IsBigEndian] in {
 def : Pat<(v2f64 (scalar_to_vector f64:$A)),
           (v2f64 (SUBREG_TO_REG (i64 1), $A, sub_64))>;

Added: llvm/trunk/test/CodeGen/PowerPC/vec-select.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/CodeGen/PowerPC/vec-select.ll?rev=364289&view=auto
==============================================================================
--- llvm/trunk/test/CodeGen/PowerPC/vec-select.ll (added)
+++ llvm/trunk/test/CodeGen/PowerPC/vec-select.ll Tue Jun 25 03:46:13 2019
@@ -0,0 +1,72 @@
+; NOTE: Assertions have been autogenerated by utils/update_llc_test_checks.py
+; RUN: llc -verify-machineinstrs -mtriple=powerpc64le-unknown-linux-gnu \
+; RUN:     -mcpu=pwr8 -ppc-asm-full-reg-names < %s | FileCheck %s
+define dso_local <4 x i32> @test(<4 x i32> %a, <4 x i32> %b, <4 x i32> %c) {
+; CHECK-LABEL: test:
+; CHECK:       # %bb.0: # %entry
+; CHECK-NEXT:    xxsel vs34, vs34, vs35, vs36
+; CHECK-NEXT:    blr
+entry:
+  %neg.i = xor <4 x i32> %c, <i32 -1, i32 -1, i32 -1, i32 -1>
+  %and.i = and <4 x i32> %neg.i, %a
+  %and1.i = and <4 x i32> %c, %b
+  %or.i = or <4 x i32> %and1.i, %and.i
+  ret <4 x i32> %or.i
+}
+
+define dso_local <8 x i16> @test2(<8 x i16> %a, <8 x i16> %b, <8 x i16> %c) {
+; CHECK-LABEL: test2:
+; CHECK:       # %bb.0: # %entry
+; CHECK-NEXT:    xxsel vs34, vs34, vs35, vs36
+; CHECK-NEXT:    blr
+entry:
+  %neg.i = xor <8 x i16> %c, <i16 -1, i16 -1, i16 -1, i16 -1, i16 -1, i16 -1, i16 -1, i16 -1>
+  %and.i = and <8 x i16> %a, %neg.i
+  %and1.i = and <8 x i16> %c, %b
+  %or.i = or <8 x i16> %and.i, %and1.i
+  ret <8 x i16> %or.i
+}
+
+define dso_local <16 x i8> @test3(<16 x i8> %a, <16 x i8> %b, <16 x i8> %c) {
+; CHECK-LABEL: test3:
+; CHECK:       # %bb.0: # %entry
+; CHECK-NEXT:    xxsel vs34, vs34, vs35, vs36
+; CHECK-NEXT:    blr
+entry:
+  %neg.i = xor <16 x i8> %c, <i8 -1, i8 -1, i8 -1, i8 -1, i8 -1, i8 -1, i8 -1, i8 -1, i8 -1, i8 -1, i8 -1, i8 -1, i8 -1, i8 -1, i8 -1, i8 -1>
+  %and.i = and <16 x i8> %neg.i, %a
+  %and1.i = and <16 x i8> %c, %b
+  %or.i = or <16 x i8> %and.i, %and1.i
+  ret <16 x i8> %or.i
+}
+
+define dso_local <2 x i64> @test4(<2 x i64> %a, <2 x i64> %b, <2 x i64> %c) {
+; CHECK-LABEL: test4:
+; CHECK:       # %bb.0: # %entry
+; CHECK-NEXT:    xxsel vs34, vs34, vs35, vs36
+; CHECK-NEXT:    blr
+entry:
+  %neg.i = xor <2 x i64> %c, <i64 -1, i64 -1>
+  %and.i = and <2 x i64> %a, %neg.i
+  %and1.i = and <2 x i64> %c, %b
+  %or.i = or <2 x i64> %and.i, %and1.i
+  ret <2 x i64> %or.i
+}
+
+; Not valid to emit XXSEL for this illegal type.
+define dso_local <4 x i1> @test5(<4 x i1> %a, <4 x i1> %b, <4 x i1> %c) {
+; CHECK-LABEL: test5:
+; CHECK:       # %bb.0: # %entry
+; CHECK-NEXT:    vspltisw v5, 1
+; CHECK-NEXT:    xxland vs0, vs36, vs35
+; CHECK-NEXT:    xxlxor vs1, vs36, vs37
+; CHECK-NEXT:    xxland vs1, vs34, vs1
+; CHECK-NEXT:    xxlor vs34, vs1, vs0
+; CHECK-NEXT:    blr
+entry:
+  %neg.i = xor <4 x i1> %c, <i1 -1, i1 -1, i1 -1, i1 -1>
+  %and.i = and <4 x i1> %a, %neg.i
+  %and1.i = and <4 x i1> %c, %b
+  %or.i = or <4 x i1> %and.i, %and1.i
+  ret <4 x i1> %or.i
+}