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

2023-02-26 Thread Arsen Arsenović via Gcc-patches
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: [PATCH v2 0/5] A small Texinfo refinement

2023-03-08 Thread Arsen Arsenović via Gcc-patches
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)

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

2023-03-08 Thread Arsen Arsenović via Gcc-patches
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 >>>> Texin

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

2023-03-09 Thread Arsen Arsenović via Gcc-patches
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

2023-03-09 Thread Arsen Arsenović via Gcc-patches
the requirement for a specific minimum version, > and make install.texi say something similar to what it says for e.g. awk -- > just use a "recent" version, and note that new versions produce better output > and very old ones may produce diagnostics. I'll add that do my own todo list. That seems reasonable, thanks. > -Sandra -- Arsen Arsenović signature.asc Description: PGP signature

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

2023-03-09 Thread Arsen Arsenović via Gcc-patches
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

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

2023-03-09 Thread Arsen Arsenović via Gcc-patches
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 definitio

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

2023-03-09 Thread Arsen Arsenović via Gcc-patches
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 insertion

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

2023-03-10 Thread Arsen Arsenović via Gcc-patches
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, fail t

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

2023-03-10 Thread Arsen Arsenović via Gcc-patches
- > Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 > München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas > Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht > München, HRB 106955 Have a lovely day! -- Arsen Arsenović signature.asc Description: PGP signature

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

2023-03-10 Thread Arsen Arsenović via Gcc-patches
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

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

2023-03-10 Thread Arsen Arsenović via Gcc-patches
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, fail t

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

2023-03-10 Thread Arsen Arsenović via Gcc-patches
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, righ

[pushed] MAINTAINERS: add myself to write after approval

2023-03-10 Thread Arsen Arsenović via Gcc-patches
. Mark G. Adams Pedro Alves Paul-Antoine Arras +Arsen Arsenović Raksit Ashok Matt Austern

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

2023-03-10 Thread Arsen Arsenović via Gcc-patches
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: >&

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

2023-03-11 Thread Arsen Arsenović via Gcc-patches
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: >&

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

2023-03-11 Thread Arsen Arsenović via Gcc-patches
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 con

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

2023-03-11 Thread Arsen Arsenović via Gcc-patches
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 pu

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

2023-03-11 Thread Arsen Arsenović via Gcc-patches
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 content

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

2023-03-12 Thread Arsen Arsenović via Gcc-patches
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 lo

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

2023-03-12 Thread Arsen Arsenović via Gcc-patches
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 go

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

