[r12-6606 Regression] FAIL: g++.dg/asan/asan_test.C -O2 (test for excess errors) on Linux/x86_64

2022-01-15 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, 9d6a0f388eb048f8d87f47af78f07b5ce513bfe6 is the first bad commit commit 9d6a0f388eb048f8d87f47af78f07b5ce513bfe6 Author: Martin Sebor Date: Sat Jan 15 16:41:40 2022 -0700 Add -Wdangling-pointer [PR63272]. caused FAIL: gcc.dg/torture/pr57147-2.c -O2 -flto -fno-use-linke

Re: [PATCH] [i386] GLC tuning: Break false dependency for dest register.

2022-01-15 Thread Hongtao Liu via Gcc-patches
On Sun, Jan 16, 2022 at 12:44 AM Uros Bizjak via Gcc-patches wrote: > > On Sat, Jan 15, 2022 at 5:39 PM Hongyu Wang wrote: > > > > Thanks for the suggestion, here is the updated patch that survived > > bootstrap/regtest. > > LGTM for me, but please get the final approval from Hongtao. > Ok, thank

[PATCH] Strengthen memory memory order for atomic::wait/notify

2022-01-15 Thread Thomas Rodgers via Gcc-patches
This patch updates the memory order of atomic accesses to the waiter's count to match libc++'s usage. It should be backported to GCC11. Tested x86_64-pc-linux-gnu. From f5ed7674f86283db4f4ff49a2cc65d4f852413a1 Mon Sep 17 00:00:00 2001 From: Thomas Rodgers Date: Sat, 15 Jan 2022 17:40:49 -0800 Sub

[PATCHv3] libiberty rust-demangle, ignore .suffix

2022-01-15 Thread Mark Wielaard
Rust symbols can have a .suffix because of compiler transformations. These can be ignored in the demangled name. Which is what this patch implements. By stopping at the first dot for v0 symbols and searching backwards to the ending 'E' for legacy symbols. An alternative implementation could be to

Re: gcc/configure: out of date

2022-01-15 Thread Martin Sebor via Gcc-patches
Martin, I've looked into removing the -Wno-error for this warning for just a subset of targets. It seems doable with some hardcoding in configure.ac but if you're planning to do the cleanup for all of them I'm wondering if we should even bother. What do you think? Martin On 1/14/22 08:46, Mart

Re: [PATCH] libiberty rust-demangle, ignore .suffix

2022-01-15 Thread Mark Wielaard
Hi Eddy, On Mon, Dec 20, 2021 at 01:50:52PM +0200, Eduard-Mihai Burtescu wrote: > Apologies for the delay, the email fell through the cracks somehow. And then I went on vacation... Sorry this fairly simple patch takes so long. > The updated patch looks like it would work alright, only needs a co

Re: [PATCH v2 1/2] add -Wuse-after-free

2022-01-15 Thread Martin Sebor via Gcc-patches
On 1/11/22 15:40, Jason Merrill wrote: On 11/30/21 17:32, Martin Sebor via Gcc-patches wrote: Attached is a revised patch with the following changes based on your comments: 1) Set and use statement uids to determine which statement     precedes which in the same basic block. 2) Avoid testing fl

Re: [PATCH] PR fortran/83079 - ICE and wrong code with TRANSFER and character(kind=4)

2022-01-15 Thread Thomas Koenig via Gcc-patches
Hi Harald, An early *ping* ... OK. Thanks for the patch! Best regards Thomas

Re: [PATCH] PR fortran/83079 - ICE and wrong code with TRANSFER and character(kind=4)

2022-01-15 Thread Harald Anlauf via Gcc-patches
An early *ping* ... Am 11.01.22 um 22:17 schrieb Harald Anlauf via Fortran: Dear Fortranners, when digging into the issue pointed out in the PR by Gerhard it turned out that there were several issues with the TRANSFER intrinsics in the case MOLD was CHARACTER(kind=4). Default CHARACTER was fin

[PATCH] i386: Improve and optimize ix86_expand_sse_movcc

2022-01-15 Thread Uros Bizjak via Gcc-patches
Modernize ix86_expand_sse_movcc to use expand_simple_{unop,binop} infrastructure to avoid manual twiddling with output registers. Also fix a couple of inconsistent vector_all_ones_operand usages, break a couple of unnecessary else-if chains, eliminate common subexpressions and do some general code

[Patch][V5][Patch 2/2]Enable -Wuninitialized + -ftrivial-auto-var-init for address taken variables.

2022-01-15 Thread Qing Zhao via Gcc-patches
Hi, Richard, This is the updated version for the change of "Enable -Wuninitialized + -ftrivial-auto-var-init for address taken variables”. Compared to the previous patch, I mainly made the following change: Delete the 4th parameter of “warn_uninit”, construct the warning message string based

Re: [PATCH] [i386] GLC tuning: Break false dependency for dest register.

2022-01-15 Thread Uros Bizjak via Gcc-patches
On Sat, Jan 15, 2022 at 5:39 PM Hongyu Wang wrote: > > Thanks for the suggestion, here is the updated patch that survived > bootstrap/regtest. LGTM for me, but please get the final approval from Hongtao. Thanks, Uros. > > Please note reg_mentioned_p in the above condition. This function > > ret

