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] Fix debug ICE on aarch64 due to bad rtl simplification (PR debug/70628)

From: Bernd Schmidt <bschmidt at redhat dot com>
To: Jakub Jelinek <jakub at redhat dot com>, Jeff Law <law at redhat dot com>, Eric Botcazou <ebotcazou at adacore dot com>
Cc: gcc-patches at gcc dot gnu dot org, Andrew Pinski <pinskia at gmail dot com>, "H.J. Lu" <hjl dot tools at gmail dot com>, Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>
Date: Tue, 12 Apr 2016 22:02:18 +0200
Subject: Re: [PATCH] Fix debug ICE on aarch64 due to bad rtl simplification (PR debug/70628)
Authentication-results: sourceware.org; auth=none
References: <20160412151011 dot GW19207 at tucnak dot redhat dot com>

On 04/12/2016 05:10 PM, Jakub Jelinek wrote:

This patch arranges for a new argument to
convert_memory_address_addr_space_1 and calls it with that new argument set
to true, to make sure it never emits instructions or creates pseudos.

I think the approach looks sensible, but I don't know if you need theextra argument. It looks like convert_memory_address_addr_space_1currently only has one user, can't you move the convert_modes call upinto that function so that convert_memory_address_addr_space_1 neveremits anything?



Bernd

Follow-Ups:
- Re: [PATCH] Fix debug ICE on aarch64 due to bad rtl simplification (PR debug/70628)
  - From: Jakub Jelinek

References:
- [PATCH] Fix debug ICE on aarch64 due to bad rtl simplification (PR debug/70628)
  - From: Jakub Jelinek

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