2023-03-13 Thread Arsen Arsenović via Gcc-patches
> release tarballs - the pre-built info format plus manual pages is > probably of less use these days? They're still widely installed and I suspect many open the manpages as a quick reference. I guess that fewer open the info pages, but it's not nobody (I certainly do, but I a

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

2023-05-17 Thread Arsen Arsenović via Gcc-patches
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 a/gcc/doc/extend.tex

[PATCH 00/10] c-family,libstdc++: P1642 and related changes

2022-09-30 Thread Arsen Arsenović via Gcc-patches
Hi, This patchset: - Implements the P1642 WG21 paper, with a fair few extensions, - Fixes libstdc++' build system on systems --without-headers, - Ports (a large chunk of) the libstdc++ testsuite to freestanding, and - Changes the semantics of `int main' in freestanding (!!). Than

[PATCH 01/10] libstdc++: Make _GLIBCXX_HOSTED respect -ffreestanding [PR103626]

2022-09-30 Thread Arsen Arsenović via Gcc-patches
i.exp: Likewise. Signed-off-by: Arsen Arsenović --- libstdc++-v3/acinclude.m4| 2 +- libstdc++-v3/configure | 2 +- libstdc++-v3/include/Makefile.am | 2 +- libstdc++-v3/include/Makefile.in | 2 +- libstdc++-v3/testsuite/lib/li

[PATCH 05/10] c-family: Implement new `int main' semantics in freestanding

2022-09-30 Thread Arsen Arsenović via Gcc-patches
plicitly-returns.c: New test. * gcc.dg/no-builtin-main.c: New test. Signed-off-by: Arsen Arsenović --- gcc/c-family/c-common.cc | 6 ++ gcc/c-family/c-common.h | 10 ++ gcc/c/c-decl.cc | 4 ++-- gcc/

[PATCH 02/10] libstdc++: Filter out unconditional default include

2022-09-30 Thread Arsen Arsenović via Gcc-patches
from ac_includes_default, when running Autoconf <2.70. * configure: Regenerate. Signed-off-by: Arsen Arsenović --- libstdc++-v3/configure.ac | 9 + 1 file changed, 9 insertions(+) diff --git a/libstdc++-v3/configure.ac b/libstdc++-v3/configure.ac index b351622bf46..c05fcdda7e

[PATCH 03/10] libstdc++: Adjust precompiled headers for freestanding

2022-09-30 Thread Arsen Arsenović via Gcc-patches
From: Jonathan Wakely Co-authored-by: Arsen Arsenović libstdc++-v3/ChangeLog: * include/precompiled/extc++.h [!_GLIBCXX_HOSTED]: Do not include headers that aren't valid for freestanding. * include/precompiled/stdc++.h [!_GLIBCXX_HOSTED]: Likewise. Signed-o

[PATCH 06/10] libstdc++: Rework how freestanding install works [PR106953]

2022-09-30 Thread Arsen Arsenović via Gcc-patches
. Signed-off-by: Arsen Arsenović --- libstdc++-v3/include/Makefile.am | 300 +++--- libstdc++-v3/include/Makefile.in | 656 +++ 2 files changed, 477 insertions(+), 479 deletions(-) diff --git a/libstdc++-v3/include/Makefile.am b/libstdc++-v3/include/Makefile.am

[PATCH 08/10] libstdc++: Add effective-target 'hosted' for testsuite [PR103626]

2022-09-30 Thread Arsen Arsenović via Gcc-patches
. Signed-off-by: Arsen Arsenović --- libstdc++-v3/testsuite/lib/libstdc++.exp | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/libstdc++-v3/testsuite/lib/libstdc++.exp b/libstdc++-v3/testsuite/lib/libstdc++.exp index d3256b6c8c4..635f16db4e8 100644 --- a/libstdc++-v3

[PATCH 04/10] libstdc++: Mark headers that must be hosted as such [PR103626]

2022-09-30 Thread Arsen Arsenović via Gcc-patches
libstdc++ tests. Signed-off-by: Arsen Arsenović --- libstdc++-v3/include/Makefile.am | 1 + libstdc++-v3/include/Makefile.in | 1 + libstdc++-v3/include/bits/algorithmfwd.h | 4 + libstdc++-v3/include/bits/concept_check.h | 2 +- libstdc++-v3

[PATCH 09/10] libstdc++: Re-enable std::hash in freestanding [PR103626]

2022-09-30 Thread Arsen Arsenović via Gcc-patches
/20_util/bitset/ext/constexpr.cc: Likewise. Signed-off-by: Arsen Arsenović --- libstdc++-v3/include/std/bitset | 9 + .../testsuite/20_util/bitset/cons/constexpr_c++23.cc | 2 ++ libstdc++-v3/testsuite/20_util/bitset/ext/constexpr.cc | 2 ++ 3 files changed

[PATCH 07/10] libstdc++: Make some tests work on freestanding [PR103626]

2022-09-30 Thread Arsen Arsenović via Gcc-patches
C_HOSTED__]: Likewise. * testsuite/std/ranges/adaptors/100577.cc [!__STDC_HOSTED__]: Likewise. Signed-off-by: Arsen Arsenović --- .../17_intro/headers/c++1998/stdc++.cc| 2 + .../17_intro/tag_type_explicit_ctor.cc| 32 +++ .../17_intro/using_namespace_std_exp_neg

[PATCH] libstdc++: Use ///< for inline documentation

2022-10-01 Thread Arsen Arsenović via Gcc-patches
ibstdc++-v3/ChangeLog: * include/std/iostream: Use ///< for inline documentation. * include/std/limits: Likewise. * include/experimental/internet: Likewise. Signed-off-by: Arsen Arsenović --- Hey, I just got reminded that I found some trivial documentation errors a few m

Re: [PATCH] libstdc++: Use ///< for inline documentation

2022-10-03 Thread Arsen Arsenović via Gcc-patches
op of that, clangd interprets it properly (it's how I stumbled upon this, cout was "Linked to standard output" in its LSP hover response). https://gcc.gnu.org/onlinedocs/libstdc++/latest-doxygen/a01635.html#a53dbc8572a84ca50272f9e55a1e23e18 -- Arsen Arsenović signature.asc Descri

[PATCH] elf: ELF toolchain --without-{headers, newlib} should provide stdint.h

2022-10-03 Thread Arsen Arsenović via Gcc-patches
stdint.h is considered a freestanding headers by C, and a valid stdint.h is required for certain parts of libstdc++' configuration, so we should simply provide one when we have no other way (i.e. newlib or user-specified sysroot) of getting one. gcc/ChangeLog: * config.gcc: --target=*-elf

Ping: [PATCH] elf: ELF toolchain --without-{headers, newlib} should provide stdint.h

2022-10-10 Thread Arsen Arsenović via Gcc-patches
Evening, Ping on this patch, it should fix some configuration issues we get on ELF targets (especially in libstdc++). https://gcc.gnu.org/pipermail/gcc-patches/2022-October/602751.html Thanks, -- Arsen Arsenović signature.asc Description: This is a digitally signed message part.

[PATCH] libstdc++: respect with-{headers, newlib} for default hosted value

2022-10-12 Thread Arsen Arsenović via Gcc-patches
This saves us a build flag when building for freestanding targets. libstdc++-v3/ChangeLog: * acinclude.m4: Default hosted to off if building without headers and without newlib. --- Tested for x86_64-elf. libstdc++-v3/acinclude.m4 | 5 - 1 file changed, 4 insertions(+), 1 del

Re: Handling of main() function for freestanding

2022-10-13 Thread Arsen Arsenović via Gcc-patches
in this test, and it doesn't even need to link, so using any ol' function should be okay; see attachment) The attached patch is also v2 of the original builtin-main one submitted earlier. Tested on x86_64-pc-linux-gnu. This revision excludes the mentioned pedwarns unless hosted. T

Re: Handling of main() function for freestanding

2022-10-13 Thread Arsen Arsenović via Gcc-patches
ed (noreturn-1.c), so maybe dropping is the right thing to do, indeed. -- Arsen Arsenović signature.asc Description: This is a digitally signed message part.

Re: Handling of main() function for freestanding

2022-10-13 Thread Arsen Arsenović via Gcc-patches
t's always marked extern "C" in the C++ frontend, AFAICT). -- Arsen Arsenović >From e60be6bb45fdba8085bde5d1883deeae640e786b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Arsen=20Arsenovi=C4=87?= Date: Thu, 13 Oct 2022 21:46:30 +0200 Subject: [PATCH v3] c-family: Implicitly return

Re: Handling of main() function for freestanding

2022-10-14 Thread Arsen Arsenović via Gcc-patches
his? Thanks, -- Arsen Arsenović >From dfc1677d1dcfa1b3b963fca8ea5eb8878c4e63d0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Arsen=20Arsenovi=C4=87?= Date: Thu, 13 Oct 2022 21:46:30 +0200 Subject: [PATCH v4] c-family: Implicitly return zero from main even on freestanding ... unless marked noreturn.

Re: Ping: [PATCH] elf: ELF toolchain --without-{headers, newlib} should provide stdint.h

2022-10-14 Thread Arsen Arsenović via Gcc-patches
On Tuesday, 11 October 2022 08:47:32 CEST Richard Biener wrote: > OK. Thanks for the review. I don't have push rights currently, so could you push this for me? Have a good evening, -- Arsen Arsenović signature.asc Description: This is a digitally signed message part.

[PATCH] libstdc++: Enable _GLIBCXX_WEAK_DEFINITION on more platforms

2022-10-19 Thread Arsen Arsenović via Gcc-patches
This should permit redefining `operator new' where-ever that is possible, and should allow libsupc++ to be more broadly useful in freestanding. libstdc++-v3/ChangeLog: * acinclude.m4: Add link test for __attribute__ ((weak)). * config.h.in: Regenerate. * config/os/bsd/darw

[PATCH] libstdc++: Don't use gstdint.h anymore

2022-10-20 Thread Arsen Arsenović via Gcc-patches
libstdc++-v3/ChangeLog: * configure.ac: Stop generating gstdint.h. * src/c++11/compatibility-atomic-c++0x.cc: Stop using gstdint.h. --- Tested on x86_64-pc-linux-gnu. libstdc++-v3/configure.ac| 6 -- libstdc++-v3/src/c++11/compatibility-atomic-c++0

[PATCH v2] libstdc++: Don't use gstdint.h anymore

2022-10-20 Thread Arsen Arsenović via Gcc-patches
libstdc++-v3/ChangeLog: * configure.ac: Stop generating gstdint.h. * src/c++11/compatibility-atomic-c++0x.cc: Stop using gstdint.h. --- > > +using guintptr_t = __UINTPTR_TYPE__; > > I think this should be local in the only function that uses it. Sure. Tested on x86_64-pc-linux-g

[PATCH] libstdc++: Make placeholders inline when inline variables are available

2022-10-20 Thread Arsen Arsenović via Gcc-patches
This slightly lowers the dependency of generated code on libstdc++.so. libstdc++-v3/ChangeLog: * include/std/functional: Make placeholders inline, if possible. --- libstdc++-v3/include/std/functional | 66 - 1 file changed, 37 insertions(+), 29 deletions(-) d

[PATCH] libstdc++: Enable building libstdc++.{a,so} when !HOSTED

2022-10-20 Thread Arsen Arsenović via Gcc-patches
This enables us to provide symbols for placeholders and numeric limits, and allows users to mess about with linker flags less. libstdc++-v3/ChangeLog: * Makefile.am [!_GLIBCXX_HOSTED]: Enable src/ subdirectory. * Makefile.in: Regenerate. * src/Makefile.am [!_GLIBCXX_HOSTED

Ping (c,c++): Handling of main() function for freestanding

2022-10-21 Thread Arsen Arsenović via Gcc-patches
ain's). This applies to both the C and C++ frontends. -- Arsen Arsenović signature.asc Description: This is a digitally signed message part.

Re: C2x features status

2022-10-21 Thread Arsen Arsenović via Gcc-patches
ls). I was intending on giving this issue a proper look in the GNU toolchain frame of reference, but never got around to it (and I kinda knocked priority down after managing to configure properly once IIRC). -- Arsen Arsenović signature.asc Description: This is a digitally signed message part.

Re: C2x features status

2022-10-21 Thread Arsen Arsenović via Gcc-patches
n and collaboration is more than welcome, though; please do reach out. Thanks, -- Arsen Arsenović signature.asc Description: This is a digitally signed message part.

Re: Ping (c,c++): Handling of main() function for freestanding

2022-10-23 Thread Arsen Arsenović via Gcc-patches
On Friday, 21 October 2022 23:02:02 CEST Joseph Myers wrote: > I have no objections to the C changes. Great! Thanks for the review. I don't have push rights currently, so I must ask that someone else pushes this patch for me. Have a great day! -- Arsen Arsenović signature.asc Des

[PATCH] coroutines: Wrap awaiting if/swich in a BIND_EXPR, if needed [PR106188]

2022-09-03 Thread Arsen Arsenović via Gcc-patches
necessary. (morph_fn_to_coro): Pass fnbody into susp_frame_data. gcc/testsuite/ChangeLog: * g++.dg/coroutines/pr106188.C: New test. Signed-off-by: Arsen Arsenović --- gcc/cp/coroutines.cc | 45 -- gcc/testsuite/g++.dg/coroutines/pr106188.C

[PATCH v2] coroutines: Ensure there's a top level bind when rewriting [PR106188]

2022-09-04 Thread Arsen Arsenović via Gcc-patches
-by: Arsen Arsenović --- gcc/cp/coroutines.cc | 10 +++ gcc/testsuite/g++.dg/coroutines/pr106188.C | 35 ++ 2 files changed, 45 insertions(+) create mode 100644 gcc/testsuite/g++.dg/coroutines/pr106188.C Hi Iain, > I can see that adding the b

Re: [PATCH v2] coroutines: Ensure there's a top level bind when rewriting [PR106188]

2022-09-05 Thread Arsen Arsenović via Gcc-patches
Hi, On Monday, 5 September 2022 00:09:51 CEST Iain Sandoe wrote: > LGTM, but I cannot actually approve it - please wait for an ack from > Jason or Nathan (or one of the other global maintainers). Will do. Thank you for your assistance. Have a good day, -- Arsen Arsenović signatu

Re: [PATCH v2] coroutines: Ensure there's a top level bind when rewriting [PR106188]

2022-09-07 Thread Arsen Arsenović via Gcc-patches
Hi, On Wednesday, 7 September 2022 17:13:03 CEST Jason Merrill wrote: > Applied with some minor tweaks, described below. > > Thanks! Got it, and noted the changes and rationale you provided Thank you! -- Arsen Arsenović signature.asc Description: This is a digitally signed message part.

[PATCH] contracts: Stop relying on mangling for naming .pre/.post clones

2022-12-10 Thread Arsen Arsenović via Gcc-patches
If the mangler is relied on, functions with extern "C" on them emit multiple definitions of the same name. gcc/cp/ChangeLog: * contracts.cc (build_contract_condition_function): Add pre/post suffixes to pre- and postcondition clones. * mangle.cc (write_encoding): Don't mang

Re: [PATCH] contracts: Stop relying on mangling for naming .pre/.post clones

2022-12-10 Thread Arsen Arsenović via Gcc-patches
Arsen Arsenović writes: > Afternoon, > > This change prevents contracts from emitting wrapper functions that have the > same name as the original function, by moving the logic that disambiguates > them > from the mangler into the build_contract_condition_function helper. >

Re: [PATCH] contracts: Stop relying on mangling for naming .pre/.post clones

2022-12-11 Thread Arsen Arsenović via Gcc-patches
ne across the board, just that this time around we don't need the mangler when it isn't used. Thanks for the review, and have a wonderful day. -- Arsen Arsenović signature.asc Description: PGP signature

Re: [PATCH 4/4] contrib: Add dg-out-generator.pl

2022-12-15 Thread Arsen Arsenović via Gcc-patches
plies with the minimal escaping style when I take the opportunity to address your other comment. (also, it just occurred to me that I forgot to escape dollar signs) Thanks, have a great day. -- Arsen Arsenović signature.asc Description: PGP signature

Re: [PATCH 1/4] contracts: Lowercase {MAYBE,NEVER}_CONTINUE

2022-12-15 Thread Arsen Arsenović via Gcc-patches
Hi, Jason Merrill writes: >> The lowercase constants are more consistent with the standard, and it is >> unlikely that the uppercase versions would've been accepted. > > OK. Thanks. Could you push this for me? I don't have write access. Have a great day. -- Ars

Re: [PATCH 2/4] libstdc++: Improve output of default contract violation handler [PR107792]

2022-12-15 Thread Arsen Arsenović via Gcc-patches
tion_name() >> +<< " at " << violation.file_name() << ':' << violation.line_number() >> +<< ": " << violation.comment() >> +<< "\n[level:" << violation.assertion_level() > > Maybe omit level/role if "default"? ACK on both, sounds good. Have a great day. -- Arsen Arsenović signature.asc Description: PGP signature

Re: [PATCH] contracts: Stop relying on mangling for naming .pre/.post clones

2022-12-15 Thread Arsen Arsenović via Gcc-patches
Hi Jason, Jason Merrill writes: > On 12/10/22 08:13, Arsen Arsenović wrote: >> If the mangler is relied on, functions with extern "C" on them emit multiple >> definitions of the same name. > > But doing it here interferes with lazy mangling. How ab

Re: [PATCH] contracts: Stop relying on mangling for naming .pre/.post clones

2022-12-17 Thread Arsen Arsenović via Gcc-patches
Jason Merrill writes: > > It's best to wrap the ChangeLog entry at column 75 so that the extra > indentation added by git log doesn't push past column 80. > > Applied with that change. > > Jason Heh, I thought I fixed my fill-column. Will check. Thanks! Have a

Re: [PATCH] contracts: Stop relying on mangling for naming .pre/.post clones

2022-12-17 Thread Arsen Arsenović via Gcc-patches
the right place, then I can update my patch for coroutines - both > Gor and Lewis replied that ‘extern “C”’ coroutines seemed reasonable to > them. Yes, that seems about right to me. Have a great day. -- Arsen Arsenović signature.asc Description: PGP signature

[PATCH 1/3] libstdc++: Improve output of default contract violation handler [PR107792]

2022-12-20 Thread Arsen Arsenović via Gcc-patches
From: Jonathan Wakely Make the output more readable. Don't output anything unless verbose termination is enabled at configure-time. libstdc++-v3/ChangeLog: PR libstdc++/107792 PR libstdc++/107778 * src/experimental/contract.cc (handle_contract_violation): Make ou

[PATCH 2/3] contracts: Update testsuite against new default viol. handler format

2022-12-20 Thread Arsen Arsenović via Gcc-patches
This change was almost entirely mechanical. Save for two files which had very short matches, these changes were produced by two seds and a Perl script, for the more involved cases. The latter will be added in a subsequent commit. The former are as follows: sed -E -i "/dg-output/s/default st

[PATCH 3/3] contrib: Add dg-out-generator.pl

2022-12-20 Thread Arsen Arsenović via Gcc-patches
) 2022 GCC Contributors. +# Contributed by Arsen Arsenović. +# +# This script is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3, or (at your option) +# any later version

