[PATCH] D100158: [SantizerCoverage] test case demonstrating PR39531

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 9 14:14:29 PDT 2021


nickdesaulniers planned changes to this revision.
nickdesaulniers added a comment.

> So basically retrieve the caller's DISubprogram, if it exists, create a line 0 DILocation with the DISubprogram as the scope.

Cool, yeah, that's what this pass already did; it just did not forsee that `EntryLoc = IP->getDebugLoc();` could return an invalid `DebugLoc` (ie. because `!dbg` metadata was missing from the insertion point).  Let me rephrase the description of this bug, then I think it's ready for review.


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