Re: [PATCH] Handle fancy_abort before diagnostic initialization [PR98586]

2021-01-11 Thread Richard Biener via Gcc-patches
On Mon, Jan 11, 2021 at 10:57 PM David Malcolm via Gcc-patches wrote: > > If fancy_abort is called before the diagnostic subsystem is initialized, > internal_error will crash internally in a way that prevents a useful > message reaching the user. > > This can happen with libgccjit in the case of

[COMMITED] MAINTAINERS: Add myself for write after approval

2021-01-11 Thread Qian Jianhua
ChangeLog: 2021-01-12 Qian Jianhua * MAINTAINERS (Write After Approval): Add myself --- MAINTAINERS | 1 + 1 file changed, 1 insertion(+) diff --git a/MAINTAINERS b/MAINTAINERS index c0aa23df57e..e88808f9fe2 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -451,6 +451,7 @@ Daniel

Re: [PATCH v4 05/29] Import replacement 'clz' functions from CM0 library

2021-01-11 Thread Daniel Engel
On Mon, Jan 11, 2021, at 8:32 AM, Richard Earnshaw wrote: > A general comment before we start: > > CLZ was added to the Arm ISA in Armv5. So all subsequent Arm versions > (and all versions implementing thumb2) will have this instruction. So > the only cases where you'll need a fallback are

Re: [PATCH] c++: -Wmissing-field-initializers in unevaluated ctx [PR98620]

2021-01-11 Thread Jason Merrill via Gcc-patches
On 1/11/21 6:19 PM, Marek Polacek wrote: This PR wants us not to warn about missing field initializers when the code in question takes places in decltype and similar. Hmm, the warning seems of questionable utility with templated code like this, but I guess this is a reasonable middle ground.

Re: [PATCH] c++: Fix ICE with CTAD in concept [PR98611]

2021-01-11 Thread Jason Merrill via Gcc-patches
On 1/11/21 8:08 PM, Patrick Palka wrote: On Mon, 11 Jan 2021, Jason Merrill wrote: On 1/9/21 5:23 PM, Patrick Palka wrote: This patch teaches find_template_parameters to visit the template represented by a CTAD placeholder, which is normally not visited by for_each_template_parm. This

[committed] analyzer: fix ICE merging dereferencing unknown ptrs [PR98628]

2021-01-11 Thread David Malcolm via Gcc-patches
Successfully bootstrapped & regrtested on x86_64-pc-linux-gnu. Pushed to master as r11-6603-gab88f3607233376c3145c320e92e71943a495bb5 gcc/analyzer/ChangeLog: PR analyzer/98628 * store.cc (binding_cluster::make_unknown_relative_to): Don't mark dereferenced unknown pointers

Re: [PATCH] c++: Fix ICE with CTAD in concept [PR98611]

2021-01-11 Thread Patrick Palka via Gcc-patches
On Mon, 11 Jan 2021, Jason Merrill wrote: > On 1/9/21 5:23 PM, Patrick Palka wrote: > > This patch teaches find_template_parameters to visit the template > > represented by a CTAD placeholder, which is normally not visited by > > for_each_template_parm. This template may be a template template >

Re: [PATCH] binuitils: Check if AR is usable for LTO build

2021-01-11 Thread Alan Modra via Gcc-patches
On Mon, Jan 11, 2021 at 02:52:43PM -0800, H.J. Lu wrote: > On Mon, Jan 11, 2021 at 1:20 PM Alan Modra wrote: > > > > On Mon, Jan 11, 2021 at 11:53:15AM -0800, H.J. Lu via Binutils wrote: > > > Check if AR is usable for LTO build with --enable-pgo-build=lto: > > > > > > checking for -plugin

Re: V2 [PATCH 1/2] GCC: Check if AR works with --plugin and rc

2021-01-11 Thread H.J. Lu via Gcc-patches
On Mon, Jan 11, 2021 at 4:22 PM Alan Modra wrote: > > On Mon, Jan 11, 2021 at 04:07:22PM -0800, H.J. Lu wrote: > > These are not fatal errors. Here is the updated patch to use > > AC_MSG_WARN instead. OK for master? > > OK by me. Please squash the two patches. > I will keep 2 separate since I

Re: V2 [PATCH 1/2] GCC: Check if AR works with --plugin and rc

2021-01-11 Thread Alan Modra via Gcc-patches
On Mon, Jan 11, 2021 at 04:07:22PM -0800, H.J. Lu wrote: > These are not fatal errors. Here is the updated patch to use > AC_MSG_WARN instead. OK for master? OK by me. Please squash the two patches. -- Alan Modra Australia Development Lab, IBM

V2 [PATCH 1/2] GCC: Check if AR works with --plugin and rc

2021-01-11 Thread H.J. Lu via Gcc-patches
On Mon, Jan 11, 2021 at 3:27 PM Alan Modra wrote: > > On Mon, Jan 11, 2021 at 08:57:05AM -0800, H.J. Lu via Binutils wrote: > > diff --git a/config/gcc-plugin.m4 b/config/gcc-plugin.m4 > > index c5b72e9a13d..798a2054edd 100644 > > --- a/config/gcc-plugin.m4 > > +++ b/config/gcc-plugin.m4 > > @@

Re: [PATCH 1/2] GCC: Check if AR works with --plugin and rc

2021-01-11 Thread Alan Modra via Gcc-patches
On Mon, Jan 11, 2021 at 08:57:05AM -0800, H.J. Lu via Binutils wrote: > diff --git a/config/gcc-plugin.m4 b/config/gcc-plugin.m4 > index c5b72e9a13d..798a2054edd 100644 > --- a/config/gcc-plugin.m4 > +++ b/config/gcc-plugin.m4 > @@ -145,6 +145,18 @@ for plugin in $plugin_names; do > break >