[PATCH 1/3] libstdc++: Improve output of default contract violation handler [PR107792]

2022-12-22 Thread Arsen Arsenović via Gcc-patches
From: Jonathan Wakely Make the output more readable. Don't output anything unless verbose termination is enabled at configure-time. libstdc++-v3/ChangeLog: PR libstdc++/107792 PR libstdc++/107778 * src/experimental/contract.cc (handle_contract_violation): Make ou

[PATCH 2/3] contracts: Update testsuite against new default viol. handler format

2022-12-22 Thread Arsen Arsenović via Gcc-patches
This change was almost entirely mechanical. Save for two files which had very short matches, these changes were produced by two seds and a Perl script, for the more involved cases. The latter will be added in a subsequent commit. The former are as follows: sed -E -i "/dg-output/s/default st

[PATCH 3/3] contrib: Add dg-out-generator.pl

2022-12-22 Thread Arsen Arsenović via Gcc-patches
..1e9247165b2 --- /dev/null +++ b/contrib/dg-out-generator.pl @@ -0,0 +1,79 @@ +#!/usr/bin/env perl +# +# Copyright (C) 2022 Free Software Foundation, Inc. +# Contributed by Arsen Arsenović. +# +# This script is free software; you can redistribute it and/or modify +# it under the terms of the GNU General

