Re: [PATCH 5/5] libgm2/libm2pim/wrapc.cc: Define NULL as nullptr

Alejandro Colomar writes: > For internal C++ code, unconditionally define NULL as nullptr. > We already require a C++11 compiler to bootstrap GCC anyway. > > Link: > Signed-off-by: Alejandro Colomar > --- > libgm2/libm2pim/wrapc.cc | 4 +--- > 1 file

Re: [PATCH] gm2: export all libc number conversion functions

Wilken Gottwalt writes: > Export all string to integral and floating point number conversion functions > (atof, atoi, atol, atoll, strtod, strtof, strtold, strtol, strtoll, strtoul, > strtoull). > > gcc/gm2: > * gm2-libs/libc.def: Export all string to number conversion functions. > > Signed

Re: [PATCH] gm2: fix bad programming practice identifier warning

Wilken Gottwalt writes: > On Tue, 23 Jul 2024 09:58:56 +0200 > Wilken Gottwalt wrote: > >> Fix using keywords as identifiers to prevent warnings coming from >> Modula-2's own libraries. >> >> m2pim/DynamicStrings.mod:1358:27: note: In procedure ‘Slice’: the symbol >> name ‘end’ is legal as an i

Re: [PATCH] gm2: fix bad programming practice warning

Wilken Gottwalt writes: > On Wed, 24 Jul 2024 12:09:46 +0200 > Wilken Gottwalt wrote: > >> Fix identifier names to be too similar to Modula-2 keywords and causing >> warnings coming from Modula-2's own libraries. >> >> m2/m2iso/StdChans.mod:54:20: note: In implementation module ‘StdChans’: >> e

Re: [PATCH] gm2: add missing debug output guard

Wilken Gottwalt writes: > On Sat, 10 Aug 2024 13:43:33 +0200 (CEST) > Gerald Pfeifer wrote: > >> On Tue, 23 Jul 2024, Gaius Mulley wrote: >> >> gcc/gm2: >> >> * gm2-libs-iso/MemStream.mod: Guard debug output. >> > many thanks! >> >&g

Re: [PATCH] doc: Rephrase GM2 Limitations section

Gerald Pfeifer writes: > I noticed a non-working link, then some other details in that section. > > Here is a suggestion to rework it a bit. > > Okay? > > Gerald > > > >>From 83e856355a94bd78afbf19eed32ca1726658f581 Mon Sep 17 00:00:00 2001 > From: Gerald Pfeifer > Date: Mon, 5 Aug 2024 21:06:20

Re: [PATCH] gm2: fix bad programming practice identifier warning

Wilken Gottwalt writes: > Fix using keywords as identifiers to prevent warnings coming from > Modula-2's own libraries. > > m2pim/DynamicStrings.mod:1358:27: note: In procedure ‘Slice’: the symbol > name ‘end’ is legal as an identifier, however as such it might cause > confusion and is considered

Re: [PATCH] gm2: add missing debug output guard

Wilken Gottwalt writes: > The Close() procedure in MemStream is missing a guard to prevent it from > printing in non-debug mode. > > gcc/gm2: > * gm2-libs-iso/MemStream.mod: Guard debug output. > > Signed-off-by: Wilken Gottwalt > --- > gcc/m2/gm2-libs-iso/MemStream.mod | 5 - > 1 fil

Re: [PATCH 06/52] m2: Replace uses of {FLOAT, {, LONG_}DOUBLE}_TYPE_SIZE