[PATCH] c++: -Wmissing-field-initializers in unevaluated ctx [PR98620]

2021-01-11 Thread Marek Polacek via Gcc-patches
This PR wants us not to warn about missing field initializers when the code in question takes places in decltype and similar. Fixed thus. Bootstrapped/regtested on x86_64-pc-linux-gnu, ok for trunk? gcc/cp/ChangeLog: PR c++/98620 * typeck2.c (process_init_constructor_record):

Re: [PATCH v4 01/29] Add and organize macros.

2021-01-11 Thread Daniel Engel
On Mon, Jan 11, 2021, at 7:21 AM, Richard Earnshaw wrote: > Some initial comments. > > On 11/01/2021 11:10, g...@danielengel.com wrote: > > From: Daniel Engel > > > > These definitions facilitate subsequent patches in this series. > > > > gcc/libgcc/ChangeLog: > > 2021-01-07 Daniel Engel >

Re: [PATCH 0/2] Check if AR works with --plugin and rc

2021-01-11 Thread H.J. Lu via Gcc-patches
On Mon, Jan 11, 2021 at 3:05 PM Alan Modra wrote: > > On Mon, Jan 11, 2021 at 08:57:04AM -0800, H.J. Lu via Binutils wrote: > > Check if AR works with --plugin and rc before passing --plugin to AR and > > RANLIB. > > Thanks for looking at this, but next time please assign the bug to > yourself.

Re: [PATCH v4 02/29] Refactor 'clz' functions into a new file.

2021-01-11 Thread Richard Earnshaw via Gcc-patches
On 11/01/2021 15:58, Daniel Engel wrote: > > On Mon, Jan 11, 2021, at 7:39 AM, Richard Earnshaw wrote: >> On 11/01/2021 15:26, Richard Earnshaw wrote: >>> On 11/01/2021 11:10, g...@danielengel.com wrote: From: Daniel Engel gcc/libgcc/ChangeLog: 2021-01-07 Daniel Engel

Re: [PATCH 0/2] Check if AR works with --plugin and rc

2021-01-11 Thread Alan Modra via Gcc-patches
On Mon, Jan 11, 2021 at 08:57:04AM -0800, H.J. Lu via Binutils wrote: > Check if AR works with --plugin and rc before passing --plugin to AR and > RANLIB. Thanks for looking at this, but next time please assign the bug to yourself. I fixed the bug too this morning, before seeing your email. --

Re: [PATCH] binuitils: Check if AR is usable for LTO build

2021-01-11 Thread H.J. Lu via Gcc-patches
On Mon, Jan 11, 2021 at 1:20 PM Alan Modra wrote: > > On Mon, Jan 11, 2021 at 11:53:15AM -0800, H.J. Lu via Binutils wrote: > > Check if AR is usable for LTO build with --enable-pgo-build=lto: > > > > checking for -plugin option... ar: no operation specified > > Failed: ar --plugin > >

[wwwdocs] Update C++ DR table with new DRs

2021-01-11 Thread Marek Polacek via Gcc-patches
I pushed this patch to update the DR table with a few new DRs. Also update some of the older ones to reflect that they're included in C++20. Marek commit 6be6626d59e1c492d5e05606e5a6902feb9e5bac Author: Marek Polacek Date: Mon Jan 11 17:18:45 2021 -0500 C++ DRs: Add new DRs, update

Re: [PATCH] libstdc++: implement locale support for AIX

2021-01-11 Thread David Edelsohn via Gcc-patches
On Mon, Jan 11, 2021 at 10:56 AM CHIGOT, CLEMENT wrote: > > >> Hi David, Clement, > >> > >>> The patch is local to libstdc++ AIX support, so I believe that I can > >>> approve it. > >> > >>have you considered merging the dragonfly and aix trees? I'm asking > >>because it seems prudent to try

Re: [PATCH] c++: ICE with constrained placeholder return type [PR98346]

2021-01-11 Thread Jason Merrill via Gcc-patches
On 1/11/21 4:40 PM, Jason Merrill wrote: On 1/7/21 4:06 PM, Patrick Palka wrote: This is essentially a followup to r11-3714 -- we ICEing from another "unguarded" call to build_concept_check, this time in do_auto_deduction, due to the presence of templated trees when !processing_template_decl.

[PATCH] Handle fancy_abort before diagnostic initialization [PR98586]

2021-01-11 Thread David Malcolm via Gcc-patches
If fancy_abort is called before the diagnostic subsystem is initialized, internal_error will crash internally in a way that prevents a useful message reaching the user. This can happen with libgccjit in the case of gcc_assert failures that occur outside of the libgccjit mutex that guards the rest

Re: [PATCH 0/8] [RS6000] rs6000_rtx_costs V2

2021-01-11 Thread Alan Modra via Gcc-patches
On Sat, Dec 05, 2020 at 07:42:07PM +1030, Alan Modra wrote: > Hi Segher, > I've been holding off pinging these knowing you had a lot of other > review work, but maybe that's settling down now? You already OK'd > 1/8, 2/8 and 6/8. Ping. > [PATCH 3/8] [RS6000] rs6000_rtx_costs tidy AND >

Re: [PATCH] c++: ICE with constrained placeholder return type [PR98346]

2021-01-11 Thread Jason Merrill via Gcc-patches
On 1/7/21 4:06 PM, Patrick Palka wrote: This is essentially a followup to r11-3714 -- we ICEing from another "unguarded" call to build_concept_check, this time in do_auto_deduction, due to the presence of templated trees when !processing_template_decl. Bootstrapped and regtested on

