PR20038: DebugInfo: Call sites without DebugLocs for temporary dtors after a conditional
authorDavid Blaikie <dblaikie@gmail.com>
Wed, 25 Jun 2014 17:57:34 +0000 (17:57 +0000)
committerDavid Blaikie <dblaikie@gmail.com>
Wed, 25 Jun 2014 17:57:34 +0000 (17:57 +0000)
commit21cd7cfd0166f292684a2748c6498a935a585cf2
tree6105c7b6d58b6acc2024501aa723e123d090eaca
parent4b07b007864ccd665e373b7e1512f28382873f54
PR20038: DebugInfo: Call sites without DebugLocs for temporary dtors after a conditional

With && at the top level of an expression, the last thing done when
emitting the expression was an unconditional jump to the cleanup block.
To reduce the amount of stepping, the DebugLoc is omitted from the
unconditional jump. This is done by clearing the IRBuilder's
"CurrentDebugLocation"*. If this is not set to some non-empty value
before the cleanup block is emitted, the cleanups don't get a location
either. If a call without a location is emitted in a function with debug
info, and that call is then inlined - bad things happen. (without a
location for the call site, the inliner would just leave the inlined
DebugLocs as they were - pointing to roots in the original function, not
inlined into the current function)

Follow up commit to LLVM will ensure that breaking the invariants of the
DebugLoc chains by having chains that don't lead to the current function
will fail assertions, so we shouldn't accidentally slip any of these
cases in anymore. Those assertions may reveal further cases that need to
be fixed in clang, though I've tried to test heavily to avoid that.

* See r128471, r128513 for the code that clears the
  CurrentDebugLocation. Simply removing this code or moving the code
  into IRBuilder to apply to all unconditional branches would regress
  desired behavior, unfortunately.

llvm-svn: 211722
clang/lib/CodeGen/CGExprScalar.cpp
clang/test/CodeGenCXX/PR20038.cpp [new file with mode: 0644]