https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68486
James Greenhalgh <jgreenhalgh at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jgreenhalgh at gcc dot gnu.org
--- Comment #16 from James Greenhalgh <jgreenhalgh at gcc dot gnu.org> ---
(In reply to Steve Kargl from comment #15)
> On Tue, Nov 24, 2015 at 07:48:19PM +0000, dominiq at lps dot ens.fr wrote:
> > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68486
> >
> > --- Comment #14 from Dominique d'Humieres <dominiq at lps dot ens.fr> ---
> > > Not sure how the gcc/testsuite got on the end. Oh well, the
> > > testcase won't hurt anything.
> >
> > [Book15] f90/bug% gfca /opt/gcc/_clean/gcc/testsuite/gfortran.dg/pr68227.f90
> > /opt/gcc/_clean/gcc/testsuite/gfortran.dg/pr68227.f90:21:0:
> >
> > forall (i=1:n) y(i)%a = x
> >
> >
> > internal compiler error: in gfc_do_allocate, at fortran/trans-stmt.c:3130
> >
> > without your fix for pr68227.
> >
>
> Of course. As I stated, it won't hurt anything.
> FAILs in the testsuite are not a problem. The
> issue with EXPR_FUNCTION and cshift is a problem.
The hurt is for those of us investigating the new regressions... That was 10
minutes of my time to investigate, multiply that by the number of people
watching for FAILs on their favourite targets (particularly with trunk as it is
at the end of stage 1) there is certainly enough hurt to go round.
Please revert the broken testcase and leave it for when you fix pr68227.