Re: [PATCH] widening_mul, i386, v2: Improve spaceship expansion on x86 [PR103973]

2022-01-15 Thread Uros Bizjak via Gcc-patches
On Sat, Jan 15, 2022 at 12:23 PM Jakub Jelinek wrote: > > On Sat, Jan 15, 2022 at 11:42:55AM +0100, Uros Bizjak wrote: > > Yes, that would be nice. XFmode is used for long double, and not obsolete. > > Ok, that seems to work. Compared to the incremental patch I've posted, I > also had to add hand

Re: [PATCH] [i386] GLC tuning: Break false dependency for dest register.

2022-01-15 Thread Hongyu Wang via Gcc-patches
Thanks for the suggestion, here is the updated patch that survived bootstrap/regtest. > Please note reg_mentioned_p in the above condition. This function > returns nonzero if register op0 appears somewhere within op1 and is > critical for the correct operation of your patch. I added reg_mentioned_

Re: [PATCH] c++: ICE with noexcept and canonical types [PR101715]

2022-01-15 Thread Patrick Palka via Gcc-patches
On Fri, 14 Jan 2022, Marek Polacek via Gcc-patches wrote: > This is a "canonical types differ for identical types" ICE, which started > with r11-4682. It's a bit tricky to explain. Consider: > > template struct S { > S bar() noexcept(T::value); // #1 > S foo() noexcept(T::value); /

[PATCH] c++, dyninit, v2: Optimize C++ dynamic initialization by constants into DECL_INITIAL adjustment [PR102876]