Re: [PATCH] binuitils: Check if AR is usable for LTO build

2021-01-11 Thread Alan Modra via Gcc-patches
On Mon, Jan 11, 2021 at 11:53:15AM -0800, H.J. Lu via Binutils wrote: > Check if AR is usable for LTO build with --enable-pgo-build=lto: > > checking for -plugin option... ar: no operation specified > Failed: ar --plugin >

Re: [PATCH v4] rs6000, vector integer multiply/divide/modulo instructions

2021-01-11 Thread will schmidt via Gcc-patches
On Mon, 2020-12-07 at 16:31 -0800, Carl Love wrote: > Will: > > I have addressed you comments with regards to the Change Log entries. > > The extra define vec_div was removed. > > Added the missing entries for DIVU_V2DI DIVS_V2DI in rs6000-call.c. > > The extra MULLD_V2DI case statement

Re: [PATCH] c++: Fix ICE with CTAD in concept [PR98611]

2021-01-11 Thread Jason Merrill via Gcc-patches
On 1/9/21 5:23 PM, Patrick Palka wrote: This patch teaches find_template_parameters to visit the template represented by a CTAD placeholder, which is normally not visited by for_each_template_parm. This template may be a template template parameter (as in the first testcase), or it may

Re: [PATCH] c++: private inheritance access diagnostics fix [PR17314]

2021-01-11 Thread Jason Merrill via Gcc-patches
On 1/8/21 7:38 PM, Anthony Sharp wrote: Hi Jason, Thank you! To start with, do you have a copyright assignment on file or in the works already? Good point. I incorrectly assumed it would only be a minor contribution copyright-wise. > Mr Edelsohn gave me a template which I've now filled out

[PATCH] binuitils: Check if AR is usable for LTO build

2021-01-11 Thread H.J. Lu via Gcc-patches
Check if AR is usable for LTO build with --enable-pgo-build=lto: checking for -plugin option... ar: no operation specified Failed: ar --plugin /usr/gcc-11.0.0-x32/libexec/gcc/x86_64-pc-linux-gnu/11.0.0/liblto_plugin.so rc no configure: error: AR with --plugin and rc is required for LTO build

Re: [PATCH] match.pd: Add ~(X - Y) -> ~X + Y simplification [PR96685]

2021-01-11 Thread Jeff Law via Gcc-patches
On 1/9/21 5:43 PM, Maciej W. Rozycki wrote: > On Mon, 21 Dec 2020, Jakub Jelinek wrote: > This patch adds the ~(X - Y) -> ~X + Y simplification requested in the PR (plus also ~(X + C) -> ~X + (-C) for constants C that can be safely negated. >>> This regresses VAX code produced

Re: [PATCH] VAX/testsuite: Remove notsi comparison elimination regressions

2021-01-11 Thread Jeff Law via Gcc-patches
On 1/10/21 7:45 AM, Maciej W. Rozycki wrote: > On Fri, 8 Jan 2021, Jeff Law wrote: > >>> gcc/testsuite/ >>> * gcc.target/vax/cmpelim-eq-notsi.c: Use subtraction from a >>> constant then rather than addition. >>> * gcc.target/vax/cmpelim-le-notsi.c: Likewise. >>> *

Re: [PATCH] tree-optimization/98221 - fix wrong unpack operation used for big-endian

2021-01-11 Thread Jeff Law via Gcc-patches
On 1/11/21 3:02 AM, Andreas Krebbel via Gcc-patches wrote: > The vec-abi-varargs-1.c testcase on IBM Z currently fails. > > While adding an SI mode vector to a DI mode vector the first is unpacked > using: > > _28 = BIT_INSERT_EXPR <{ 0, 0, 0, 0 }, _2, 0>; > _34 = [vec_unpack_lo_expr] _28;

Re: libstdc++ PR 57272 Fancy pointer support in Hashtable

2021-01-11 Thread François Dumont via Gcc-patches
Hi     I had another look to this attempt to properly support alloc fancy pointers.     I consider all your remarks appart from the big one below about all this being a waste of time :-)     I do not see why we should use the alloc fancy pointer type in our _Hashtable implementation

[pushed] aarch64: Add support for unpacked SVE ASRD

2021-01-11 Thread Richard Sandiford via Gcc-patches
This patch adds support for both conditional and unconditional unpacked ASRD. This meant adding a new define_insn for the unconditional form, instead of reusing the conditional instructions. It also meant extending the current conditional patterns to support merging with any independent value,

[pushed] aarch64: Add support for unpacked SVE conditional BIC

2021-01-11 Thread Richard Sandiford via Gcc-patches
This patch adds support for unpacked conditional BIC. The type suffix could be taken from the element size or the container size, so the patch continues to use the element size. This is consistent with the existing support for unconditional BIC. Tested on aarch64-linux-gnu and aarch64_be-elf.

[pushed] aarch64: Add support for unpacked SVE MULH

2021-01-11 Thread Richard Sandiford via Gcc-patches
This patch extends the SMULH and UMULH support to unpacked vectors. The type suffix must be taken from the element size rather than the container size. The main use of these patterns is to support division and modulus by a constant. The conditional forms would be hard to trigger from non-ACLE

[pushed] aarch64: Add support for unpacked SVE ABD

2021-01-11 Thread Richard Sandiford via Gcc-patches
This patch adds support for unpacked SVE SABD and UABD. It also rewrites the patterns so that they match as combine patterns without the need for REG_EQUAL notes. Finally, there was no pattern for merging with the second input, which can be handled by reversing the operands. The type suffix