Re: [PATCH 3/3] contrib: Add dg-out-generator.pl

2022-12-22 Thread Arsen Arsenović via Gcc-patches
t could easily use the simpler [\r\n]* form too: % regexp {^[\r\n]*$} "\r\n\n\n" 1 Feel free to swap that in too. Thanks, have a great night. -- Arsen Arsenović signature.asc Description: PGP signature

Re: [PATCH 3/3] contrib: Add dg-out-generator.pl

2022-12-22 Thread Arsen Arsenović via Gcc-patches
e. I'm not sure why, the original handler didn't emit double newlines or something either I don't think. Thanks for handling that anyway. -- Arsen Arsenović signature.asc Description: PGP signature

[PATCH 2/2] libstdc++: Fix a few !HOSTED test regressions

2023-01-10 Thread Arsen Arsenović via Gcc-patches
libstdc++-v3/ChangeLog: * testsuite/20_util/to_chars/version.cc: Mark hosted-only. * testsuite/20_util/uses_allocator/lwg3677.cc: Ditto. * testsuite/20_util/weak_ptr/cons/self_move.cc: Ditto. * testsuite/std/ranges/adaptors/as_rvalue/1.cc: Replace usage of s

[PATCH 1/2] libstdc++: Enable string_view in freestanding

2023-01-10 Thread Arsen Arsenović via Gcc-patches
This enables the default contract handler in freestanding environments, and, of course, provides freestanding users with string_view. libstdc++-v3/ChangeLog: * include/Makefile.am: Install bits/char_traits.h, std/string_view * include/Makefile.in: Regenerate. * inc

