[PATCH] D155377: [SanitizerCoverage] Add fallback DebugLocation for instrumented calls
Marco Elver via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jul 17 02:07:31 PDT 2023
melver added inline comments.
================
Comment at: llvm/test/Instrumentation/SanitizerCoverage/missing_dbg.ll:5
+
+define i32 @with_dbg(ptr %a, ptr %b) sanitize_address !dbg !3 {
+entry:
----------------
jkl wrote:
> melver wrote:
> > Is sanitize_address needed for this test?
> oh, no. It's just a leftover from copying the other test. LKM if I should send an update or if you just want to fix it when committing
I assume it's trivial to fix, so just fixing it before committing is fine.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155377/new/
https://reviews.llvm.org/D155377
More information about the llvm-commits
mailing list