]> granicus.if.org Git - llvm/commit
PR20038: DebugInfo: Inlined call sites where the caller has debug info but the call...
authorDavid Blaikie <dblaikie@gmail.com>
Wed, 25 Jun 2014 18:03:10 +0000 (18:03 +0000)
committerDavid Blaikie <dblaikie@gmail.com>
Wed, 25 Jun 2014 18:03:10 +0000 (18:03 +0000)
commitc39f36df61fd69fc856c92ce8e6700b6b5357c20
treec6e87c7fac100d5f224fee9d4cab650df5342192
parentd5a8fa72bb41eedbfde7e8124414123c49a2351c
PR20038: DebugInfo: Inlined call sites where the caller has debug info but the call itself has no debug location.

This situation does bad things when inlined, so I've fixed Clang not to
produce inlinable call sites without locations when the caller has debug
info (in the one case where I could find that this occurred). This
updates the PR20038 test case to be what clang now produces, and readds
the assertion that had to be removed due to this bug.

I've also beefed up the debug info verifier to help diagnose these
issues in the future, and I hope to add checks to the inliner to just
assert-fail if it encounters this situation. If, in the future, we
decide we have to cope with this situation, the right thing to do is
probably to just remove all the DebugLocs from the inlined instructions.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@211723 91177308-0d34-0410-b5e6-96231b3b80d8
lib/CodeGen/AsmPrinter/DwarfDebug.cpp
lib/CodeGen/LexicalScopes.cpp
lib/IR/DebugInfo.cpp
test/DebugInfo/PR20038.ll