[pshed] testsuite, C++, Darwin: Skip cxa_atexit-6, which is not applicable.

ite/ChangeLog: * g++.dg/tree-ssa/cxa_atexit-6.C: Skip for Darwin. Signed-off-by: Iain Sandoe --- gcc/testsuite/g++.dg/tree-ssa/cxa_atexit-6.C | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/gcc/testsuite/g++.dg/tree-ssa/cxa_atexit-6.C b/gcc/testsuite/g++.dg/tree-

[pushed] testsuite, darwin: Compile a test without unwind frames.

t-3.c: Suppress unwind frames. Signed-off-by: Iain Sandoe --- gcc/testsuite/gcc.dg/darwin-weakimport-3.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/gcc/testsuite/gcc.dg/darwin-weakimport-3.c b/gcc/testsuite/gcc.dg/darwin-weakimport-3.c index a15b5b0e7cb..7e83987b

Re: Ping [PATCH/RFC] target, hooks: Allow a target to trap on unreachable [PR109267].

> On 14 May 2024, at 14:29, Richard Biener wrote: > > On Wed, May 8, 2024 at 9:37 PM Iain Sandoe wrote: >> >> Hi Folks, >> >> I’d like to land a viable solution to this issue if possible, (it is a show- >> stopper for the aarch64-darwin development b

Re: Fix gnu versioned namespace mode 00/03

> On 13 May 2024, at 06:06, François Dumont wrote: > > > On 07/05/2024 18:15, Iain Sandoe wrote: >> Hi François >> >>> On 4 May 2024, at 22:11, François Dumont wrote: >>> >>> Here is the list of patches to restore gnu versioned names

Ping [PATCH/RFC] target, hooks: Allow a target to trap on unreachable [PR109267].

Hi Folks, I’d like to land a viable solution to this issue if possible, (it is a show- stopper for the aarch64-darwin development branch). > On 9 Apr 2024, at 14:55, Iain Sandoe wrote: > > So far, tested lightly on aarch64-darwin; if this is acceptable then > it will be possible

Re: Fix gnu versioned namespace mode 00/03

Hi François > On 4 May 2024, at 22:11, François Dumont wrote: > > Here is the list of patches to restore gnu versioned namespace mode. > > 1/3: Bump gnu version namespace > > This is important to be done first so that once build of gnu versioned > namespace is fixed there is no chance to

Re: Fix gnu versioned namespace mode 01/03

Hi François As you know I am keen to see this land - but having had some experience with applying previous patches to actual toolchain builds .. > On 4 May 2024, at 22:11, François Dumont wrote: > > libstdc++: Bump gnu versioned namespace to __9 I think that the namespace version should be

Re: Trait built-in naming convention

> On 2 May 2024, at 20:30, Ken Matsui wrote: > > On Thu, May 2, 2024 at 10:54 AM Marek Polacek wrote: >> >> On Thu, May 02, 2024 at 08:37:53PM +0300, Ville Voutilainen wrote: >>> On Thu, 2 May 2024 at 20:25, Ken Matsui wrote: > There was some discussion of how to name the built-ins

[pushed] Objective-C, NeXT, v2: Correct a regression in code-gen.

nge. (build_v2_protocol_list_address_table): Likewise. Signed-off-by: Iain Sandoe --- gcc/objc/objc-next-runtime-abi-02.cc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/gcc/objc/objc-next-runtime-abi-02.cc b/gcc/objc/objc-next-runtime-abi-02.cc index cdf559b9bea..248ef641281 100

Re: [PATCH v2] [testsuite] require sqrt_insn effective target where needed

Hi Folks, > On 23 Apr 2024, at 09:59, Kewen.Lin wrote: > > Hi, > > on 2024/4/22 17:56, Alexandre Oliva wrote: >> This patch takes feedback received for 3 earlier patches, and adopts a >> simpler approach to skip the still-failing tests, that I believe to be >> in line with ppc maintainers'

[PATCH] libgfortran: Fix up the autoreconf warnings.

rate. Signed-off-by: Iain Sandoe --- libgfortran/Makefile.am | 1431 +++--- libgfortran/Makefile.in | 9848 ++- 2 files changed, 4126 insertions(+), 7153 deletions(-) diff --git a/libgfortran/Makefile.am b/libgfortran/Makefile.am index 9f8a4f69863..8bef1

Re: Request for testing on non-Linux targets; remove special casing of /usr/lib and /lib from the driver

Hi Andrew, > On 17 Apr 2024, at 14:59, Rainer Orth wrote: >> The driver currently will remove "/lib" and "/usr/lib" from the library >> path that gets passed to the linker because it considers them as paths that >> the linker will already known to search. But this is not true for newer >>

Re: [PATCH] c++/modules: optimize tree flag streaming

Hi Patrick, > On 10 Apr 2024, at 17:33, Jason Merrill wrote: > > On 4/10/24 11:26, Patrick Palka wrote: >> On Wed, 10 Apr 2024, Patrick Palka wrote: >>> >>> On Tue, 9 Apr 2024, Jason Merrill wrote: >>> On 2/16/24 10:06, Patrick Palka wrote: > On Thu, 15 Feb 2024, Patrick Palka wrote:

Re: [PATCH] libstdc++: Compile std::allocator instantiations as C++20

> On 11 Apr 2024, at 18:33, Ville Voutilainen > wrote: > > On Thu, 11 Apr 2024 at 20:22, Jonathan Wakely wrote: >> >> I'm considering this late patch for gcc-14 to workaround an issue >> discovered by a recent Clang change. >> >> I'm not yet sure if Clang is right to require these

Re: [PATCH/RFC] On the use of -funreachable-traps to deal with PR 109627

> On 9 Apr 2024, at 08:53, Iain Sandoe wrote: > > > >> On 9 Apr 2024, at 08:48, Jakub Jelinek wrote: >> >> On Tue, Apr 09, 2024 at 09:44:01AM +0200, Richard Biener wrote: >>> (why not do it at each such switch?) >> >> Because the tr

[PATCH/RFC] target, hooks: Allow a target to trap on unreachable [PR109267].

New. * doc/invoke.texi: Document -munreachable-traps. * doc/tm.texi: Regenerate. * doc/tm.texi.in: Document TARGET_UNREACHABLE_SHOULD_TRAP. * target.def (TARGET_UNREACHABLE_SHOULD_TRAP): New hook. Signed-off-by: Iain Sandoe --- gcc/builtins.cc| 7 +++ gcc/con

Re: [PATCH] build: Check for cargo when building rust language

Hi Arthur, > On 9 Apr 2024, at 13:01, Arthur Cohen wrote: > > On 4/9/24 10:55, Iain Sandoe wrote: >> Hi Arthur, >>> On 9 Apr 2024, at 11:40, Arthur Cohen wrote: >>> On 4/9/24 09:47, John Paul Adrian Glaubitz wrote: >>>> Hello, >>>&

Re: [PATCH] build: Check for cargo when building rust language

Hi Arthur, > On 9 Apr 2024, at 11:40, Arthur Cohen wrote: > On 4/9/24 09:47, John Paul Adrian Glaubitz wrote: >> Hello, >> On Mon, 2024-04-08 at 18:33 +0200, pierre-emmanuel.pa...@embecosm.com wrote: >>> The rust frontend requires cargo to build some of it's components, >>> it's presence was

Re: [PATCH/RFC] On the use of -funreachable-traps to deal with PR 109627

> On 9 Apr 2024, at 08:48, Jakub Jelinek wrote: > > On Tue, Apr 09, 2024 at 09:44:01AM +0200, Richard Biener wrote: >> (why not do it at each such switch?) > > Because the traps would then be added even to the bbs which later > end up in the middle of the function. If we defer the

[PATCH/RFC] On the use of -funreachable-traps to deal with PR 109627

Hi PR 109627 is about functions that have had their bodies completely elided, but still have the wrappers for EH frames (either .cfi_xxx or LFSxx/LFExx). These are causing issues for some linkers because such functions result in FDEs with a 0 code extent. The simplest representation of this

[pushed] Darwin: Sync coverage specs with gcc/gcc.cc.

ate coverage specs. Signed-off-by: Iain Sandoe --- gcc/config/darwin.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/config/darwin.h b/gcc/config/darwin.h index 31019a0c49d..c09b9e9dc94 100644 --- a/gcc/config/darwin.h +++ b/gcc/config/darwin.h @@ -406,7 +40

[pushed] testsuite, Darwin: Account for block labels in function body scans.

Log: * lib/scanasm.exp: Add 'LB*' to the local labels that are ignored for Darwin. Signed-off-by: Iain Sandoe --- gcc/testsuite/lib/scanasm.exp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/testsuite/lib/scanasm.exp b/gcc/testsuite/lib/scanasm.exp index 741a5a04

Re: [PATCH] libstdc++: Fix infinite loop in std::istream::ignore(n, delim) [PR93672]

> On 4 Apr 2024, at 16:29, Jonathan Wakely wrote: > > I would appreciate more eyes on this to confirm my conclusions about > negative int_type values, and the proposed fix, make sense. > > Tested x86_64-linux. > > -- >8 -- > > A negative value for the delim value passed to

Re: [PATCH] Include safe-ctype.h after C++ standard headers, to avoid over-poisoning

Hi Richard, > On 7 Mar 2024, at 13:40, FX Coudert wrote: > >> I think it's an obvious change ... > > Thanks, pushed. > > Dimitry, I suggest you post the second patch for review. Given that the two patches here (for https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111632) were considered obvious

[PATCH] libphobos, Darwin: Enable libphobos for most Darwin.

t;= 12. Signed-off-by: Iain Sandoe --- libphobos/configure.tgt | 9 + 1 file changed, 9 insertions(+) diff --git a/libphobos/configure.tgt b/libphobos/configure.tgt index 13879380416..7159688 100644 --- a/libphobos/configure.tgt +++ b/libphobos/configure.tgt @@ -27,6 +27,9 @@ case &qu

Ping: [PATCH] jit: Ensure ssize_t is defined.

> On 29 Jan 2024, at 11:26, Iain Sandoe wrote: > I guess the solution here depends on the scope over which we expect > the header to be used. > >> On 28 Jan 2024, at 23:13, Iain Sandoe wrote: >>> On 28 Jan 2024, at 21:25, Eric Gallager wrote: >>> On Sun,

Re: [PATCH] jit, Darwin: Implement library exports list.

Hi David, > On 25 Jan 2024, at 10:16, Iain Sandoe wrote: > >> On 24 Jan 2024, at 18:31, David Malcolm wrote: >> >> On Tue, 2024-01-16 at 11:10 +, Iain Sandoe wrote: >>> Tested on x86_64, i686 Darwin and x86_64 Linux, >>> OK for trunk? when ? &

[pushed] testsuite: Remove duplicate -lgcov [PR114034]

so there is no need to put an extra one in the options. PR testsuite/114034 gcc/testsuite/ChangeLog: * g++.dg/gcov/gcov-dump-1.C: Remove extra -lgcov. * g++.dg/gcov/gcov-dump-2.C: Likewise. Signed-off-by: Iain Sandoe --- gcc/testsuite/g++.dg/gcov/gcov-dump-1.C | 2 +-

[pushed] testsuite, Darwin: Allow for an undefined symbol [PR114036].

ote that the symbol is expected to be undefined. PR testsuite/114036 gcc/testsuite/ChangeLog: * gcc.misc-tests/gcov-14.c: Allow for 'Foo' to be undefined on Darwin link lines. Signed-off-by: Iain Sandoe --- gcc/testsuite/gcc.misc-tests/gcov-14.c | 2 +- 1 file changed

[pushed] Darwin: Correct a version check.

so use 700 in the check. gcc/ChangeLog: * config/darwin.cc (darwin_override_options): Update the clang major version value in the dsymutil check. Signed-off-by: Iain Sandoe --- gcc/config/darwin.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/con

[pushed] Darwin: Do not emit .macinfo when dsymutil cannot consume it.

den by -gstrict-dwarf (although the objects will contain macinfo; dsymutil will not produce a .dSYM with it). gcc/ChangeLog: * config/darwin.cc (darwin_override_options): Reduce the debug level to 2 if dsymutil cannot handle .macinfo sections. Signed-off-by: Iain Sandoe --- gcc/con

[pushed] testsuite, Darwin: Update bad-mapper-1 after libiberty changes.

gcc/testsuite/ChangeLog: * g++.dg/modules/bad-mapper-1.C: Update expected test output for earlier Darwin. Signed-off-by: Iain Sandoe --- gcc/testsuite/g++.dg/modules/bad-mapper-1.C | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gcc/testsuite/g++.dg/modules/bad-map

[pushed] testsuite, Darwin: Use the IOKit framework in framework-1.c [PR114049].

Kit. PR target/114049 gcc/testsuite/ChangeLog: * gcc.dg/framework-1.c: Use an IOKit header instead of a Kernel one. Signed-off-by: Iain Sandoe --- gcc/testsuite/gcc.dg/framework-1.c | 8 +++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/gcc/testsuite/gcc

[pushed] libstdc++: Sync the atomic_link_flags implementation with GCC.

led libatomic. Signed-off-by: Iain Sandoe --- libstdc++-v3/testsuite/lib/dg-options.exp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libstdc++-v3/testsuite/lib/dg-options.exp b/libstdc++-v3/testsuite/lib/dg-options.exp index bc387d17ed7..00ca678a53a 100644 --- a/libstdc++

[PATCH] libstdc++, Darwin: Do not use dev/null as the file for executables.

-v3/ChangeLog: * testsuite/lib/libstdc++.exp (v3_target_compile): Instead of /dev/null, use a temporary file for test executables on Darwin. Signed-off-by: Iain Sandoe --- libstdc++-v3/testsuite/lib/libstdc++.exp | 20 +++- 1 file changed, 19 insertions(+), 1 delet

[pushed] Darwin: Fix bootstrap break on X86.

off-by: Iain Sandoe --- gcc/config/i386/darwin.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gcc/config/i386/darwin.h b/gcc/config/i386/darwin.h index 2f773924d6e..12cdc34a19e 100644 --- a/gcc/config/i386/darwin.h +++ b/gcc/config/i386/darwin.h @@ -109,8 +109,8 @@ along with

Re: [PATCH] Fix libcc1plugin and libc1plugin to avoid poisoned identifiers

Hi Dimitry, > On 7 Mar 2024, at 16:48, Dimitry Andric wrote: > > Ref: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=111632 > > Use INCLUDE_VECTOR before including system.h, instead of directly > including , to avoid running into poisoned identifiers. I would say that the patch itself is

Re: [PATCH] Include safe-ctype.h after C++ standard headers, to avoid over-poisoning

Hi Dimitry, folks, > On 6 Mar 2024, at 23:02, Dimitry Andric wrote: > > On 6 Mar 2024, at 15:57, FX Coudert wrote: >> >>> Hmm I recall trying it and finding a problem - was there some different fix >>> applied >>> in the end? >> >> The bug is still open, I don’t think a patch was applied,

Re: [PATCH] Include safe-ctype.h after C++ standard headers, to avoid over-poisoning

> On 6 Mar 2024, at 13:54, Sam James wrote: > > FX Coudert writes: > >> I would like to patch this patch from September 2023: >> https://gcc.gnu.org/pipermail/gcc-patches/2023-September/631611.html >> >> This bug is now hitting macOS in the latest version of Xcode (it was >> originally

Re: [C++ coroutines] Initial implementation pushed to master.

> On 5 Mar 2024, at 17:31, H.J. Lu wrote: > > On Sat, Jan 18, 2020 at 4:54 AM Iain Sandoe wrote: >> >> 2020-01-18 Iain Sandoe >> >>* Makefile.in: Add coroutine-passes.o. >>* builtin-types.def (BT_CONST_SIZE

Re: [PATCH] lto, Darwin: Fix offload section names.

Hi Thomas, > On 29 Feb 2024, at 14:37, Thomas Schwinge wrote: > On 2024-01-16T15:00:16+, Iain Sandoe wrote: >> Currently, these section names have wrong syntax for Mach-O. >> Although they were added some time ago; recently added tests are >> now emitting t

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

> On 21 Feb 2024, at 23:36, Iain Sandoe wrote: > >> On 21 Feb 2024, at 23:06, Jason Merrill wrote: >> >> On 2/20/24 00:45, Alexandre Oliva wrote: >>> On Feb 16, 2024, Jason Merrill wrote: >>>> So, for stage2+, let's add just prev- libgcc. >&

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

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 with rust - >> since the rust folks are talking a

Re: [PATCH v1 03/13] aarch64: Mark x18 register as a fixed register for MS ABI

> On 22 Feb 2024, at 17:45, Andrew Pinski wrote: > > On Thu, Feb 22, 2024 at 3:56 AM Richard Earnshaw (lists) > wrote: >> >> On 21/02/2024 18:30, Evgeny Karpov wrote: >>> >> +/* X18 reserved for the TEB on Windows. */ >> +#ifdef TARGET_ARM64_MS_ABI >> +# define FIXED_X18 1 >> +# define

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

> On 21 Feb 2024, at 23:06, Jason Merrill wrote: > > On 2/20/24 00:45, Alexandre Oliva wrote: >> On Feb 16, 2024, Jason Merrill wrote: >>> So, for stage2+, let's add just prev- libgcc. >> I'm pretty sure this will break bootstrap-lean where libgcc_s isn't a >> system library, and we're

Re: [PATCH] libgccjit: Add option to allow special characters in function names

> On 20 Feb 2024, at 20:50, David Malcolm wrote: > > On Thu, 2024-02-15 at 17:08 -0500, Antoni Boucher wrote: >> Hi. >> This patch adds a new option to allow special characters like . and $ >> in function names. >> This is useful to allow for mangling using those characters. >> Thanks for the

[PATCH] libgcc, aarch64: Allow for BE platforms in heap trampolines.

ons as a byte array so that the order is independent of memory byte order. (struct aarch64_trampoline): Likewise. Signed-off-by: Iain Sandoe --- libgcc/config/aarch64/heap-trampoline.c | 30 - 1 file changed, 15 insertions(+), 15 deletions(-) diff --git a/lib

[PATCH] aarch64: Allow aarch64-linux-muscl for heap trampolines [PR113971].

gcc/ChangeLog: * config/aarch64/heap-trampoline.c: Allow all linux variants. Signed-off-by: Iain Sandoe --- libgcc/config/aarch64/heap-trampoline.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/libgcc/config/aarch64/heap-trampoline.c b/libgcc/config/aarch64/h

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

> On 16 Feb 2024, at 21:05, Jason Merrill wrote: > > On 2/14/24 18:33, Iain Sandoe wrote: >>> On 14 Feb 2024, at 22:59, Iain Sandoe wrote: >>>> On 12 Feb 2024, at 19:59, Jason Merrill wrote: >>>> >>>> On 2/10/24 07:30, Iain Sandoe

[PATCH] libstdc++, Darwin: Handle a linker warning [PR112397].

.am: If we are building for Darwin, then suppress hot/cold partitioning for the array allocators. * libsupc++/Makefile.in: Regenerated. Signed-off-by: Iain Sandoe Co-authored-by: Jonathan Wakely --- libstdc++-v3/configure | 35 +++--- libstdc++

[PATCH] libiberty: Fix error return value in pex_unix_exec_child [PR113957].

utcome for the routine. Signed-off-by: Iain Sandoe --- libiberty/pex-unix.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libiberty/pex-unix.c b/libiberty/pex-unix.c index af98062a94c..f3a1cc95ada 100644 --- a/libiberty/pex-unix.c +++ b/libiberty/pex-unix.c @@ -695,6 +695,7 @@ pex_unix_exec

Re: [PATCH] aarch64, acle header: Cast uint64_t pointers to DIMode.

> On 15 Feb 2024, at 18:05, Richard Sandiford wrote: > > Iain Sandoe writes: >>> On 5 Feb 2024, at 14:56, Iain Sandoe wrote: >>> >>> Tested on aarch64-linux,darwin and a cross from aarch64-darwin to linux, >>> OK for trunk, or some alternative i

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

> On 14 Feb 2024, at 22:59, Iain Sandoe wrote: >> On 12 Feb 2024, at 19:59, Jason Merrill wrote: >> >> On 2/10/24 07:30, Iain Sandoe wrote: >>>> On 10 Feb 2024, at 12:07, Jason Merrill wrote: >>>> >>>> On 2/10/24 05:46, Iain Sandoe

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

> On 12 Feb 2024, at 19:59, Jason Merrill wrote: > > On 2/10/24 07:30, Iain Sandoe wrote: >>> On 10 Feb 2024, at 12:07, Jason Merrill wrote: >>> >>> On 2/10/24 05:46, Iain Sandoe wrote: >>>>> On 9 Feb 2024, at 23:21, Iain Sandoe wrote:

Re: [PATCH v2] x86: Support x32 and IBT in heap trampoline

> On 14 Feb 2024, at 18:12, H.J. Lu wrote: > > On Tue, Feb 13, 2024 at 8:46 AM Jakub Jelinek wrote: >> >> On Tue, Feb 13, 2024 at 08:40:52AM -0800, H.J. Lu wrote: >>> Add x32 and IBT support to x86 heap trampoline implementation with a >>> testcase. >>> >>> 2024-02-13 Jakub Jelinek >>>

Re: [PATCH] x86, libgcc: Implement ia32 basic heap trampoline [PR113855].

> On 10 Feb 2024, at 17:46, Jakub Jelinek wrote: > > On Sat, Feb 10, 2024 at 05:14:44PM +0000, Iain Sandoe wrote: >> PR target/113855 >> >> gcc/ChangeLog: >> >> * config/i386/darwin.h (DARWIN_HEAP_T_LIB): Moved to be >> availa

[PATCH] x86, libgcc: Implement ia32 basic heap trampoline [PR113855].

I have so far tested this on i686-darwin17 and on x86_64-linux (with 32b multilib )with the following permutations: C (dg.exp=*nest*), Ada : \{-m64,-m32\}\{,-ftrampoline-impl=heap\}\{,-shared-libgcc\} Fortran \{-m64,-m32\}\{,-ftrampoline-impl=heap\} The only fails [gnat] are expected (scanning

Re: [PATCH] Darwin, testsuite: -multiply_defined is obsolete

> On 10 Feb 2024, at 14:54, FX Coudert wrote: > > With Xcode 15, gcc.dg/ssp-2.c fails due to a warning: -multiply_defined is > obsolete > > The patches ignores the warning when present. > OK to push? yes, thanks. Iain … although part of me is curious about whether we still have any

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

> On 10 Feb 2024, at 12:07, Jason Merrill wrote: > > On 2/10/24 05:46, Iain Sandoe wrote: >>> On 9 Feb 2024, at 23:21, Iain Sandoe wrote: >>> >>> >>> >>>> On 9 Feb 2024, at 10:56, Iain Sandoe wrote: >>>>> On 8 Feb 2

Re: [PATCH] Darwin, testsuite: -bind_at_load is deprecated

Hi FX, > On 10 Feb 2024, at 11:58, FX Coudert wrote: > > With Xcode 15, gcc.dg/darwin-ld-2.c fails due to a warning: > ld: warning: -bind_at_load is deprecated on macOS > > The patches ignores the warning when present. > OK to push? Yes, thanks. Iain I guess for GCC-15 we might need to see

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

> On 9 Feb 2024, at 23:21, Iain Sandoe wrote: > > > >> On 9 Feb 2024, at 10:56, Iain Sandoe wrote: >>> On 8 Feb 2024, at 21:44, Jason Merrill wrote: >>> >>> On 2/8/24 12:55, Paolo Bonzini wrote: >>>> On 2/8/24 18:16, Jason Merrill

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

> On 9 Feb 2024, at 10:56, Iain Sandoe wrote: >> On 8 Feb 2024, at 21:44, Jason Merrill wrote: >> >> On 2/8/24 12:55, Paolo Bonzini wrote: >>> On 2/8/24 18:16, Jason Merrill wrote: >>>>>> >>>>> >>>>> Hmm. In

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

> On 8 Feb 2024, at 21:44, Jason Merrill wrote: > > On 2/8/24 12:55, Paolo Bonzini wrote: >> On 2/8/24 18:16, Jason Merrill wrote: > Hmm. In stage 1, when we build with the system gcc, I'd think we want the just-built gnat1 to find the system libgcc. In stage

[pushed] libgcc, Darwin: Update symbol exports to include bitint and bf.

Tested on i686-darwin8, x86_64-darwin{14,17,19,21,23} pushed to trunk. thanks, Iain --- 8< --- Some exports were missed from the GCC-13 cycle, these are added here along with the bitint-related ones added in GCC-14. libgcc/ChangeLog: * config/i386/libgcc-darwin.ver: Export bf and

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

> On 8 Feb 2024, at 19:25, Jason Merrill wrote: > > On 2/8/24 12:51, Iain Sandoe wrote: >>> On 8 Feb 2024, at 17:16, Jason Merrill wrote: >>> >>> On 2/8/24 12:12, Jason Merrill wrote: >>>> On 2/8/24 10:04, Iain Sandoe wrote: >>>>

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

> On 8 Feb 2024, at 17:16, Jason Merrill wrote: > > On 2/8/24 12:12, Jason Merrill wrote: >> On 2/8/24 10:04, Iain Sandoe wrote: >>> Hi Jason, >>> >>> I have tested this on modern Darwin (with libc++ as the system library) and >>> on &g

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

Hi Jason, I have tested this on modern Darwin (with libc++ as the system library) and on older Darwin, where we do see the issue - because the system linker is written in C++ and links with libstdc++ (so sometimes we get a crash, or worse unpredictable beahviour). - For modern Darwin [ >

Re: [PATCH] aarch64, acle header: Cast uint64_t pointers to DIMode.

> On 5 Feb 2024, at 14:56, Iain Sandoe wrote: > > Tested on aarch64-linux,darwin and a cross from aarch64-darwin to linux, > OK for trunk, or some alternative is needed? Hmm.. apparently, this fails the linaro pre-commit CI for g++ with: error: invalid conversion from 'long in

[PATCH] aarch64, acle header: Cast uint64_t pointers to DIMode.

Tested on aarch64-linux,darwin and a cross from aarch64-darwin to linux, OK for trunk, or some alternative is needed? thanks Iain --- 8< --- Currently, most of the acle tests fail on the Darwin port because DI mode is "long" and uint64 is "long long". The fix for this used in other headers is

[pushed] testsuite, Darwin: Allow for undefined symbols in shared test.

tested on i686, x86_64 Darwin, x86_64, aarch64 linux, pushed to trunk, thanks, Iain --- 8< --- Darwin's linker defaults to error on undefined (which makes it look as if we do not support shared, leading to tests being marked incorrectly as unsupported). This fixes the issue by allowing the

Re: [PATCH] libgcc: Avoid warnings on __gcc_nested_func_ptr_created [PR113402]

> On 1 Feb 2024, at 08:51, Richard Biener wrote: > > On Thu, Feb 1, 2024 at 9:23 AM Jakub Jelinek wrote: >> >> On Wed, Jan 31, 2024 at 01:04:20PM +0100, Jakub Jelinek wrote: >>> On Sun, Jan 28, 2024 at 11:02:33AM +, Iain Sandoe wrote: >>>>

Re: [PATCH] libgcc: Fix up i386/t-heap-trampoline [PR113403]

> On 1 Feb 2024, at 08:22, Jakub Jelinek wrote: > > On Wed, Jan 31, 2024 at 12:59:27PM +0100, Jakub Jelinek wrote: >> On Sun, Jan 28, 2024 at 02:07:32PM +0000, Iain Sandoe wrote: >>> --- a/libgcc/config/aarch64/t-heap-trampoline >>> +++ b/libgcc/config/aarch

Re: [PATCH] libstdc++: Enable std::text_encoding for darwin and FreeBSD

Hi Jonathan, > On 30 Jan 2024, at 15:02, Jonathan Wakely wrote: > > This should fix the std/text_encoding/* FAILs that Iain sees on darwin. > I assume it will make it work for FreeBSD too. > > I won't push this until I hear it works for at least one of those. It works on

[PATCH] testsuite, ubsan: Add libstdc++ deps where required.

tested on i686, x86_64 (and aarch64) Darwin, x86_64, aarch64 Linux, OK for trunk? thanks Iain --- 8< --- We use the ubsan tests from both C, C++, D and Fortran. the sanitizer libraries link to libstdc++. When we are using the C/gdc/gfortran driver, and the target might require a path to the

[PATCH] testsuite, asan, hwsan: Add libstdc++ deps where required.

tested on i686, x86_64 (and aarch64) Darwin, x86_64, aarch64 Linux, OK for trunk? thanks Iain --- 8< --- We use the shared asan/hwasan from both C,C++,D and Fortran. The sanitizer libraries link to libstdc++. When we are using the C/gdc/gfortran driver, and the target might require a path to

Re: [PATCH] jit: Ensure ssize_t is defined.

Hi David, I guess the solution here depends on the scope over which we expect the header to be used. > On 28 Jan 2024, at 23:13, Iain Sandoe wrote: >> On 28 Jan 2024, at 21:25, Eric Gallager wrote: >> On Sun, Jan 28, 2024 at 6:45 AM Iain Sandoe wrote: >>> >>&

Re: [PATCH] jit: Ensure ssize_t is defined.

> On 28 Jan 2024, at 21:25, Eric Gallager wrote: > > On Sun, Jan 28, 2024 at 6:45 AM Iain Sandoe wrote: >> >> Tested on i686, x86_64 Darwin, x86_64 Linux, >> OK for trunk? >> >> --- 8< --- >> >> On some targets it seems that ssize_

[PATCH] testsuite, Objective-C++: Update link flags [PR112863].

Tested on i686, x86_64, aarch64 Darwin, x86_64, aarch64 Linux, OK for trunk? thanks, Iain --- 8< --- These regressions are caused by missing or duplicate runpaths which now fire linker warnings. We need to add options to locate libobjc (and on Darwin libobjc-gnu) along with libstdc++. Usually

[PATCH] testsuite, libphobos: Update link flags [PR112864].

Tested on i686, x86_64, aarch64 Darwin, x86_64, aarch64 Linux, OK for trunk? thanks, Iain --- 8< --- The regressions here are primarily from duplicated '-B' additions. We remove the duplicate on the link line. We also make sure that platforms with extensions other than .so for shared libs will

[PATCH] testsuite, gfortan: Update link flags [PR112862].

Tested on i686, x86_64, aarch64 Darwin, x86_64, aarch64 Linux, OK for trunk? thanks, Iain --- 8< --- The regressions here are caused by two issues: 1. In some cases there is no generated runpath for libatomic 2. In other cases there are duplicate paths. This patch simplifies the addition of

[PATCH] testsuite, GDC: Update link flags [PR112861].

Tested on i686, x86_64, aarch64 Darwin, x86_64, aarch64 Linux, OK for trunk? thanks, Iain --- 8< --- The regressions here are because we do not generate a runpath for the uninstalled libstdc++. This patch updates the link flags handling to simplify it. We need to add options to locate both

[PATCH 2/2] libgcc: Make heap trampoline support dynamic [PR113403].

In order to handle system security constraints during GCC build and test and that most platform versions cannot link to libgcc_eh since the unwinder there is incompatible with the system one. 1. We make the support functions weak definitions. 2. We include them as a CRT for platform conditions

[PATCH 1/2] libgcc: Make heap trampoline support dynamic [PR113403].

This removes the heap trampoline support functions from libgcc.a and adds them to libgcc_eh.a. They are also present in libgcc_s. PR libgcc/113403 libgcc/ChangeLog: * config/aarch64/t-heap-trampoline: Move the heap trampoline support functions from libgcc.a to

[PATCH 0/2] libgcc: Make heap trampoline support dynamic [PR113403].

This series follows Jakub's suggestion in the PR (for Linux in the first patch) and handles Darwin-specific cases (in the second). Sorry this has taken a while, the Darwin permutations had some glitches which necessitated re-tests on several OS versions. Tested on x86_64 (and aarch64) Darwin,

[pushed] Objective-C, Darwin: Do not overalign CFStrings and Objective-C metadata.

ise. (build_ehtype): Likewise. * objc-runtime-shared-support.cc (generate_strings): Likewise. Signed-off-by: Iain Sandoe --- gcc/config/darwin.cc| 1 + gcc/objc/objc-next-runtime-abi-02.cc| 18 +++--- gcc/objc/objc-runtime-shared-support.cc | 4 3 files chan

[PATCH] jit: Ensure ssize_t is defined.

ble since that is where Posix specifies that ssize_t is defined. gcc/jit/ChangeLog: * libgccjit.h: Conditionally include where it is available to ensure declaration of ssize_t. Signed-off-by: Iain Sandoe --- gcc/jit/libgccjit.h | 3 +++ 1 file changed, 3 insertions(+) diff --

[pushed] testsuite, Objective-C: Fix duplicate libobjc cases.

ear on the link line leading to spurious warnings from Darwin's new linker. gcc/testsuite/ChangeLog: * obj-c++.dg/encode-10.mm: Remove unneeded '-lobjc' option addition. * obj-c++.dg/encode-9.mm: Likewise. Signed-off-by: Iain Sandoe --- gcc/testsuite/obj-c++.dg/encode-10.mm | 1 -

Re: [PATCH v3] Fix __builtin_nested_func_ptr_{created, deleted} symbol versions [PR113402]

> On 18 Jan 2024, at 15:05, Jakub Jelinek wrote: > > On Thu, Jan 18, 2024 at 02:59:23PM +0000, Iain Sandoe wrote: >> --- a/gcc/builtins.cc >> +++ b/gcc/builtins.cc >> @@ -8416,6 +8416,11 @@ expand_builtin (tree exp, rtx target, rtx subtarget, >&g

Re: [PATCH] jit, Darwin: Implement library exports list.

Hi David, > On 24 Jan 2024, at 18:31, David Malcolm wrote: > > On Tue, 2024-01-16 at 11:10 +0000, Iain Sandoe wrote: >> Tested on x86_64, i686 Darwin and x86_64 Linux, >> OK for trunk? when ? >> thanks, >> Iain > > Hi Iain, thanks for the patch. > >

Re: [PATCH] testsuite, jit: Stabilize error output.

gentle ping, with the increasing use of CI, it seems an idea to tackle this sooner rather than later. thanks Iain > On 16 Jan 2024, at 11:12, Iain Sandoe wrote: > > Tested on x86_64, i686 Darwin, x86_64 Linux, > OK for trunk? When? > thanks > Iain > > --- 8< ---

Re: [PATCH] jit, Darwin: Implement library exports list.

gentle ping, this fixes quite a few of the new jit fails on darwin. thanks Iain > On 16 Jan 2024, at 11:10, Iain Sandoe wrote: > > Tested on x86_64, i686 Darwin and x86_64 Linux, > OK for trunk? when ? > thanks, > Iain > > --- 8< --- > > Currently, we have no

[pushed] Darwin, configure: Handle a missing substitution.

exp when ENABLE_DARWIN_AT_RPATH_TRUE is not '#'. Signed-off-by: Iain Sandoe --- gcc/Makefile.in | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/Makefile.in b/gcc/Makefile.in index deb12e17d25..95caa54a52b 100644 --- a/gcc/Makefile.in +++ b/gcc/Makefile.in @@ -4303,7 +430

[PATCH v3] Fix __builtin_nested_func_ptr_{created, deleted} symbol versions [PR113402]

cc-std.ver.in (GCC_7.0.0): Likewise and then move __gcc_nested_func_ptr_created and __gcc_nested_func_ptr_deleted from this symbol version to ... (GCC_14.0.0): ... this one. Signed-off-by: Iain Sandoe Co-authored-by: Jakub Jelinek --- gcc/builtins.cc |

[pushed] Objective-C/C++: Ensure sufficient setup for the preprocessor.

res that these are initialized before pre-process-only jobs are run. gcc/objc/ChangeLog: * objc-act.cc (objc_init): Initialize interface and class name hash maps before the preprocessor uses them. Signed-off-by: Iain Sandoe --- gcc/objc/objc-act.cc | 7 +-- 1 file changed, 5 insertions(+)

[pushed] Darwin: Suppress adding embedded rpaths for earlier OS versions.

. gcc/ChangeLog: * config/darwin.h (DARWIN_RPATH_SPEC): Arrange for the %P spec to be conditional on macosx-version-min. Signed-off-by: Iain Sandoe --- gcc/config/darwin.h | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/gcc/config/darwin.h b/gcc/config/da

[pushed] Darwin: Fix a typo in Objective-C meta-data.

rom the properties of the DECLs. gcc/ChangeLog: * config/darwin.cc (darwin_objc1_section): Use the correct meta-data version for constant strings. (machopic_select_section): Assert if we fail to handle CFString sections as Obejctive-C meta-data or drectly. Signed-off-by: I

[pushed] Darwin: Fix constant CFString code-gen [PR105522].

ode. (darwin_libc_has_function): Likewise. (darwin_build_constant_cfstring): Create an anonymous var to hold each CFString. * config/darwin.h (ASM_OUTPUT_LABELREF): Handle constant CFstrings. Signed-off-by: Iain Sandoe --- gcc/config/darwin.cc|

Re: [COMMITTED] rust_debug: Cast size_t values to unsigned long before printing.

Hi Arthur, > On 18 Jan 2024, at 10:30, Arthur Cohen wrote: > On 1/18/24 10:13, Rainer Orth wrote: >> Arthur Cohen writes: >>> Using %lu to format size_t values breaks 32 bit targets, and %zu is not >>> supported by one of the hosts GCC aims to support - HPUX >> But we do have uses of %zu in

Re: [PATCH v2] Fix __builtin_nested_func_ptr_{created, deleted} symbol versions [PR113402]

> On 17 Jan 2024, at 08:55, Iain Sandoe wrote: > > Tested on x86_64, aarch64 Darwin21 (which default to heap-based trampolines) > and on x86_64 Darwin19 and Linux (which default to executable stack > trampolines). > OK for trunk? Hmm.. maybe this is not right and the bui

[PATCH v2] Fix __builtin_nested_func_ptr_{created, deleted} symbol versions [PR113402]

ise and then move __gcc_nested_func_ptr_created and __gcc_nested_func_ptr_deleted from this symbol version to ... (GCC_14.0.0): ... this one. Signed-off-by: Iain Sandoe Co-authored-by: Jakub Jelinek --- gcc/builtins.def| 4 ++-- gcc/doc/invoke.t

  1   2   3   4   5   6   7   8   9   10   >