[PATCH] D124490: [InstrProf] Minimal Block Coverage

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 4 12:50:34 PDT 2022


MaskRay added inline comments.


================
Comment at: llvm/lib/Transforms/Instrumentation/PGOInstrumentation.cpp:265
+static cl::opt<bool> PGOBlockCoverage(
+    "pgo-block-coverage", cl::init(false), cl::Hidden, cl::ZeroOrMore,
+    cl::desc(
----------------
MaskRay wrote:
> Delete `, cl::init(false)`
Also delete `cl::ZeroOrMore`. I have changed `cl::opt` that the option can be specified multiple times without an error.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124490/new/

https://reviews.llvm.org/D124490



More information about the llvm-commits mailing list