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 PR tree-opt/16808, Vops are not being renamed

From: Andrew Pinski <pinskia at physics dot uc dot edu>
To: Diego Novillo <dnovillo at redhat dot com>
Cc: "gcc-patches at gcc dot gnu dot org Patches" <gcc-patches at gcc dot gnu dot org>
Date: Mon, 1 Nov 2004 23:35:01 -0500
Subject: Re: [PATCH] Fix PR tree-opt/16808, Vops are not being renamed
References: <117384CA-2B60-11D9-A2F5-000A95D692F4@physics.uc.edu> <1099326518.7545.41.camel@localhost.localdomain>

On Nov 1, 2004, at 11:28 AM, Diego Novillo wrote:

On Sun, 2004-10-31 at 12:12, Andrew Pinski wrote:

	* tree-ssa.c (mark_all_v_defs): New function.
	(replace_immediate_uses): Call the new function.
	(pass_redundant_phi): Add TODO_fix_def_def_chains.

This is wrong.  TODO_rename_vars and TODO_fix_def_def_chains do
essentially the same thing.  If we have disappearing operands, you
should call mark_new_vars_to_rename.

Ok, what about this?

ChangeLog:
	* tree-ssa.c (replace_immediate_uses): If we call fold_stmt, make sure
	that the vops get marked for renaming.

Attachment: temp.diff.txt
Description: Text document

Thanks,
Andrew Pinski

Follow-Ups:
- Re: [PATCH] Fix PR tree-opt/16808, Vops are not being renamed
  - From: Diego Novillo

References:
- Re: [PATCH] Fix PR tree-opt/16808, Vops are not being renamed
  - From: Diego Novillo

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