diff options
| author | Adrian Prantl <aprantl@apple.com> | 2014-04-24 17:41:45 +0000 |
|---|---|---|
| committer | Adrian Prantl <aprantl@apple.com> | 2014-04-24 17:41:45 +0000 |
| commit | f4223918de0aa08443ef9910e7237cfecdca60ac (patch) | |
| tree | 5a62fe92a3da5cea1d12823a51ef21ae69147215 /llvm/lib/CodeGen/InlineSpiller.cpp | |
| parent | 6bfbdc55f86c9100a94930bdbb29dff7c8590e8d (diff) | |
| download | bcm5719-llvm-f4223918de0aa08443ef9910e7237cfecdca60ac.tar.gz bcm5719-llvm-f4223918de0aa08443ef9910e7237cfecdca60ac.zip | |
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
llvm-svn: 207130
Diffstat (limited to 'llvm/lib/CodeGen/InlineSpiller.cpp')
0 files changed, 0 insertions, 0 deletions

