[pushed] ICE when attempting to fold HIGH from an unbounded array in a nested procedure [PR modula2/121354]

function. Signed-off-by: Gaius Mulley --- gcc/m2/gm2-compiler/M2GenGCC.mod | 51 +--- 1 file changed, 33 insertions(+), 18 deletions(-) diff --git a/gcc/m2/gm2-compiler/M2GenGCC.mod b/gcc/m2/gm2-compiler/M2GenGCC.mod index 2507c536181..2440b2acf66 100644 --- a/gcc/m2/gm2

[pushed] PR modula2/121314: quotes appearing in concatenated error strings

. * gm2/errors/fail/badindrtype2.mod: New test. Signed-off-by: Gaius Mulley --- gcc/m2/mc-boot/GFormatStrings.cc | 4 +- gcc/m2/mc-boot/GM2EXCEPTION.cc| 6 +- gcc/m2/mc-boot/GSFIO.cc | 20 ++ gcc/m2/mc-boot/GSFIO.h

[pushed][modula2] Tidyup remove unused local variables

. Signed-off-by: Gaius Mulley --- gcc/m2/gm2-compiler/M2GenGCC.mod | 5 - gcc/m2/gm2-compiler/M2Range.mod | 2 -- 2 files changed, 7 deletions(-) diff --git a/gcc/m2/gm2-compiler/M2GenGCC.mod b/gcc/m2/gm2-compiler/M2GenGCC.mod index 4a9ced32dcf..2507c536181 100644 --- a/gcc/m2/gm2-compiler

[pushed] PR modula2/121289 Poor warning location when using Wstyle option

: Gaius Mulley --- gcc/m2/gm2-compiler/M2Students.def| 2 +- gcc/m2/gm2-compiler/M2Students.mod| 16 --- gcc/m2/gm2-compiler/P2SymBuild.mod| 2 +- gcc/m2/gm2-libs-iso/LowLong.mod | 10 ++--- gcc/m2/gm2-libs-iso/LowReal.mod | 14

[pushed-15][PATCH] [modula2] Add return to remove build warning

This patch adds a return statement to M2Exception which removes a build warning. gcc/m2/ChangeLog: * gm2-libs/M2EXCEPTION.mod (M2Exception): Add return exException in case Raise completes. (cherry picked from commit db8b92d8d61de408e14a4aebf5a34936699d) Signed-off-by: Gaius

[pushed-15][PATCH] PR modula2/121164: Modula 2 build failure followup

picked from commit ab5a89c0b4f1ead202dee072e16690607b810111) Signed-off-by: Gaius Mulley --- gcc/m2/gm2-compiler/P1SymBuild.mod| 45 +- gcc/m2/gm2-compiler/P2SymBuild.mod| 26 +++--- gcc/m2/gm2-compiler/P3SymBuild.mod| 2 +- gcc/m2/gm2-compiler

[pushed][PATCH] [modula2] Add return to remove build warning

This patch adds a return statement to M2Exception which removes a build warning. gcc/m2/ChangeLog: * gm2-libs/M2EXCEPTION.mod (M2Exception): Add return exException in case Raise completes. Signed-off-by: Gaius Mulley --- gcc/m2/gm2-libs/M2EXCEPTION.mod | 5 +++-- 1 file

[pushed-15] [PATCH] PR modula2/121164 Modula 2 build failure

This patch fixes the 2nd parameter name mismatch in ARRAYOFCHAR.mod. gcc/m2/ChangeLog: PR modula2/121164 * gm2-libs/ARRAYOFCHAR.mod (Write): Rename 2nd parameter name a to str. (cherry picked from commit 22d8b89689769e5efefd2c4e6dda88d9f0b2a945) Signed-off-by: Gaius

[pushed] [PATCH] PR modula2/121164 Modula 2 build failure

This patch fixes the 2nd parameter name mismatch in ARRAYOFCHAR.mod. gcc/m2/ChangeLog: PR modula2/121164 * gm2-libs/ARRAYOFCHAR.mod (Write): Rename 2nd parameter name a to str. Signed-off-by: Gaius Mulley --- gcc/m2/gm2-libs/ARRAYOFCHAR.mod | 6 +++--- 1 file changed

