diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 46f6141d22..c5c962a11c 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2014-03-28 Joel Brobecker + + * varobj.c (varobj_value_has_mutated): If NEW_VALUE is + a reference, strip the reference layer before calling + the lang_ops value_has_mutated callback. + 2014-03-27 Sergio Durigan Junior Remove some globals from our parser. diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 062d014a71..092cde8ab3 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2014-03-28 Joel Brobecker + + * gdb.ada/mi_dyn_arr: New testcase. + 2014-03-27 Doug Evans * gdb.dwarf2/dw2-abs-hi-pc.exp: Build tests with "nodebug". diff --git a/gdb/testsuite/gdb.ada/mi_dyn_arr.exp b/gdb/testsuite/gdb.ada/mi_dyn_arr.exp new file mode 100644 index 0000000000..434c21dcfa --- /dev/null +++ b/gdb/testsuite/gdb.ada/mi_dyn_arr.exp @@ -0,0 +1,52 @@ +# Copyright 2014 Free Software Foundation, Inc. +# +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see . + +load_lib "ada.exp" + +standard_ada_testfile foo + +if {[gdb_compile_ada "${srcfile}" "${binfile}" executable [list debug]] != "" } { + return -1 +} + +load_lib mi-support.exp +set MIFLAGS "-i=mi" + +gdb_exit +if [mi_gdb_start] { + continue +} + +mi_delete_breakpoints +mi_gdb_reinitialize_dir $srcdir/$subdir +mi_gdb_load ${binfile} + +if ![mi_run_to_main] then { + fail "Cannot run to main, testcase aborted" + return 0 +} + +set bp_location [gdb_get_line_number "STOP" ${testdir}/foo.adb] +mi_continue_to_line \ + "foo.adb:$bp_location" \ + "stop at start of main Ada procedure" + +mi_gdb_test "-var-create bt * bt" \ + "\\^done,name=\"bt\",numchild=\"3\",.*" \ + "Create bt varobj" + +mi_gdb_test "-var-update 1 *" \ + "\\^done,changelist=\\\[\\\]" \ + "list ggg1's children" diff --git a/gdb/testsuite/gdb.ada/mi_dyn_arr/foo.adb b/gdb/testsuite/gdb.ada/mi_dyn_arr/foo.adb new file mode 100644 index 0000000000..4a9f3581c5 --- /dev/null +++ b/gdb/testsuite/gdb.ada/mi_dyn_arr/foo.adb @@ -0,0 +1,24 @@ +-- Copyright 2014 Free Software Foundation, Inc. +-- +-- This program is free software; you can redistribute it and/or modify +-- it under the terms of the GNU General Public License as published by +-- the Free Software Foundation; either version 3 of the License, or +-- (at your option) any later version. +-- +-- This program is distributed in the hope that it will be useful, +-- but WITHOUT ANY WARRANTY; without even the implied warranty of +-- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +-- GNU General Public License for more details. +-- +-- You should have received a copy of the GNU General Public License +-- along with this program. If not, see . + +with Pck; use Pck; +procedure Foo is + -- The goal here is to have an array whose bounds are not + -- known at compile time. + BT : Bounded := New_Bounded (Low => 1, High => 3); +begin + Do_Nothing (BT'Address); -- STOP +end Foo; + diff --git a/gdb/testsuite/gdb.ada/mi_dyn_arr/pck.adb b/gdb/testsuite/gdb.ada/mi_dyn_arr/pck.adb new file mode 100644 index 0000000000..c77c31ed18 --- /dev/null +++ b/gdb/testsuite/gdb.ada/mi_dyn_arr/pck.adb @@ -0,0 +1,30 @@ +-- Copyright 2014 Free Software Foundation, Inc. +-- +-- This program is free software; you can redistribute it and/or modify +-- it under the terms of the GNU General Public License as published by +-- the Free Software Foundation; either version 3 of the License, or +-- (at your option) any later version. +-- +-- This program is distributed in the hope that it will be useful, +-- but WITHOUT ANY WARRANTY; without even the implied warranty of +-- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +-- GNU General Public License for more details. +-- +-- You should have received a copy of the GNU General Public License +-- along with this program. If not, see . + +package body Pck is + function New_Bounded (Low, High : Integer) return Bounded is + Result : Bounded (Low .. High); + begin + for J in Low .. High loop + Result (J) := J; + end loop; + return Result; + end New_Bounded; + + procedure Do_Nothing (A : System.Address) is + begin + null; + end Do_Nothing; +end Pck; diff --git a/gdb/testsuite/gdb.ada/mi_dyn_arr/pck.ads b/gdb/testsuite/gdb.ada/mi_dyn_arr/pck.ads new file mode 100644 index 0000000000..2519fdd322 --- /dev/null +++ b/gdb/testsuite/gdb.ada/mi_dyn_arr/pck.ads @@ -0,0 +1,21 @@ +-- Copyright 2014 Free Software Foundation, Inc. +-- +-- This program is free software; you can redistribute it and/or modify +-- it under the terms of the GNU General Public License as published by +-- the Free Software Foundation; either version 3 of the License, or +-- (at your option) any later version. +-- +-- This program is distributed in the hope that it will be useful, +-- but WITHOUT ANY WARRANTY; without even the implied warranty of +-- MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +-- GNU General Public License for more details. +-- +-- You should have received a copy of the GNU General Public License +-- along with this program. If not, see . + +with System; +package Pck is + type Bounded is array (Integer range <>) of Integer; + function New_Bounded (Low, High : Integer) return Bounded; + procedure Do_Nothing (A : System.Address); +end Pck; diff --git a/gdb/varobj.c b/gdb/varobj.c index bbe42137a1..10ef5b7406 100644 --- a/gdb/varobj.c +++ b/gdb/varobj.c @@ -1649,7 +1649,16 @@ varobj_value_has_mutated (struct varobj *var, struct value *new_value, return 0; if (var->root->lang_ops->value_has_mutated) - return var->root->lang_ops->value_has_mutated (var, new_value, new_type); + { + /* The varobj module, when installing new values, explicitly strips + references, saying that we're not interested in those addresses. + But detection of mutation happens before installing the new + value, so our value may be a reference that we need to strip + in order to remain consistent. */ + if (new_value != NULL) + new_value = coerce_ref (new_value); + return var->root->lang_ops->value_has_mutated (var, new_value, new_type); + } else return 0; }