[llvm] [ICP] Add a few tunings to indirect-call-promotion (PR #149892)

Mingming Liu via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 23 10:10:16 PDT 2025


================
@@ -63,8 +63,8 @@ extern cl::opt<bool> EnableVTableProfileUse;
 
 // Command line option to disable indirect-call promotion with the default as
 // false. This is for debug purpose.
-static cl::opt<bool> DisableICP("disable-icp", cl::init(false), cl::Hidden,
-                                cl::desc("Disable indirect call promotion"));
+cl::opt<bool> DisableICP("disable-icp", cl::init(false), cl::Hidden,
----------------
mingmingl-llvm wrote:

nit: afaikt it's more conventional to have `static` keyword for an `cl::opt` when the option is only used in one cpp file. It seems this option isn't used in other files in this PR, and I think we should re-add the `static` keyword.

https://github.com/llvm/llvm-project/pull/149892


More information about the llvm-commits mailing list