Re: [PATCH 1/2] libstdc++: Enable string_view in freestanding

2023-01-10 Thread Arsen Arsenović via Gcc-patches
.cc index dbe1745c89d..a02a61c9b4c 100644 --- a/libstdc++-v3/testsuite/21_strings/basic_string_view/operations/substr/wchar_t.cc +++ b/libstdc++-v3/testsuite/21_strings/basic_string_view/operations/substr/wchar_t.cc @@ -20,9 +20,12 @@ // basic_string_view::substr #include -#include #include

Re: [PATCH,WWWDOCS] htdocs: add an Atom feed for GCC news

2023-01-17 Thread Arsen Arsenović via Gcc-patches
C4287 appendix B (<https://datatracker.ietf.org/doc/html/rfc4287#appendix-B>). >> What do others think? >> >> Gerald Hope that helps! -- Arsen Arsenović signature.asc Description: PGP signature

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

2023-09-05 Thread Arsen Arsenović via Gcc-patches
iar with the full script. -- Arsen Arsenović signature.asc Description: PGP signature

Re: Question on -fwrapv and -fwrapv-pointer

2023-09-15 Thread Arsen Arsenović via Gcc-patches
where overflow is unexpected. One could easily imagine a scenario where overflow is not expected in some region of code but is in the larger application. -- Arsen Arsenović signature.asc Description: PGP signature

[PATCH] doc: Use a separate directory for new modules we add to PATH

2022-11-09 Thread Arsen Arsenović via Gcc-patches
ChangeLog: * doc/baseconf.py: Inject dirname(__file__)/'modules' to path instead of just ``.''. * doc/gcc_sphinx.py: Moved to... * doc/modules/gcc_sphinx.py: ...here. --- Evening, This patch addresses a path mixup and provides a nice compromise between convenience

[PATCH] libstdc++: Fix python/ not making install directories

2022-11-13 Thread Arsen Arsenović via Gcc-patches
I'm unsure why this issue only started manifesting now with how old this code is, but this should fix it. libstdc++-v3/ChangeLog: * python/Makefile.am: Call mkinstalldirs before INSTALL_DATA when installing gdb scripts. * python/Makefile.in: Regenerate. --- Hi, Someone sp

Re: [PATCH] libstdc++: Fix python/ not making install directories

2022-11-14 Thread Arsen Arsenović via Gcc-patches
y ;) Was the omission of the mkdir $(DESTDIR)$(toolexeclibdir) intentional? I only see TELD/debug in your revision of the patch. Chances are, it gets created elsewhere (my test was just install-target-libstdc++-v3, so not even the full install), but it might be worth being conservative about

