Re: Question on -fwrapv and -fwrapv-pointer

Qing Zhao writes: > Even though unsigned integer overflow is well defined, it might be > unintentional, shall we warn user about this? This would be better addressed by providing operators or functions that do overflow checking in the language, so that they can be explicitly used where

Re: [Patch] contrib/gcc-changelog: Check whether revert-commit exists

Tobias Burnus writes: > Attached an old patch. See attached patch for the current one. > > Difference is one line: the warning that is shown in the example output > below. Python-wise, the changes seem fine. Unsure if it does the right thing, though, since I'm not familiar with the full

Re: [PATCH] libstdc++ Add cstdarg to freestanding

Jonathan Wakely writes: > On Fri, 21 Jul 2023 at 22:23, Paul M. Bendixen via Libstdc++ > wrote: >> >> P1642 includes the header cstdarg to the freestanding implementation. >> This was probably left out by accident, this patch puts it in. >> Since this is one of the headers that go in whole

Re: [PATCH v2 2/2] libstdc++: Replace all manual FTM definitions and use

Jonathan Wakely writes: > [..snip..] > Thanks for adding the comments like "// C++ < 20". > > I think in the comment on the #endif can be just __cpp_lib_any > rather than defined(__cpp_lib_any). Similarly for > __cpp_lib_atomic_float in . Oh, and __cpp_lib_atomic_ref. And > in , and several

Re: [PATCH v2 1/2] libstdc++: Implement more maintainable header

Arsen Arsenović writes: > This commit replaces the ad-hoc logic in with an AutoGen > database that (mostly) declaratively generates a version.h bit which > combines all of the FTM logic across all headers together. > > This generated header defines macros of the form __glibcxx_foo, >

[PATCH v2 1/2] libstdc++: Implement more maintainable header

This commit replaces the ad-hoc logic in with an AutoGen database that (mostly) declaratively generates a version.h bit which combines all of the FTM logic across all headers together. This generated header defines macros of the form __glibcxx_foo, equivalent to their __cpp_lib_foo variants,

Re: [PATCHv2] Use toplevel configure for GMP and MPFR for gdb

Matthias Klose via Gcc-patches writes: > On 10.11.22 20:05, apinski--- via Binutils wrote: >> From: Andrew Pinski >> This patch uses the toplevel configure parts for GMP/MPFR for >> gdb. The only thing is that gdb now requires MPFR for building. >> Before it was a recommended but not required

Re: [PATCH 00/24] Sync shared build infrastructure with binutils-gdb

Joseph Myers writes: > On Tue, 8 Aug 2023, Arsen Arsenović via Gcc-patches wrote: > >> Yes. Libtool was forked over a decade ago. My next project is syncing >> upstream and us back up. Unsure about pkg.m4. > > Note as per previous discuss

Re: [PATCH 00/24] Sync shared build infrastructure with binutils-gdb

Eric Gallager writes: > Hi, with the updates to libtool.m4 and pkg.m4, those files originally > come from upstream libtool and pkg-config, correct? Won't patching > GCC's local copies make re-syncing them with upstream > libtool/pkg-config more difficult, or have these patches already been >

Re: [PATCH 24/24] toplevel: Makefile.def: add install-strip dependency on libsframe

Jeff Law writes: > OK. > > I know I skipped over a few. I'll try to get those wrapped up shortly. Thanks, I've pushed all of those (since I received all the reviews). > While I strongly dislike git submodules, this may be one of those occasions > where their use might be justified.

Re: [PATCH 2/2] libstdc++: Replace all manual FTM definitions and use

Jonathan Wakely writes: > On Sat, 29 Apr 2023 at 11:24, Arsen Arsenović via Libstdc++ < > libstd...@gcc.gnu.org> wrote: > >> libstdc++-v3/ChangeLog: >> >> * libsupc++/typeinfo: Switch to bits/version.h for >> __cpp_lib_constexpr_typeinfo. >> >> > Does this change have an impact

Re: [PATCH 2/2] libstdc++: Replace all manual FTM definitions and use

Jonathan Wakely writes: > On Sat, 29 Apr 2023 at 11:24, Arsen Arsenović via Libstdc++ < > libstd...@gcc.gnu.org> wrote: > >> libstdc++-v3/ChangeLog: >> >> * libsupc++/typeinfo: Switch to bits/version.h for >> __cpp_lib_constexpr_typeinfo. >> >> > Does this change have an impact

