public inbox for gcc@gcc.gnu.org
 help / color / mirror / Atom feed
From: Gaius Mulley <gaiusmod2@gmail.com>
To: Richard Biener <richard.guenther@gmail.com>,
	gcc@gcc.gnu.org, dje.gcc@gmail.com
Subject: Proposal for the merger and patch set
Date: Wed, 18 May 2022 14:47:06 +0100	[thread overview]
Message-ID: <87tu9n6jhh.fsf_-_@debian> (raw)
In-Reply-To: <C66F1ACC-8A6F-4673-886F-79A9C232A7B4@gmail.com> (Richard Biener's message of "Sat, 14 May 2022 08:15:42 +0200")

Richard Biener <richard.guenther@gmail.com> writes:

> I suggest you post merge patches where the branch touches generic code
> for review again, indicating parts that have been approved in the
> past.

Hello Richard, David and the GCC Steering Committee,

Firstly thank you for the release of gcc-12.1 and secondly thank you
for taking the time to review the Modula-2 front end and for valuable
review feedback over the years.  I hope you have all managed to have a
break after the 12.1 release.

Here for reference are the previous patch emails and reviews

[For historical completeness
https://gcc.gnu.org/pipermail/gcc-patches/2019-November/534683.html
]

Jan 18 2021 patches:

https://gcc.gnu.org/pipermail/gcc-patches/2021-January/563732.html

Richard's <rguenther@suse.de> review of the patches last year.

https://gcc.gnu.org/pipermail/gcc-patches/2021-January/563747.html


June 2021 patches:

https://gcc.gnu.org/pipermail/gcc-patches/2021-June/573103.html

Richard's <rguenther@suse.de> review of the June patches last year.

https://gcc.gnu.org/pipermail/gcc-patches/2021-June/573147.html

Richard Guenther  <rguenther@suse.de> writes:

> I suggest you post merge patches where the branch touches generic
> code for review again, indicating parts that have been approved in
> the past.

Sure I'll try and group the subsequent email patches into the following categories:

  1.  all GCC files which have been patched.
  2.  the top level /gm2tools contents.
  3.  the top level /libgm2 contents.
  4.  the glue code (between Modula-2 and GCC) contained in /gcc/m2/gm2-gcc.
      (these have not been reviewed before).  These will be posted in
      3 sub parts.
  5.  all the driver code /gcc/m2.
  6.  /gcc/m2/Make-lang.in
      (not been reviewed before)
  7.  /gcc/doc/gm2.texi
      (not been reviewed before)

The full source code is available via:

  $ git clone git://gcc.gnu.org/git/gcc.git gcc-git-devel-modula2
  $ cd gcc-git-devel-modula2
  $ git checkout devel/modula-2

currently it is being merged every 1-3 days or so.  One of the
recommendations after the last review was to maintain this branch.
Over the last year the changes include many bug porting fixes, removal
of compiler build warnings, migrating the code base to C++, improving
accuracy/detail of compiler error messages and improving error recovery.

It builds on gnu/linux/amd64, gnu/linux/aarch64, gnu/linux/sparc64,
solaris/sparc64, gnu/linux/i386, gnu/linux/ppc64le, gnu/linux/power7
(and probably others).  Also built with profiled lto on amd64 and
aarch64.

Richard Guenther  <rguenther@suse.de> writes

> There’s at least BSD in the list of primary targets. It would be
> nice to enable more but that can be done during stage1.

and now FreeBSD 13 i586 providing a patch was applied for C++.
[Interestingly csinl, ctanl and friends are missing on FreeBSD 13
which cause all runtime tests to fail].

(Test results posted to gcc-testresults).

On amd64-gnu-linux it builds as a cross compiler for aarch64.
http://floppsie.comp.glam.ac.uk/Southwales/gaius/web/bare-metal-m2.html

I've built GCC from the master branch enabled c,c++,fortran,go,d
and then built GCC from devel/modula-2 enabled c,c++,fortran,go,d,m2
and no further regressions were seen.

Anyway I suspect this email is long enough for now - feel free to ask
questions, 9 patches posted to the gcc-patches list.  Let me know if you
require the patch set and/or new files in a different format,

regards,
Gaius

      parent reply	other threads:[~2022-05-18 13:47 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-13 22:56 Proposal for the merger of devel/modula-2 into master Gaius Mulley
2022-05-14  6:15 ` Richard Biener
2022-05-14 12:06   ` Gaius Mulley
2022-05-18 13:47   ` Gaius Mulley [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87tu9n6jhh.fsf_-_@debian \
    --to=gaiusmod2@gmail.com \
    --cc=dje.gcc@gmail.com \
    --cc=gcc@gcc.gnu.org \
    --cc=richard.guenther@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).