Re: [PATCH] libstdc++: Fix python/ not making install directories

2022-11-14 Thread Arsen Arsenović via Gcc-patches
eans it has the problems that mkinstalldirs is > supposed to solve, but is that only relevant for Solaris 8, i.e. not > relevant? Ah, sorry. I didn't notice that it did that at the top. Yes, mkdir -p is at least as good then. Thanks, -- Arsen Arsenović signature.asc Description: PGP signature

Ping: [PATCH] libstdc++: Enable building libstdc++.{a, so} when !HOSTED

2022-11-14 Thread Arsen Arsenović via Gcc-patches
Evening, Since S1 is closed now, best to ping this patch before it drifts into GCC14. Archive link: https://gcc.gnu.org/pipermail/gcc-patches/2022-October/604031.html Have a great evening! -- Arsen Arsenović signature.asc Description: PGP signature

Re: [PATCH] libstdc++: Enable _GLIBCXX_WEAK_DEFINITION on more platforms

2022-11-14 Thread Arsen Arsenović via Gcc-patches
Evening, Similar to the other patch, pinging this before it drifts unreasonably far into S3, and hence into GCC14. Archive: https://gcc.gnu.org/pipermail/gcc-patches/2022-October/603931.html Have a great evening :) -- Arsen Arsenović signature.asc Description: PGP signature

