[PATCH] D42616: [X86] Emit 11-byte or 15-byte NOPs on recent AMD targets, else default to 10-byte NOPs (PR22965)

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 28 23:48:46 PST 2018


craig.topper added inline comments.


================
Comment at: lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp:357
 
-  // 15 is the longest single nop instruction.  Emit as many 15-byte nops as
-  // needed, then emit a nop of the remaining length.
+  // Emit as many MaxNopLengthnops as needed, then emit a nop of the remaining
+  // length.
----------------
Minor. Missing space between MaxNopLength and nops


Repository:
  rL LLVM

https://reviews.llvm.org/D42616





More information about the llvm-commits mailing list