diff options
author | Simon Marchi <simon.marchi@ericsson.com> | 2017-11-22 15:51:44 -0500 |
---|---|---|
committer | Simon Marchi <simon.marchi@ericsson.com> | 2017-11-22 15:51:44 -0500 |
commit | 41bd68f52c05f5654bed49f312f6562c8d048897 (patch) | |
tree | 28168b31a3fbc4f23f10286ce3e5f9c94c17f658 /gdb/testsuite/gdb.opt/inline-locals.exp | |
parent | 7e2fd2f47b37185505f68e45c5cf016c7a16e1ce (diff) | |
download | fsf-binutils-gdb-41bd68f52c05f5654bed49f312f6562c8d048897.zip fsf-binutils-gdb-41bd68f52c05f5654bed49f312f6562c8d048897.tar.gz fsf-binutils-gdb-41bd68f52c05f5654bed49f312f6562c8d048897.tar.bz2 |
Show optimized out local variables in "info locals"
Currently, optimized out variables are not shown when doing "info
locals". Some users found that confusing, thinking GDB forgot to print
their variable. This patch adds them to the "info locals" output. I
added a test in gdb.dwarf2 to test for that behavior. I think doing a
synthetic DWARF test is the easiest way to have an optimized out local
variable for sure.
However, this change reveals what I think is a bug in GDB, see:
http://lists.dwarfstd.org/pipermail/dwarf-discuss-dwarfstd.org/2017-September/004394.html
This patch marks the tests in inline-locals.exp that start failing as
KFAIL. I'd like to tackle this bug eventually, but I don't have the
time right now. I think it's still better to show an extra erroneous
entry than to not show the optimized out variables at all. I haven't
created a bug in bugzilla yet, but if we agree it's indeed a bug, I'll
create one and update the setup_kfail lines with the actual bug number
before pushing.
gdb/ChangeLog:
* stack.c (iterate_over_block_locals): Add LOC_OPTIMIZED_OUT
case in switch.
gdb/testsuite/ChangeLog:
* gdb.opt/inline-locals.exp: Mark tests as KFAIL.
* gdb.dwarf2/info-locals-optimized-out.exp: New file.
* gdb.dwarf2/info-locals-optimized-out.c: New file.
Diffstat (limited to 'gdb/testsuite/gdb.opt/inline-locals.exp')
-rw-r--r-- | gdb/testsuite/gdb.opt/inline-locals.exp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/gdb/testsuite/gdb.opt/inline-locals.exp b/gdb/testsuite/gdb.opt/inline-locals.exp index 76463a6..7245df7 100644 --- a/gdb/testsuite/gdb.opt/inline-locals.exp +++ b/gdb/testsuite/gdb.opt/inline-locals.exp @@ -43,6 +43,7 @@ if { ! $no_frames } { "backtrace from bar 2" gdb_test "up" "#1 .*func1 .* at .*" "up from bar 2" gdb_test "info frame" ".*inlined into frame.*" "func1 inlined 2" + setup_kfail "gdb/xyz" *-*-* gdb_test "info locals" "array = {.*}" "info locals above bar 2" set msg "info args above bar 2" @@ -82,6 +83,7 @@ if { ! $no_frames } { "backtrace from bar 3" gdb_test "up" "#1 .*func1 .* at .*" "up from bar 3" gdb_test "info frame" ".*inlined into frame.*" "func1 inlined 3" + setup_kfail "gdb/xyz" *-*-* gdb_test "info locals" "array = {.*}" "info locals above bar 3" set msg "info args above bar 3" |