[llvm] [ctx_prof] Add Inlining support (PR #106154)

Mingming Liu via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 29 21:10:46 PDT 2024


================
@@ -74,6 +74,11 @@ class PGOCtxProfContext final {
     Iter->second.emplace(Other.guid(), std::move(Other));
   }
 
+  void ingestAllContexts(uint32_t CSId, CallTargetMapTy &&Other) {
+    auto [_, Inserted] = callsites().try_emplace(CSId, std::move(Other));
+    assert(Inserted);
----------------
minglotus-6 wrote:

nit: put some error messages in the assert statement here and elsewhere. See https://llvm.org/docs/CodingStandards.html#assert-liberally

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


More information about the llvm-commits mailing list