This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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] Kill TYPE_METHODS debug 1/9

From: Nathan Sidwell <nathan at acm dot org>
To: GCC Patches <gcc-patches at gcc dot gnu dot org>
Cc: Jason Merrill <jason at redhat dot com>, Richard Guenther <richard dot guenther at gmail dot com>, Jim Wilson <wilson at tuliptree dot org>, Jan Hubicka <hubicka at ucw dot cz>
Date: Fri, 14 Jul 2017 12:48:55 -0400
Subject: Re: [PATCH] Kill TYPE_METHODS debug 1/9
Authentication-results: sourceware.org; auth=none
References: <8377f2cd-577c-60b2-63ac-f2619ebe63aa@acm.org>

This changes dbxout and dwarf2out.

Rather than iterate over the TYPE_METHODS, they now need to deal with member fnsin the regular TYPE_FIELDS iteration.dbxout was a little weirdly convoluted, apparently presuming that functions withthe same name are all together. That's not true, so other than maybe slightdebug bloat in cases when they happen to be adjacent, it seems more sensible tohandle each member function as a separate item.

The dwarf2out changes are just moving the processing to the TYPE_FIELDS loop,and thereby deleting some duplicate code.


I'd appreciate a review of this patch.

Oh, the patch series survived a bootstrap on x86_64-linux.

nathan

--
Nathan Sidwell

Attachment: method-debug.diff
Description: Text document

Follow-Ups:
- Re: [PATCH] Kill TYPE_METHODS debug 1/9
  - From: Jim Wilson

References:
- [PATCH] Kill TYPE_METHODS 0/9
  - From: Nathan Sidwell

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