[PATCH] D100158: [SantizerCoverage] handle missing DBG MD when inserting libcalls
David Blaikie via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 12 13:55:39 PDT 2021
dblaikie added inline comments.
================
Comment at: llvm/test/Instrumentation/SanitizerCoverage/crit-edge-sancov.ll:24
+; CHECK-NEXT: call void @__sanitizer_cov_trace_pc() #[[ATTR0]], !dbg [[DBG7:![0-9]+]]
+; CHECK: [[DBG6]] = !DILocation(line: {{.*}}, scope: !3)
+entry:
----------------
nickdesaulniers wrote:
> dblaikie wrote:
> > Probably good to check this is actually line 0, since that's what the code does intentionally - it'd be weird/problematic if we invented/used some other location.
> I think it will be `scopeLine` of parent plus `0`.
hmm, wait, that doesn't sound right - sorry, didn't realize that's what the code is doing. The code should be setting the line to 0, not the scopeLine. (line 0 is for lines that aren't related to any particular/identifiable line of source code)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100158/new/
https://reviews.llvm.org/D100158
More information about the llvm-commits
mailing list