Debug info for optimized code: Support variables that are on the stack and
authorAdrian Prantl <aprantl@apple.com>
Thu, 24 Apr 2014 17:41:45 +0000 (17:41 +0000)
committerAdrian Prantl <aprantl@apple.com>
Thu, 24 Apr 2014 17:41:45 +0000 (17:41 +0000)
commitf28f471f1600d3226f6ac96e7d3b8fa27344d11f
tree2364fbb03766ff57d29cc9f4385b591aea3425e3
parent35f9e1aa4990a20245cb7c39eb01f9feffc97853
Debug info for optimized code: Support variables that are on the stack and
described by DBG_VALUEs during their lifetime.

Previously, when a variable was at a FrameIndex for any part of its
lifetime, this would shadow all other DBG_VALUEs and only a single
fbreg location would be emitted, which in fact is only valid for a small
range and not the entire lexical scope of the variable. The included
dbg-value-const-byref testcase demonstrates this.

This patch fixes this by
Local
- emitting dbg.value intrinsics for allocas that are passed by reference
- dropping all dbg.declares (they are now fully lowered to dbg.values)
SelectionDAG
- renamed constructors for SDDbgValue for better readability.
- fix UserValue::match() to handle indirect values correctly
- not inserting an MMI table entries for dbg.values that describe allocas.
- lowering dbg.values that describe allocas into *indirect* DBG_VALUEs.
CodeGenPrepare
- leaving dbg.values for an alloca were they are (see comment)
Other
- regenerated/updated instcombine-intrinsics testcase and included source

rdar://problem/16679879
http://reviews.llvm.org/D3374

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@207130 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm/CodeGen/SelectionDAG.h
lib/CodeGen/CodeGenPrepare.cpp
lib/CodeGen/LiveDebugVariables.cpp
lib/CodeGen/SelectionDAG/InstrEmitter.cpp
lib/CodeGen/SelectionDAG/SDNodeDbgValue.h
lib/CodeGen/SelectionDAG/SelectionDAG.cpp
lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
lib/Transforms/Utils/Local.cpp
test/DebugInfo/X86/dbg-value-const-byref.ll
test/DebugInfo/X86/instcombine-instrinsics.ll