This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [Patch, Fortran, OOP] PR 64230: [4.9/5 Regression] Invalid memory reference in a compiler-generated finalizer for allocatable component
- From: Janus Weil <janus at gcc dot gnu dot org>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: Andreas Schwab <schwab at suse dot de>, gfortran <fortran at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 27 Jan 2015 19:57:33 +0100
- Subject: Re: [Patch, Fortran, OOP] PR 64230: [4.9/5 Regression] Invalid memory reference in a compiler-generated finalizer for allocatable component
- Authentication-results: sourceware.org; auth=none
- References: <CAKwh3qipOj3hjqLpQdw7_HZx_QOZF9pyDU91ZyEpp_FhjTxJQg at mail dot gmail dot com> <mvmzj94mv8g dot fsf at hawking dot suse dot de> <20150127093028 dot GC1746 at tucnak dot redhat dot com> <CAKwh3qgUY9HhQMKdmk1VrFQB7WXQtjUOvE9JDMdps3z_p-=ojQ at mail dot gmail dot com> <20150127182338 dot GL1746 at tucnak dot redhat dot com>
2015-01-27 19:23 GMT+01:00 Jakub Jelinek <jakub@redhat.com>:
> On Tue, Jan 27, 2015 at 07:20:10PM +0100, Janus Weil wrote:
>> 2015-01-27 10:30 GMT+01:00 Jakub Jelinek <jakub@redhat.com>:
>> > Yeah, if you want to add ubsan tests, you need to add gfortran.dg/ubsan/
>> > directory and hack up ubsan.exp in there
>>
>> Thanks for the remark, I was suspecting something like that. However,
>> for this case it's not really worth the hassle. In fact the test case
>> does not really need the sanitizer and should also work without it. So
>> I'll just remove the -fsanitize option:
>>
>> Index: gcc/testsuite/gfortran.dg/class_allocate_18.f90
>> ===================================================================
>> --- gcc/testsuite/gfortran.dg/class_allocate_18.f90 (Revision 220180)
>> +++ gcc/testsuite/gfortran.dg/class_allocate_18.f90 (Arbeitskopie)
>> @@ -1,5 +1,4 @@
>> ! { dg-do run }
>> -! { dg-options "-fsanitize=undefined" }
>> !
>> ! PR 64230: [4.9/5 Regression] Invalid memory reference in a
>> compiler-generated finalizer for allocatable component
>> !
>
> LGTM.
Good, committed as r220181. Since I had already backported the
original patch to 4.9 yesterday, I'll do the same there ...
Cheers,
Janus
- References:
- [Patch, Fortran, OOP] PR 64230: [4.9/5 Regression] Invalid memory reference in a compiler-generated finalizer for allocatable component
- Re: [Patch, Fortran, OOP] PR 64230: [4.9/5 Regression] Invalid memory reference in a compiler-generated finalizer for allocatable component
- Re: [Patch, Fortran, OOP] PR 64230: [4.9/5 Regression] Invalid memory reference in a compiler-generated finalizer for allocatable component
- Re: [Patch, Fortran, OOP] PR 64230: [4.9/5 Regression] Invalid memory reference in a compiler-generated finalizer for allocatable component
- Re: [Patch, Fortran, OOP] PR 64230: [4.9/5 Regression] Invalid memory reference in a compiler-generated finalizer for allocatable component