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: [Fortran] Clean-up gfc_trans_where_assign

From: Paul Thomas <paulthomas2 at wanadoo dot fr>
To: Roger Sayle <roger at eyesopen dot com>
Cc: fortran at gcc dot gnu dot org, gcc-patches at gcc dot gnu dot org
Date: Mon, 20 Feb 2006 07:20:31 +0100
Subject: Re: [Fortran] Clean-up gfc_trans_where_assign
References: <Pine.LNX.4.44.0602182009190.13575-100000@www.eyesopen.com>

Roger Sayle wrote:

The following patch is a clean-up to gfc_trans_where_assign resulting
from the recent reorganization of gfc_trans_where_2.  We now always
construct a "ready-to-use" execution mask, so the original code to
traverse a linked list of masks and AND them together can be removed
from gfc_trans_where_assign.  This change was kept separate to simplify
the review of the previous patch.  This'll also please the middle-end
folks, as its one less instance of TREE_CHAIN abuse.  I've not added
a gcc_asert that TREE_CHAIN (mask) is NULL_TREE, as this would preserve
this misuse :-)

The following patch has been tested on x86_64-unknown-linux-gnu with
a top-level bootstrap, including fortran, and regression tested with
a top-level "make -k check" with no new failures.

Ok for mainline?

OK - it patches cleanly, regtests fine and does what you advertise.

Paul

References:
- [Fortran] Clean-up gfc_trans_where_assign
  - From: Roger Sayle

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