[RFC] Migrating g77 testsuite to gfortran

[Paul Brook]

On Sunday 11 July 2004 03:58, Billinghurst, David (CALCRTS) wrote:
> In the last few days I have copied some of the existing g77 tests from
> g77.dg to gfortran.dg/g77.  To complete this, I would like to delete
> those tests from g77.dg (patch below).
>
> This raises the question of what do to with the rest of the g77 testsuite.
>
> 1. I assume that gcc-3.5.0 will not contain a g77 testsuite, so we will
> have to remove the g77.dg and g77.f-torture directories at some stage.

We could actually remove these directories now as g77 is already gone.
They'll still be in CVS, so it's not like we'll loose them forever. I don't 
think I have authority to approve this though.

> 2. As a fortran user, I don't want regressions from g77 to gfortran (or
> at least I want to know about them) so the tests in the g77 testsuite
> should be retained in there original condition (where possible) or
> with minimum change (if supported).

Maybe. My impression was what that we now favour the dg testsuites over the 
torture tests. This would be a convenient opportunity to convert these tests 
if this is indeed the case. FOr most tests this should just be a case of 
adding dg-do compie or dg-do run to the top of the test.

Anyone else have any opinions on dg vs. torture?

> 3.  Some tests are for g77 options that are not supported by gfortran,
> for example "-fone-trip".  These have to go.

Or implement these options in gfortran.

> Here is a plan for discussion. I am happy to work on this, as I can do it
> in tiny steps as I find the time.
>
> A.  Create directories:
>     - gfortran-fortran-torture/compile/g77 for files in
> g77.f-torture/compile - gfortran-fortran-torture/execute/g77 for files in
> g77.f-torture/execute and patch the dejagnu files so that they work as
> expected

See above. It may be better to convert these tests to the dg harness.

> B.  Move tests in the testsuite that "just work".
>
> C.  Move, after trivial "obvious" patches, tests that just need tweaks for
>     changed warnings, etc.
>
> D.  For tests that now fail:
>      - if they can be XFAILed: move them, xfail them and log PRs
>      - if they can't be XFAILed: log PR and ...?
>
> E.  For tests that aren't supported by gfortran, submit patches to remove
> them from the g77 testsuite.  Patch review will sort out any issues.
>
> F.  Review the (hopefully empty set) of cases not covered by A-E.

I think we should also create a README.g77 file in the testsuite directory 
that lists all the tests covered by F, E, and possibly D. The will help keep 
track of what has/hasn't been converted, and record the decisions for 
posterity.

Paul