Re: [PATCH 1/3] testsuite: Unbork multilib testing on RISC-V (and any target really)

2023-05-31 Thread Iain Sandoe via Gcc-patches
> On 31 May 2023, at 18:57, Jeff Law via Gcc-patches > wrote: > > > > On 5/31/23 10:25, Vineet Gupta wrote: >> Multilib testing on trunk is currently busted (and surprisingly this >> affects any/all targets but it seems nobody cares). We currently get the >> following splat: > I wouldn't sa

[pushed] Darwin, PPC: Fix struct layout with pragma pack [PR110044].

2023-06-02 Thread Iain Sandoe via Gcc-patches
@David: I am not sure what sets the ABI on AIX (for Darwin, it is effectively "whatever the system compiler [Apple gcc-4] does") but from an inspection of the code, it seems that (if the platform should honour #pragma pack) a similar effect could be present there too. Tested on powerpc-apple-darwi

[pushed] Darwin: Use -platform_version when available [PR110624].

2023-07-13 Thread Iain Sandoe via Gcc-patches
tested on i688, x86_64 Darwin versions with/without support for the platform_version flag. Checked by Rainer on macOS14. Pushed to trunk thanks, Iain --- 8< --- Later versions of the static linker support a more flexible flag to describe the OS, OS version and SDK used to build the code. This r

[pushed] Darwin: Handle linker '-demangle' option.

2023-07-21 Thread Iain Sandoe via Gcc-patches
Tested with Darwin linker versions that do/do not support the option and on x86_64-linux-gnu, pushed to trunk, thanks Iain --- 8< --- Most of the Darwin linkers in use support this option which we will now pass by default (matching the Xcode clang impl.)> Signed-off-by: Iain Sandoe gcc/ChangeL

Re: [PING][PATCH][gdb/build] Fix build breaker with --enabled-shared

2022-07-12 Thread Iain Sandoe via Gcc-patches
Hi Tom > On 12 Jul 2022, at 14:42, Tom de Vries via Gcc-patches > wrote: > > [ dropped gdb-patches, since already applied there. ] > > On 6/27/22 15:38, Tom de Vries wrote: >> On 6/27/22 15:03, Tom de Vries wrote: >>> Hi, >>> >>> When building gdb with --enabled-shared, I run into: >>> ... >>

Re: [PATCH 1/2] Allow subtarget customization of CC1_SPEC

2022-07-24 Thread Iain Sandoe via Gcc-patches
Hi Sebastian, > On 25 Jul 2022, at 06:59, Sebastian Huber > wrote: > On 22/07/2022 16:02, Jose E. Marchesi wrote: >> I find "subtarget" confusing in this context. >> If it is about rtems.h, linux.h or sol2.h, wouldn't this be better >> called OS_CC1_SPEC or similar? These files specify configu

Re: [PATCH] tree-optimization/105679 - disable backward threading of unlikely entry

2022-07-31 Thread Iain Sandoe via Gcc-patches
Hi Richi, > On 29 Jul 2022, at 09:54, Richard Biener via Gcc-patches > wrote: > > The following makes the backward threader reject threads whose entry > edge is probably never executed according to the profile. That in > particular, for the testcase, avoids threading the irq == 1 check > on th

Re: [PATCH] tree-optimization/105679 - disable backward threading of unlikely entry

2022-08-01 Thread Iain Sandoe via Gcc-patches
> On 1 Aug 2022, at 09:21, Richard Biener via Gcc-patches > wrote: > > On Sun, 31 Jul 2022, Iain Sandoe wrote: > >> Hi Richi, >> >>> On 29 Jul 2022, at 09:54, Richard Biener via Gcc-patches >>> wrote: >>> >>> The following makes the backward threader reject threads whose entry >>> edge i

[pushed] modula-2, plugin: Fix Darwin bootstrap issues.

2023-08-13 Thread Iain Sandoe via Gcc-patches
Tested on x86_64 Darwin and x86_64 Linux, pushed to master, will also backport to 13 if it is needed there, thanks Iain --- 8< --- This corrects some typos in the suffix of the m2rte pluing that lead to a bootstrap fail on Darwin, where the suffix is not '.so'. On some versions of Darwin, the li

Re: Darwin: Replace environment runpath with embedded [PR88590]

2023-08-18 Thread Iain Sandoe via Gcc-patches
Hi Joseph, > On 18 Aug 2023, at 21:17, Joseph Myers wrote: > > On Tue, 15 Aug 2023, FX Coudert via Gcc-patches wrote: > >> I am currently retesting the patches on various archs (Linux and Darwin) >> after a final rebase, but various previous versions were >> regression-tested, and have been s

Re: Darwin: Replace environment runpath with embedded [PR88590]

2023-08-18 Thread Iain Sandoe via Gcc-patches
> On 18 Aug 2023, at 23:59, Joseph Myers wrote: > > On Fri, 18 Aug 2023, Iain Sandoe via Gcc-patches wrote: > >> There is quite extensive Apple Developer documentation on delivering >> packages with co-installed libraries using @rpath (that is the intended >>

Re: [PATCH] Testsuite: fix analyzer tests on Darwin

2023-08-19 Thread Iain Sandoe via Gcc-patches
Hi FX, thanks for chasing these fails down, > On 19 Aug 2023, at 22:28, FX Coudert wrote: > > gcc.dg/analyzer/ currently has 80 failures on Darwin (both > x86_64-apple-darwin and aarch64-apple-darwin). All those come from two issues: > > 1. Many tests use memset() without including the head

[pushed] testsuite,objective-c++: Fix imported NSObjCRuntime.h.

2023-06-23 Thread Iain Sandoe via Gcc-patches
Tested on x86_64-darwin, pushed to trunk, thanks Iain --- 8< --- We have imported some headers from the GNUStep project to allow us to maintain the testsuite independent to changing versions of system headers. One of these headers has a macro that (now we have support for __has_feature) expands

[pushed] configure, Darwin: Ensure overrides to host-pie are passed to gcc configure.

2023-06-25 Thread Iain Sandoe via Gcc-patches
Tested on i686, powerpc, x86_64 and aarch64 darwin (for versions requiring PIE and also some that default to non-PIE). Also tested on x86_64-linux-gnu with and without --enable-host-pie. pushed to master, thanks Iain --- 8< --- The latest versions of Darwin on the Aarch64 platform mandate PIE

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

2023-06-25 Thread Iain Sandoe via Gcc-patches
Tested on various affected Darwin versions and on x86_64-linux-gnu OK for trunk? OK for 13.2? thanks Iain --- 8< --- When we make a select() that fails, there is an attempt to (a) diagnose why and (b) make a fallback. These actions are causing some tests to hang on some Darwin versions, this is

Re: [PATCH] configure: Implement --enable-host-bind-now

2023-06-27 Thread Iain Sandoe via Gcc-patches
> On 27 Jun 2023, at 16:31, Marek Polacek via Gcc-patches > wrote: > > On Tue, Jun 27, 2023 at 01:39:16PM +0200, Martin Jambor wrote: >> Hello, >> >> On Tue, May 16 2023, Marek Polacek via Gcc-patches wrote: >>> As promised in the --enable-host-pie patch, this patch adds another >>> configur

[pushed] libphobos, testsuite: Disable forkgc2 on Darwin [PR103944]

2023-07-01 Thread Iain Sandoe via Gcc-patches
From: Iain Sandoe This has been in use for some time across all the Darwin version supported by D. It has also been tested on x86_64-linux-gnu. Approved on irc by Iain Buclaw, pushed to main (and will be backported). thanks Iain --- 8< --- It hangs the testsuite (requiring manual intervention

[PATCH] libphobos: Handle Darwin Arm and AArch64 in fibre context asm.

2023-07-02 Thread Iain Sandoe via Gcc-patches
Tested on AArch64 (Arm64) Darwin on 11.x, 13.x and master, OK for trunk? and backports? thanks Iain --- 8< --- This code currently fails to build because it contains ELF- specific directives. This patch excludes those directives when the platform is Darwin. We do not expect switching fibres bet

[pushed] Darwin, Objective-C: Support -fconstant-cfstrings [PR108743].

2023-07-02 Thread Iain Sandoe via Gcc-patches
Tested on x86_64-darwin and with make pdf and an examination of the revised output. Pushed to trunk, thanks Iain --- 8< --- This support the -fconstant-cfstrings option as used by clang (and expect by some build scripts) as an alias to the target-specific -mconstant-cfstrings. The documentation

[pushed] testsuite, Darwin: Remove an unnecessary flags addition.

2023-07-03 Thread Iain Sandoe via Gcc-patches
This has been in use for some time in the Darwin branches that are used by downstream distributions. Re-tested on x86_64-darwin, pushed to trunk, thanks, Iain --- 8< --- The addition of the multiply_defined suppress flag has been handled for some considerable time now in the Darwin specs; remove

[pushed] testsuite, objective-c: Fix a testcase on Windows.

2023-02-15 Thread Iain Sandoe via Gcc-patches
tested by 'nightstrike' on Windows, and on x86_64-darwin21, pushed to master, thanks, Iain --- 8< --- Windows needs to use uintptr_t to represent an integral pointer type (long is not the right type there). Patch from 'nightstike'. Signed-off-by: Iain Sandoe gcc/testsuite/ChangeLog:

[pushed] testsuite, objective-c: Cater for Windows intptr type.

2023-02-16 Thread Iain Sandoe via Gcc-patches
Tested by nightstrike on Windows, and on x86_64 Linux and Darwin and i686-darwin, pushed to trunk, thanks Iain --- 8< --- This adjusts the diagnostic output matches to cater for the differences in intptr types on Windows. Patch from 'nightstrike'. Signed-off-by: Iain Sandoe gcc/testsuite/Chan

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

2023-03-10 Thread Iain Sandoe via Gcc-patches
Hi all, > On 9 Mar 2023, at 23:35, Sandra Loosemore via Gcc-patches > wrote: > > On 3/9/23 01:26, Richard Biener wrote: > >> SLES 12 has texinfo 4.13a, SLES 15 has texinfo 6.5. We still provide >> up-to-date GCC for SLES 12 but we can probably manage in some ways >> when the texinfo requireme

[pushed] Darwin, libgcc : Adjust min version supported for the OS.

2023-05-19 Thread Iain Sandoe via Gcc-patches
Tested across the Darwin range (this patch has been on the WIP branches for some time) and on x86_64-linux-gnu, for reference. pushed to trunk, thanks Iain --- 8< --- Tools from later versions of the OS deprecate or fail to support earlier OS revisions. Signed-off-by: Iain Sandoe libgcc/Change

[pushed] libobjc: Add local macros to support encode generation [P109913].

2023-05-22 Thread Iain Sandoe via Gcc-patches
This fixes bootstrap on powerpc-darwin, and was tested there for both GNU and NeXT Objective-C testsuite. Approved in the PR by Andrew, pushed to trunk, thanks, Iain --- 8< --- r14-976-g9907413a3a6aa3 alters code to use the preferred _P-style macros rather than direct comparisons of (for example

Re: [PATCH] Objective-C: don't require redundant -fno-objc-sjlj-exceptions for the NeXT v2 ABI

2021-08-03 Thread Iain Sandoe via Gcc-patches
> On 2 Aug 2021, at 22:37, Matt Jacobson via Gcc-patches > wrote: > >> On Aug 2, 2021, at 5:09 PM, Eric Gallager wrote: >> >> On Wed, Jul 28, 2021 at 11:36 PM Matt Jacobson via Gcc-patches >> wrote: >>> >>> As is, an invocation of GCC with -fnext-runtime -fobjc-abi-version=2 >>> crashes,

Re: [Patch v3 Fortran] Fix c_float128 and c_float128_complex on targets with 128-bit long double.

2021-08-11 Thread Iain Sandoe via Gcc-patches
Hi Folks > On 11 Aug 2021, at 11:55, Segher Boessenkool > wrote: > On Tue, Aug 10, 2021 at 04:46:11PM -0600, Sandra Loosemore wrote: >> OK. I used your wording verbatim for the first one. For the second >> one, I'm still pretty confused as I think it is at least theoretically >> possible on

Re: [PATCH] Objective-C: fix crash with -fobjc-nilcheck

2021-08-14 Thread Iain Sandoe via Gcc-patches
Hi Matt, > On 14 Aug 2021, at 09:14, Matt Jacobson via Gcc-patches > wrote: > > When -fobjc-nilcheck is enabled, messages that result in a struct type should > yield a zero-initialized struct when sent to nil. Currently, the frontend > crashes when it encounters this situation. This patch f

Re: [PATCH] configure: Allow a host makefile fragment to override PIE flag settings.

2021-08-25 Thread Iain Sandoe via Gcc-patches
Hi, > On 20 Aug 2021, at 11:29, Richard Sandiford wrote: > >>> Maybe it would be easier to have the makefile fragments determine >>> something like CODE_MODEL_CFLAGS, which can be "-fPIC", "-mdynamic-no-pic", >>> etc., and use: >>> >>> COMPILER += $(NO_PIE_CFLAGS) $(CODE_MODEL_CFLAGS) >> >> OK

Re: [PATCH] configure: Allow a host makefile fragment to override PIE flag settings.

2021-08-25 Thread Iain Sandoe via Gcc-patches
> On 25 Aug 2021, at 18:56, H.J. Lu via Gcc-patches > wrote: > > On Wed, Aug 25, 2021 at 10:51 AM H.J. Lu wrote: >> >> On Wed, Aug 25, 2021 at 10:42 AM Iain Sandoe wrote: >>> >>> Hi, >>> On 20 Aug 2021, at 11:29, Richard Sandiford wrote: >> Maybe it would be easier

Re: [PATCH] configure: Allow a host makefile fragment to override PIE flag settings.

2021-08-25 Thread Iain Sandoe via Gcc-patches
> On 25 Aug 2021, at 19:22, H.J. Lu wrote: > > On Wed, Aug 25, 2021 at 11:10 AM Iain Sandoe wrote: >> >> >> >>> On 25 Aug 2021, at 18:56, H.J. Lu via Gcc-patches >>> wrote: >>> >>> On Wed, Aug 25, 2021 at 10:51 AM H.J. Lu wrote: On Wed, Aug 25, 2021 at 10:42 AM Iain Sandoe

Re: [r11-3641 Regression] FAIL: gcc.dg/torture/pta-ptrarith-1.c -Os scan-tree-dump alias "ESCAPED = {[^\n}]* i f [^\n}]*}" on Linux/x86_64 (-m32 -march=cascadelake)

2020-10-04 Thread Iain Sandoe via Gcc-patches
H.J. Lu via Gcc-patches wrote: On Sat, Oct 3, 2020 at 5:57 PM Segher Boessenkool wrote: On Sat, Oct 03, 2020 at 12:21:04PM -0700, sunil.k.pandey via Gcc-patches wrote: On Linux/x86_64, c34db4b6f8a5d80367c709309f9b00cb32630054 is the first bad commit commit c34db4b6f8a5d80367c709309f9b00cb3

Re: Make 'autoreconf' work for 'gcc', 'libobjc' (was: [PATCH] regenerate configure files and config.h.in files)

2022-09-20 Thread Iain Sandoe via Gcc-patches
Hi! > On 20 Sep 2022, at 14:37, Thomas Schwinge wrote: > On 2022-08-25T11:42:01+0200, Martin Liška wrote: >> I wrote a scipt that runs autoconf in all folders that have configure.ac >> file and same for autoheader (where AC_CONFIG_HEADERS is present) and >> this is the output. >> >> The script

Re: [PATCH] Avoid depending on destructor order

2022-09-26 Thread Iain Sandoe via Gcc-patches
> On 26 Sep 2022, at 12:49, Thomas Neumann via Gcc-patches > wrote: > > Hi Claudiu, > >> This change prohibits compiling of ARC backend: >>> + gcc_assert (in_shutdown || ob); >> in_shutdown is only defined when ATOMIC_FDE_FAST_PATH is defined, >> while gcc_assert is outside of any ifdef. Pl

[PATCH] coroutines: Use cp_build_init_expr consistently.

2022-10-11 Thread Iain Sandoe via Gcc-patches
Tested on x86_64-darwin19, OK for master? thanks Iain -- >8 -- Now we have the TARGET_EXPR_ELIDING_P flag, it is important to ensure it is set properly on target exprs. The code here has a mixture of APIs used to build inits. This patch changes that to use cp_build_init_expr() where possible, s

[pushed] Darwin, libgcc: Fix build errors on powerpc-darwin8.

2022-03-04 Thread Iain Sandoe via Gcc-patches
PowerPC Darwin8 is the last version to use an unwind frame fallback routine. This had been omitted from the new shared EH library, along with one more header dependency that only fires there. tested on x86_64-darwin18, powerpc-darwin9 and cross to powerpc-darwin8 pushed to master, thanks Iain Sig

[pushed] Darwin: Fix a type mismatch warning for a non-GCC bootstrap compiler.

2022-03-04 Thread Iain Sandoe via Gcc-patches
DECL_MD_FUNCTION_CODE() returns an int, on one particular compiler the code in darwin_fold_builtin() triggers a warning. Fixed thus. tested on x86_64-darwin18, powerpc-darwin9, cross to powerpc-darwin8. pushed to master, thanks Iain Signed-off-by: Iain Sandoe gcc/ChangeLog: * config/d

[pushed] Darwin: Address a translation comment [PR104552].

2022-03-08 Thread Iain Sandoe via Gcc-patches
This amends an error message to correct punctuation and a little better wording. bootstrapped on x86_64-darwin18, pushed to master, thanks, Iain Signed-off-by: Iain Sandoe PR translation/104552 gcc/ChangeLog: * config/host-darwin.cc (darwin_gt_pch_get_address): Amend t

[PATCH 1/2] libcody: Do not use a dummy port number in getaddrinfo().

2022-03-13 Thread Iain Sandoe via Gcc-patches
The getaddrinfo() call requires either a non-null name for the server or a port service / number. In the code that opens a connection we have been calling this with a dummy port number of "0". Unfortunately this triggers a crashing bug in some BSD versions (and OSes importing that code). In this

[PATCH 2/2] c++tools: Work around a BSD bug in getaddrinfo().

2022-03-13 Thread Iain Sandoe via Gcc-patches
Some versions of the BSD getaddrinfo() call do not work with the specific input of "0" for the servname entry (a segv results). Since we are making the call with a dummy port number, the value is actually not important, other than it should be in range. Work around the BSD bug by using "1" instea

Re: [PATCH] doc: Document Solaris D bootstrap requirements [PR 103528]

2022-03-16 Thread Iain Sandoe via Gcc-patches
> On 16 Mar 2022, at 13:28, Rainer Orth wrote: > > This patch documents the Solaris-specific D bootstrap requirements. > > Tested by building and inspecting gccinstall.{pdf,info}. > > Ok for trunk? > > I've omitted the Darwin-specific stuff so far documented in PRs d/103577 > and d/103578:

Re: [PATCH] doc: Document Solaris D bootstrap requirements [PR 103528]

2022-03-16 Thread Iain Sandoe via Gcc-patches
Hi Rainer, > On 16 Mar 2022, at 15:55, Rainer Orth wrote: > >>> I've omitted the Darwin-specific stuff so far documented in PRs d/103577 >>> and d/103578: >>> >>> * needs --enable-libphobos >>> * top of gcc-11 branch only >>> * backport of -static-libphobos patch >>> * Darwin/i386 doesn't work

[pushed] testsuite, modules, Darwin: Adjust expected output for older OS versions.

2022-03-18 Thread Iain Sandoe via Gcc-patches
Darwin versions <= 10 (macOS 10.6) emit different diagnostics for the failure case being tested by bad-mapper-1.C. Adjust the dg- expressions to reflect this. tested on powerpc,i686-darwin9, x86-64-darwin10,17,20 powerpc64le,powerpc64,x86_64-linux-gnu, pushed to master, thanks Iain Signed-off-

[pushed] libgccjit: Fix a bootstrap break for some targets.

2022-04-14 Thread Iain Sandoe via Gcc-patches
Some targets use 'long long unsigned int' for unsigned HW int, and this leads to a Werror=format= fail for two print cases in jit-playback.cc introduced in r12-8117-g30f7c83e9cfe (Add support for bitcasts [PR104071]) As discussed on IRC, casting to (long) seems entirely reasonable for the values (

[PATCH] c++, coroutines: Make sure our temporaries are in a bind expr [PR105287]

2022-04-18 Thread Iain Sandoe via Gcc-patches
There are a few cases where we can generate a temporary that does not need to be added to the coroutine frame (i.e. these are genuinely ephemeral). The intent was that unnamed temporaries should not be 'promoted' to coroutine frame entries. However there was a thinko and these were not actually e

[PATCH] c++, coroutines: Account for overloaded promise return_value() [PR105301].

2022-04-18 Thread Iain Sandoe via Gcc-patches
Whether it was intended or not, it is possible to define a coroutine promise with multiple return_value() methods [which need not even have the same type]. We were not accounting for this possibility in the check to see whether both return_value and return_void are specifier (which is prohibited b

[PATCH] c++, coroutines: Improve check for throwing final await [PR104051].

2022-04-18 Thread Iain Sandoe via Gcc-patches
We check that the final_suspend () method returns a sane type (i.e. a class or structure) but, unfortunately, that check has to be later than the one for a throwing case. If the user returns some nonsensical type from the method, we need to handle that in the checking for noexcept. tested on x86_

[pushed] LRA, rs6000, Darwin: Amend lo_sum use for forced constants [PR104117].

2022-02-11 Thread Iain Sandoe via Gcc-patches
Two issues resulted in this PR, which manifests when we force a constant into memory in LRA (in PIC code on Darwin). The presence of such forced constants is quite dependent on other RTL optimisations, and it is easy for the issue to become latent for a specific case. First, in the Darwin-specifi

[PATCH] DWARF: Match behaviour of .cfi_xxx when doing manual frame output.

2021-11-17 Thread Iain Sandoe via Gcc-patches
At present, for several reasons, it is not possible to switch Darwin to use .cfi instructions for frame output. When GCC uses .cfi_ instructions, the behaviour w.r.t frame sections (for a target with unwind frames by defaults): (no options ) .eh_frame (-g ) .eh_frame (-g -fno-unwind-tables -fno-a

[PATCH 1/4] Driver : Provide a spec to insert rpaths for compiler lib dirs.

2021-11-17 Thread Iain Sandoe via Gcc-patches
This provides a spec to insert "-rpath DDD" for each DDD corresponding to a compiler startfile directory. This allows a target to use @rpath as the install path for libraries, and have the compiler provide the necessary rpath to handle this. gcc/ChangeLog: * gcc.c (RUNPATH_OPTION): New.

[PATCH 2/4] Darwin : Handle rpaths given on the command line.

2021-11-17 Thread Iain Sandoe via Gcc-patches
We want to produce a situation where a default rpath can be added to each executable (or dylib), but that can be overridden by any specific rpath provided by the user. gcc/ChangeLog: * config.gcc: Include rpath.opt * config/darwin-driver.c (darwin_driver_init): Detect cases

[PATCH 4/4] Darwin, Ada : Add loader path as a default rpath.

2021-11-17 Thread Iain Sandoe via Gcc-patches
Allow the Ada runtimes to find GCC runtimes relative to their non- standard install positions. gcc/ada/ * gcc-interface/Makefile.in: Add @loader_path runpaths to the libgnat and libgnarl shared library builds. --- gcc/ada/gcc-interface/Makefile.in | 2 ++ 1 file changed,

[PATCH 0/4] Darwin: Replace environment runpath with embedded [PR88590].

2021-11-17 Thread Iain Sandoe via Gcc-patches
This is a fairly long explanation of the problems being addressed by the patch set. Most of the changes are Darwin-specific - a change to the libtool component allowing for this @rpath and some minor additions to makefiles where libtool is not in use. At present, this seems pretty specific to the

[PATCH 3/4] Darwin : Allow for configuring Darwin to use embedded runpath.

2021-11-17 Thread Iain Sandoe via Gcc-patches
Recent Darwin versions place contraints on the use of run paths specified in environment variables. This breaks some assumptions in the GCC build. This change allows the user to configure a Darwin build to use '@rpath/libraryname.dylib' in library names and then to add an embedded runpath to exec

Re: [PATCH 3/4] Darwin : Allow for configuring Darwin to use embedded runpath.

2021-11-17 Thread Iain Sandoe via Gcc-patches
> On 17 Nov 2021, at 22:50, Joseph Myers wrote: > > On Wed, 17 Nov 2021, Iain Sandoe via Gcc-patches wrote: > >> * libtool.m4: Add 'enable-darwin-at-runpath'. Act on the >> enable flag to alter Darwin libraries to use @rpath names. > > To

Re: [PATCH] darwin, d: Support outfile substitution for liphobos

2021-11-19 Thread Iain Sandoe via Gcc-patches
Hi Iain > On 19 Nov 2021, at 08:32, Iain Buclaw wrote: > This patch fixes a stage2 bootstrap failure in the D front-end on > darwin due to libgphobos being dynamically linked despite > -static-libphobos being on the command line. > > In the gdc driver, this takes the previous fix for the Darwin

[PATCH] libphobos, testsuite: Add prune clauses for two Darwin cases.

2021-11-19 Thread Iain Sandoe via Gcc-patches
Depending on the permutation of CPU, OS version and shared/non- shared library inclusion, we get can get two warnings from the external tools (ld64, dsymutil) which are not actually GCC issues but relate to the external tools. These are already pruned in the main testsuite, this adds them to the l

[PATCH] libstdc++, testsuite: Add a prune expression for external tool bug.

2021-11-19 Thread Iain Sandoe via Gcc-patches
Depending on the permutation of CPU, OS version and shared/non- shared library inclusion, we get can get warnings from the external tools (ld64, dsymutil) which are not actually libstdc++ issues but relate to the external tools themselves. This is already pruned in the main testsuite, this adds it

[PATCH] gcc, doc: Fix Darwin bootstrap: Amend an @option command to elide a space.

2021-11-19 Thread Iain Sandoe via Gcc-patches
At least some version(s) of makeinfo (4.8) do not like @option {-} the brace has to follow the @option without any whitespace. makeinfo 4.8 is installed on Darwin systems and this breaks bootstrap. The amendment follows the style of the surrounding code. tested on Darwin9, 18, 19, 20, 21, pus

[pushed] Darwin: Rework handling for unwinder code in libgcc_s and specs [PR80556].

2021-11-20 Thread Iain Sandoe via Gcc-patches
This addresses a long-standing problem where a work-around for an unwinder issue (also a regression) regresses other functionality. The patch replaces several work-arounds with a fix for PR80556 and a work-around for PR88590. * The fix for PR80556 requires a bump to the SO name for libgcc_s, sinc

[pushed] libitm: Fix bootstrap for targets without HAVE_ELF_STYLE_WEAKREF.

2021-11-26 Thread Iain Sandoe via Gcc-patches
Recent improvements to null address warnings notice that for targets that do not support HAVE_ELF_STYLE_WEAKREF the dummy stub implementation of __cxa_get_globals() means that the address can never be null. Fixed by removing the test for such targets. tested on i686, x86_64, powerpc-darwin and x8

[pushed ] Darwin: Rewrite host PCH support [PR 55610].

2021-12-02 Thread Iain Sandoe via Gcc-patches
We need to revise the PCH memory allocation scheme to enable support for PIE on aarch64. The rewrite uses a similar scheme to the one used on Linux. We attempt to identify VM segments for each arch/OS version that are always available to the compiler (note this is not general, it only needs to wo

[pushed] Objective-C, NeXT: Reorganise meta-data declarations.

2021-12-05 Thread Iain Sandoe via Gcc-patches
This moves the GTY declaration of the meta-data indentifier array into the header that enumerates these and provides shorthand defines for them. This avoids a problem seen with a relocatable PCH implementation. Tested on x86_64, i686 Darwin, pushed to master, thanks Iain Signed-off-by: Iain Sand

Re: [PATCH] pch: Add support for relocation of the PCH data [PR71934]

2021-12-08 Thread Iain Sandoe via Gcc-patches
> On 7 Dec 2021, at 14:50, Jakub Jelinek via Gcc-patches > wrote: > > On Tue, Dec 07, 2021 at 10:55:07AM +0100, Jakub Jelinek via Gcc-patches wrote: >> So, this patch instead builds a relocation table (sorted list of addresses >> in the blob which needs relocation) at PCH save time, stores it

[pushed] Darwin: Amend pie options when linking mdynamic-no-pic.

2021-12-08 Thread Iain Sandoe via Gcc-patches
On i686 Darwin from macOS 10.7 onwards the default is to link executables as PIE, which conflicts with code generated using mdynamic-no-pic. Rather than warn about this and then get the user to add -Wl,-no_pie, we can inject this in the link specs. tested on i686,powerpc-darwin9,i686-darwin17, x8

[pushed] libgcc, Darwin: Build a libgcc_s.1 for backwards compatibility.

2021-12-08 Thread Iain Sandoe via Gcc-patches
In order to reslve a long-standing issue with inter-operation with libSystem, we have bumped the SO name for libgcc_s. Distributions might wish to install this new version into a structure where exisiting code is already linked with the compiler-local libgcc_s.1 (providing symbols exported by the

Re: [PATCH] pch: Add support for relocation of the PCH data [PR71934]

2021-12-09 Thread Iain Sandoe via Gcc-patches
> On 9 Dec 2021, at 14:59, Jakub Jelinek wrote: > > On Wed, Dec 08, 2021 at 08:00:03AM +, Iain Sandoe wrote: >>> On 7 Dec 2021, at 14:50, Jakub Jelinek via Gcc-patches >>> wrote: >> The attached patch should be applied before (or merged with) the change for >> relocation when it is appli

[pushed] libgcc, Darwin: Update darwin10 unwinder shim dependencies.

2021-12-10 Thread Iain Sandoe via Gcc-patches
We include libgcc_tm.h to provide a prototype for this shim so add that to the make dependencies. tested on x86_64-darwin, pushed to master, thanks Iain Signed-off-by: Iain Sandoe libgcc/ChangeLog: * config/t-darwin: Add libgcc_tm.h to the dependencies for darwin10-unwind-find-

[PATCH] configure: Account CXXFLAGS in gcc-plugin.m4.

2021-12-10 Thread Iain Sandoe via Gcc-patches
While doing tests of the PCH changes, I noticed that all the plugin tests were being omitted from m32 Darwin under some permutations of flags. It turned out to be a broken config test - it was not removing -mdynamic-no-pic properly. We now use a C++ compiler so that we need to process CXXFLAGS as

Re: [PATCH 0/6] Add Intel Sierra Forest Instructions

2022-10-14 Thread Iain Sandoe via Gcc-patches
Hi Haochen > On 14 Oct 2022, at 08:54, Haochen Jiang via Gcc-patches > wrote: > > These six patches aimed to add Intel Sierra Forest instructions, including > AVX-IFMA, AVX-VNNI0INT8, AVX-NE-CONVERT, CMPccXADD. We also added intrinsic > for vector __bf16 in this series of patch and Sierra Fore

Re: [PATCH 0/6] Add Intel Sierra Forest Instructions

2022-10-14 Thread Iain Sandoe via Gcc-patches
> On 14 Oct 2022, at 09:20, Hongtao Liu wrote: > > On Fri, Oct 14, 2022 at 4:14 PM Iain Sandoe via Gcc-patches > wrote: >> >> Hi Haochen >> >>> On 14 Oct 2022, at 08:54, Haochen Jiang via Gcc-patches >>> wrote: >>> >> >

Re: [PATCH 0/6] Add Intel Sierra Forest Instructions

2022-10-14 Thread Iain Sandoe via Gcc-patches
> On 14 Oct 2022, at 09:30, Hongtao Liu wrote: > > On Fri, Oct 14, 2022 at 4:24 PM Iain Sandoe wrote: >> >> >> >>> On 14 Oct 2022, at 09:20, Hongtao Liu wrote: >>> >>> On Fri, Oct 14, 2022 at 4:14 PM Iain Sandoe via Gcc-patches &

Re: [PATCH 0/6] Add Intel Sierra Forest Instructions

2022-10-19 Thread Iain Sandoe via Gcc-patches
Hi Hongtao > On 17 Oct 2022, at 02:56, Hongtao Liu wrote: > > On Mon, Oct 17, 2022 at 9:30 AM Bernhard Reutner-Fischer > wrote: >> >> On 17 October 2022 03:02:22 CEST, Hongtao Liu via Gcc-patches >> >> Do you have this series as a branch somewhere that I can try on one of >> the

Re: [PATCH 0/6] Add Intel Sierra Forest Instructions

2022-10-20 Thread Iain Sandoe via Gcc-patches
> On 20 Oct 2022, at 10:09, Hongtao Liu via Gcc-patches > wrote: > > On Thu, Oct 20, 2022 at 9:11 AM Hongtao Liu wrote: >> >> On Wed, Oct 19, 2022 at 7:09 PM Iain Sandoe wrote: >>> >>> Hi Hongtao >>> On 17 Oct 2022, at 02:56, Hongtao Liu wrote: On Mon, Oct 17, 2022 at 9:

Re: Patch ping (was Re: [PATCH] libstdc++: Clear padding bits in atomic compare_exchange)

2022-09-09 Thread Iain Sandoe via Gcc-patches
> On 9 Sep 2022, at 19:36, Rainer Orth wrote: > >> Here's a complete patch that combines the various incremental patches >> that have been going around. I'm testing this now. >> >> Please take a look. > > unfortunately, this patch broke macOS bootstrap (seen on > x86_64-apple-darwin11.4.2):

Re: [PATCH v2, rs6000] Change insn condition from TARGET_64BIT to TARGET_POWERPC64 for VSX scalar extract/insert instructions

2022-09-13 Thread Iain Sandoe via Gcc-patches
Hi > On 13 Sep 2022, at 03:34, HAO CHEN GUI via Gcc-patches > wrote: > On 7/9/2022 上午 1:19, Segher Boessenkool wrote: >> make -k -j60 check >> RUNTESTFLAGS="--target_board=unix'{-m64,-m32,-m32/-mpowerpc64}'" >> >> It is fine to not test -m32/-mpowerpc64 so often, and certaionly not >> someth

[pushed] testsuite: Add target requires for ifuncs to mv31.C.

2022-04-28 Thread Iain Sandoe via Gcc-patches
g++.target/i386/mv31.C fails on targets without ifuncs support so add the necessary target supports guard. tested on x86_64 linux and darwin. pushed to master, thanks, Iain Signed-off-by: Iain Sandoe gcc/testsuite/ChangeLog: * g++.target/i386/mv31.C: Add target supports guard for ifunc

[pushed] testsuite,X86: Fix missing USER_LABEL_PREFIX cases.

2022-04-28 Thread Iain Sandoe via Gcc-patches
Yet another set of testcases that do not account for targets that use __USER_LABEL_PREFIX__. tested on x86_64 Linux and Darwin, pushed to master, thanks, Iain Signed-off-by: Iain Sandoe gcc/testsuite/ChangeLog: * gcc.target/i386/memcpy-strategy-10.c: Account for __USER_LABEL_PR

[PATCH] c++, coroutines: Partial reversion of r12-8308-g15a176a833f23e [PR105426].

2022-04-28 Thread Iain Sandoe via Gcc-patches
The changes to fix PR 105287 included a tightening of the constraints on which variables are promoted to frame copies. This has exposed that we are failing to name some variables that should be promoted. The long-term fix is to address the cases where the naming has been missed, but for the short

[pushed] Objective-C, NeXT: Adjust symbol marking to match host tools.

2022-05-03 Thread Iain Sandoe via Gcc-patches
Current host tools mark some additional symbols as 'no dead strip' and also expose one additional group to the linker. This does not affect older Darwin versions or x86_64, but omitting these changes results in link errors for aarch64. Tested on i686, x86_64 and a cross to aarch64 Darwin. pushed

Re: [PATCH] Fold GCC_VERSION >= $old_version to TRUE, otherwise to FALSE.

2022-05-05 Thread Iain Sandoe via Gcc-patches
> On 5 May 2022, at 19:50, Martin Liška wrote: > > On 5/5/22 20:35, Andrew Pinski wrote: >> GCC_VERSION will be 0 if GCC is not being used. >> So you need to audit these better really. > > Ah, I see. So it basically means all the non-GCC conditional > code needs to remain and I can replace >

Re: [PATCH v3 1/2] rs6000: Move g++.dg powerpc PR tests to g++.target

2022-05-12 Thread Iain Sandoe via Gcc-patches
Hi > On 12 May 2022, at 23:38, Segher Boessenkool > wrote: > On Mon, Apr 18, 2022 at 12:15:34PM -0500, Paul A. Clarke wrote: >> -/* { dg-skip-if "" { powerpc*-*-darwin* } } */ >> +/* Never tested on darwin, so skip there. */ >> +/* { dg-skip-if "" { *-*-darwin* } } */ > > That is probably the

Re: [COMMITTED] Export global ranges during the VRP block walk.

2022-05-14 Thread Iain Sandoe via Gcc-patches
Hi Andrew > On 13 May 2022, at 14:58, Andrew MacLeod via Gcc-patches > wrote: > > VRP currently searches the ssa_name list for globals to exported after it > finishes running. This change simply exports globals as they are calculated > for the final time during the DOM walk. > > This avoid

Re: [COMMITTED] Export global ranges during the VRP block walk.

2022-05-14 Thread Iain Sandoe via Gcc-patches
Hi Andrew, Toon, > On 14 May 2022, at 09:09, Toon Moene wrote: > On 5/14/22 10:00, Iain Sandoe via Gcc-patches wrote: >>> On 13 May 2022, at 14:58, Andrew MacLeod via Gcc-patches >>> wrote: >>> >>> VRP currently searches the ssa_name list for g

Re: [PATCH] bootstrap/97666 - fix array of bool allocation

2020-11-03 Thread Iain Sandoe via Gcc-patches
Martin Sebor via Gcc-patches wrote: On 11/3/20 4:13 AM, Richard Biener wrote: This fixes the bad assumption that sizeof (bool) == 1 Bootstrap / regtest running on x86_64-unknown-linux-gnu. 2020-11-03 Richard Biener PR bootstrap/97666 * tree-vect-slp.c (vect_build_slp_tree_2)

Re: [PATCH] x86: Adjust keylocker testcases for fail on darwin

2020-11-09 Thread Iain Sandoe via Gcc-patches
Uros Bizjak via Gcc-patches wrote: On Mon, Nov 9, 2020 at 11:50 AM Hongyu Wang wrote: Hi According to the discussion in https://gcc.gnu.org/pipermail/gcc/2020-November/234096.html, The testcase for keylocker-* is too strict for darwin target. This patch adjusted the regex, and add a missing

Re: [PATCH] libstdc++: Fix error shown during Solaris build

2020-11-13 Thread Iain Sandoe via Gcc-patches
Jonathan Wakely wrote: On 13/11/20 11:07 +, Jonathan Wakely wrote: Currently this is shown when building libstdc++ on Solaris: -lrt: open: No such file or directory The error comes from the make_sunver.pl script which tries to open each of its arguments. The arguments are passed by this

Re: [PATCH] warn for more impossible null pointer tests

2021-09-01 Thread Iain Sandoe via Gcc-patches
Hi Paul, > On 1 Sep 2021, at 20:28, Koning, Paul via Gcc-patches > wrote: > >> On Sep 1, 2021, at 3:08 PM, Jeff Law via Gcc-patches >> wrote: >> On 9/1/2021 12:57 PM, Koning, Paul wrote: >>> On Sep 1, 2021, at 1:35 PM, Jeff Law via Gcc-patches wrote: Generally OK. The

Re: [PATCH V3 0/6] Initial support for AVX512FP16

2021-09-02 Thread Iain Sandoe via Gcc-patches
Hi Hongtao. > On 2 Sep 2021, at 07:06, Hongtao Liu via Gcc-patches > wrote: > > I'm going to check in the first 3 patches which are already approved. > > Update hf soft-fp from glibc. > [i386] Enable _Float16 type for TARGET_SSE2 and above. > [i386] libgcc: Enable hfmode soft-sf/df/xf/tf ex

Re: [PATCH V3 0/6] Initial support for AVX512FP16

2021-09-02 Thread Iain Sandoe via Gcc-patches
Patch below fixes bootstrap, OK if it passes testing on x86_64 darwin/linux? (if !OK .. then suggestions welcome) thanks Iain > On 2 Sep 2021, at 16:18, Hongtao Liu wrote: > > > > On Thursday, September 2, 2021, Iain Sandoe wrote: > Hi Hongtao. > > > On 2 Sep 2021, at 07:06, Hongtao Liu vi

Re: [PATCH V3 0/6] Initial support for AVX512FP16

2021-09-03 Thread Iain Sandoe via Gcc-patches
Hi Joseph, > On 2 Sep 2021, at 21:03, Joseph Myers wrote: > > On Thu, 2 Sep 2021, Iain Sandoe via Gcc-patches wrote: > >> diff --git a/libgcc/soft-fp/eqdf2.c b/libgcc/soft-fp/eqdf2.c >> index 2a44ee377ce..a3bb664f5f1 100644 >> --- a/libgcc/soft-fp/eqdf2.c &

Re: [PATCH] Explicitly add -msse2 to compile HF related libgcc source file.

2021-09-03 Thread Iain Sandoe via Gcc-patches
> On 3 Sep 2021, at 10:00, Jakub Jelinek via Gcc-patches > wrote: > > On Fri, Sep 03, 2021 at 03:41:13PM +0800, liuhongt via Gcc-patches wrote: >> --- a/libgcc/config/i386/64/t-softfp >> +++ b/libgcc/config/i386/64/t-softfp >> @@ -1 +1,6 @@ >> softfp_extras := fixhfti fixunshfti floattihf flo

Re: [PATCH] Explicitly add -msse2 to compile HF related libgcc source file.

2021-09-03 Thread Iain Sandoe via Gcc-patches
> On 3 Sep 2021, at 13:33, Jakub Jelinek via Gcc-patches > wrote: > > On Fri, Sep 03, 2021 at 01:32:14PM +0100, Iain Sandoe via Gcc-patches wrote: >> diff --git a/libgcc/libgcc-std.ver.in b/libgcc/libgcc-std.ver.in >> index cea33267e53..f5c1bc1aa6c 100644 >>

Re: [PATCH] Explicitly add -msse2 to compile HF related libgcc source file.

2021-09-06 Thread Iain Sandoe via Gcc-patches
> On 6 Sep 2021, at 08:08, Jakub Jelinek via Gcc-patches > wrote: > > On Mon, Sep 06, 2021 at 09:42:32AM +0800, Hongtao Liu wrote: >>> Also, shouldn't the *hf* and *hc* APIs be exported from libgcc_s.so.1? >>> >>> 254: 00011960 968 FUNCLOCAL DEFAULT 13 __floattihf >>> 25

Re: [PATCH] testsuite: Fix c-c++-common/auto-init-* tests

2021-09-16 Thread Iain Sandoe via Gcc-patches
> On 16 Sep 2021, at 18:11, Qing Zhao via Gcc-patches > wrote: > > > >> On Sep 16, 2021, at 10:47 AM, Jakub Jelinek wrote: >> >> On Thu, Sep 16, 2021 at 03:39:46PM +, Qing Zhao wrote: Even -mtune= is needed if you want to stay safe, otherwise people testing with --target_boa

[PATCH] coroutines: Only set parm copy guard vars if we have exceptions [PR 102454].

2021-09-27 Thread Iain Sandoe via Gcc-patches
For coroutines, we make copies of the original function arguments into the coroutine frame. Normally, these are destroyed on the proper exit from the coroutine when the frame is destroyed. However, if an exception is thrown before the first suspend point is reached, the cleanup has to happen in t

[pushed] libgcc, X86, Darwin: Export cpu_model and indicator.

2021-09-28 Thread Iain Sandoe via Gcc-patches
Hi, These two symbols have been emitted since 4.8, but were not added to the Darwin exports, so we have been using the ones from libgcc.a. Added to libgcc_s now. tested on i686 and x86_64-darwin, pushed to master, thanks Iain Signed-off-by: Iain Sandoe libgcc/ChangeLog: * config/i386

[pushed] Darwin, PPC : Fix R13 for PPC64.

2021-09-28 Thread Iain Sandoe via Gcc-patches
Hi, We have a somewhat unusual situation in that for PPC64, R13 is both reserved for future use by the ABI document and callee-saved. In fact, it is already used internally by the pthreads implementation to contain pthread_self. So add R13 to the fixed regs, but also keep it in the callee- saved

[pushed] Darwin, D : Add .d suffix to the list for invoking dsymutil.

2021-09-28 Thread Iain Sandoe via Gcc-patches
Hi, Recognise .d for D source files on the command line. This will trigger an invocation of dsymutil when a D source is present. tested along with D patches on i686, powerpc and x86_64 darwin, pushed to master, thanks, Iain gcc/ChangeLog: * config/darwin.h (DSYMUTIL_SPEC): Recognize D

[PATCH 0/4] config: Allow a host to opt out of PCH.

2021-11-04 Thread Iain Sandoe via Gcc-patches
GCC (currently) has an implementation of pre-compiled-headers, that relies on being able to launch the compiler executable at the same address each time. This constraint is not permitted by some system security models. The facility is an optimisation; saving the output of parsing a covering heade

  1   2   3   >