Re: [PATCH] libstdc++: Enable building libstdc++.{a,so} when !HOSTED

2022-11-15 Thread Arsen Arsenović via Gcc-patches
Jonathan Wakely writes: > On Thu, 20 Oct 2022 at 16:53, Arsen Arsenović via Libstdc++ > wrote: >> >> This enables us to provide symbols for placeholders and numeric limits, > > > I'm not convinced this is worth doing. > > The placeholders and the

Re: [PATCH] doc: tfix

2023-04-20 Thread Arsen Arsenović via Gcc-patches
@var{level}. Please include a ChangeLog like the following: gcc/ChangeLog: * doc/extend.texi (Common Function Attributes): Remove duplicate word. I can add that and push for you, if you agree. Thanks, have a most lovely night! -- Arsen Arsenović signature.asc Description: PGP signature

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

2023-04-20 Thread Arsen Arsenović via Gcc-patches
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 over

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

2023-04-21 Thread Arsen Arsenović via Gcc-patches
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: >> >>

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

2023-04-21 Thread Arsen Arsenović via Gcc-patches
$(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 mak

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

2023-04-21 Thread Arsen Arsenović via Gcc-patches
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

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

2023-04-21 Thread Arsen Arsenović via Gcc-patches
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ć

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

2023-04-25 Thread Arsen Arsenović via Gcc-patches
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

[PATCH 0/2] Unify and deduplicate FTM code

2023-04-29 Thread Arsen Arsenović via Gcc-patches
d=gnu++23}{-fno-freestanding,-ffreestanding}" (without the line breaks) to find no relevant failures. OK for trunk? Thanks in advance, have a lovely day. [1] https://git.sr.ht/~arsen/scripts/tree/master/item/difall.bash [2] https://git.sr.ht/~arsen/scripts/tree/master/item/vercmp.bash Arsen Arse

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

