[llvm] [profcheck] Annotate `select` instructions (PR #152171)

[Aiden Grossman via llvm-commits]

================
@@ -26,6 +26,12 @@ using namespace llvm;
 static cl::opt<int64_t>
     DefaultFunctionEntryCount("profcheck-default-function-entry-count",
                               cl::init(1000));
+static cl::opt<bool> AnnotateSelect("profcheck-annotate-select",
+                                    cl::init(true));
+static cl::opt<uint32_t> SelectTrueWeight("profcheck-select-true-weight",
----------------
boomanaiden154 wrote:

I'm not sure we're talking about the same thing?

I'm not asking for anything to be changed in the injector.

I'm wondering why you've made these `cl::opt<uint32_t>`s instead of `constexpr uint32_t`. I don't see the benefit in making these configurable.

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