[PATCH] D17573: [X86] PR26554: Enable using of true long nops for x86-64 for every CPU

Bruno Cardoso Lopes via llvm-commits llvm-commits at lists.llvm.org
Wed May 11 09:08:17 PDT 2016


bruno added a comment.

Hi Andrey,


================
Comment at: lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp:353
@@ -345,3 +352,3 @@
   // Select the right NOP table.
   // FIXME: Can we get if CPU supports long nops from the subtarget somehow?
   const uint8_t (*Nops)[10] = HasNopl ? TrueNops : AltNops;
----------------
Any reason why this shouldn't be done as a target feature as suggested in the FIXME above?


http://reviews.llvm.org/D17573





More information about the llvm-commits mailing list