[PATCH] D87295: Prefer vpxor over vpxorps for AVX2 PR36127

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/X86/X86InstrInfo.cpp:7456
+  case X86::VXORPSYrr:
+    return SetBlendDomain(14, true);
   case X86::VPANDDZ128rr:  case X86::VPANDDZ128rm:
----------------
I don't understand how this code works. The first thing SetBlendDomain does is check if the last operand is an immediate which should be false for XOR. And what does the 14 mean?


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

https://reviews.llvm.org/D87295