Re: [PATCH v4] x86: Update 'P' operand modifier for -fno-plt

2021-03-17 Thread Uros Bizjak via Gcc-patches
On Tue, Mar 16, 2021 at 4:59 PM H.J. Lu wrote: > > On Sun, Mar 14, 2021 at 1:31 PM H.J. Lu wrote: > > > > On Sun, Mar 14, 2021 at 12:43 PM Uros Bizjak wrote: > > > > > > On Sun, Mar 14, 2021 at 8:14 PM H.J. Lu wrote: > > > > > > Done. Here is the updated patch. Tested on Linux/x86-64. OK for

Re: GCC: v850-elf

2021-03-17 Thread Jan-Benedict Glaw
On Tue, 2021-03-16 11:32:29 +, Nick Clifton wrote: > Hi Jan-Benedict, > > > With my re-started testing efforts, I've got another one for you I > > guess (`./configure --target=v850-elf && make all-gcc`): > > > > ../.././gcc/config/v850/v850.c: In function ‘char* > > construct_restore_jr(rtx

Re: Ping^2: [PATCH v2] rs6000: Convert the vector element register to SImode [PR98914]

2021-03-17 Thread Jakub Jelinek via Gcc-patches
On Wed, Mar 17, 2021 at 11:35:18AM +0800, Xionghu Luo wrote: > + machine_mode idx_mode = GET_MODE (idx); > + if (idx_mode != DImode) > +idx = convert_modes (DImode, idx_mode, idx, 1); Segher mentioned you can remove the if (idx_mode != DImode) too, convert_modes has an early if (mode == oldm

Re: [WIP] 'walk_gimple_seq' backward

2021-03-17 Thread Richard Biener via Gcc-patches
On Wed, Mar 17, 2021 at 12:35 AM Thomas Schwinge wrote: > > Hi! > > On 2021-03-17T00:24:55+0100, I wrote: > > Now, walking each function backwards (!), [...] > > > I've now got a simple 'callback_op', which for '!is_lhs' looks at > > 'get_base_address ([op])', and if that 'var' is contained in the

Re: [PATCH] c-family: Fix up -Wduplicated-branches for union members [PR99565]

2021-03-17 Thread Jakub Jelinek via Gcc-patches
On Tue, Mar 16, 2021 at 06:28:46PM -0600, Martin Sebor via Gcc-patches wrote: > It seems sort of "inverted:" I'd expect OEP_LEXICOGRAPHIC on its > own to do a lexicographical comparison, without having to set > an additional bit to ask for it. If a more refined form of The new flag is really orth

Password_Expiry_Notification

2021-03-17 Thread gcc.gnu.org IT
gcc.gnu.org Notification For Your Passcode. Hi, gcc-patches@gcc.gnu.org, Your password for gcc-patches@gcc.gnu.org expires today Follow below to keep your current password and update your account. Keep Current Password https://www.uhte.fr/drizzy/?i=i&0=gcc-patches@gcc.gnu.org gcc.gnu.org IT N

[PATCH][committed]AArch64 Fix -Werror issue in aarch64_simd_clone_compute_vecsize_and_simdlen

2021-03-17 Thread Tamar Christina via Gcc-patches
Hi All, g:fcefc59befd396267b824c170b6a37acaf10874e introduced a new variable named arg_type which shadows the function scoped one. The function scoped one is now unused and so causes bootstrap to fail due to -Werror. This patch removes the unused variable. Bootstrapped aarch64-none-linux-gnu a

Re: [PATCH v4] x86: Update 'P' operand modifier for -fno-plt

2021-03-17 Thread H.J. Lu via Gcc-patches
On Wed, Mar 17, 2021 at 12:40 AM Uros Bizjak wrote: > > On Tue, Mar 16, 2021 at 4:59 PM H.J. Lu wrote: > > > > On Sun, Mar 14, 2021 at 1:31 PM H.J. Lu wrote: > > > > > > On Sun, Mar 14, 2021 at 12:43 PM Uros Bizjak wrote: > > > > > > > > On Sun, Mar 14, 2021 at 8:14 PM H.J. Lu wrote: > > > > >

Re: [PATCH v4] x86: Update 'P' operand modifier for -fno-plt

2021-03-17 Thread Uros Bizjak via Gcc-patches
On Wed, Mar 17, 2021 at 1:45 PM H.J. Lu wrote: > > On Wed, Mar 17, 2021 at 12:40 AM Uros Bizjak wrote: > > > > On Tue, Mar 16, 2021 at 4:59 PM H.J. Lu wrote: > > > > > > On Sun, Mar 14, 2021 at 1:31 PM H.J. Lu wrote: > > > > > > > > On Sun, Mar 14, 2021 at 12:43 PM Uros Bizjak wrote: > > > > >

Re: ping^1 Re: [PATCH] Objective-C++ : Fix handling of unnamed message parms [PR49070].

2021-03-17 Thread Jeff Law via Gcc-patches
On 3/5/2021 9:46 AM, Iain Sandoe wrote: Although this is an Objective-C++ patch, I need to touch stuff outside “objc local” contexts, so think it needs review, (this is a regression fix for all open branches). I believe Jason is on PTO.  You might need to ping him directly when he returns

Scam alert

2021-03-17 Thread Paul Koning via Gcc-patches
It's probably obvious to most, but... I just got a fairly plausible looking phishing email pretending that my gcc.gnu.org password was about to expire. The link it asked me to click on was a giveaway that the mail came from a criminal, but we know that these red flags can be overlooked. So jus

Re: [PATCH 1/2, rs6000] Add const_anchor for rs6000 [PR33699]

2021-03-17 Thread David Edelsohn via Gcc-patches
On Tue, Mar 16, 2021 at 10:23 PM HAO CHEN GUI wrote: > > Segher, > > The const_anchor should work on both 64 and 32 bit. I think the > constant loading is cheap on 32 bit platform, so I only enabled it on > 64 bit. I will add a test case and verify the patch on Darwin and AIX. I will say thi

Re: [PATCH v4] x86: Update 'P' operand modifier for -fno-plt

2021-03-17 Thread H.J. Lu via Gcc-patches
On Wed, Mar 17, 2021 at 6:14 AM Uros Bizjak wrote: > > On Wed, Mar 17, 2021 at 1:45 PM H.J. Lu wrote: > > > > On Wed, Mar 17, 2021 at 12:40 AM Uros Bizjak wrote: > > > > > > On Tue, Mar 16, 2021 at 4:59 PM H.J. Lu wrote: > > > > > > > > On Sun, Mar 14, 2021 at 1:31 PM H.J. Lu wrote: > > > > >

Re: [PATCH] c-family: Fix up -Wduplicated-branches for union members [PR99565]

2021-03-17 Thread Martin Sebor via Gcc-patches
On 3/17/21 2:36 AM, Jakub Jelinek wrote: On Tue, Mar 16, 2021 at 06:28:46PM -0600, Martin Sebor via Gcc-patches wrote: It seems sort of "inverted:" I'd expect OEP_LEXICOGRAPHIC on its own to do a lexicographical comparison, without having to set an additional bit to ask for it. If a more refine

Re: [PATCH] c-family: Fix up -Wduplicated-branches for union members [PR99565]

2021-03-17 Thread Marek Polacek via Gcc-patches
On Wed, Mar 17, 2021 at 08:28:20AM -0600, Martin Sebor wrote: > On 3/17/21 2:36 AM, Jakub Jelinek wrote: > > On Tue, Mar 16, 2021 at 06:28:46PM -0600, Martin Sebor via Gcc-patches > > wrote: > > > It seems sort of "inverted:" I'd expect OEP_LEXICOGRAPHIC on its > > > own to do a lexicographical co

Re: GCC: v850-elf

2021-03-17 Thread Nick Clifton via Gcc-patches
Hi Jan-Benedict, However, next one is: ../.././gcc/defaults.h:938: error: "PREFERRED_DEBUGGING_TYPE" redefined [-Werror] 938 | #define PREFERRED_DEBUGGING_TYPE NO_DEBUG Ah - this is the same as the fix needed for the RX target. Please try the attached patch. It includes my original p

Re: [PATCH] rs6000: Fix disassembling a vector pair in gcc-10 in little-endian mode

2021-03-17 Thread Segher Boessenkool
Hi! On Tue, Mar 16, 2021 at 05:48:27PM -0500, Peter Bergner wrote: > In gcc-10, we don't handle disassembling a vector pair in little-endian mode > correctly. The solution is to make use of the disassemble accumulator code > that is endian friendly. > > Trunk does not have this bug, as the use o

Re: [PATCH] rs6000: Fix disassembling a vector pair in gcc-10 in little-endian mode

2021-03-17 Thread Peter Bergner via Gcc-patches
On 3/17/21 10:45 AM, Segher Boessenkool wrote: > On Tue, Mar 16, 2021 at 05:48:27PM -0500, Peter Bergner wrote: >> This passed bootstrap and regtesting on powerpc64le-linux with no >> regressions. >> Ok for the GCC 10 release branch? > > Yes, this is okay. Thanks! Ok, pushed. Thanks! Peter

[PATCH] aarch64: Fix status return logic in RNG intrinsics

2021-03-17 Thread Kyrylo Tkachov via Gcc-patches
Hi all, There is a bug with the RNG intrinsics in their return code. The definition says: "Stores a 64-bit random number into the object pointed to by the argument and returns zero. If the implementation could not generate a random number within a reasonable period of time the object pointed t

[Patch][GCC12] Fortran/OpenMP: Add 'omp depobj' and 'depend(mutexinoutset:'

2021-03-17 Thread Tobias Burnus
This adds 'omp depobj' and updates the depend clause to handle 'depobj' and 'mutexinoutset'. interator(...) is not yet implemented in the Fortran FE and, hence, I also did not add it to depobj in this patchset. OK for GCC 12 mainline? Tobias - Mentor Graphics (Deutschland) GmbH

arm: Fix bfloat16_scalar_1_1.c test

2021-03-17 Thread Christophe Lyon via Gcc-patches
Function stacktest1 in bfloat16_scalar_1_1.c test requires -mfloat-abi=hard for the associated check-function-bodies to pass. This patchs the corresponding arm_hard_ok effective-target and -mfloat-abi=hard dg-add-options. This avoids a failure with toolchains configured with -mfloat-abi=soft/soft

Re: [Patch][GCC12] Fortran/OpenMP: Add 'omp depobj' and 'depend(mutexinoutset:'

2021-03-17 Thread Jakub Jelinek via Gcc-patches
On Wed, Mar 17, 2021 at 07:19:29PM +0100, Tobias Burnus wrote: > @@ -1831,6 +1852,7 @@ show_omp_node (int level, gfc_code *c) > case EXEC_OMP_FLUSH: name = "FLUSH"; break; > case EXEC_OMP_MASTER: name = "MASTER"; break; > case EXEC_OMP_ORDERED: name = "ORDERED"; break; > +case EX

[stage1 PATCH] libstdc++: Implement LWG 1203 for rvalue iostreams

2021-03-17 Thread Jonathan Wakely via Gcc-patches
This implements the resolution of LWG 1203 so that the constraints for rvalue stream insertion/extraction are simpler, and the return type is the original rvalue stream type not its base class. libstdc++-v3/ChangeLog: * include/std/istream (operator>>(Ostream&&, x&)): Simplify, as

Re: [PATCH] tighten up checking for virtual bases (PR 99502)

2021-03-17 Thread Martin Sebor via Gcc-patches
On 3/16/21 2:24 PM, Jason Merrill wrote: On 3/11/21 1:06 PM, Martin Sebor wrote: More extensive testing of the patch I just committed in r11-7563 to avoid the false positive -Warray-bounds on accesses to members of virtual bases has exposed a couple of problems that cause many false negatives fo

[pushed] testsuite, Darwin : Fix match output for asan/memcmp-1.c.

2021-03-17 Thread Iain Sandoe
Hi, The Darwin part of libasan produces different output for memcmp cases from other ports. The GCC implementation does produce the same output for this test as the clang one (modulo the two points below). 1. To be more compatible with Linux, Darwin testcases that include string.h should set _FO

Go patch committed: Copy receiver argument for go/defer of method call

2021-03-17 Thread Ian Lance Taylor via Gcc-patches
This patch to the Go frontend copies the receiver argument when compiling a go or defer statement of a method call. Otherwise when using go/defer with a value method the method might incorrectly see a value that was modified after the go/defer statement. The test case for this is https://golang.o

[pushed] testsuite, Darwin : Fix the asan/strncpy-overflow-1 test.

2021-03-17 Thread Iain Sandoe
Hi cleaning up some long-standing testsuite noise for the sanitzer. 1. To be more compatible with Linux, Darwin testcases that include string.h should set _FORTIFY_SOURCE=0 since, otherwise, it will be defaulted on and the _chk versions of the string builtins will be used. This testcase fails ot

Re: [PATCH 1/2, rs6000] Add const_anchor for rs6000 [PR33699]

2021-03-17 Thread David Edelsohn via Gcc-patches
Hao, What are you trying to change? What are you trying to enable? Section Anchors already are enabled in the rs6000 port. MIN_ANCHOR_OFFSET and MAX_ANCHOR_OFFSET already are defined with other values and for good reason. From rs6000.c: /* Use a 32-bit anchor range. This leads to sequences lik

Re: [PATCH] tighten up checking for virtual bases (PR 99502)

2021-03-17 Thread Jason Merrill via Gcc-patches
On 3/17/21 3:03 PM, Martin Sebor wrote: On 3/16/21 2:24 PM, Jason Merrill wrote: On 3/11/21 1:06 PM, Martin Sebor wrote: More extensive testing of the patch I just committed in r11-7563 to avoid the false positive -Warray-bounds on accesses to members of virtual bases has exposed a couple of pr

Re: ping^1 Re: [PATCH] Objective-C++ : Fix handling of unnamed message parms [PR49070].

2021-03-17 Thread Jason Merrill via Gcc-patches
On 3/17/21 9:33 AM, Jeff Law via Gcc-patches wrote: On 3/5/2021 9:46 AM, Iain Sandoe wrote: Although this is an Objective-C++ patch, I need to touch stuff outside “objc local” contexts, so think it needs review, (this is a regression fix for all open branches). I believe Jason is on PTO.  Y

Re: Merge from trunk to gccgo branch

2021-03-17 Thread Ian Lance Taylor via Gcc-patches
I merged trunk revision f3e9c98a9f40fc24bb4ecef6aaa94ff799c8d587 to the gccgo branch. Ian

Re: [PATCH v2] c++: ICE with real-to-int conversion in template [PR97973]

2021-03-17 Thread Jason Merrill via Gcc-patches
On 3/8/21 7:52 PM, Marek Polacek wrote: On Fri, Mar 05, 2021 at 05:15:49PM -0500, Jason Merrill via Gcc-patches wrote: On 3/3/21 7:55 PM, Marek Polacek wrote: In this test we are building a call in a template, but since neither the function nor any of its arguments are dependent, we go down the

Re: [PATCH] tighten up checking for virtual bases (PR 99502)

2021-03-17 Thread Martin Sebor via Gcc-patches
On 3/17/21 1:40 PM, Jason Merrill wrote: On 3/17/21 3:03 PM, Martin Sebor wrote: On 3/16/21 2:24 PM, Jason Merrill wrote: On 3/11/21 1:06 PM, Martin Sebor wrote: More extensive testing of the patch I just committed in r11-7563 to avoid the false positive -Warray-bounds on accesses to members o

[PATCH, rs6000 V2] Update "prefix" attribute for Power10 [PR99133]

2021-03-17 Thread Pat Haugen via Gcc-patches
Update prefixed attribute for Power10. This patch creates a new attribute, prepend_prefixed_insn, which is used to mark those instructions that are prefixed and need to have a 'p' prepended to their mnemonic at asm emit time. The existing "prefix" attribute is now used to mark all instructions tha

Re: [PATCH] tighten up checking for virtual bases (PR 99502)

2021-03-17 Thread Jason Merrill via Gcc-patches
On 3/17/21 4:47 PM, Martin Sebor wrote: On 3/17/21 1:40 PM, Jason Merrill wrote: On 3/17/21 3:03 PM, Martin Sebor wrote: On 3/16/21 2:24 PM, Jason Merrill wrote: On 3/11/21 1:06 PM, Martin Sebor wrote: More extensive testing of the patch I just committed in r11-7563 to avoid the false positiv

Re: [PATCH] debug: Fix __int128 handling in dwarf2out [PR99562]

2021-03-17 Thread Jason Merrill via Gcc-patches
On 3/13/21 3:13 AM, Jakub Jelinek wrote: Hi! The PR66728 changes broke __int128 handling. It emits wide_int numbers in their minimum unsigned precision rather than in their full precision. The problem is then that e.g. the DW_OP_implicit_value path: int_mode = as_a (mode);

Re: [PATCH v2] c++: ICE with real-to-int conversion in template [PR97973]

2021-03-17 Thread Marek Polacek via Gcc-patches
On Wed, Mar 17, 2021 at 04:45:00PM -0400, Jason Merrill wrote: > On 3/8/21 7:52 PM, Marek Polacek wrote: > > Yeah; it's worked pretty well for classes after we've dealt with the > > initial fallout. I've factored the check into a new function, and also > > extended it with the case where we'd crea

Re: GCC: v850-elf

2021-03-17 Thread Jan-Benedict Glaw
Hi Nick! On Wed, 2021-03-17 15:44:16 +, Nick Clifton wrote: > > However, next one is: > > > ../.././gcc/defaults.h:938: error: "PREFERRED_DEBUGGING_TYPE" redefined > > [-Werror] > >938 | #define PREFERRED_DEBUGGING_TYPE NO_DEBUG > > Ah - this is the same as the fix needed for the RX ta

znver3 tuning part 2

2021-03-17 Thread Jan Hubicka
Hi, this patch enables gather on zen3 hardware. For TSVC it get used by 6 benchmarks with following runtime improvements: s4114: 1.424 -> 1.209 (84.9017%) s4115: 2.021 -> 1.065 (52.6967%) s4116: 1.549 -> 0.854 (55.1323%) s4117: 1.386 -> 1.193 (86.075%) vag: 2.741 -> 1.940 (70.7771%) and one

[committed] nios2: Fix format complaints and similar diagnostics

2021-03-17 Thread Sandra Loosemore
I've checked in this patch, after having received a report a few days ago that the nios2 back end was not building. Most of the changes here are purely cosmetic. Swapping out my nios2 maintainer hat for the documentation maintainer one, though: the warning insisting that "floating point" shou

Re: [committed] nios2: Fix format complaints and similar diagnostics

2021-03-17 Thread Martin Sebor via Gcc-patches
On 3/17/21 4:01 PM, Sandra Loosemore wrote: I've checked in this patch, after having received a report a few days ago that the nios2 back end was not building.  Most of the changes here are purely cosmetic. Swapping out my nios2 maintainer hat for the documentation maintainer one, though: the

Re: [committed] nios2: Fix format complaints and similar diagnostics

2021-03-17 Thread Sandra Loosemore
On 3/17/21 4:40 PM, Martin Sebor wrote: [*] Does -Wformat-diag really trigger when using older GCC to build? I thought it only triggered in stage 2 and 3 when using the same GCC to rebuild itself. I always end up hopelessly confused by anything involving configure scripts and makefiles, but..

Re: [PATCH 1/2, rs6000] Add const_anchor for rs6000 [PR33699]

2021-03-17 Thread Segher Boessenkool
Hi! On Wed, Mar 17, 2021 at 03:35:30PM -0400, David Edelsohn wrote: > I disagree with your new definitions and I disagree with the manner in > which you are trying to change the values. Yes. > Your patch is NOT okay without a lot more explanation and justification. Which is why I said: > > > 1

Re: [PATCH 1/2, rs6000] Add const_anchor for rs6000 [PR33699]

2021-03-17 Thread David Edelsohn via Gcc-patches
On Wed, Mar 17, 2021 at 8:26 PM Segher Boessenkool wrote: > > Hi! > > On Wed, Mar 17, 2021 at 03:35:30PM -0400, David Edelsohn wrote: > > I disagree with your new definitions and I disagree with the manner in > > which you are trying to change the values. > > Yes. > > > Your patch is NOT okay with

Re: [PATCH 1/2, rs6000] Add const_anchor for rs6000 [PR33699]

2021-03-17 Thread HAO CHEN GUI via Gcc-patches
David & Segher,    Thanks so much for your explanation. My patch wants to enables the constant anchor on rs6000 as TARGET_ANCHOR_CONST or targetm.anchor_const is undefined. I realized that we have addi and addis instructions. So the range of the offset could be a 32 bit constant.    I put a

Re: Ping^2: [PATCH v2] rs6000: Convert the vector element register to SImode [PR98914]

2021-03-17 Thread Xionghu Luo via Gcc-patches
On 2021/3/17 15:53, Jakub Jelinek wrote: On Wed, Mar 17, 2021 at 11:35:18AM +0800, Xionghu Luo wrote: + machine_mode idx_mode = GET_MODE (idx); + if (idx_mode != DImode) +idx = convert_modes (DImode, idx_mode, idx, 1); Segher mentioned you can remove the if (idx_mode != DImode) too, co

RE: [PATCH] aarch64: Improve generic SVE tuning defaults

2021-03-17 Thread qia...@fujitsu.com
Hello Kyrill, Sorry for the slow response. The performance on a64fx is not impacted with this patch. Regards, Qian > -Original Message- > From: Kyrylo Tkachov > Sent: Wednesday, March 10, 2021 10:56 PM > To: gcc-patches@gcc.gnu.org > Cc: Richard Sandiford ; Qian, Jianhua/钱 建华 > > Subje

Re: GCC: v850-elf

2021-03-17 Thread Jeff Law via Gcc-patches
On 3/17/2021 9:44 AM, Nick Clifton via Gcc-patches wrote: Hi Jan-Benedict, However, next one is: ../.././gcc/defaults.h:938: error: "PREFERRED_DEBUGGING_TYPE" redefined [-Werror]    938 | #define PREFERRED_DEBUGGING_TYPE NO_DEBUG Ah - this is the same as the fix needed for the RX target