Re: [PATCH 1/2] libstdc++: Implement more maintainable header

Jonathan Wakely writes: > [...snip...] > It looks like this will regenerate the bits/version.h file if it's older > than the definitions or the autogen template, right? > > Generally we don't want to touch anything in the source tree as part of a > normal build. It's OK to do that when

[PATCH 22/24] libtool.m4: augment symcode for Solaris 11

From: Nick Alcock This reports common symbols like GNU nm, via a type code of 'C'. ChangeLog: * libtool.m4 (lt_cv_sys_global_symbol_pipe): Augment symcode for Solaris 11. gcc/ChangeLog: * configure: Regenerate. libatomic/ChangeLog: * configure: Regenerate.

[PATCH 19/24] Deprecate a.out support for NetBSD targets.

From: John Ericson As discussed previously, a.out support is now quite deprecated, and in some cases removed, in both Binutils itself and NetBSD, so this legacy default makes little sense. `netbsdelf*` and `netbsdaout*` still work allowing the user to be explicit about there choice.

[PATCH 14/24] libtool.m4: fix nm BSD flag detection

From: Nick Alcock Libtool needs to get BSD-format (or MS-format) output out of the system nm, so that it can scan generated object files for symbol names for -export-symbols-regex support. Some nms need specific flags to turn on BSD-formatted output, so libtool checks for this in its

[PATCH 24/24] toplevel: Makefile.def: add install-strip dependency on libsframe

From: Indu Bhagat As noted in PR libsframe/30014 - FTBFS: install-strip fails because bfdlib relinks and fails to find libsframe, the install time dependencies of libbfd need to be updated. ChangeLog: * Makefile.def: Reflect that libsframe needs to installed before libbfd.

[PATCH 23/24] bfd: linker: merge .sframe sections

From: Indu Bhagat The linker merges all the input .sframe sections. When merging, the linker verifies that all the input .sframe sections have the same abi/arch. The linker uses libsframe library to perform key actions on the .sframe sections - decode, read, and create output data. This

[PATCH 16/24] Add support for the haiku operating system

From: Alexander von Gluck IV These are the os support patches we have been grooming and maintaining for quite a few years over on git.haiku-os.org. All of these architectures are working and most have been stable for quite some time. ChangeLog: * configure: Regenerate. *

[PATCH 13/24] configure: reinstate 32b PA-RISC HP-UX target in toplevel

The Binutils still support this target. ChangeLog: * configure.ac: Reinstate 32b PA-RISC HP-UX targets * configure: Regenerate. --- configure| 10 ++ configure.ac | 10 ++ 2 files changed, 20 insertions(+) diff --git a/configure b/configure index

[PATCH 15/24] libtool.m4: fix the NM="/nm/over/here -B/option/with/path" case

From: Nick Alcock My previous nm patch handled all cases but one -- if the user set NM in the environment to a path which contained an option, libtool's nm detection tries to run nm against a copy of nm with the options in it: e.g. if NM was set to "nm --blargle", and nm was found in /usr/bin,

[PATCH 10/24] gprofng: a new GNU profiler

From: Vladimir Mezentsev ChangeLog: * Makefile.def: Add gprofng module. * configure.ac: Add --enable-gprofng option. * Makefile.in: Regenerate. * configure: Regenerate. include/ChangeLog: * collectorAPI.h: New file. * libcollector.h: New file.

[PATCH 17/24] egrep in binutils

From: Alan Modra Apparently some distros have a nagging egrep that helpfully tells you egrep is deprecated and to use "grep -E". The nag message causes a ld testsuite failure. What's more the advice isn't that good. The "-E" flag may not be available with older versions of grep. This patch

[PATCH 20/24] PKG_CHECK_MODULES: Check if $pkg_cv_[]$1[]_LIBS works

From: "H.J. Lu" It is quite normal to have headers without library on multilib OSes. Add AC_TRY_LINK to PKG_CHECK_MODULES to check if $pkg_cv_[]$1[]_LIBS works. config/ChangeLog: * pkg.m4 (PKG_CHECK_MODULES): Add AC_TRY_LINK to check if $pkg_cv_[]$1[]_LIBS works. ---

[PATCH 21/24] PKG_CHECK_MODULES: Properly check if $pkg_cv_[]$1[]_LIBS works

From: "H.J. Lu" There is no need to check $pkg_cv_[]$1[]_LIBS works if package check failed. config/ChangeLog: * pkg.m4 (PKG_CHECK_MODULES): Use AC_TRY_LINK only if $pkg_failed = no. --- config/pkg.m4 | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git