[pushed] aarch64: Add support for unpacked SVE ADR

2021-01-11 Thread Richard Sandiford via Gcc-patches
This patch extends the ADR patterns to handle unpacked vectors. They would work with both elements and containers, but since the instructions only support .s and .d, we get more coverage by using containers. Tested on aarch64-linux-gnu and aarch64_be-elf. Pushed to trunk. Richard gcc/

[pushed] aarch64: Add general unpacked SVE conditional binary arithmetic

2021-01-11 Thread Richard Sandiford via Gcc-patches
This patch adds support for conditional binary ADD, SUB, MUL, SMAX, UMAX, SMIN, UMIN, LSL, LSR, ASR, AND, ORR and EOR. It's not really possible to split it up further given how the patterns are written. Min, max and right-shift need the element size rather than the container size. The others

[pushed] aarch64: Add support for unpacked SVE mult, max and min

2021-01-11 Thread Richard Sandiford via Gcc-patches
This patch makes the SVE_INT_BINARY_IMM patterns support unpacked arithmetic, covering MUL, SMAX, SMIN, UMAX and UMIN. For min and max, the type suffix must be taken from the element size rather than the container size. The XFAILs are due to PR98602. Tested on aarch64-linux-gnu and

[pushed] aarch64: Add support for unpacked SVE shifts

2021-01-11 Thread Richard Sandiford via Gcc-patches
This patch adds support for unpacked SVE LSL, ASR and LSR. For right shifts, the type suffix needs to be taken from the element size rather than the container size. Tested on aarch64-linux-gnu and aarch64_be-elf. Pushed to trunk. Richard gcc/ * config/aarch64/aarch64-sve.md (3)

Re: [PATCH] Properly release symtab::m_clones.

2021-01-11 Thread Martin Liška
On 1/11/21 5:55 PM, Jan Hubicka wrote: The patch is about not using delete for a memory that is allocated by GGC. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? Thanks, Martin gcc/ChangeLog: PR jit/98615 * symtab-clones.h

Re: [patch] fix -Wformat-diag warnings in rs6000-call.c

2021-01-11 Thread Martin Sebor via Gcc-patches
On 1/11/21 9:30 AM, Matthias Klose wrote: On 1/10/21 10:18 PM, Martin Sebor wrote: On 1/10/21 3:29 AM, Matthias Klose wrote: is the newline intended? It's followed by a debug_rtx call. To avoid the warning there shouldn't be any trailing punctuation or whitespace in the message.  The GCC

[gcc-10 backport] libiberty: Support the new ("v0") mangling scheme in rust-demangle

2021-01-11 Thread Amanieu d'Antras via Gcc-patches
Could this commit be backported to the gcc-10 branch? We would like to switch the default name mangling scheme in Rust but are currently blocked on upstream tool support. commit 84096498a7bd788599d4a7ca63543fc7c297645e libiberty: Support the new ("v0") mangling scheme in rust-demangle

[PATCH 0/2] Check if AR works with --plugin and rc

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

[PATCH 2/2] Binutils: Check if AR works with --plugin and rc

2021-01-11 Thread H.J. Lu via Gcc-patches
Check if AR works with --plugin and rc before passing --plugin to AR and RANLIB. bfd/ PR ld/27173 binutils/ PR ld/27173 * configure: Regenerated. gas/ PR ld/27173 * configure: Regenerated. gprof/ PR ld/27173 * configure: Regenerated.

[PATCH 1/2] GCC: Check if AR works with --plugin and rc

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

Re: [PATCH] Properly release symtab::m_clones.

2021-01-11 Thread Jan Hubicka
> The patch is about not using delete for a memory that > is allocated by GGC. > > Patch can bootstrap on x86_64-linux-gnu and survives regression tests. > > Ready to be installed? > Thanks, > Martin > > gcc/ChangeLog: > > PR jit/98615 > * symtab-clones.h (clone_info::release):

[PATCH] Properly release symtab::m_clones.

2021-01-11 Thread Martin Liška
The patch is about not using delete for a memory that is allocated by GGC. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Ready to be installed? Thanks, Martin gcc/ChangeLog: PR jit/98615 * symtab-clones.h (clone_info::release): Release

Re: [PATCH 1/3] GCC: Pass --plugin to AR and RANLIB

2021-01-11 Thread Hans-Peter Nilsson via Gcc-patches
> From: Luis Machado via Gcc-patches > Date: Mon, 11 Jan 2021 15:58:43 +0100 > This seems to have broken the builds on AArch64-Linux Ubuntu 18.04. > > make[2]: Entering directory 'binutils-gdb-master-bionic/libiberty' > rm -f ./libiberty.a pic/./libiberty.a noasan/./libiberty.a > ar --plugin

Re: [PATCH] libphobos: Allow building libphobos using Solaris/x86 assembler

2021-01-11 Thread Iain Buclaw via Gcc-patches
Excerpts from Iain Buclaw's message of January 7, 2021 6:48 pm: > Excerpts from Rainer Orth's message of January 7, 2021 5:17 pm: >> Hi Iain, >> The Solaris assemblers don't support UTF-8 identifiers. Unless gdc can encode them in some way for toolchains like this (no idea if this is

Re: [PATCH v3] libgcc: Thumb-1 Floating-Point Library for Cortex M0

2021-01-11 Thread Christophe Lyon via Gcc-patches
On Mon, 11 Jan 2021 at 17:18, Daniel Engel wrote: > > On Mon, Jan 11, 2021, at 8:07 AM, Christophe Lyon wrote: > > On Sat, 9 Jan 2021 at 14:09, Christophe Lyon > > wrote: > > > > > > On Sat, 9 Jan 2021 at 13:27, Daniel Engel wrote: > > > > > > > > On Thu, Jan 7, 2021, at 4:56 AM, Richard