[pushed-15] [PATCH] PR modula2/120912: Request for a procedure to obtain a file from an IOChan

. (cherry picked from commit 15670d4477ce219c017bd52417a6074b981fb197) Signed-off-by: Gaius Mulley --- gcc/m2/gm2-libs-iso/IOChanUtils.def | 8 gcc/m2/gm2-libs-iso/IOChanUtils.mod | 12 +++- 2 files changed, 19 insertions(+), 1 deletion(-) diff --git a/gcc/m2/gm2-libs-iso

[pushed-15] [PATCH] [modula2] Comment tidyup in gm2-compiler/M2GCCDeclare.mod

7a7cc65b8987b9b05fb8fb75824e2000861e6c30) Signed-off-by: Gaius Mulley --- gcc/m2/gm2-compiler/M2GCCDeclare.mod | 16 ++-- 1 file changed, 6 insertions(+), 10 deletions(-) diff --git a/gcc/m2/gm2-compiler/M2GCCDeclare.mod b/gcc/m2/gm2-compiler/M2GCCDeclare.mod index be80695d3e8..860a89ac8b0 100644 --- a/gcc

[pushed-15] [PATCH] PR modula2/120673: Mutually dependent types crash the compiler

: Gaius Mulley --- gcc/m2/gm2-compiler/M2GCCDeclare.mod| 94 +++-- gcc/m2/gm2-compiler/M2MetaError.mod | 37 +++- gcc/m2/gm2-compiler/P2SymBuild.mod | 31 --- gcc/testsuite/gm2/pim/fail/badmodvar.mod| 7 ++ gcc/testsuite/gm2/pim/fail/cyclictypes.mod

[pushed-15] [PATCH] PR modula2/120606: FOR loop ICE if the last expression uses an array

end expressions e1 and e2 respectively. gcc/testsuite/ChangeLog: PR modula2/120606 * gm2/pim/pass/forarray.mod: New test. (cherry picked from commit 639a147414ab2b870f9482123fcaa1821e0d5475) Signed-off-by: Gaius Mulley --- gcc/m2/gm2-compiler/M2Quads.mod | 2 ++ gcc

[pushed-15] [PATCH] [PR modula2/119650, PR modula2/117203]: WriteString and Delete are missing from base libraries

/testwritechar.mod: New test. (cherry picked from commit d1c3cfa3296ae5010c514d67f57acf144a299c7a) Signed-off-by: Gaius Mulley --- gcc/m2/gm2-compiler/P2Build.bnf | 79 +++--- gcc/m2/gm2-compiler/P3Build.bnf | 99 gcc/m2/gm2-compiler/PCBuild.bnf

[pushed-15] [PATCH] PR modula2/120497: error is generated for good code when returning a pointer var variable

. * gm2/pim/pass/ReturnType2.mod: New test. (cherry picked from commit 170717fa243ef466a99498113167627539af4553) Signed-off-by: Gaius Mulley --- gcc/m2/gm2-compiler/M2Range.mod| 64 -- gcc/testsuite/gm2/pim/pass/ReturnType.mod | 17 ++ gcc/testsuite/gm2/pim

[pushed-15] [PATCH] PR modula2/120389 Assigning wrong type to an array causes an ICE

/testcharint.mod: New test. * gm2/pim/fail/testindrx.mod: New test. * gm2/pim/pass/testxindr.mod: New test. * gm2/pim/pass/testxindr2.mod: New test. * gm2/pim/pass/testxindr3.mod: New test. Signed-off-by: Gaius Mulley --- gcc/m2/gm2-compiler/M2Check.def| 3 +- gcc

[pushed-15] [PATCH] PR modula2/120389 ICE if assigning a constant char to an integer array

is assignment compatible with the type of right. gcc/testsuite/ChangeLog: PR modula2/120389 * gm2/iso/fail/badarray3.mod: New test. (cherry picked from commit 895a8abad245365940939911e3d0de850522791e) Signed-off-by: Gaius Mulley --- gcc/m2/gm2-compiler/M2GenGCC.mod