"Kewen.Lin" writes: > Hi Gaius, > >>> static tree >>> build_m2_short_real_node (void) >>> { >>> - tree c; >>> - >>> - /* Define `REAL'. */ >>> - >>> - c = make_node (REAL_TYPE); >>> - TYPE_PRECISION (c) = FLOAT_TYPE_SIZE; >>> - layout_type (c); >>> - return c; >>> + /* Define `SHORTREA

Re: [PATCH 06/52] m2: Replace uses of {FLOAT, {, LONG_}DOUBLE}_TYPE_SIZE

"Kewen.Lin" writes: Hi Kewen, > Nice! Looking forward to you pushing this new one (I'm withdrawing the > original > patch). all pushed now - thanks for the original patch! regards, Gaius

Re: [PATCH 06/52] m2: Replace uses of {FLOAT, {, LONG_}DOUBLE}_TYPE_SIZE

"Kewen.Lin" writes: > Hi Joseph and Gaius, > > on 2024/6/4 02:02, Joseph Myers wrote: >> On Sun, 2 Jun 2024, Kewen Lin wrote: >> >>> diff --git a/gcc/m2/gm2-gcc/m2type.cc b/gcc/m2/gm2-gcc/m2type.cc >>> index 571923c08ef..d52cbdf0b99 100644 >>> --- a/gcc/m2/gm2-gcc/m2type.cc >>> +++ b/gcc/m2/gm2-

Re: [PATCH 06/52] m2: Replace uses of {FLOAT, {, LONG_}DOUBLE}_TYPE_SIZE

Kewen Lin writes: > Joseph pointed out "floating types should have their mode, > not a poorly defined precision value" in the discussion[1], > as he and Richi suggested, the existing macros > {FLOAT,{,LONG_}DOUBLE}_TYPE_SIZE will be replaced with a > hook mode_for_floating_type. To be prepared f

Re: [PATCH] modula2: Fully respect DESTDIR in texi rule

Sam James writes: > This was originally reported in Gentoo at https://bugs.gentoo.org/930014. > > 2024-05-20 Sam James > gcc/m2/ > * Make-lang.in (m2.install-info): Pass --destdir for dir index. > --- > gcc/m2/Make-lang.in | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > dif

Re: [PATCH] libgm2: re-generate with autoreconf

Simon Marchi writes: > > I don't have access to the gcc repo, so could you please push the patch > on my behalf? all done - many thanks for the patch! regards, Gaius

Re: [PATCH] testsuite: gm2: Remove timeout overrides [PR114886]

Rainer Orth writes: > Hi Gaius, > >> yes this looks good to me please apply. Thanks for the rationale > > done for trunk. I guess it's ok to apply to the gcc-14 branch after the > release and some soak time? many thanks - and yes certainly also to gcc-14 (after an appropriate amount of delay),

[pushed] wwwdocs: modula2 update for changes.html, index.html, readings.html and frontends.html

Pushed the commit c74a573fa888f3970b6b38d57020f0160e49e58a frontends.html: Mention modula-2 was merged during gcc-13. gcc-14/changes.html: New section heading for modula-2 and populate. index.html: Add modula-2 to the list of languages supported by GCC. readings.html (Modula 2 inform

Re: [PATCH] libgm2: re-generate with autoreconf

Christophe Lyon writes: > On Tue, 30 Apr 2024 at 04:01, Simon Marchi wrote: >> >> I get a diff when running "autoreconf" in this directory. I think that >> the current state is erroneous: it appears to have been generated using >> >> aclocal -I ../config -I .. >> >> even though configure.ac

Re: [PATCH] testsuite: gm2: Remove timeout overrides [PR114886]

Rainer Orth writes: > A large number of gm2 tests are timing out even on current Solaris/SPARC > systems. As detailed in the PR, the problem is that the gm2 testsuite > artificially lowers many timeouts way below the DejaGnu default of 300 > seconds, often as short as 10 seconds. The problem li

Re: [committed] Further spelling fixes in translatable strings

Jakub Jelinek writes: > On Tue, Apr 23, 2024 at 11:32:08AM +0100, Jonathan Wakely wrote: >> On Mon, 22 Apr 2024 at 22:30, Jakub Jelinek wrote: >> Yup: >> https://gcc.gnu.org/codingconventions.html#Spelling >> >> That spelling is explicitly mentioned at the link above, so they >> should be "ize"

[pushed] modula2: add modula-2 language section to languages supported by GCC

Pushed to trunk as r14-9915-g0dc39dee836761f1bc993d760f4ed5f3d127897a This patch introduces a small modula-2 language section to the Language Standards Supported by GCC node. gcc/ChangeLog: * doc/standards.texi (Language Standards Supported by GCC): Add Modula-2 language sectio

[pushed] modula2: add modula-2 language section to languages supported by GCC

Pushed to trunk as r14-9915-g0dc39dee836761f1bc993d760f4ed5f3d127897a This patch introduces a small modula-2 language section to the Language Standards Supported by GCC node. gcc/ChangeLog: * doc/standards.texi (Language Standards Supported by GCC): Add Modula-2 language sectio

[PATCH][wwwdocs] modula2 update for changes.html, index.html, readings.html and frontends.html

Hello, Here are a proposed set of patches to highlight the changes in the modula-2 frontend for GCC 14.1. The patches also contain updates for: * readings.html: Add Modula-2 language standards. * index.html: Modula-2 listed in the languages supported by GCC. * frontends.html: Update now

Re: [PATCH] modula2: Add m2.install-dvi in gcc/m2/Make-lang.in

Christophe Lyon writes: > m2 has a m2.dvi build rule, but lacks the m2.install-dvi one. > > 2024-04-04 Christophe Lyon > > gcc/m2/ > * Make-lang.in (m2.install-dvi): New rule. > --- > gcc/m2/Make-lang.in | 12 > 1 file changed, 12 insertions(+) > > diff --git a/gcc/m2

Re: [PATCH 2/2] modula2: Fix m2.install-info in gcc/m2/Make-lang.in

Christophe Lyon writes: > Fix a few typos: the generated filename is m2.info (not gm2.info, and > gm2$(exeext) is a file not a directory (so test -d would always fail). > > 2024-03-29 Christophe Lyon > > gcc/m2/ > * Make-lang.in (m2.install-info): Fix rule. > --- > gcc/m2/Make-lan

Re: [PATCH 1/2] modula2: Add m2.install-html rule to gcc/m2/Make-lang.in

Christophe Lyon writes: > This rule was missing, and 'make install-html' was failing. > It is copied from the corresponding one in fortran. > > 2024-03-29 Christophe Lyon > > gcc/m2/ > * Make-lang.in (install-html): New rule. > --- > gcc/m2/Make-lang.in | 19 +++ >

Re: [PATCH RFA] build: drop target libs from LD_LIBRARY_PATH [PR105688]

Iain Sandoe writes: > Hi Gaius, > >> On 22 Feb 2024, at 18:06, Gaius Mulley wrote: >> >> Iain Sandoe writes: >> >>> Right now, AFAIK the only target runtimes used by host tools are >>> libstdc++, libgcc and libgnat. I agree that might change

Re: [PATCH RFA] build: drop target libs from LD_LIBRARY_PATH [PR105688]

Iain Sandoe writes: > Right now, AFAIK the only target runtimes used by host tools are > libstdc++, libgcc and libgnat. I agree that might change with rust - > since the rust folks are talking about using one of the runtimes in > the FE, I am not aware of other language FEs requiring their targt

Re: [PATCH] libgm2: Fix libm2iso/wraptime.cc compilation on Solaris

Rainer Orth writes: > As it turned out, my patch to complete the libgm2 autoconf macros works > on both Linux/sparc64 and Linux/x86_64, but breaks Solaris bootstrap: > > /vol/gcc/src/hg/master/local/libgm2/libm2iso/wraptime.cc: In function 'int > m2iso_wraptime_gettimeofday(void*, timezone*)': >

Re: [PATCH] libgm2: Define missing configure macros [PR113888]

Rainer Orth writes: > As detailed in PR modula2/113888, the 32-bit libm2pim/target.c doesn't > assemble in a Linux/sparc64 multilib build. However, the root cause is > that the HAVE_EXP10* macros are never defined. While I was at it, I > checked for other cases where the code uses HAVE_* macros

[PATCH] PR modula2/113836 gm2 does not dump gimple or quadruples to a file

98da2a54d --- /dev/null +++ b/gcc/m2/gm2-compiler/M2LangDump.def @@ -0,0 +1,65 @@ +(* M2LangDump.def provides support routines for the -flang-dump. + +Copyright (C) 2024 Free Software Foundation, Inc. +Contributed by Gaius Mulley . + +This file is part of GNU Modula-2. + +GNU Modula-2 is free softwa

Re: [PATCH] m2: Use time_t in time and don't redefine alloca

"H.J. Lu" writes: > Fix the m2 build warning and error: > > [...] > ../../src/gcc/m2/mc/mc.flex:32:9: warning: "alloca" redefined >32 | #define alloca __builtin_alloca > | ^~ > In file included from /usr/include/stdlib.h:587, > from :22: > /usr/include/alloc

Re: [PATCH] modula2: Many powerpc platforms do _not_ have support for IEEE754 long double [PR111956]

Richard Biener writes: > On Thu, Jan 18, 2024 at 1:58 AM Gaius Mulley wrote: >> >> >> ok for master ? >> >> Bootstrapped on power8 (cfarm135), power9 (cfarm120) and >> x86_64-linux-gnu. > > OK. many thanks! > I wonder what this does to the lib

[PATCH] modula2: Many powerpc platforms do _not_ have support for IEEE754 long double [PR111956]

ok for master ? Bootstrapped on power8 (cfarm135), power9 (cfarm120) and x86_64-linux-gnu. --- This patch corrects commit r14-4149-g81d5ca0b9b8431f1bd7a5ec8a2c94f04bb0cf032 which assummed all powerpc platforms would have IEEE754 long double. The patch ensures that cc1gm2 obtains the default I

Re: Modula-2: Support '-isysroot [...]'

Thomas Schwinge writes: > Hi! > > OK to push the attached "Modula-2: Support '-isysroot [...]'"? > > This greatly improves test results for the cross configurations I've > tested, but I don't know if any real handling needs to be implemented, or > this should be done differently altogether? > > >

Re: [PATCH] gm2: Fix mc/mc.flex compilation on Solaris

Rainer Orth writes: > The recent warning changes broke gm2 bootstrap on Solaris: > > /vol/gcc/src/hg/master/local/gcc/m2/mc/mc.flex: In function 'handleFile': > /vol/gcc/src/hg/master/local/gcc/m2/mc/mc.flex:297:21: error: implicit > declaration of function 'alloca' [-Wimplicit-function-declarat

Re: [PATCH] m2, build: Use LDLFAGS for mklink

Rainer Orth writes: > When trying to bootstrap current trunk on macOS 14.0 beta 3 with Xcode > 15 beta 4, the build failed running mklink in stage 2: > > unset CC ; m2/boot-bin/mklink -s --langc++ --exit --name m2/mc-boot/main.cc > /vol/gcc/src/hg/master/darwin/gcc/m2/init/mcinit > dyld[55825]:

Re: [PATCH] modula-2: Amend the handling of failed select() calls in RTint [PR108835].

Iain Sandoe writes: > Tested on various affected Darwin versions and on x86_64-linux-gnu > OK for trunk? > OK for 13.2? > thanks > Iain lgtm - many thanks, regards, Gaius

Re: [PATCH] modula2: Fix bootstrap

Jakub Jelinek writes: > Hi! > > internal-fn.h since yesterday includes insn-opinit.h, which is a generated > header. > One of my bootstraps today failed because some m2 sources started compiling > before insn-opinit.h has been generated. > > Normally, gcc/Makefile.in has > # In order for parallel

Re: [PATCH 05/14] m2: use _P() defines from tree.h

Bernhard Reutner-Fischer via Gcc-patches writes: > From: Bernhard Reutner-Fischer > > gcc/m2/ChangeLog: > > * gm2-gcc/m2builtins.cc (doradix): Use _P defines from tree.h. > (doplaces): Ditto. > (doexponentmin): Ditto. > (doexponentmax): Ditto. > (dolarge): Ditto. >

Re: [PATCH 05/14] m2: use _P() defines from tree.h

Bernhard Reutner-Fischer via Gcc-patches writes: > From: Bernhard Reutner-Fischer > > gcc/m2/ChangeLog: > > * gm2-gcc/m2builtins.cc (doradix): Use _P defines from tree.h. > (doplaces): Ditto. > (doexponentmin): Ditto. > (doexponentmax): Ditto. > (dolarge): Ditto. >

Re: libgm2: Remove 'autogen.sh'

Thomas Schwinge writes: > Hi! > > On 2023-04-14T13:49:20+0100, Gaius Mulley via Gcc-patches > wrote: >> Thomas Schwinge writes: >>> Separately, given that plain 'autoreconf' works, why have 'autogen.sh' at >>> all? >> >> If

Re: [PATCH] gcc/m2: Drop references to $(P)

Arsen Arsenović writes: > $(P) seems to have been a workaround for some old, proprietary make > implementations that we no longer support. It was removed in > r0-31149-gb8dad04b688e9c. > > gcc/m2/ChangeLog: > > * Make-lang.in: Remove references to $(P). > * Make-maintainer.in: Ditto.

Re: libgm2: Adjust 'autogen.sh' to 'ACLOCAL_AMFLAGS', and simplify

Thomas Schwinge writes: > Hi! > > On 2022-12-06T14:47:26+0000, Gaius Mulley via Gcc-patches > wrote: >> This patch set consists of the libgm2 makefile, autoconf sources >> necessary to build the libm2pim, libm2iso, libm2min, libm2cor >> and libm2log. > > N

Re: [PATCH] Modula-2: fix documentation layout

Eric Botcazou writes: > Hi Gaius, > >> yes indeed and thanks for the patch! > > You're welcome. The documentation was slightly broken again in the meantime, > but nothing really serious this time. > > Again tested with a modern and an old version of Makeinfo. OK for mainline? > > > 2023-03-27

Re: [wwwdocs] document modula-2 in gcc-13/changes.html (and index.html)

Gerald Pfeifer writes: > The patch is now in, alas all the GNU M2 manual links now point to > non-existant locations. > > Does maintainer-scripts/update_web_docs_git require an update to cover > Modula-2 and actually build the manual we are now linking to (or rather > trying to)? > > Gerald Ap

Re: [pushed] wwwdocs: gcc-13: Escape < and > as < and >

Gerald Pfeifer writes: > Note that in HTML < and > have a special meaning, so we cannot simply > write "<* noreturn *>", but need to escape it as "<* noreturn *>". > > Pushed. > > Gerald apologies and thanks for correcting this bug regards, Gaius

Re: [wwwdocs] document modula-2 in gcc-13/changes.html (and index.html)

Gerald Pfeifer writes: > Hi Gaius, > > apologies, I thought you had pushed the updated patch and only now > realized it's not in yet. > > Please look into the few bits below and then go ahead and push. > > On Mon, 6 Feb 2023, Gaius Mulley wrote: >> * htdo

[PATCH] Allow front ends to register spec functions gcc/{gcc.cc,gcc.h} [PR108261]

Hello, bootstrapped on gcc master x86_64 and no extra failures generated on all front ends. Would this be ok for trunc? regards, Gaius Allow front ends to register spec functions gcc/{gcc.cc,gcc.h} [PR108261] This patch allows front ends to register spec functions. It is motivated by PR108

[PATCH] Allow front ends to register spec functions gcc/{gcc.cc,gcc.h} [PR108261]

Hello, bootstrapped on gcc master x86_64 and no extra failures generated on all front ends. Would this be ok for trunc? regards, Gaius Allow front ends to register spec functions gcc/{gcc.cc,gcc.h} [PR108261] This patch allows front ends to register spec functions. It is motivated by PR108

Re: [wwwdocs] document modula-2 in gcc-13/changes.html (and index.html)

Gerald Pfeifer writes: > On Fri, 3 Feb 2023, Gaius Mulley wrote: >> The following patch provides a summary of the modula-2 front end >> and also contains links to the online modula-2 documentation in >> index.html. > >> +Modula-2 >> + >> + Support for

[wwwdocs] document modula-2 in gcc-13/changes.html (and index.html)

Hello, The following patch provides a summary of the modula-2 front end and also contains links to the online modula-2 documentation in index.html. [I'm just about to git push fixes so that modula-2 builds html, info and pdf documentation into the standard directories.] regards, Gaius dif

Re: For Modula-2 build-tree testing, also set up paths to compiler libraries

Thomas Schwinge writes: > Hi! > > On 2022-10-10T16:31:26+0100, Gaius Mulley via Gcc-patches > wrote: >> Here are the dejagnu expect library scripts for the gm2 >> testsuite. > > This (or some variant thereof; haven't checked

Re: [PATCH] modula2/108462 - duplicate install of static modula2 target libs

Richard Biener writes: > On Mon, 23 Jan 2023, Richard Biener wrote: > >> The following addresses the fact that libgm2 installs static libraries >> into two places, one performed by >> >> toolexeclib_LTLIBRARIES = libm2cor.la >> >> and one performed as part of the install-data-local rule to a >>

Re: [COMMITTED] testsuite: Restore TORTURE_OPTIONS in gm2/warnings/returntype/fail/warnings-returntype-fail.exp

Rainer Orth writes: > Between 20230126 and 20230127, 1100+ gm2 tests started to FAIL for the > non-default multilib, e.g. > > FAIL: gm2/calling-c/datatypes/unbounded/run/pass/m.mod compilation, > {additional_flags= -O0 -g -Werror=return-type } timeout=10 > UNRESOLVED: gm2/calling-c/datatypes/unb

Re: [PATCH] modula-2: Fixes for preprocessing [PR102343, PR108182]

Iain Sandoe writes: > Tested on x86_64, powerpc64 - linux-gnu (with 32b multilibs), > i686, powerpc darwin (with 64b multilibs) x86_64 darwin (with and without > 32b multilib). > OK for trunk? > thanks > Iain > > --- 8< --- > > Modula-2 uses the C preprocessor to implement handling for conditiona

Re: [PATCH] modula-2: Fix stack size request in initPreemptive [PR108405]

Iain Sandoe writes: > Given that, currently, this value is not configurable per target the > short-term solution is to avoid a bad request. > > Tested on x86_64-darwin21, OK for trunk? > thanks > Iain Hi Iain, yes this is fine. LGTM - thanks regards, Gaius

Re: [PATCH] modula-2, driver, Front end: Revise handling of I and L paths [PR108182].

Iain Sandoe writes: > Tested on x86_64-linux-gnu (with a 32b multilib), powerpc, i686 and > x86_64-darwin. OK for trunk? > thanks, > Iain > > --- 8< --- > > The adds the includes in the FE as done in other GCC languages. > It also revises the library handling to avoid additional -L options > fro

Re: [PATCH] Modula-2, testsuite: Remove use of concatenated paths.

Iain Sandoe writes: > tested on x86_64-darwin21 with an updated compiler that does not support > the path concatentation and on an unpatched trunk. > OK for trunk? > thanks > Iain > > --- 8< --- > > The original implementation for Modula-2 search paths allows things like > '-I/path/a:/path/b'. S

Re: [PATCH] modula2/108144 - Fix multilib install of libgm2

Richard Biener writes: > The following adjusts libgm2 to properly use the multilib build > infrastructure, thereby fixing the install with > --enable-version-specific-runtime-libs > > In particular config-ml.pl needs to be applied to generated Makefiles > as documented in the manual and we have t

Re: [PATCH] modula-2, testsuite: Make libs and interfaces consistent.

Iain Sandoe writes: > Tested on x86_64-linux-gnu (with a 32b multilib), powerpc, i686 and > x86_64-darwin. OK for trunk? > thanks, > Iain LGTM, thank you regards, Gaius

Re: [PATCH] modula2/108144 - fix --enable-version-specific-runtime-libs

Richard Biener writes: > The following fixes --enable-version-specific-runtime-libs for > the modula2 target libraries. The issue is that the install > happens via for example > > toolexeclib_LTLIBRARIES = libm2cor.la > > and toolexeclibdir is set to $(toolexecdir)/$(gcc_version)$(MULTISUBDIR) >

Re: [PATCH] contrib: Partial fix for failed update-copyright --this year [PR108413]

Jakub Jelinek writes: > Hi! > > As mentioned on IRC or in PR108413, the last update-copyright.py --this year > failed and that is why we are in a strange state where some copyrights have > been updated and others have not. > The full list of errors I got was I think: > gcc/m2/mc-boot/GmcOptions.c

Re: [PATCH] modula-2: Handle pass '-v' option to the compiler.

Iain Sandoe writes: > Tested on x86-64-darwin21. > OK for trunk? > Iain yes LGTM, thanks, Gaius > --- 8< --- > > Somehow this setting had been missed, and we really need the verbose > flag to enable useful debug output. > > Signed-off-by: Iain Sandoe > > gcc/m2/ChangeLog: > > * gm2-gcc/

Re: [PATCH v3 17/19] modula2 front end: dejagnu expect library scripts

Jason Merrill writes: > On 12/6/22 09:47, Gaius Mulley via Gcc-patches wrote: >> Here are the dejagnu expect library scripts for the gm2 >> testsuite. > > A couple of weeks ago I noticed on a testrun that the modula tests > didn't seem to be timing out properl

Re: [PATCH, Modula2] PR-108142 Many empty directories created in the build directory

Jakub Jelinek writes: > On Tue, Jan 10, 2023 at 11:16:28AM +0100, Richard Biener via Gcc-patches > wrote: >> > @@ -424,7 +388,7 @@ override PLUGINCFLAGS := $(filter-out >> > -mdynamic-no-pic,$(PLUGINCFLAGS)) >> > >> > plugin/m2rte$(soext): $(srcdir)/m2/plugin/m2rte.cc >> > $(GCC_HEADER_DEPEND

Re: Missing dependencies in m2/ ?

Jeff Law writes: > I'm still seeing it as of about 2 hours ago: > > http://law-sandy.freeddns.org:8080/job/avr-elf/2125/console > > A good run (yesterday): > > http://law-sandy.freeddns.org:8080/job/avr-elf/2124/console > > Hi Jeff, many thanks for the urls above - useful I'll attempt to reprodu

[PATCH, Modula2] PR-108142 Many empty directories created in the build directory

PR-108142 Modula-2 configure generates many subdirectories in the top build directory. This patch dynamically creates subdirectories under gcc/m2 if and when required. Bootstrapped on x86_64 gnu/linux, ok for master? regards, Gaius gcc/m2/ChangeLog: * Make-lang.in (GM2_1): Change -B

Re: [PATCH] Modula-2: fix documentation layout

Eric Botcazou writes: > Hi, > > the Modula-2 documentation is rejected by older versions of Makeinfo because > the web of @node markers is fairly broken, apparently some subsections were > moved around, most notably between the Overview and Using sections, and the > @node markers were not (pro

Re: Missing dependencies in m2/ ?

Jeff Law via Gcc-patches writes: > I've been getting sporatic errors like this since the introduction of > the modula-2 front-end: > >> In file included from ../../..//gcc/gcc/m2/mc-boot/GSFIO.c:29: >> ../../..//gcc/gcc/system.h:556:20: error: conflicting declaration of C >> function 'const char

[PATCH, modula2] v2 PR-108182 gm2 driver mishandles target and multilib options

Hi, This is version 2 with the SKIPOPT applied to every option given to save_switch. ok for master? Bootstrapped on x86_64 gnu/linux and darwin. regards, Gaius PR-108182 gm2 driver mishandles target and multilib options here are some patches which attempt to allow target specific include

Re: [PATCH] modula-2, libm2min: Declare abort and exit as expected.

Iain Sandoe writes: > Tested with a build on x86_64-darwin21, the build warnings are gone and > the build completed as normal. > OK for master? > thanks > Iain > > --- 8< --- > > The build is currently emitting a warning that abort() is declared differently > from the built-in. This updates the

[PATCH, modula2] PR-108182 gm2 driver mishandles target and multilib options

ChangeLog entry follows: regards, Gaius [PATCH, modula2] PR-108182 gm2 driver mishandles target and multilib options here are some patches which attempt to allow target specific include paths and library paths in the gm2 driver. I admit that the patch has flaws in that it only processes op

[PATCH, modula2] PR-108182 gm2 driver mishandles target and multilib options

Hi, here are some patches which attempt to allow target specific include paths and library paths in the gm2 driver. I admit that the patch has flaws in that it only processes options -f, -m in the lang_specific_driver. [Called after driver::set_up_specs but before read_specs is called]. I susp

Re: [PATCH] modula-2: Remove uses of scalb*() and significand*() [PR107631]

Iain Sandoe writes: > Tested on x86_64-darwin21 and x86_64-linux-gnu without any m2 regressions. > OK for trunk? > thanks > Iain yes sure LGTM thank you! Gaius

Re: [PATCH] modula-2, driver: Implement handling for -static-libgm2.

Iain Sandoe writes: > Hi Gaius, > >> On 4 Jan 2023, at 12:11, Gaius Mulley wrote: >> >> Iain Sandoe writes: >> >>> tested on x86_64-linux-gnu, x86_64,aarch64-darwin21, > >> >> yes LGTM - it was unimplemented - thanks! > > My apolo

Re: [PATCH 4/n] modula-2, driver: Handle static-libstd++ for targets without static/dynamic

Iain Sandoe writes: > Oops pressed ‘send' too soon - this is part of the series for Darwin: > > There are several modula-2 issues on Darwin, some blocking bootstrap on > one or more system versions. > > This has been tested on powerpc/i688-darwin9 .. x86_64-darwin10,17,21 and > the prototype aarc

[PATCH modula2] Add missing declarations to gcc/m2/gm2-libs-min/M2RTS.def

version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU Ge

Re: [PATCH] modula-2, driver: Implement handling for -static-libgm2.

Iain Sandoe writes: > tested on x86_64-linux-gnu, x86_64,aarch64-darwin21, > OK for trunk? > thanks, > Iain > > --- 8< --- > > This was unimplemented so far. > > gcc/ChangeLog: > > * common.opt: Add -static-libgm2. > * config/darwin.h (LINK_SPEC): Handle static-libgm2. > > gcc/m2/Chan

Re: [PATCH] modula-2: Module registration constructors need to be visible [PR108259].

Iain Sandoe writes: > Tested on x86_64-linux-gnu, x86_64,aarch64-darwin21. > There remain issues with shared libraries, but the link fails are fixed > by this. > > OK for master? > Thanks > Iain > > --- 8< --- > > In the current design the main executable links explicitly to the module > registra

Re: [PATCH] modula-2: Fix registration of modules via constructors [PR108183].

Iain Sandoe writes: > When I first made this patch I had a question as to what should be > done for registration CTORs generated by the compiler for .mod files. > I've now answered that question (the code that makes the GCC decl > has also be updated in a separately posted patch). > > teste

Re: [PATCH] modula-2, doc: Build dvi, ps and pdf doc in the gcc/doc directory.

Iain Sandoe writes: > Tested on darwin21 with "make m2.pdf" and "make m2.dvi". > OK for trunk? > thanks. > Iain > > --- 8< --- > > This also uses the configured $(TEXI2DVI) and $(TEXI2PDF) to deal with those > targets (since we cannot assume to know what the user might have installed). > > gcc/m2

Re: [PATCH] Modula-2, testsuite: No 96 bit floating type on Darwin.

Iain Sandoe writes: > Tested on x86_64 and aarch64 Darwin, > OK for master? > thanks > Iain > > --- 8< --- > > The realbitscast.mod is currently failing on x86_64 and aarch64 > Darwin since they do not have a 96b floating type. Disable the > type for all Darwin arches. > > gcc/testsuite/ChangeL

Re: [PATCH 3/n] modula2: Ensure that module registration constructors are 'extern' [PR108183].

Iain Sandoe writes: > There are several modula-2 issues on Darwin, some blocking bootstrap on > one or more system versions. > > This has been tested on powerpc/i688-darwin9 .. x86_64-darwin10,17,21 and > the prototype aarch64-darwin branch on darwin21. > > OK for trunk? > thanks > Iain > > NOTE:

Re: [PATCH modula2] PR-108142 Remove empty directories created in the build directory

heck cxxobj1 having been rebuilt by the relevant g++/gcc. I propose to rework the patch (renaming the directories) and ensure that Canadian cross works (currently it doesn't) - although it was tested for classic cross (build == host != target). regards, Gaius >> On 26 Dec 2022, at 1

[PATCH modula2] PR-108142 Remove empty directories created in the build directory

Hello, Bootstrapped on amd64 GNU/Linux. Ok for trunk? regards, Gaius --- 8< --- PR-108142 Remove empty directories created in the build directory. This patch removes empty directories created in the build directory. Directories are only created if required and all modula-2 build output is c

Re: [PATCH] modula2: PR-108119 Disable m2 plugin m2rte

Rainer Orth writes: > Hi Gaius, > > Btw., you've got a couple of formatting errors in your ChangeLog entires: > ChangeLog: * Makefile.def (extra_configure_flags): Add @enable_m2plugin@. * Makefile.in : Rebuilt. > ^ no blank here. Besid

Re: [PATCH] modula2: PR-108119 Disable m2 plugin m2rte

Richard Biener writes: >> >> ChangeLog: >> >> * Makefile.def (extra_configure_flags): Add @enable_m2plugin@. >> * Makefile.in : Rebuilt. >> * configure : Rebuilt. >> * configure.ac (host_tools): Remove unused gm2tools. >> (m2plugin) New AC_ARG_ENABLE. >>

Re: [committed] modula2: Fix lto profiledbootstrap on powerpc64le-linux and s390x-linux [PR108153]

Jakub Jelinek writes: > Hi! > > Lto profiledbootstrap was failing for me on {powerpc64le,s390x}-linux with > modula 2 enabled, with: > cc1gm2: internal compiler error: the location value is corrupt > 0x11a3d2d m2assert_AssertLocation(unsigned int) > ../../gcc/m2/gm2-gcc/m2assert.cc:40 > 0

[PATCH] modula2: PR-108119 Disable m2 plugin m2rte

Hello, PR-108119 Disable m2 plugin m2rte (provide --enable-m2plugin configure option). The m2 plugin m2rte attempts to find reachable calls to the m2 exception handler, but it identifies the m2 exception calls by procedure name. As this won't work with other languages it should be disabled by d

Re: [PATCH] modula2: Don't treat % in Modula 2 messages specially

Jakub Jelinek writes: > Hi! > > On top of the just posted patch, this patch makes sure that > any % chars in message strings aren't treated as format chars. > None of these functions take variable number of arguments, so for > most of format specifiers there is nowhere to take arguments from, > i

Re: [PATCH] modula2: Fix up bootstrap on powerpc64le-linux [PR108147]

Jakub Jelinek writes: > Hi! > > As mentioned in the PR, bootstrap with m2 enabled currently fails > on powerpc64le-linux, we get weird ICE after printing some diagnostics. > The problem is that mc creates from *.def prototypes like > extern void m2linemap_WarningAtf (m2linemap_location_t location

[PATCH, m2]: PR-108122 Reduce sleep times in gm2/pimcoroutines/run/pass/testtime.mod

PR-108122 Reduce sleep times in gm2/pimcoroutines/run/pass/testtime.mod Change time unit to 1 jiffy (with respect to TimerHandler.def) rather than a second. gcc/testsuite/ChangeLog: * gm2/pimcoroutines/run/pass/testtime.mod: Reduce sleep times in the test by a factor of 25. Te

[PATCH, m2] Add missing m2.stage{profile,feedback} to Make-lang.in

Add missing profile and feedback hooks consistent with all other frontends. gcc/m2/ChangeLog: * Make-lang.in (m2.stageprofile): Added. (m2.stagefeedback) Added. Bootstrapped and tested using configure --with-build-config=bootstrap-lto-lean --enable-languages=m2 make profiledbo

[committed, pushed] PR-107607 m2: Remove bdepend on realpath, cut and echo

It can be replaced by a subshell'd cd just fine. (cd gcc/m2; autoconf-2.69) gcc/m2/ChangeLog: * configure.ac: Stop probing for realpath. * tools-src/calcpath: Break dependency on realpath, cut and echo. * configure: Rebuilt --- gcc/m2/config

Re: [PATCH v5 1/19] modula2 front end: Fixes, improvements detecting python3 and documentation generation (shorter).

Jakub Jelinek writes: > On Wed, Dec 14, 2022 at 08:35:07AM +0000, Gaius Mulley via Gcc-patches wrote: >> thanks - this is the last patch tick. So I'll actually do the merge now :-) > > I've committed following patch to fix up formatting of ChangeLog entries. Many

Re: [PATCH v5 1/19] modula2 front end: Fixes, improvements detecting python3 and documentation generation (shorter).

Richard Biener writes: > On Wed, Dec 14, 2022 at 8:48 AM Gaius Mulley wrote: >> >> >> >> This patch set adds a re-exp ACX_CHECK_PROG_VER to detect python3. >> HAVE_PYTHON is then checked in gcc/m2/Make-lang.in to generate library >> chapters if python3 i

Re: [PATCH v4 1/19] modula2 front end: changes outside gcc/m2, libgm2 and gcc/testsuite.

Richard Biener writes: > On Sat, Dec 10, 2022 at 1:49 AM Gaius Mulley via Gcc-patches > wrote: >> >> >> While writing the ChangeLog entries git gcc-verify spotted an oversight >> with v3 of this patch set. I had forgotten to post gm2.texi and also a >> t

Re: [PATCH v5 1/19] modula2 front end: changes outside gcc/m2, libgm2 and gcc/testsuite. Addendum.

Richard Biener writes: > On Sat, Dec 10, 2022 at 12:29 PM Gaius Mulley via Gcc-patches > wrote: >> >> >> This patch contains a replacement doc/gm2.texi which uses American >> spelling, all references to gm2 outside the gcc tree purged and the >> News/

[PATCH v5 1/19] modula2 front end: changes outside gcc/m2, libgm2 and gcc/testsuite. Addendum.

evelopment +@direntry +* gm2: (gm2). A GCC-based compiler for the Modula-2 language +@end direntry +@end format + +@insertcopying +@end ifinfo + +@titlepage +@title The GNU Modula-2 Compiler +@versionsubtitle +@author Gaius Mulley + +@page +@vskip 0pt plus 1filll +Published by the Free Software

[PATCH v4 1/19] modula2 front end: changes outside gcc/m2, libgm2 and gcc/testsuite.

anguage +@end direntry +@end format + +@insertcopying +@end ifinfo + +@titlepage +@title The GNU Modula-2 Compiler +@versionsubtitle +@author Gaius Mulley + +@page +@vskip 0pt plus 1filll +Published by the Free Software Foundation @* +51 Franklin Street, Fifth Floor@* +Boston, MA 02110-1301, USA@* +@s

  1   2   3   >