[PATCH 04/24] Sync with binutils: GCC: Pass --plugin to AR and RANLIB

From: "H.J. Lu" Sync with binutils for building binutils with LTO: 50ad1254d50 GCC: Pass --plugin to AR and RANLIB Detect GCC LTO plugin. Pass --plugin to AR and RANLIB to support LTO build. ChangeLog: * Makefile.tpl (AR): Add @AR_PLUGIN_OPTION@ (RANLIB): Add

[PATCH 18/24] PR27116, Spelling errors found by Debian style checker

From: Alan Modra config/ChangeLog: * override.m4: Correct comment grammar. --- config/override.m4 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/override.m4 b/config/override.m4 index d3e815bb1b4..5a1bffe6018 100644 --- a/config/override.m4 +++

[PATCH 12/24] Pass PKG_CONFIG_PATH down from top-level Makefile

From: Simon Marchi [Sending to binutils, gdb-patches and gcc-patches, since it touches the top-level Makefile/configure] I have my debuginfod library installed in a non-standard location (/opt/debuginfod), which requires me to set PKG_CONFIG_PATH=/opt/debuginfod/lib/pkg-config. If I just set

[PATCH 05/24] GCC: Check if AR works with --plugin and rc

From: "H.J. Lu" AR from older binutils doesn't work with --plugin and rc: [hjl@gnu-cfl-2 bin]$ touch foo.c [hjl@gnu-cfl-2 bin]$ ar --plugin /usr/libexec/gcc/x86_64-redhat-linux/10/liblto_plugin.so rc libfoo.a foo.c [hjl@gnu-cfl-2 bin]$ ./ar --plugin

[PATCH 07/24] binutils, gdb: support zstd compressed debug sections

From: Fangrui Song PR29397 PR29563: Add new configure option --with-zstd which defaults to auto. If pkgconfig/libzstd.pc is found, define HAVE_ZSTD and support zstd compressed debug sections for most tools. * bfd: for addr2line, objdump --dwarf, gdb, etc * gas: support

[PATCH 08/24] configure: require libzstd >= 1.4.0

From: Christophe Lyon gas uses ZSTD_compressStream2 which is only available with libzstd >= 1.4.0, leading to build errors when an older version is installed. This patch updates the check libzstd presence to check its version is >= 1.4.0. However, since gas seems to be the only component

[PATCH 02/24] PR29961, plugin-api.h: "Could not detect architecture endianess"

From: Alan Modra Found when attempting to build binutils on sparc sunos-5.8 where sys/byteorder.h defines _BIG_ENDIAN but not any of the BYTE_ORDER variants. This patch adds the extra tests to cope with the old machine, and tidies the header a little. include/ChangeLog: *

[PATCH 06/24] toplevel: Recover tilegx/tilepro targets

These are still supported in Binutils. ChangeLog: * configure: Regenerate. * configure.ac: Recover tilegx/tilepro targets. --- configure| 5 - configure.ac | 5 - 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/configure b/configure index

[PATCH 11/24] Disable year 2038 support on 32-bit hosts by default

From: Luis Machado With a recent import of gnulib, code has been pulled that tests and enables 64-bit time_t by default on 32-bit hosts that support it. Although gdb can use the gnulib support, bfd doesn't use gnulib and currently doesn't do these checks. As a consequence, if we have a 32-bit

[PATCH 03/24] gcc-4.5 build fixes

From: Alan Modra Trying to build binutils with an older gcc currently fails. Working around these gcc bugs is not onerous so let's fix them. include/ChangeLog: * xtensa-dynconfig.h (xtensa_isa_internal): Delete unnecessary forward declaration. --- include/xtensa-dynconfig.h |

[PATCH 09/24] add --enable-default-compressed-debug-sections-algorithm configure option

From: Martin Liska ChangeLog: * configure.ac: Add --enable-default-compressed-debug-sections-algorithm. * configure: Regenerate. --- configure| 13 + configure.ac | 6 ++ 2 files changed, 19 insertions(+) diff --git a/configure b/configure index

[PATCH 01/24] toplevel: Substitute GDCFLAGS instead of using CFLAGS

r14-2875-g1ed21e23d6d4da ("Use substituted GDCFLAGS") already implemented this change, but only on the generated file rather than in the template it is generated from. ChangeLog: * Makefile.tpl: Substitute @GDCFLAGS@ instead of using $(CFLAGS). --- Makefile.tpl | 2 +- 1 file

