[PATCH] D92993: [X86] WIP Swizzle the execution domains for AVX2 targets to prefer integer over floating point

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added a comment.

Are you happy to continue work on this or should someone else commandeer it?



================
Comment at: llvm/lib/Target/X86/X86InstrInfo.cpp:7624
+
+  if (Res.first && Subtarget.hasAVX()) {
+    // Rotate the domain encodings 1-3 so that 3 becomes 1 to make integer
----------------
This probably be set to hasAVX2/hasInt256


================
Comment at: llvm/lib/Target/X86/X86InstrInfo.cpp:7688
+  assert(Domain>0 && Domain<4 && "Invalid execution domain");
+  if (Subtarget.hasAVX()) {
+    // Rotate the domain encoding so that domain 1 becomes 3. So that we prefer
----------------
This probably be set to hasAVX2/hasInt256


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

https://reviews.llvm.org/D92993