This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH][RFC] Fix PR71632, remove parts of TER
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Richard Biener <rguenther at suse dot de>, Jeff Law <law at redhat dot com>, Michael Matz <matz at suse dot de>, Andrew MacLeod <amacleod at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Fri, 1 Jul 2016 09:39:23 +0200
- Subject: Re: [PATCH][RFC] Fix PR71632, remove parts of TER
- Authentication-results: sourceware.org; auth=none
- References: <alpine dot LSU dot 2 dot 11 dot 1606301502410 dot 29772 at t29 dot fhfr dot qr>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Thu, Jun 30, 2016 at 03:51:20PM +0200, Richard Biener wrote:
> The following patch fixes PR71632 by removing delayed expansion of
> TERed defs. Instead it adds code to apply the scheduling effect
> to the GIMPLE IL (so you also get better interleaved GIMPLE stmt
> / generated RTL dumps in .expand).
Does anything from TER survive after this patch?
I thought the whole point was that the expansion can see through
the SSA_NAMEs and optimize based on that, by not seeing through
them it doesn't, or if it somewhere still uses get_gimple_for_ssa_name,
if the definition will be already expanded, it might expand stuff multiple
times.
Jakub