2022-01-15 Thread Jakub Jelinek via Gcc-patches
Hi! Sorry for the delay in response. On Thu, Dec 02, 2021 at 02:35:25PM +0100, Richard Biener wrote: > So with > > +/* Mark start and end of dynamic initialization of a variable. */ > +DEF_INTERNAL_FN (DYNAMIC_INIT_START, ECF_LEAF | ECF_NOTHROW, ". r ") > +DEF_INTERNAL_FN (DYNAMIC_INIT_END, ECF

Re: [PATCH] gcc: pass-manager: Fix memory leak. [PR jit/63854]

2022-01-15 Thread Marc Nieper-Wißkirchen
Jeff, David, do you need any more input from my side? -- Marc Am Sa., 8. Jan. 2022 um 17:32 Uhr schrieb Jeff Law : > > > > On 1/6/2022 6:53 AM, David Malcolm via Gcc-patches wrote: > > On Sun, 2021-12-19 at 22:30 +0100, Marc Nieper-Wißkirchen wrote: > >> This patch fixes a memory leak in the pass

[r12-6420 Regression] FAIL: g++.dg/vect/pr92595.cc -std=c++2a (test for excess errors) on Linux/x86_64

2022-01-15 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, d3ff7420e941931d32ce2e332e7968fe67ba20af is the first bad commit commit d3ff7420e941931d32ce2e332e7968fe67ba20af Author: Andre Vieira Date: Thu Dec 2 14:34:15 2021 + [vect] Re-analyze all modes for epilogues caused FAIL: gcc.dg/tree-ssa/gen-vect-26.c (internal compil

[r12-6560 Regression] FAIL: gfortran.dg/gomp/allocate-2.f90 -O (test for errors, line 36) on Linux/x86_64

2022-01-15 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, 69561fc781aca3dea3aa4d5d562ef5a502965924 is the first bad commit commit 69561fc781aca3dea3aa4d5d562ef5a502965924 Author: Hafiz Abid Qadeer Date: Fri Sep 24 10:04:12 2021 +0100 Add support for allocate clause (OpenMP 5.0). caused FAIL: gfortran.dg/gomp/allocate-2.f90 -O

[r12-6404 Regression] FAIL: gfortran.dg/ieee/signaling_1.f90 -Os (test for excess errors) on Linux/x86_64

2022-01-15 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, 492954263e39346287a5a2a32bcc5312466a0ee1 is the first bad commit commit 492954263e39346287a5a2a32bcc5312466a0ee1 Author: Francois-Xavier Coudert Date: Sun Jan 2 11:36:23 2022 +0100 Fortran: Allow IEEE_CLASS to identify signaling NaNs caused FAIL: gfortran.dg/ieee/signali

[r12-6586 Regression] FAIL: g++.dg/torture/pr57993-2.C -Os (test for excess errors) on Linux/x86_64

2022-01-15 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, 74abb0beb420830e52dfc6b3ee74e77dae8e31a3 is the first bad commit commit 74abb0beb420830e52dfc6b3ee74e77dae8e31a3 Author: Martin Liska Date: Fri Jan 14 15:21:40 2022 +0100 testsuite: rename 2 files. caused FAIL: g++.dg/torture/pr57993-2.C -O0 (test for excess errors) F

[r12-6581 Regression] FAIL: g++.old-deja/g++.robertl/eb43.C -std=c++2a (test for excess errors) on Linux/x86_64

2022-01-15 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, de196e5dd8ea4d0ed01a8c265afdd3676e27545b is the first bad commit commit de196e5dd8ea4d0ed01a8c265afdd3676e27545b Author: Jonathan Wakely Date: Tue Jan 11 18:42:38 2022 + libstdc++: Add attribute to features deprecated in C++17 [PR91260] caused FAIL: g++.old-deja/g++.

[r12-6517 Regression] FAIL: gfortran.dg/ieee/signaling_1.f90 -Os execution test on Linux/x86_64

2022-01-15 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, 6b14100b9504800768da726dcb81f1857db3b493 is the first bad commit commit 6b14100b9504800768da726dcb81f1857db3b493 Author: Francois-Xavier Coudert Date: Wed Jan 12 11:19:37 2022 +0100 Fortran: fix testcase compiler flags caused FAIL: gfortran.dg/ieee/signaling_1.f90 -O0

[r12-6578 Regression] FAIL: g++.dg/cpp0x/constexpr-compare2.C -std=c++2a (test for excess errors) on Linux/x86_64

2022-01-15 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, d686d5d85c23451c03799dc55e456b73065f7333 is the first bad commit commit d686d5d85c23451c03799dc55e456b73065f7333 Author: Jakub Jelinek Date: Fri Jan 14 12:07:49 2022 +0100 c++: Reject in constant evaluation address comparisons of start of one var and end of another [PR890

[r12-6585 Regression] FAIL: c-c++-common/Walloca-larger-than.c -Wc++-compat (test for excess errors) on Linux/x86_64

2022-01-15 Thread sunil.k.pandey via Gcc-patches
On Linux/x86_64, d8b64476138671f3d89cd66f224a9b59e465631b is the first bad commit commit d8b64476138671f3d89cd66f224a9b59e465631b Author: Martin Liska Date: Fri Jan 14 15:04:33 2022 +0100 testsuite: rename files in c-c++-common. caused FAIL: c-c++-common/Walloca-larger-than.c -std=gnu++

Re: [PATCH] OpenMP front-end: allow requires dynamic_allocators

2022-01-15 Thread Jakub Jelinek via Gcc-patches
On Mon, Dec 20, 2021 at 03:16:23PM +, Andrew Stubbs wrote: > This patch removes the "sorry" message for the OpenMP "requires > dynamic_allocators" feature in C, C++ and Fortran. > > The clause is supposed to state that the user code will not work without the > omp_alloc/omp_free and omp_init_a

[PATCH] widening_mul, i386, v2: Improve spaceship expansion on x86 [PR103973]

2022-01-15 Thread Jakub Jelinek via Gcc-patches
On Sat, Jan 15, 2022 at 11:42:55AM +0100, Uros Bizjak wrote: > Yes, that would be nice. XFmode is used for long double, and not obsolete. Ok, that seems to work. Compared to the incremental patch I've posted, I also had to add handling of the case where we have just x == y ? 0 : x < y ? -1 : 1 (b

Re: [PATCH] widening_mul, i386: Improve spaceship expansion on x86 [PR103973]

2022-01-15 Thread Uros Bizjak via Gcc-patches
On Sat, Jan 15, 2022 at 10:56 AM Jakub Jelinek wrote: > > On Sat, Jan 15, 2022 at 09:29:05AM +0100, Uros Bizjak wrote: > > > --- gcc/config/i386/i386.md.jj 2022-01-14 11:51:34.432384170 +0100 > > > +++ gcc/config/i386/i386.md 2022-01-14 18:22:41.140906449 +0100 > > > @@ -23886,6 +23886,18 @@

Re: [PATCH] widening_mul, i386: Improve spaceship expansion on x86 [PR103973]

2022-01-15 Thread Jakub Jelinek via Gcc-patches
On Sat, Jan 15, 2022 at 09:29:05AM +0100, Uros Bizjak wrote: > > --- gcc/config/i386/i386.md.jj 2022-01-14 11:51:34.432384170 +0100 > > +++ gcc/config/i386/i386.md 2022-01-14 18:22:41.140906449 +0100 > > @@ -23886,6 +23886,18 @@ (define_insn "hreset" > >[(set_attr "type" "other") > > (

Re: [PATCH] widening_mul, i386: Improve spaceship expansion on x86 [PR103973]

2022-01-15 Thread Uros Bizjak via Gcc-patches
On Fri, Jan 14, 2022 at 11:56 PM Jakub Jelinek wrote: > > Hi! > > C++20: > #include > auto cmp4way(double a, double b) > { > return a <=> b; > } > expands to: > ucomisd %xmm1, %xmm0 > jp .L8 > movl$0, %eax > jne .L8 > .L2: > ret > .p2

Re: [PATCH] c++: error message for dependent template members [PR70417]

2022-01-15 Thread Anthony Sharp via Gcc-patches
Hi Jason, Hope you are well. Apologies, I've not had time to sit down and look at this since last month I quit my old job, then I had family around for the whole of the Christmas period, and then even more recently I've had to start my new job. In any case happy that you managed to figure it all