This is the mail archive of the fortran@gcc.gnu.org mailing list for the GNU Fortran 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]

[Patch, fortran] PR33370 - committed as 'obvious'

From: "Paul Richard Thomas" <paul dot richard dot thomas at gmail dot com>
To: "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>, "gcc-patches List" <gcc-patches at gcc dot gnu dot org>, "Mark Mitchell" <mark at codesourcery dot com>, "Thomas Koenig" <tkoenig at alice-dsl dot net>
Date: Mon, 10 Sep 2007 10:05:51 +0200
Subject: [Patch, fortran] PR33370 - committed as 'obvious'
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=beta; h=domainkey-signature:received:received:message-id:date:from:to:subject:mime-version:content-type; bh=dgU89HvxcpyA9ylxLboYMdWmxCKu9Wd3R56eqLWKkcc=; b=eIuxqNBCwdtR/LrIZecjqSekIaANOhD8YYamuriuWJ19rrAGo3n8W1OfoCtlfiBYFcxUqKp1JZTdm4HRiCCKOirlwY/kZ+YVrglCF8ltTofZA6md3rAK3BnWRIM2iM3qpAqFZ5NUhOcK00JW8W2NZEnEhooCfiT2EVwR+EDUj8A=
Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:message-id:date:from:to:subject:mime-version:content-type; b=qrcqYDJQE+AqoOGkvRBfH2TyQi9b9qKiFm4g3VkDpV/0qnCNk8NA+gh1dMNz47FNkffqKtqxjSzdbdMwvZYUipNvlpuwa6XgWhgHToS+Ed9NgJSnvnM71pFiu6ZJXEQ7tL3tFyf+PM1KSmupJEG9bRiS66E2YErNRIzp+OTN9Ag=

In testing my patch for subreference array pointers, Thomas Koenig
found a regression in another part of gfortran that was serious enough
as to warrant immediate action.  I therefore committed, as obvious,
the patch below.  I felt that this was safe because the conditions
that caused the regression have been made more restrictive and will
push compilation towards the scalarizer, which works correctly.

Bootstrapped and regtested on x86_ia64/FC5

2007-09-10  Paul Thomas  <pault@gcc.gnu.org>

	PR fortran/33370
	* trans-expr.c (copyable_array_p):  Add tests that expression
	is a variable, that it has no subreferences and that it is a
	full array.
	(gfc_trans_assignment): Change conditions to suit modifications
	to copyable_array_p.

2007-09-10  Paul Thomas  <pault@gcc.gnu.org>

	PR fortran/33370
	* gfortran.dg/array_memcpy_5.f90:  New test.

Paul

Attachment: check.diff
Description: Text document

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