Re: [PATCH v4 06/29] Import replacement 'ctz' functions from CM0 library

2021-01-11 Thread Richard Earnshaw via Gcc-patches
On 11/01/2021 11:10, g...@danielengel.com wrote: > From: Daniel Engel > > This version combines __ctzdi2() with __ctzsi2() into a single object with > an efficient tail call. The former implementation of __ctzdi2() was in C. > > On architectures without a clz instruction, this version merges

Re: [PATCH v4 05/29] Import replacement 'clz' functions from CM0 library

2021-01-11 Thread Richard Earnshaw via Gcc-patches
A general comment before we start: CLZ was added to the Arm ISA in Armv5. So all subsequent Arm versions (and all versions implementing thumb2) will have this instruction. So the only cases where you'll need a fallback are armv6m (and derivatives) and pre-armv5 (Arm or thumb1). So there's no

Re: [patch] fix -Wformat-diag warnings in rs6000-call.c

2021-01-11 Thread Matthias Klose
On 1/10/21 10:18 PM, Martin Sebor wrote: > On 1/10/21 3:29 AM, Matthias Klose wrote: >> is the newline intended? It's followed by a debug_rtx call. > > To avoid the warning there shouldn't be any trailing punctuation > or whitespace in the message.  The GCC quoting directives should > be

Re: [PATCH v3] libgcc: Thumb-1 Floating-Point Library for Cortex M0

2021-01-11 Thread Daniel Engel
On Mon, Jan 11, 2021, at 8:07 AM, Christophe Lyon wrote: > On Sat, 9 Jan 2021 at 14:09, Christophe Lyon > wrote: > > > > On Sat, 9 Jan 2021 at 13:27, Daniel Engel wrote: > > > > > > On Thu, Jan 7, 2021, at 4:56 AM, Richard Earnshaw wrote: > > > > On 07/01/2021 00:59, Daniel Engel wrote: > > > >

Re: [PATCH v3] libgcc: Thumb-1 Floating-Point Library for Cortex M0

2021-01-11 Thread Christophe Lyon via Gcc-patches
On Sat, 9 Jan 2021 at 14:09, Christophe Lyon wrote: > > On Sat, 9 Jan 2021 at 13:27, Daniel Engel wrote: > > > > On Thu, Jan 7, 2021, at 4:56 AM, Richard Earnshaw wrote: > > > On 07/01/2021 00:59, Daniel Engel wrote: > > > > --snip-- > > > > > > > > On Wed, Jan 6, 2021, at 9:05 AM, Richard

Re: [PATCH v4 02/29] Refactor 'clz' functions into a new file.

2021-01-11 Thread Daniel Engel
On Mon, Jan 11, 2021, at 7:39 AM, Richard Earnshaw wrote: > On 11/01/2021 15:26, Richard Earnshaw wrote: > > On 11/01/2021 11:10, g...@danielengel.com wrote: > >> From: Daniel Engel > >> > >> gcc/libgcc/ChangeLog: > >> 2021-01-07 Daniel Engel > >> > >>* config/arm/lib1funcs.S: Move

Re: [PATCH] libstdc++: implement locale support for AIX

2021-01-11 Thread CHIGOT, CLEMENT via Gcc-patches
>> Hi David, Clement, >> >>> The patch is local to libstdc++ AIX support, so I believe that I can >>> approve it. >> >>have you considered merging the dragonfly and aix trees? I'm asking >>because it seems prudent to try and avoid creating more and more >>almost-but-not-quite-similar

Re: [PATCH] libstdc++: implement locale support for AIX

2021-01-11 Thread Jonathan Wakely via Gcc-patches
On 11/01/21 16:35 +0100, Rainer Orth wrote: Hi David, Clement, The patch is local to libstdc++ AIX support, so I believe that I can approve it. have you considered merging the dragonfly and aix trees? I'm asking because it seems prudent to try and avoid creating more and more

Re: [PATCH v4 02/29] Refactor 'clz' functions into a new file.

2021-01-11 Thread Richard Earnshaw via Gcc-patches
On 11/01/2021 15:26, Richard Earnshaw wrote: > On 11/01/2021 11:10, g...@danielengel.com wrote: >> From: Daniel Engel >> >> gcc/libgcc/ChangeLog: >> 2021-01-07 Daniel Engel >> >> * config/arm/lib1funcs.S: Move __clzsi2() and __clzdi2() to >> * config/arm/bits/clz2.S: New file. > > No,

Re: [PATCH] libstdc++: implement locale support for AIX

2021-01-11 Thread Rainer Orth
Hi Clement, > Would "Skip if target = aix" be ok, especially for the few that I have > absolutely no idea why they are failing ? you should try to avoid such skips if at all possible. Should the tests start to PASS in the future, either due to AIX changes or changes in libstdc++, this would go

Re: [PATCH] libstdc++: implement locale support for AIX

2021-01-11 Thread Rainer Orth
Hi David, Clement, > The patch is local to libstdc++ AIX support, so I believe that I can approve > it. have you considered merging the dragonfly and aix trees? I'm asking because it seems prudent to try and avoid creating more and more almost-but-not-quite-similar configurations (Solaris

Re: [PATCH v4 04/29] Refactor 64-bit shift functions into a new file.

