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 0/5] RFC: Overhaul of diagnostics (v2)

From: Jeff Law <law at redhat dot com>
To: Michael Matz <matz at suse dot de>, Richard Biener <richard dot guenther at gmail dot com>
Cc: David Malcolm <dmalcolm at redhat dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Wed, 23 Sep 2015 09:08:49 -0600
Subject: Re: [PATCH 0/5] RFC: Overhaul of diagnostics (v2)
Authentication-results: sourceware.org; auth=none
References: <1442957171-22904-1-git-send-email-dmalcolm at redhat dot com> <alpine dot LSU dot 2 dot 20 dot 1509231508490 dot 31674 at wotan dot suse dot de> <CAFiYyc3GNP7TAJzKFMybKdVLiaz_jQ4pO-DxMkCZT_oe1EJ3cQ at mail dot gmail dot com> <alpine dot LSU dot 2 dot 20 dot 1509231545580 dot 31674 at wotan dot suse dot de>

On 09/23/2015 07:47 AM, Michael Matz wrote:

Hi,

On Wed, 23 Sep 2015, Richard Biener wrote:

The issue we have with LTO is that the linemap gets populated in quite
random order and thus we repeatedly switch files (we've mitigated this
somewhat for GCC 5).


Yes.

We also considered dropping column info (and would drop range info) as
diagnostics are from optimizers only with LTO and we keep locations
merely for debug info.


That would be the obvious mitigations, yes.  I do like the fact that we'd
be able to do all this without enlarging location_t.

That's the hope.

However, I did ask David to ponder the effects if ultimately we did needto extend location_t to 64 bits.

Jff

References:
- [PATCH 0/5] RFC: Overhaul of diagnostics (v2)
  - From: David Malcolm
- Re: [PATCH 0/5] RFC: Overhaul of diagnostics (v2)
  - From: Michael Matz
- Re: [PATCH 0/5] RFC: Overhaul of diagnostics (v2)
  - From: Richard Biener
- Re: [PATCH 0/5] RFC: Overhaul of diagnostics (v2)
  - From: Michael Matz

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