This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH v3 PR gdb/16841] virtual inheritance via typedef cannot find base
On 9/18/2018 4:59 AM, Tom Tromey wrote:
">" == Weimin Pan <weimin.pan@oracle.com> writes:
+2018-09-12 Weimin Pan <weimin.pan@oracle.com>
+
+ PR gdb/16841
This should read "PR c++/16841". The robot doesn't care but I suppose
it's a bit clearer for humans.
Done.
+ PR gdb/16841
Here too.
Done.
+ gdb_test "print ${scope}d" " = 6.5999999999999996"
It's best with floats to use a number that is exact.
So, instead of 6.6, 6.25 or something like that.
Using 6.6 will result in an inexact number as well. Will change it to
"6.25".
The code has a loop to deal with multiple inheritance but the test
doesn't exercise this case. I think an additional for that would be
good.
Sorry, which case is not exercised? Could it be printing superbase class
member in base?
+ if (get_baseclass_offset (domain, curtype, v, &boff,
+ &isvirt))
+ mem_offset += boff;
+ else
+ {
+ struct type *t = check_typedef (value_type (this_v));
+ t = check_typedef (TYPE_TARGET_TYPE (t));
+ if (get_baseclass_offset (t, curtype, this_v,
+ &boff, &isvirt))
+ mem_offset += boff;
Could you explain this "else" block? I did not understand the reason
for it. I suppose it could use a comment.
There is a brief comment:
/* Find class offset of type CURTYPE from either its
parent type DOMAIN or the type of implied
this. */
i.e. trying to find the class offset for CURTYPE first from its parent
type DOMAIN.
If failed, try to find it from the type of the implied this.
Thanks for your comments.
Weimin
thanks,
Tom