This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH 3/4] aix: add support for dwarf2.

From: Tom Tromey <tromey at redhat dot com>
To: Tristan Gingold <gingold at adacore dot com>
Cc: gdb-patches at sourceware dot org
Date: Mon, 23 May 2011 09:42:03 -0600
Subject: Re: [PATCH 3/4] aix: add support for dwarf2.
References: <1306146544-3925-1-git-send-email-gingold@adacore.com> <1306146544-3925-4-git-send-email-gingold@adacore.com> <m3aaed4eib.fsf@fleche.redhat.com>

Tom> I don't like this approach, because it requires some non-obvious code
Tom> synchronization between this struct and dwarf2read.

No, I'm pretty wrong here.  Reordering the struct would indeed cause
problems, because all the fields have the same type -- but this is also
not a very likely transformation.  Adding or removing a field should be
ok, which is what matters.  I don't think this is such a big deal after
all.

Tom

Follow-Ups:
- Re: [PATCH 3/4] aix: add support for dwarf2.
  - From: Tristan Gingold

References:
- [RFA] Add dwarf2 support to xcoff (aix)
  - From: Tristan Gingold
- [PATCH 3/4] aix: add support for dwarf2.
  - From: Tristan Gingold
- Re: [PATCH 3/4] aix: add support for dwarf2.
  - From: Tom Tromey

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]