From 3433cfa51f6397231ffe2b2c69298eff89179769 Mon Sep 17 00:00:00 2001 From: Siva Chandra Date: Thu, 16 Oct 2014 07:14:13 -0700 Subject: [PATCH] Guard a call to TYPE_TARGET_TYPE in gnuv3_pass_by_reference. gdb/ChangeLog: * gnu-v3-abi.c (gnuv3_pass_by_reference): Call TYPE_TARGET_TYPE on the arg type of a constructor only if it is of reference type. gdb/testsuite/ChangeLog: * gdb.cp/non-trivial-retval.cc: Add a test case. * gdb.cp/non-trivial-retval.exp: Add a test. --- gdb/ChangeLog | 5 +++++ gdb/gnu-v3-abi.c | 12 ++++++----- gdb/testsuite/ChangeLog | 5 +++++ gdb/testsuite/gdb.cp/non-trivial-retval.cc | 33 +++++++++++++++++++++++++++++ gdb/testsuite/gdb.cp/non-trivial-retval.exp | 1 + 5 files changed, 51 insertions(+), 5 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 370a7ad..c366ab7 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2014-10-24 Siva Chandra Reddy + + * gnu-v3-abi.c (gnuv3_pass_by_reference): Call TYPE_TARGET_TYPE + on the arg type of a constructor only if it is of reference type. + 2014-10-23 Sandra Loosemore * nios2-tdep.c (nios2_analyze_prologue): Use new instruction field diff --git a/gdb/gnu-v3-abi.c b/gdb/gnu-v3-abi.c index a6c6f9f..b960aa3 100644 --- a/gdb/gnu-v3-abi.c +++ b/gdb/gnu-v3-abi.c @@ -1320,13 +1320,15 @@ gnuv3_pass_by_reference (struct type *type) if (TYPE_NFIELDS (fieldtype) == 2) { struct type *arg_type = TYPE_FIELD_TYPE (fieldtype, 1); - struct type *arg_target_type; - arg_target_type = check_typedef (TYPE_TARGET_TYPE (arg_type)); + if (TYPE_CODE (arg_type) == TYPE_CODE_REF) + { + struct type *arg_target_type; - if (TYPE_CODE (arg_type) == TYPE_CODE_REF - && class_types_same_p (arg_target_type, type)) - return 1; + arg_target_type = check_typedef (TYPE_TARGET_TYPE (arg_type)); + if (class_types_same_p (arg_target_type, type)) + return 1; + } } } diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 3b7bb46..b6e3505 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2014-10-24 Siva Chandra Reddy + + * gdb.cp/non-trivial-retval.cc: Add a test case. + * gdb.cp/non-trivial-retval.exp: Add a test. + 2014-10-20 Yao Qi * gdb.python/py-objfile-script-gdb.py.in: Rename it to ... diff --git a/gdb/testsuite/gdb.cp/non-trivial-retval.cc b/gdb/testsuite/gdb.cp/non-trivial-retval.cc index 8382f40..fd1b695 100644 --- a/gdb/testsuite/gdb.cp/non-trivial-retval.cc +++ b/gdb/testsuite/gdb.cp/non-trivial-retval.cc @@ -63,6 +63,39 @@ f2 (int i1, int i2) return b; } +class B1 +{ +public: + B1 () {} + /* This class exists to test that GDB does not trip on other + constructors (not copy constructors) which take one + argument. Hence, put this decl before the copy-ctor decl. + If it is put after copy-ctor decl, then the decision to mark + this class as non-trivial will already be made and GDB will + not look at this constructor. */ + B1 (int i); + B1 (const B1 &obj); + + int b1; +}; + +B1::B1 (const B1 &obj) +{ + b1 = obj.b1; +} + +B1::B1 (int i) : b1 (i) { } + +B1 +f22 (int i1, int i2) +{ + B1 b1; + + b1.b1 = i1 + i2; + + return b1; +} + class C { public: diff --git a/gdb/testsuite/gdb.cp/non-trivial-retval.exp b/gdb/testsuite/gdb.cp/non-trivial-retval.exp index 7934946..3450a94 100644 --- a/gdb/testsuite/gdb.cp/non-trivial-retval.exp +++ b/gdb/testsuite/gdb.cp/non-trivial-retval.exp @@ -32,5 +32,6 @@ gdb_continue_to_breakpoint "Break here" gdb_test "p f1 (i1, i2)" ".* = {a = 123}" "p f1 (i1, i2)" gdb_test "p f2 (i1, i2)" ".* = {b = 123}" "p f2 (i1, i2)" +gdb_test "p f22 (i1, i2)" ".* = {b1 = 123}" "p f22 (i1, i2)" gdb_test "p f3 (i1, i2)" ".* = {.* c = 123}" "p f3 (i1, i2)" gdb_test "p f4 (i1, i2)" ".* = {.* e = 123}" "p f4 (i1, i2)" -- 2.7.4