[PATCH] D91813: [PGO] verify BFI counts after loading profile data
Rong Xu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 20 12:20:16 PST 2020
xur added inline comments.
================
Comment at: llvm/lib/Transforms/Instrumentation/PGOInstrumentation.cpp:262
+ "pgo-verify-bfi", cl::init(false), cl::Hidden,
+ cl::desc("Print out non-match BFI counts after setting profile metadata."));
+
----------------
davidxl wrote:
> Add a description here that -Rpass-analysis=PGO is needed for messages.
Sure.
================
Comment at: llvm/lib/Transforms/Instrumentation/PGOInstrumentation.cpp:1704
+ ORE.emit([&]() {
+ return OptimizationRemark(DEBUG_TYPE, "bfi-verify", F.getSubprogram(),
+ &F.getEntryBlock())
----------------
davidxl wrote:
> Remark or RemarkAnalysis?
I thought you suggested Remark. I'm OK either way.
I will change to RemarkAnalysis.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91813/new/
https://reviews.llvm.org/D91813
More information about the llvm-commits
mailing list