[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