[PATCH] D75300: Support prefix padding for alignment purposes (Relaxable instructions only)

Kan Shengchen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 14 09:39:00 PDT 2020


skan added inline comments.


================
Comment at: llvm/lib/Target/X86/MCTargetDesc/X86AsmBackend.cpp:110
+    "x86-pad-max-prefix-size", cl::init(0),
+    cl::desc("Maximum number of redundant prefixes to use for padding"));
+
----------------
Before landing this patch, could you make this description better? I don't think the word "redundant" is appropriate here since `X86PadMaxPrefixSize` counts  in the existing prefixes of the instructions,which may be useful, e.g. escape prefix.


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

https://reviews.llvm.org/D75300





More information about the llvm-commits mailing list