2023-04-29 Thread Arsen Arsenović via Gcc-patches
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, acco

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

2023-05-03 Thread Arsen Arsenović via Gcc-patches
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 d6093397a61.

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

2023-05-08 Thread Arsen Arsenović via Gcc-patches
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

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

2023-01-22 Thread Arsen Arsenović via Gcc-patches
--- Evening, I've gone through lists.html and documented the Sourceware public-inbox instance, since it seems to be commonly missed, yet some might find useful. This patch includes: - A brief summary of what the public-inbox provides, - A brief summary on alternative methods of accessing the ar

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

2023-01-22 Thread Arsen Arsenović via Gcc-patches
Arsen Arsenović writes: > --- > Evening, > > I've gone through lists.html and documented the Sourceware public-inbox > instance, since it seems to be commonly missed, yet some might find useful. > > This patch includes: > - A brief summary of what the public

[PATCH] libstdc++: Document P1642 and extensions

2023-01-23 Thread Arsen Arsenović via Gcc-patches
libstdc++-v3/ChangeLog: * doc/xml/manual/using.xml: Document newly-freestanding headers and the effect of the -ffreestanding flag. * doc/xml/manual/status_cxx2023.xml: Document P1642R11 as completed. * doc/xml/manual/configure.xml: Document that hosted insta

<    1   2   3   4   >