[PATCH 00/24] Sync shared build infrastructure with binutils-gdb

Hello, This patch set, combined with a sibling patch set sent on the binutils and GDB MLs, bring up the shared infrastructure between the two projects in sync again. It largely consists of cherry-picks from various people which have been reviewed and accepted on the binutils-gdb side, as well as

[pushed] extend.texi: index __auto_type

gcc/ChangeLog: * doc/extend.texi: Add @cindex on __auto_type. --- Pushed as obvious. gcc/doc/extend.texi | 1 + 1 file changed, 1 insertion(+) diff --git a/gcc/doc/extend.texi b/gcc/doc/extend.texi index 093bd97ba4d..ec9ffa3c86e 100644 --- a/gcc/doc/extend.texi +++

Re: [PATCH v2 0/5] A small Texinfo refinement

Hi, NightStrike writes: > Can you include texinfo in a top-level directory and have it built with the > build, like you can with dejagnu? That's generally a decent path for > building new gcc on older systems. Note that the Texinfo requirement wasn't bumped, so nothing changed in this regard,

[pushed] doc: Fix a pinch of typos in extend.texi

gcc/ChangeLog: * doc/extend.texi (C++ Concepts) : Remove extraneous parenthesis. Fix misnamed index entry. : Fix misnamed index entry. --- Pushed as obvious. gcc/doc/extend.texi | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git

Ping: [PATCH 0/2] Unify and deduplicate FTM code

Ping. This patch rebases cleanly as of right now. No new changes to review. Have a lovely day! -- Arsen Arsenović signature.asc Description: PGP signature

[PUSHED gcc-11] extend.texi: replace @itemx not preceded by @item.

gcc/ChangeLog: * doc/extend.texi: Replace @itemx not being preceded by @item. --- Fixes errors raised by newer Texinfo. Pushed as obvious. gcc/doc/extend.texi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/doc/extend.texi b/gcc/doc/extend.texi index

[PATCH 1/2] libstdc++: Implement more maintainable header

This commit replaces the ad-hoc logic in with an AutoGen database that (mostly) declaratively generates a version.h bit which combines all of the FTM logic across all headers together. This generated header defines macros of the form __glibcxx_foo, equivalent to their __cpp_lib_foo variants,

[PATCH 0/2] Unify and deduplicate FTM code

Greetings! This patch set replaces all code that involves defining feature test macros based on loosely put together conditionals in the standard library with a unified helper for specifying and requiring feature test macros, as well as updating most usage sites, many of which have been migrated

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

Richard Biener writes: > If it doesn't show any issues after a while on trunk it's OK for 13.2 I > think. Thanks, I'll re-propose it later then. Have a lovely day! -- Arsen Arsenović signature.asc Description: PGP signature

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

Jakub Jelinek writes: > Doesn't fix any regression, so not ok for 13.1 and I wouldn't bother > for 13.2 either. Okay, but it can affect --enable-languages=all in a slim edge case. Why not 13.2? It seems sufficiently simple. Thanks, have a lovely night! -- Arsen Arsenović signature.asc

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

Hi Gaius, Gaius Mulley writes: > yes certainly this is fine. lgtm. Thanks for spotting and the patch Sure. Will push to master and wait for a RM to weigh in on 13. Thanks! -- Arsen Arsenović signature.asc Description: PGP signature

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

