[PATCH] D157913: [Coverage] Allow Clang coverage to be used with debug info correlation.

Zequan Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 15 12:23:59 PDT 2023


zequanwu marked an inline comment as done.
zequanwu added inline comments.


================
Comment at: llvm/lib/ProfileData/InstrProfCorrelator.cpp:42
+const char *InstrProfCorrelator::CovFunctionNameAttributeName =
+    "Cov Function Name";
 
----------------
phosek wrote:
> I missed this earlier, but I think that spelling this out in full, that is `Coverage Function Name`, would be more consistent with the `Coverage Type`.
Renamed at: https://github.com/llvm/llvm-project/commit/37402c34918a4627de77c3b20964207892383242


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157913



More information about the cfe-commits mailing list