[PATCH] D120975: [BOLT] Strip redundant AdSize override prefix
Rafael Auler via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 16 05:38:29 PDT 2022
rafauler accepted this revision.
rafauler added a comment.
This revision is now accepted and ready to land.
LGTM (nit below)
================
Comment at: bolt/lib/Target/X86/X86MCPlusBuilder.cpp:39-40
+
+static cl::opt<bool> X86StripRedundantAdSize(
+ "x86-strip-redundant-adsize",
+ cl::desc("Remove redundant Address-Size override prefix"), cl::init(true),
----------------
Can we call this (...)AddressSize and -address-size? I find adsize to be a bit confusing.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D120975/new/
https://reviews.llvm.org/D120975
More information about the llvm-commits
mailing list