[PATCH] D29370: [X86] Don't base domain decisions on VEXTRACTF128/VINSERTF128 if only AVX1 is available.

Elena Demikhovsky via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 9 00:28:53 PST 2017


delena added inline comments.


================
Comment at: lib/Target/X86/X86InstrInfo.cpp:8969
+
+  switch (opcode) {
+  case X86::VEXTRACTF128mr:
----------------
I don't think that moving instructions mapping from tables to "switch" is a good idea.


https://reviews.llvm.org/D29370





More information about the llvm-commits mailing list