[committed] PR modula2/120253: Error message column numbers should start at 1 not 0

. (m2flex_GetColumnNo): Return FIRST_COLUMN if currentLine is NULL. (m2flex_GetLineNo): Rewrite for positive logic. (m2flex_GetLocation): Ditto. (cherry picked from commit 9a485b83e177cb742be17faf20ac5cc7db14fee3) Signed-off-by: Gaius Mulley --- gcc/m2/m2.flex | 25

[PATCH] [PR modula2/120761] GM2_FOR_BUILD is not substituted in the toplevel Makefile

Ok for master? Bootstrapped on x86_64 gnu/linux regards, Gaius --- [PR modula2/120761] GM2_FOR_BUILD is not substituted in the toplevel Makefile This patch removes the unused GM2_FOR_BUILD in the toplevel Makefile.tpl and removes any reference to GM2_FOR_BUILD in libgm2. The only reference

Re: [committed] modula2: Fix a comment typo

Jakub Jelinek writes: > Hi! > > During make gcc.pot I've noticed among tons of other warnings (e.g. because > can't appears in non-C/C++ style comment and so gettext considers it > unterminated) a warning where the lack of " looked unintentional. > > Committed to trunk as obvious. > > 2025-04-09

Re: [PATCH] testsuite, gm2: Use -B option for libstdc++ where required.

Iain Sandoe writes: > TL;DR without this change we get circa 2500 new fails in the gm2 testsuite > when PR119172 is fixed. > > Recent increases in security constraints on macOS14+ (resulting in PR119172) > mean that the system will not fall back to using alternate shared libs than > the ones actu

Re: [PATCH v2 7/7] Alpha: Add option to avoid data races for partial writes [PR117759]

"Maciej W. Rozycki" writes: ... > There are notable regressions between a plain `-mno-bwx' configuration > and a `-mno-bwx -msafe-partial' one: > > FAIL: gm2/iso/run/pass/strcons.mod execution, -g > FAIL: gm2/iso/run/pass/strcons.mod execution, -O > FAIL: gm2/iso/run/pass/strcons.mod executi

Re: [PATCH v2 7/7] Alpha: Add option to avoid data races for partial writes [PR117759]

Jeff Law writes: > On 1/6/25 6:03 AM, Maciej W. Rozycki wrote: ... >> There are notable regressions between a plain `-mno-bwx' >> configuration >> and a `-mno-bwx -msafe-partial' one: >> FAIL: gm2/iso/run/pass/strcons.mod execution, -g >> FAIL: gm2/iso/run/pass/strcons.mod execution, -O >> FAI

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

Wilken Gottwalt writes: > Fix identifier names to be too similar to Modula-2 keywords and causing > warnings coming from Modula-2's own libraries. > > m2/m2log/InOut.mod:51:18: note: In implementation module ‘InOut’: > either the identifier has the same name as a keyword or alternatively a > keyw

Re: [PATCH] Remove the rest of INCLUDE_MEMORY

Andrew Pinski writes: > I missed these in r15-5603-gb3f1b9e2aa079f8ec73e because they > were no in a `.cc` or `.h` file or they were outside of the gcc > subdirectory. > > Bootstrapped and tested on x86_64-linux-gnu. > > gcc/m2/ChangeLog: > > * mc/keyc.mod: Don't print `#define INCLUDE_MEMO

Re: [PATCH] build: Remove INCLUDE_MEMORY [PR117737]

Andrew Pinski writes: > Since diagnostic.h is included in over half of the sources, requiring to > `#define INCLUDE_MEMORY` > does not make sense. Instead lets unconditionally include memory in system.h. > > The majority of this patch is just removing `#define INCLUDE_MEMORY` from the > sources

Re: [PATCH] m2: Fix up dependencies some more

Jakub Jelinek writes: > Hi! > > Anyway, bootstrapped/regtested successfully on x86_64-linux and i686-linux, > ok for trunk? That doesn't mean all dependencies are correct, just that > this change didn't make things worse. > > 2024-11-06 Jakub Jelinek > > gcc/m2/ > * Make-lang.in (m2_OBJ

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

  1   2   3   >