This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch, fortran] PR84219 - [8 Regression] ICE: Invalid expression in gfc_target_interpret_expr
- From: Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>
- To: Christophe Lyon <christophe dot lyon at linaro dot org>
- Cc: "fortran at gcc dot gnu dot org" <fortran at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Fri, 2 Mar 2018 08:53:15 +0000
- Subject: Re: [Patch, fortran] PR84219 - [8 Regression] ICE: Invalid expression in gfc_target_interpret_expr
- Authentication-results: sourceware.org; auth=none
- References: <CAGkQGiKMohYfm_cN+80TmNp4eueZ6gUom__qdkHOzh0ydfQnyQ@mail.gmail.com> <CAKdteObwoRzF0J+6_XrO642JHayjXTZRYDC6WTs+FTMhNdvMDQ@mail.gmail.com> <CAGkQGiJTxk8sZf_TywOm78KPZj=ZHV+znh4d16VH_zt+88BG-A@mail.gmail.com>
Hi Christophe,
I committed the wrong test :-( Corrected r258128.
Many thanks
Paul
On 2 March 2018 at 08:38, Paul Richard Thomas
<paul.richard.thomas@gmail.com> wrote:
> Hi Christophe,
>
> I was not aware. Thanks for letting me know.
>
> It's wierd that there is no error message or an ICE. Previously, an
> ICE would occur instead of the intended error message.
>
> I do not have access to either targets. Would you be so kind as to run
> coarray_47.f90 separately to see what happens? Note that 7-branch
> behaves correctly.
>
> Cheers
>
> Paul
>
> On 2 March 2018 at 08:26, Christophe Lyon <christophe.lyon@linaro.org> wrote:
>> On 1 March 2018 at 14:28, Paul Richard Thomas
>> <paul.richard.thomas@gmail.com> wrote:
>>> Committed as 'obvious' in revision 258098.
>>>
>>> Paul
>>>
>>> 2018-03-01 Paul Thomas <pault@gcc.gnu.org>
>>>
>>> PR fortran/84219
>>> * target-memory.c (gfc_interpret_derived): Assert that BT_VOID
>>> components are caf tokens.
>>> (gfc_target_interpret_expr): Treat BT_VOID expressions as
>>> integers.
>>>
>>> 2018-03-01 Paul Thomas <pault@gcc.gnu.org>
>>>
>>> PR fortran/84219
>>> * gfortran.dg/coarray_47.f90: New test.
>>
>>
>> Hi Paul,
>>
>> You may already be aware of that, but the new test fails at least on
>> aarch64 and arm targets:
>> FAIL: gfortran.dg/coarray_47.f90 -O (test for errors, line 12)
>> As a matter of fact, I can see no error message in gfortran.log, hence
>> dg-error fails to match.
>>
>> Christophe
>
>
>
> --
> "If you can't explain it simply, you don't understand it well enough"
> - Albert Einstein
--
"If you can't explain it simply, you don't understand it well enough"
- Albert Einstein