- Currently, the default instructions will use Clang's frontend PGO feature.
Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/docs/TestSuiteGuide.md | ||
---|---|---|
355 | I would suggest either making TEST_SUITE_USE_IR_PGO=On default, or at minimum setting it to On in this example (e.g. line 340) and then flipping the comment on line 354. |
LGTM, but I'd wait to see if anyone has a strong dependency on the flag being the other way around.
llvm/docs/TestSuiteGuide.md | ||
---|---|---|
355 | +1, just change the default in llvm-test-suite and remove -DTEST_SUITE_USE_IR_PGO=ON from the docs |
llvm/docs/TestSuiteGuide.md | ||
---|---|---|
355 | Thanks for the input! Given that front-end instrumentation (i.e., fprofile-instr-generate) is mentioned in user-manual (https://clang.llvm.org/docs/UsersManual.html#differences-between-sampling-and-instrumentation) and option-handling, I'm leaning towards keeping the default in case people rely on it in the daily testing workflow. |
llvm/docs/TestSuiteGuide.md | ||
---|---|---|
355 | I thought we were trying to move away from -fprofile-instr-generate for any sort of PGO and only use it for code coverage, but that requires documentation updating |
I would suggest either making TEST_SUITE_USE_IR_PGO=On default, or at minimum setting it to On in this example (e.g. line 340) and then flipping the comment on line 354.