[gomp] Fortran OpenMP parser

Paul Brook paul@codesourcery.com
Mon Sep 19 19:41:00 GMT 2005


On Monday 19 September 2005 20:31, Tobias Schlüter wrote:
> Jakub Jelinek wrote:
> > On Mon, Sep 19, 2005 at 08:36:29PM +0200, Tobias Schl?ter wrote:
> > I'd move just the largest contiguous chunks.
> > I think that is:
> > 1) 500 lines from match.c
> > 2) potentially big resolve_omp_directive and its subroutines
> > 3) potentially big gfc_trans_omp_directive and its subroutines
> > Or do you prefer to move parse_omp_* and decode_omp_* as well?
> > The parse.c stuff uses various parse.c local macros, static functions
> > etc.
> >
> > Do you want all of 1), 2) and 3) to go into one single file (openmp.c),
> > or separate match-openmp.c, resolve-openmp.c and trans-openmp.c?
>
> I'm perfectly fine with a single file; if both the parsing and the
> translation part are going to significantly grow, it might make sense to
> follow the logic used in the rest of the FE, e.g. stmt.c vs. trans-stmt.c,
> but I can't tell if this will be worth the effort.

I think it's worth splitting the pure frontend bits (match/resolve) bits from 
the gcc tree generation bits (trans-*).

Paul



More information about the Gcc-patches mailing list