2021-01-11 Thread Richard Earnshaw via Gcc-patches
On 11/01/2021 11:10, g...@danielengel.com wrote: > From: Daniel Engel > > gcc/libgcc/ChangeLog: > 2021-01-07 Daniel Engel > > * config/arm/lib1funcs.S: Move __ashldi3(), __ashrdi3(), __lshldi3() to > * config/arm/bits/shift.S: New file. Again, drop the bits/ R. > --- >

Re: [PATCH v4 03/29] Refactor 'ctz' functions into a new file.

2021-01-11 Thread Richard Earnshaw via Gcc-patches
On 11/01/2021 11:10, g...@danielengel.com wrote: > From: Daniel Engel > > gcc/libgcc/ChangeLog: > 2021-01-07 Daniel Engel > > * config/arm/lib1funcs.S: Move __ctzsi2() to > * config/arm/bits/ctz2.S: New file. Similar to the previous patch, just use ctz.S without the bits subdir.

Re: [PATCH v4 02/29] Refactor 'clz' functions into a new file.

2021-01-11 Thread Richard Earnshaw via Gcc-patches
On 11/01/2021 11:10, g...@danielengel.com wrote: > From: Daniel Engel > > gcc/libgcc/ChangeLog: > 2021-01-07 Daniel Engel > > * config/arm/lib1funcs.S: Move __clzsi2() and __clzdi2() to > * config/arm/bits/clz2.S: New file. No, please don't push these down into a subdirectory.

Re: [PATCH] libstdc++: implement locale support for AIX

2021-01-11 Thread CHIGOT, CLEMENT via Gcc-patches
Hi David, Alright, I'll see what I can do. Would "Skip if target = aix" be ok, especially for the few that I have absolutely no idea why they are failing ? Clément From: David Edelsohn Sent: Monday, January 11, 2021 4:03 PM To: CHIGOT, CLEMENT Cc:

Re: [PATCH v4 01/29] Add and organize macros.

2021-01-11 Thread Richard Earnshaw via Gcc-patches
Some initial comments. On 11/01/2021 11:10, g...@danielengel.com wrote: > From: Daniel Engel > > These definitions facilitate subsequent patches in this series. > > gcc/libgcc/ChangeLog: > 2021-01-07 Daniel Engel > > * config/arm/t-elf: Organize functions into logical groups. > *

Re: [PATCH] libstdc++: implement locale support for AIX

2021-01-11 Thread David Edelsohn via Gcc-patches
Hi, Clement The patch is local to libstdc++ AIX support, so I believe that I can approve it. libstdc++ loads the gcc testsuite target-supports.exp extensions. The patch needs to update the libstdc++ testcases to require wchar32 or utf-8 as appropriate to avoid the failures. Thanks, David On

Re: [PATCH] [WIP] openmp: Add OpenMP 5.0 task detach clause support

2021-01-11 Thread Kwok Cheung Yeung
Hello Thanks for the review. Due to the Christmas holidays I have not finished addressing all these issues yet, but I expect to be done by the end of this week. Can this patch still make it for GCC 10, as I believe stage 4 is starting soon? Thanks Kwok On 10/12/2020 2:38 pm, Jakub Jelinek

Re: [PATCH 1/3] GCC: Pass --plugin to AR and RANLIB

2021-01-11 Thread Luis Machado via Gcc-patches
This seems to have broken the builds on AArch64-Linux Ubuntu 18.04. make[2]: Entering directory 'binutils-gdb-master-bionic/libiberty' rm -f ./libiberty.a pic/./libiberty.a noasan/./libiberty.a ar --plugin /usr/lib/gcc/aarch64-linux-gnu/7/liblto_plugin.so rc ./libiberty.a \ ./regex.o

Re: --enable-link-serialization=1 doesn't work

2021-01-11 Thread Matthias Klose
On 1/9/21 7:52 PM, Matthias Klose wrote: > The attached patch makes the link targets a little bit more verbose. Ok to > commit? approved by Jakub on IRC, checked in. > It shows that --enable-link-serialization=1 doesn't work: > > $ grep ^Linking ../log > Linking gnat1 |==-- |

Re: [PATCH v3] libgcc: Thumb-1 Floating-Point Library for Cortex M0

2021-01-11 Thread Richard Earnshaw via Gcc-patches
On 09/01/2021 18:04, Daniel Engel wrote: > On Sat, Jan 9, 2021, at 5:09 AM, Christophe Lyon wrote: >> On Sat, 9 Jan 2021 at 13:27, Daniel Engel wrote: >>> >>> On Thu, Jan 7, 2021, at 4:56 AM, Richard Earnshaw wrote: On 07/01/2021 00:59, Daniel Engel wrote: > --snip-- > > On Wed,

RE: [PATCH 1/8 v9]middle-end slp: Support optimizing load distribution

2021-01-11 Thread Richard Biener
On Mon, 11 Jan 2021, Tamar Christina wrote: > Hi Richi, > > Attached is the updated patch. > > Note that testcases for all of these will be committed with the patch but I'm > Finishing up the 32-bit Arm changes to mirror the changes the AArch64 > maintainer > wanted and then have to do

Re: [PATCH] Optimize macro: make it more predictable

2021-01-11 Thread Martin Liška
I'm suggesting postponing this to GCC 12 as I'm planning a bigger target/optimize attribute (pragma) overhaul. Martin On 12/7/20 12:03 PM, Martin Liška wrote: PING^2 On 11/26/20 2:56 PM, Martin Liška wrote: PING^1 On 11/9/20 11:35 AM, Martin Liška wrote: On 11/3/20 2:34 PM, Jakub Jelinek

Re: [PATCH] Fix UBSAN bootstrap

