[PATCH] D157632: [Profile] Allow online merging with debug info correlation.

Arthur Eubanks via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 21 17:06:44 PDT 2023


aeubanks added inline comments.


================
Comment at: compiler-rt/lib/profile/InstrProfilingMerge.c:146
+    if (!(__llvm_profile_get_version() & VARIANT_MASK_DBG_CORRELATE)) {
+      PROF_ERR("%s\n", "Missing profile data section.");
+      return 1;
----------------
aeubanks wrote:
> we're running into this error message even though we don't have debug info correlation turned on, is that expected in some configurations? I'm still trying to understand how we could end up in this code path without debug info correlation
> 
> https://crbug.com/1473935
also it seems like this code path isn't tested?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157632



More information about the cfe-commits mailing list