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: Many regressions with: [patch] Cleanup fortran/convert.c

From: Thomas Koenig <tkoenig at netcologne dot de>
To: Tobias Burnus <burnus at net-b dot de>
Cc: Steven Bosscher <stevenb dot gcc at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, fortran at gcc dot gnu dot org
Date: Sat, 10 Mar 2012 18:22:20 +0100
Subject: Re: Many regressions with: [patch] Cleanup fortran/convert.c
References: <CABu31nO+9AY7BEizP5=SPJqCb2vdKHorrzcZeULvJWEgw5dR5w@mail.gmail.com> <4F5B2AAD.2040009@net-b.de>

Am 10.03.2012 11:19, schrieb Tobias Burnus:

Steven Bosscher wrote:

This cleans up some remnants of the ancestors of fortran's convert.c,
which was copied from GNAT IIRC. I would bootstrap&test this, but
trunk appears to be broken for x86_64-linux right now (ICE in
patch_jump_insn). But I can post this
for review, at least.
OK for trunk, after bootstrap+test?


Your patch seems to have caused many Fortran regressions. At least I see
with 185156 only one (known) failure, cf.
http://gcc.gnu.org/ml/gcc-testresults/2012-03/msg01069.html

While starting with 185160 there are many, many gfortran failures, cf.
http://gcc.gnu.org/ml/gcc-testresults/2012-03/msg01073.html

One error from the log is

/home/tkoenig/trunk/gcc/testsuite/gfortran.dg/gnu_logical_1.F:4:0: error: type mismatch in binary expression logical(kind=8)

logical(kind=8)

logical(kind=4)

D.2303 = k8 & l1.26;

Thomas

References:
- [patch] Cleanup fortran/convert.c
  - From: Steven Bosscher
- Many regressions with: [patch] Cleanup fortran/convert.c
  - From: Tobias Burnus

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