[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