Re: [patch] testsuite/libgomp.fortran/pr66199-2.f90 – remove dg-options

2019-11-04 Thread Jakub Jelinek
On Mon, Nov 04, 2019 at 02:02:38PM +0100, Tobias Burnus wrote: > On 11/4/19 1:29 PM, Jakub Jelinek wrote: > > Note, pr66199-1.f90 has the same issue. And I'd probably go for the > > removal of dg-do run instead. > > Well, there are even more (ignoring those which have -std=legacy or > -fno-inline

Re: [patch] testsuite/libgomp.fortran/pr66199-2.f90 – remove dg-options

2019-11-04 Thread Tobias Burnus
On 11/4/19 1:29 PM, Jakub Jelinek wrote: Note, pr66199-1.f90 has the same issue. And I'd probably go for the removal of dg-do run instead. Well, there are even more (ignoring those which have -std=legacy or -fno-inline or … in dg-options): libgomp.fortran/examples-4/declare_target-5.f90, lib

Re: [patch] testsuite/libgomp.fortran/pr66199-2.f90 – remove dg-options

2019-11-04 Thread Jakub Jelinek
On Mon, Nov 04, 2019 at 01:18:58PM +0100, Tobias Burnus wrote: > This test case had from the beginning both > ! { dg-do run } > ! { dg-options "-O2" } > > However, 'dg-do run' implies torture, i.e. multiple optimization options, > while 'dg-options "-O2"' implies -O2, only. > > This means that

[patch] testsuite/libgomp.fortran/pr66199-2.f90 – remove dg-options

2019-11-04 Thread Tobias Burnus
This test case had from the beginning both ! { dg-do run } ! { dg-options "-O2" } However, 'dg-do run' implies torture, i.e. multiple optimization options, while 'dg-options "-O2"' implies -O2, only. This means that the test case is compiled 4 times with "-O0", "-O1", "-O3 -f…" and "-Os" – fol