Re: GCC/gfortran: asking for permission to merge devel/gfortran-test into master, adding a shared memory implementation of coarrays.
Tobias Burnus <[email protected]>
| Newsgroups | gmane.comp.gcc.patches,gmane.comp.gcc.fortran |
|---|---|
| Message-ID | <[email protected]> |
Hi Jerry, hi all, Jerry D wrote: > Here is a current patch set. Can this be put into some proper patch series? I mean there are usually patches posted as 1/10 to 10/10 patches and I see the following in the patches you posted: * The runtime library itself – consisting one the files into the runtime library, the configure.ac/Makefile.ac integration, and fixes (unused args) to get it running. — And some follow-up fixes. * Front-end changes – those seem to be generic changes. I hope that there is somewhere a testcase for this, but it seems as those should be separately reviewed. * Testsuite changes – including enabling the new library. The first two items seem to be rather independent. In principle, I think it would be good to have the FE fixes with testcases in first, then the library integration and then the testsuite enabling. But in any case, I think for the FE part, it makes sense to have a proper patch email per patch – such that it can be reviewed one by one. Tobias