[clang] [llvm] [InstrPGO] Support cold function coverage instrumentation (PR #109837)

[Lei Wang via cfe-commits]

================
@@ -1182,8 +1187,13 @@ 
PassBuilder::buildModuleSimplificationPipeline(OptimizationLevel Level,
   const bool IsCtxProfUse =
       !UseCtxProfile.empty() && Phase == ThinOrFullLTOPhase::ThinLTOPreLink;
 
+  // Enable cold function coverage instrumentation if
+  // InstrumentColdFuncOnlyPath is provided.
+  const bool IsColdFuncOnlyInstrGen = PGOInstrumentColdFunctionOnly =
----------------
wlei-llvm wrote:

Thanks for pointing out. It's intentional to set the global variable, I wasn't 
aware it will cause data race. I will try to change it to local one. 

https://github.com/llvm/llvm-project/pull/109837
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits