This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, Fortran, Coarray, v1] Add support for failed images
- From: Andre Vehreschild <vehre at gmx dot de>
- To: Christophe Lyon <christophe dot lyon at linaro dot org>
- Cc: Jerry DeLisle <jvdelisle at charter dot net>, GCC-Patches-ML <gcc-patches at gcc dot gnu dot org>, GCC-Fortran-ML <fortran at gcc dot gnu dot org>, Alessandro Fanfarillo <elfanfa at ucar dot edu>, Damian Rouson <damian at rouson dot net>
- Date: Thu, 9 Mar 2017 09:45:02 +0100
- Subject: Re: [PATCH, Fortran, Coarray, v1] Add support for failed images
- Authentication-results: sourceware.org; auth=none
- References: <20170304185848.08865575@vepi2> <7d51d255-1747-55e8-35fe-29d1316bf1c0@charter.net> <20170305124110.4bfb1ff4@vepi2> <CAKdteOZV6uQ_-g8orqTcT+2yv5zK=GZYefKLbs5xFiG1qFWU6Q@mail.gmail.com>
Hi Christophe,
you are right, that error message does not help a bit. Can you manually compile
and execute the testcase? Does it print "ERROR STOP: This statement should not
be reached."?
If not what does
fail_image_2 && echo "yes" || echo "no"
print? It should be "yes"
- Andre
On Wed, 8 Mar 2017 23:58:10 +0100
Christophe Lyon <christophe.lyon@linaro.org> wrote:
> Hi,
>
> On 5 March 2017 at 12:41, Andre Vehreschild <vehre@gmx.de> wrote:
> > Hi Jerry,
> >
> > thanks for seconding my read of the standard and reviewing so quickly.
> > Committed as r245900.
> >
>
> I've noticed that the new test:
> gfortran.dg/coarray/fail_image_2.f08 -fcoarray=single -O2 -latomic
> execution test
> fails on arm and aarch64.
> I'm using qemu if it matters, and my gfortran.log has:
> spawn /XXX/qemu-wrapper.sh ./fail_image_2.exe
> FAIL: gfortran.dg/coarray/fail_image_2.f08 -fcoarray=single -O2
> -latomic execution test
>
> that is, no obvious error message :-(
>
> Am I the only one seeing this?
>
> Thanks,
>
> Christophe
>
> > Regards,
> > Andre
> >
> > On Sat, 4 Mar 2017 15:06:25 -0800
> > Jerry DeLisle <jvdelisle@charter.net> wrote:
> >
> >> On 03/04/2017 09:58 AM, Andre Vehreschild wrote:
> >> > Hi all,
> >> >
> >> > attached patch polishes the one begun by Alessandro. It adds
> >> > documentation and fixes the style issues. Furthermore did I try to
> >> > interpret the standard according to the FAIL IMAGE statement. IMHO
> >> > should it just quit the executable without any error code. The
> >> > caf_single library emits "FAIL IMAGE" to stderr, while in coarray=single
> >> > mode it just quits. What do you think?
> >> >
> >> > Bootstraps and regtests ok on x86_64-linux/f25. Ok for trunk? (May be
> >> > later).
> >> >
> >> > Gruß,
> >> > Andre
> >> >
> >>
> >> From my read:
> >>
> >> "A failed image is usually associated with a hardware failure of the
> >> processor, memory system, or interconnection network"
> >>
> >> Since the FAIL IMAGE statement is intended to simulate such failure, I
> >> agree with your interpretation as well, it just stops execution.
> >>
> >> Yes OK for trunk now.
> >>
> >> Jerry
> >
> >
> > --
> > Andre Vehreschild * Email: vehre ad gmx dot de
--
Andre Vehreschild * Email: vehre ad gmx dot de
- References:
- [PATCH, Fortran, Coarray, v1] Add support for failed images
- Re: [PATCH, Fortran, Coarray, v1] Add support for failed images
- Re: [PATCH, Fortran, Coarray, v1] Add support for failed images
- Re: [PATCH, Fortran, Coarray, v1] Add support for failed images