2021-01-11 Thread Nathan Sidwell
On 1/11/21 5:38 AM, Jakub Jelinek wrote: On Mon, Jan 11, 2021 at 11:26:42AM +0100, Martin Liška wrote: Problem here was that GCC-related options are not applied in stage2 (and later stages). It's caused by fact that CXX is xg++ in stage2 (and later stages). Fixed with the following patch.

[PATCH] tree-optimization/91403 - avoid excessive code-generation

2021-01-11 Thread Richard Biener
The vectorizer, for large permuted grouped loads, generates inefficient intermediate code (cleaned up only later) that runs into complexity issues in SCEV analysis and elsewhere. For the non-single-element interleaving case we already put a hard limit in place, this applies the same limit to the

Re: [PATCH v2] aarch64: Add cpu cost tables for A64FX

2021-01-11 Thread Richard Sandiford via Gcc-patches
"Qian, Jianhua" writes: > Hi Richard > >> -Original Message- >> From: Richard Sandiford >> Sent: Friday, January 8, 2021 7:04 PM >> To: Qian, Jianhua/钱 建华 >> Cc: gcc-patches@gcc.gnu.org >> Subject: Re: [PATCH v2] aarch64: Add cpu cost tables for A64FX >> >> Qian Jianhua writes: >> >

Re: [PATCH v2] testsuite: Fix test failures from outputs.exp [PR98225]

2021-01-11 Thread Rainer Orth
Hi Bernd, > On 1/8/21 8:27 PM, David Edelsohn wrote: >> Hi, Bernd >> >> Thanks for investigating this and creating a revised version of the >> patch. With the second patch, the gcc.misc-test/outputs.exp results >> are clean on AIX. >> > > Many thanks for confirming that the patch works. > > Is

Re: [PATCH] testsuite: Fix test failures from outputs.exp [PR98225]

2021-01-11 Thread Rainer Orth
Hi Alexandre, > On Jan 7, 2021, Bernd Edlinger wrote: > >> I don't know why that is there in the first place, as there >> are no C++ test cases, these files should not be created at all. > > collect2, on platforms that use it, create .cdtor files even for C. > David Edelsohn told me so back

RE: [PATCH 5/8 v9]middle-end slp: support complex multiply and complex multiply conjugate

