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: Jakub Jelinek <jakub at redhat dot com>
- To: Janus Weil <janus at gcc dot gnu dot org>
- 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:23:38 +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>
- Reply-to: Jakub Jelinek <jakub at redhat dot 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.
Jakub