[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