2021-01-11 Thread Richard Biener
On Mon, 11 Jan 2021, Tamar Christina wrote: > Hi Richi, > > This adds support for complex multiply and complex multiply and accumulate to > the vect pattern detector. > > Bootstrapped Regtested on aarch64-none-linux-gnu, x86_64-pc-linux-gnu > and no issues. > > Ok for master? (I replied to a

Re: [PATCH] [x86]Delete dead code in ix86_expand_sse_comi.[PR98612]

2021-01-11 Thread Uros Bizjak via Gcc-patches
On Mon, Jan 11, 2021 at 11:28 AM Hongtao Liu wrote: > > Hi: > d->flag is always 0 for builtins located in > BDESC_FIRST (comi,COMI,...) > ... > BDESC_END (COMI, PCMPESTR) > So the condition can be deleted, so did BUILTIN_DESC_SWAP_OPERANDS. > > Regtested and bootstrapped on

[PATCH] tree-optimization/98526 - fix vectorizer reduction cost

2021-01-11 Thread Richard Biener
This fixes a double-counting in the reduction cost when vectorizing the reduction through the regular vectorizable_* functions. Bootstrapped / tested on x86_64-unknown-linux-gnu, pushed. 2021-01-11 Richard Biener PR tree-optimization/98526 * tree-vect-loop.c

Re: [committed] d: Support deprecated, @disable, and user-defined attributes on enum members

2021-01-11 Thread Iain Buclaw via Gcc-patches
Excerpts from Andreas Schwab's message of January 10, 2021 9:08 am: > ../../gcc/d/dmd/parse.c: In member function 'Dsymbols* > Parser::parseDeclDefs(int, Dsymbol**, PrefixAttributes*)': > ../../gcc/d/dmd/parse.c:647:29: error: unused variable 'e' > [-Werror=unused-variable] > 647 |

[committed] d: Remove visibility and lookup deprecation

2021-01-11 Thread Iain Buclaw via Gcc-patches
Hi, This patch merges the D front-end implementation with upstream dmd 2d3d13748, removing the visibility and lookup deprecation. The deprecation phase for access checks is finished. The `-ftransition=import` and `-ftransition=checkimports` switches no longer have an effect and are now removed.

Re: [PATCH v2] testsuite: Fix test failures from outputs.exp [PR98225]

2021-01-11 Thread Bernd Edlinger
On 1/8/21 8:27 PM, David Edelsohn wrote: > Hi, Bernd > > Thanks for investigating this and creating a revised version of the > patch. With the second patch, the gcc.misc-test/outputs.exp results > are clean on AIX. > Many thanks for confirming that the patch works. Is it OK to push? Thanks

[PATCH v4 04/29] Refactor 64-bit shift functions into a new file.

2021-01-11 Thread gnu
From: Daniel Engel gcc/libgcc/ChangeLog: 2021-01-07 Daniel Engel * config/arm/lib1funcs.S: Move __ashldi3(), __ashrdi3(), __lshldi3() to * config/arm/bits/shift.S: New file. --- libgcc/config/arm/bits/shift.S | 102

[PATCH v4 17/29] Import replacement 64-bit division functions from the CM0 library.

2021-01-11 Thread gnu
From: Daniel Engel gcc/libgcc/ChangeLog: 2021-01-07 Daniel Engel * config/arm/bpabi.c: Deleted unused file. * config/arm/eabi/ldiv.S: Replaced the __aeabi_ldivmod() and __aeabi_uldivmod() wrapper functions with a full implementation. * config/arm/t-bpabi:

[PATCH v4 24/29] Import single precision division from the CM0 library.

2021-01-11 Thread gnu
From: Daniel Engel gcc/libgcc/ChangeLog: 2021-01-08 Daniel Engel * config/arm/eabi/fdiv.S: New file for __divsf3(). * config/arm/lib1funcs.S: #include eabi/fdiv.S (v6m only). * config/arm/t-elf: Add _divsf3 and _fp_divloopf3 to LIB1ASMFUNCS. ---

[PATCH v4 26/29] Import float-to-integer conversion functions from the CM0 library.

2021-01-11 Thread gnu
From: Daniel Engel gcc/libgcc/ChangeLog: 2021-01-08 Daniel Engel * config/arm/bpabi-lib.h: Remove obsolete RENAME_LIBRARY directives. * config/arm/eabi/ffixed.S: New file for __float[un]sisf/disf(). * config/arm/lib1funcs.S: #include eabi/ffixed.S (v6m only). *

[PATCH v4 23/29] Import single precision multiplication from the CM0 library.

2021-01-11 Thread gnu
From: Daniel Engel gcc/libgcc/ChangeLog: 2021-01-08 Daniel Engel * config/arm/eabi/fmul.S: New file for __mulsf3(). * config/arm/lib1funcs.S: #include eabi/fmul.S (v6m only). * config/arm/t-elf: Move _mulsf3 to global scope in LIB1ASMFUNCS (this object was

[PATCH v4 22/29] Import single precision addition and subtraction from the CM0 library.

2021-01-11 Thread gnu
From: Daniel Engel Since this is the first phase of the floating point functions, some common parsing and formatting routines and also included. These common routines will be referenced by other functions in subsequent commits. However, even if the size penalty is accounted entirely to

[PATCH v4 25/29] Import integer-to-float conversion functions from the CM0 library.

2021-01-11 Thread gnu
From: Daniel Engel gcc/libgcc/ChangeLog: 2021-01-08 Daniel Engel * config/arm/bpabi-lib.h: Remove obsolete RENAME_LIBRARY directives. * config/arm/eabi/ffloat.S: New file for __float[un]sisf/disf(). * config/arm/lib1funcs.S: #include eabi/ffloat.S (v6m only). *

[PATCH v4 27/29] Import float<->double conversion functions from the CM0 library.

2021-01-11 Thread gnu
From: Daniel Engel gcc/libgcc/ChangeLog: 2021-01-08 Daniel Engel * config/arm/eabi/fcast.S: New file for __aeabi_f2d/__extendsfdf2() __aeabi_d2f(), __truncdfsf2(). * config/arm/lib1funcs.S: #include eabi/fcast.S (v6m only). * config/arm/t-elf: Add _arm_d2f and

[PATCH v4 28/29] Import float<->__fp16 conversion functions from the CM0 library.

2021-01-11 Thread gnu
From: Daniel Engel gcc/libgcc/ChangeLog: 2021-01-09 Daniel Engel * config/arm/eabi/fcast.S: Add __aeabi_f2h*() and __aeabi_h2f*(). * config/arm/fp16: Disable duplicate C routines (v6m only). * config/arm/t-bpabi: Add _arm_f2h* and _arm_h2f* objects to LIB1ASMFUNCS. ---

[PATCH v4 18/29] Import new integer multiplication functions from the CM0 library.

2021-01-11 Thread gnu
From: Daniel Engel gcc/libgcc/ChangeLog: 2021-01-07 Daniel Engel * config/arm/eabi/lmul.S: New file for __muldi3(), __mulsidi3(), and __umulsidi3(). * config/arm/lib1funcs.S: #eabi/lmul.S (v6m only). * config/arm/t-elf: Add the new objects to LIB1ASMFUNCS. ---

[PATCH v4 21/29] Refactor Thumb-1 floating point subtraction into a new file.

2021-01-11 Thread gnu
From: Daniel Engel gcc/libgcc/ChangeLog: 2021-01-07 Daniel Engel * config/arm/bpabi-v6m.S: Moved __aeabi_frsub() to * config/arm/eabi/fadd.S: New file. * config/arm/lib1funcs.S: #include eabi/fadd.S (v6m only). --- libgcc/config/arm/bpabi-v6m.S | 16

[PATCH v4 20/29] Import single precision comparison functions from the CM0 library.

2021-01-11 Thread gnu
From: Daniel Engel These functions are significantly smaller and faster than the wrapper functions and soft-float implementation they replace. Using the first comparison operator (e.g. '<=') costs about 70 bytes initially, but every additional operator incrementally adds just 4 bytes. NOTE: It

[PATCH v4 19/29] Refactor Thumb-1 single precision comparison functions into a new file.

2021-01-11 Thread gnu
From: Daniel Engel gcc/libgcc/ChangeLog: 2021-01-07 Daniel Engel * config/arm/bpabi-v6m.S: Moved __aeabi_fcmp*() functions to * config/arm/eabi/fcmp.S: New file. * config/arm/lib1funcs.S: #include eabi/fcmp.S (v6m only). --- libgcc/config/arm/bpabi-v6m.S | 63

[PATCH v4 07/29] Import replacement 64-bit shift functions from CM0 library.

2021-01-11 Thread gnu
From: Daniel Engel In Thumb mode, the new functions are each 1-2 instructions smaller and faster, and branchless when the IT instruction is available. The ARM versions were converted to the "xxl/xxh" big-endian register convention, but are otherwise unchanged. gcc/libgcc/ChangeLog: 2021-01-07

  1   2   >