[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