$(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. --- Hi, We spotted that the m2

Re: [PATCH] update_web_docs_git: Add updated Texinfo to PATH

Gerald Pfeifer writes: > On Thu, 20 Apr 2023, Arsen Arsenović wrote: >>> I understand, just am wondering whether and why the : is required? I >>> don't think we are using this construct anywhere else? >> Without them, this would happen: >> >> ~$ "${foo:=foo}" >> bash: foo: command not

Re: [PATCH] update_web_docs_git: Add updated Texinfo to PATH

Gerald Pfeifer writes: > Hi Arsen, > > On Fri, 14 Apr 2023, Arsen Arsenović wrote: >>> Did you intentionally not implement the following part of my suggestion >>> >>>if [ x${MAKEINFO}x = xx ]; then >>>: >> > that is, allowing to override from the command-line (or crontab)? >> (answering

Re: [PATCH] doc: tfix

Alejandro Colomar via Gcc-patches writes: > Remove repeated word (typo). > > Signed-off-by: Alejandro Colomar > --- > gcc/doc/extend.texi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/gcc/doc/extend.texi b/gcc/doc/extend.texi > index fd3745c5608..cdfb25ff272 100644

Re: [PATCH] update_web_docs_git: Add updated Texinfo to PATH

Gerald Pfeifer writes: > On Tue, 11 Apr 2023, Arsen Arsenović wrote: >> Ah! Good idea. What do you think of the following? > > Did you intentionally not implement the following part of my suggestion > >if [ x${MAKEINFO}x = xx ]; then >: > > that is, allowing to override from the

Re: [PATCH] update_web_docs_git: Add updated Texinfo to PATH

Gerald Pfeifer writes: > On Thu, 6 Apr 2023, Arsen Arsenović wrote: >> maintainer-scripts/ChangeLog: >> >> * update_web_docs_git: Add updated Texinfo to PATH > > Do we really need to adjust PATH, or could we just introduce a MAKEINFO > variable, something like > > if [ x${MAKEINFO}x =

[PATCH] update_web_docs_git: Add updated Texinfo to PATH

maintainer-scripts/ChangeLog: * update_web_docs_git: Add updated Texinfo to PATH --- Hi, I'm posting this as a ping and a patch necessary to get the wwwdocs building with the new Texinfo version that's installed on gcc.gnu.org. It would be nice to do this ahead of the GCC 13 release. I

Re: [PATCH 2/4] libstdc++: Add a test for FTM redefinitions

Arsen Arsenović via Libstdc++ writes: > Good idea. Both the positive and negative cases work when implemented > like this: > > From 98d302bab9a87d3f2daecf9d54d62f33ba216673 Mon Sep 17 00:00:00 2001 > From: =?UTF-8?q?Arsen=20Arsenovi=C4=87?= > Date: Wed, 8 Mar 2023 17:01:24 +0100 > Subject:

Re: [PATCH 1/4] libstdc++: Harmonize and other headers

Jonathan Wakely writes: >> [snip...] >>+/* Duplicate definition with unique_ptr.h. */ >>+#if __cplusplus > 202002L && defined(__cpp_constexpr_dynamic_alloc) >>+# define __cpp_lib_constexpr_memory 202202L >>+#elif __cplusplus > 201703L >>+# include > > Including is unrelated to the FTM, and

Re: [PATCH 2/4] libstdc++: Add a test for FTM redefinitions

Jonathan Wakely writes: > On 05/04/23 01:09 +0200, Arsen Arsenović wrote: >>This test detects redefinitions by compiling stdc++.h with >>-Wsystem-headers. Thanks Patrick Palka for the suggestion. >> >>libstdc++-v3/ChangeLog: >> >> * testsuite/17_intro/versionconflict.cc: New test. >>---

Re: [PATCH 4/4] libstdc++: Fix some freestanding test failures

Jonathan Wakely writes: > On Wed, 5 Apr 2023 at 00:13, Arsen Arsenović via Libstdc++ > wrote: >> >> At some point, was added to the non-hosted bit of the C++17 >> block, which induced failures in many tests. >> >> In addition, some tests also lacked a dg-require-effective-target hosted >>

[PATCH 1/4] libstdc++: Harmonize and other headers

Due to recent, large changes in libstdc++, the feature test macros declared in got out of sync with the other headers that possibly declare them. libstdc++-v3/ChangeLog: * include/bits/unique_ptr.h (__cpp_lib_constexpr_memory): Synchronize the definition block with... *

[PATCH 3/4] libstdc++: Downgrade DEBUG to ASSERTIONS when !HOSTED

Supporting the debug mode in freestanding is a non-trivial job, so instead, as a best-effort, enable assertions, which are light and easy. libstdc++-v3/ChangeLog: * include/bits/c++config: When __STDC_HOSTED__ is zero, disable _GLIBCXX_DEBUG and, if it was set, enable

[PATCH 4/4] libstdc++: Fix some freestanding test failures

At some point, was added to the non-hosted bit of the C++17 block, which induced failures in many tests. In addition, some tests also lacked a dg-require-effective-target hosted tag. libstdc++-v3/ChangeLog: * include/precompiled/stdc++.h (C++17): Don't double-include , once

[PATCH 2/4] libstdc++: Add a test for FTM redefinitions

This test detects redefinitions by compiling stdc++.h with -Wsystem-headers. Thanks Patrick Palka for the suggestion. libstdc++-v3/ChangeLog: * testsuite/17_intro/versionconflict.cc: New test. --- libstdc++-v3/testsuite/17_intro/versionconflict.cc | 6 ++ 1 file changed, 6

Re: [PATCH] Introduce -nolibstdc++ option

Hi Alexandre, Alexandre Oliva via Gcc-patches writes: > Hello, Gerald, > > On Feb 11, 2023, Gerald Pfeifer wrote: > >> On Thu, 23 Jun 2022, Alexandre Oliva via Gcc-patches wrote: >>> Introduce -nostdlib++ option >>> >>> Using g++ to link without libstdc++, as in g++.dg/abi/pure-virtual1.C,

Re: Ping (gcc/configure.ac, docs): [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias

Arsen Arsenović via Gcc-patches writes: > Thank you! I'll do a final rebase and push in the morning. Done! Gerald, please update the scripts when you get a chance (but back the old ones up just in case!) If makeinfo is updated as I've asked in one of the other emails, will the scr

Re: Ping (gcc/configure.ac, docs): [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias

Sandra Loosemore writes: > On 3/18/23 14:14, Arsen Arsenović via Gcc-patches wrote: >> Evening, >> Ping on this patch series. >> I believe we're close to being able to apply it, with contents of the >> documentation changes being approved, and only a small build sys

Re: Ping (gcc/configure.ac, docs): [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias

Joseph Myers writes: > On Mon, 20 Mar 2023, Sandra Loosemore wrote: > >> Joseph, could you maybe review the last piece? A direct pointer to it in >> Arsen's git is >> >> https://git.sr.ht/~arsen/gcc/commit/bc734311cbca1085a1728f79b7eebef8cc7aeac3 > > That's OK, assuming I understand correctly

Ping (gcc/configure.ac, docs): [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias

Evening, Ping on this patch series. I believe we're close to being able to apply it, with contents of the documentation changes being approved, and only a small build system change and a revised patch series "shape" left to review. The former change of these two is in:

Re: [PATCH v2 0/5] A small Texinfo refinement

Richard Biener writes: > On Fri, Mar 10, 2023 at 8:00 PM Gerald Pfeifer wrote: >> >> On Fri, 10 Mar 2023, Sandra Loosemore wrote: >> > AFAIK we have not knowingly changed any specific requirements beyond the >> > stated 4.7 and 4.9 for PDF output, but it concerns me that nobody is >> > likely

[PATCH gcc-{11, 12}] c++: top level bind when rewriting coroutines [PR106188]

In the edge case of a coroutine not containing any locals, the ifcd/switch temporaries would get added to the coroutine frame, corrupting its layout. To prevent this, we can make sure there is always a BIND_EXPR at the top of the function body, and thus, always a place for our new temporaries to

Re: [PATCH v2 1/5] docs: Create Indices appendix

Gerald Pfeifer writes: > On Sat, 11 Mar 2023, Arsen Arsenović wrote: >> Sandra Loosemore writes: >>> Hmmm, first off, I think somebody other than me needs to approve the >>> configure and makefile pieces, as well as the maintainer-scripts part. >>> It looks conceptually right to me > > Per

[PATCH v2] html: Set CONTENTS_OUTPUT_LOCATION=inline if makeinfo supports it

This flag allows us to restore old (pre-6.8) behavior of the @{summary,}content commands, so that texi2any continues to emit summarycontents first. maintainer-scripts/ChangeLog: * update_web_docs_git: Set CONTENTS_OUTPUT_LOCATION=inline in order to put @shortcontents above

Re: [PATCH v2 1/5] docs: Create Indices appendix

Gerald Pfeifer writes: > On Sat, 11 Mar 2023, Arsen Arsenović wrote: >> OK, changed up a bit, what do you think of this: > >> maintainer-scripts/ChangeLog: >> >> * update_web_docs_git: Set CONTENTS_OUTPUT_LOCATION=inline in >> order to put @shortcontents above contents. See >>

Re: [PATCH v2 1/5] docs: Create Indices appendix

Sandra Loosemore writes: > On 3/11/23 05:22, Arsen Arsenović wrote: >> OK, changed up a bit, what do you think of this: >> maintainer-scripts/ChangeLog: >> * update_web_docs_git: Set CONTENTS_OUTPUT_LOCATION=inline in >> order to put @shortcontents above contents. See >>

Re: [PATCH v2 1/5] docs: Create Indices appendix

Sandra Loosemore writes: > On 3/9/23 13:38, Arsen Arsenović wrote: >> Found the change. HTML got support for CONTENTS_OUTPUT_LOCATION, >> which defaults to after_top, which ignores the inline location of >> these elements. Here's a patch: >> maintainer-scripts/ChangeLog: >> *

Re: [PATCH v2 1/5] docs: Create Indices appendix

Sandra Loosemore writes: > On 3/9/23 13:38, Arsen Arsenović wrote: >> Found the change. HTML got support for CONTENTS_OUTPUT_LOCATION, >> which defaults to after_top, which ignores the inline location of >> these elements. Here's a patch: >> maintainer-scripts/ChangeLog: >> *

[pushed] MAINTAINERS: add myself to write after approval

ChangeLog: * MAINTAINERS (Write After Approval): Add myself. --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index a61d3ae06df..3c533cb651d 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -318,6 +318,7 @@ from other maintainers or reviewers.

Re: [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias

Hi Gerald, Gerald Pfeifer writes: > On Fri, 10 Mar 2023, Arsen Arsenović wrote: >> Thanks, Thomas. I'd be happy to undergo this process later today. If I >> understood right, I should fill out >> https://sourceware.org/cgi-bin/pdw/ps_form.cgi and name you, right? > > Yes. Thanks, done.

Re: [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias

Afternoon, Sandra Loosemore writes: > On 2/23/23 03:27, Arsen Arsenović via Gcc-patches wrote: >> The @gol macro appears to have existed as a workaround for a bug in old >> versions of makeinfo and/or texinfo.tex, where they would, in some types >> of output, fai

Re: [PATCH v2 5/5] update_web_docs_git: Update CSS reference to new manual CSS

Sandra Loosemore writes: > On 2/23/23 03:27, Arsen Arsenović via Gcc-patches wrote: >> maintainer-scripts/ChangeLog: >> * update_web_docs_git (CSS): Update CSS reference to point to >> /texinfo-manuals.css. > > I'm going to defer to Gerald on this one,

Re: [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias

Hi Gerald, Thomas, Thomas Schwinge writes: > Hi! > > On 2023-03-10T01:17:04+0100, Gerald Pfeifer wrote: >> On Thu, 9 Mar 2023, Sandra Loosemore wrote: >>> This is OK, but I'd like to see this patch split into two separate >>> commits as well -- one for the texinfo.tex import, and one for the

Re: [PATCH v2 4/5] Update texinfo.tex, remove the @gol macro/alias

Morning, Sandra Loosemore writes: > On 2/23/23 03:27, Arsen Arsenović via Gcc-patches wrote: >> The @gol macro appears to have existed as a workaround for a bug in old >> versions of makeinfo and/or texinfo.tex, where they would, in some types >> of output, fai

[PATCH 2/2] libstdc++: Add a test for FTM redefinitions

This test detects redefinitions by compiling stdc++.h with -Wsystem-headers. Thanks Patrick Palka for the suggestion. libstdc++-v3/ChangeLog: * testsuite/17_intro/versionconflict.cc: New test. --- libstdc++-v3/testsuite/17_intro/versionconflict.cc | 6 ++ 1 file changed, 6

[PATCH 1/2] libstdc++: Harmonize and other headers

Due to recent, large changes in libstdc++, the feature test macros declared in got out of sync with the other headers that possibly declare them. This patch resolves that. libstdc++-v3/ChangeLog: * include/bits/unique_ptr.h (__cpp_lib_constexpr_memory): Synchronize the

Re: [PATCH v2 1/5] docs: Create Indices appendix

Sandra Loosemore writes: > On 2/23/23 03:27, Arsen Arsenović via Gcc-patches wrote: >> The GCC manual has multiple indices. By creating an appendix which >> lists them, we help makeinfo present a more accessible way for the >> reader to see all the indices. >> g

Re: [PATCH v2 0/5] A small Texinfo refinement

Sandra Loosemore writes: >> As an example, let's take this link: >> https://gcc.gnu.org/onlinedocs/gcc-12.2.0/gcc/Warning-Options.html#index-Wpedantic >> This should place you below the item line this index entry refers to, >> and there aren't any copiable anchors (see equivalent in my render

Re: [PATCH v2 1/5] docs: Create Indices appendix

Sandra Loosemore writes: > On 2/23/23 03:27, Arsen Arsenović via Gcc-patches wrote: >> The GCC manual has multiple indices. By creating an appendix which >> lists them, we help makeinfo present a more accessible way for the >> reader to see all the indices. >> g

Re: [PATCH v2 0/5] A small Texinfo refinement

Sandra Loosemore writes: > On 3/8/23 02:11, Arsen Arsenović wrote: >> Sandra Loosemore writes: >> >>> On 2/23/23 03:27, Arsen Arsenović via Gcc-patches wrote: >>>> I've rerendered the updated documentation with latest development >>>> Texinfo

Re: [PATCH v2 0/5] A small Texinfo refinement

Sandra Loosemore writes: > On 2/23/23 03:27, Arsen Arsenović via Gcc-patches wrote: >> I've rerendered the updated documentation with latest development >> Texinfo (as some of the changes I made for the purposes of the GCC >> manual still aren't in releases) at: >&g

Re: [PATCH v2 0/5] A small Texinfo refinement

Hi Thomas, Thomas Schwinge writes: > Hi Arsen! > > Thanks for caring about the documentation infrastructure! > > On 2023-02-23T11:27:09+0100, Arsen Arsenović via Gcc-patches > wrote: >> The commit "docs: Reorder @opindex to be before corresponding options&

Re: [wwwdocs] Add revised Texinfo manual CSS

Morning, Gerald Pfeifer writes: > On Fri, 27 Jan 2023, Arsen Arsenović via Gcc-patches wrote: >> htdocs/texinfo-manuals.css | 129 + > > Thank you - I pushed this. > > +/* Texinfo documentation stylesheet. > + Inspired by the G

[PATCH v2 3/5] doc: Add @defbuiltin family of helpers, set documentlanguage

The @defbuiltin{,x} macros are convenience macros for the often-repeated task of defining a built-in function in extend.texi. Usage of this macro should lead to a higher degree of consistency across pieces of text written by different people, and provide a better reading experience, as they

[PATCH v2 5/5] update_web_docs_git: Update CSS reference to new manual CSS

maintainer-scripts/ChangeLog: * update_web_docs_git (CSS): Update CSS reference to point to /texinfo-manuals.css. --- maintainer-scripts/update_web_docs_git | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/maintainer-scripts/update_web_docs_git

[PATCH v2 1/5] docs: Create Indices appendix

The GCC manual has multiple indices. By creating an appendix which lists them, we help makeinfo present a more accessible way for the reader to see all the indices. gcc/ChangeLog: * doc/gcc.texi: Add the Indices appendix, to make texinfo generate nice indices overview page.

[PATCH v2 0/5] A small Texinfo refinement

Morning, (as said in ) This is a rebase + slight amendment of the previous patch series visible at: https://inbox.sourceware.org/gcc-patches/cover.1674777149.git.ar...@aarsen.me/ The commit "docs: Reorder @opindex to be before

Re: (rebased patchset) [PATCH 3/7] **/*.texi: Reorder index entries

Hi Gerald, Thanks for the review. Gerald Pfeifer writes: > I was going to push this, alas there are now rejects for > fortran/invoke.texi, gcc/doc/invoke.texi and gcc/doc/cppopts.texi. > > If you can get me a rebased version I'll give it a try again. (Or > are there some earlier dependencies?

Re: [PATCH 3/7] **/*.texi: Reorder index entries

Hi Thomas, Thomas Schwinge writes: > Hi! > > On 2023-01-27T01:18:31+0100, Arsen Arsenović via Gcc-patches > wrote: >> Much like the previous commit, this change is mostly mechanical > > I find it helpful to see in the Git commit log some kind of rationale, > instead

Ping^2: [PATCH+wwwdocs 0/8] A small Texinfo refinement

Ping. Like last time, I rebased the series. The first two times around, I did not notice there's dedicated maintainers for the documentation component, and so, I am adding Gerald, Joseph and Sandra to CC this time. Apologies for the omission. Range-diff since last ping: 1: 6eba1548dfe = 1:

Ping: [wwwdocs] lists: Add documentation about the Sourceware public-inbox

Ping: https://inbox.sourceware.org/gcc-patches/20230123012100.4021860-1-ar...@aarsen.me/ No further changes happened to the file between the post and today. Have a lovely night :-) -- Arsen Arsenović signature.asc Description: PGP signature

Ping: [PATCH+wwwdocs 0/8] A small Texinfo refinement

Ping on this patch. I took the liberty to rebase it. The changes are minimal, so I didn't want to resend the entire patchset. I included a range diff and a pull request for your convenience. The render is also updated, and ``make all && make html'' passes (which is something I forgot to check

  1   2   >