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)
commit442584588abd0549308bc675a51807cdf4d65744
tree1f2de97a1fda52e4baa996a7ad3a8164cebf48a1
parent21cd7cfd0166f292684a2748c6498a935a585cf2
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.

llvm-svn: 211723
llvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
llvm/lib/CodeGen/LexicalScopes.cpp
llvm/lib/IR/DebugInfo.cpp
llvm/test/DebugInfo/PR20038.ll