[PATCH] D37981: AMDGPU: Start selecting s_xnor_{b32, b64}

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 18 10:40:38 PDT 2017


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

LGTM



================
Comment at: lib/Target/AMDGPU/SIInstrInfo.cpp:3822
+  MachineBasicBlock::iterator MII = Inst;
+  DebugLoc DL = Inst.getDebugLoc();
+
----------------
const reference


================
Comment at: test/CodeGen/AMDGPU/xnor.ll:18
+; GCN-LABEL: {{^}}scalar_xnor_i32_mul_use
+; GCN-NOT: s_xnor_b32
+define amdgpu_kernel void @scalar_xnor_i32_mul_use(
----------------
The positive checks for the expected expanded instructions are generally more reliable


https://reviews.llvm.org/D37981





More information about the llvm-commits mailing list