2015-01-27 19:23 GMT+01:00 Jakub Jelinek <ja...@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 <ja...@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