[committed] libphobos: Always build with warning flags enabled

Hi, This patch moves WARN_DFLAGS from GDCFLAGS to AM_DFLAGS so it is always included in the build and testsuite of libphobos. Currently, this doesn't happen as GDCFLAGS is overriden by it being set at the top-level. Bootstrapped and regression tested on x86_64-linux-gnu, and committed to mainlin

Re: [PATCH] Check DECL_CONTEXT of new/delete operators.

On Tue, Apr 7, 2020 at 5:17 PM Jonathan Wakely wrote: > > On Tue, 7 Apr 2020 at 12:57, Richard Biener > wrote: > > > > On Tue, Apr 7, 2020 at 1:46 PM Jonathan Wakely > > wrote: > > > > > > On Tue, 7 Apr 2020 at 12:40, Richard Biener > > > wrote: > > > > > > > > On Tue, Apr 7, 2020 at 1:30 PM

Re: [PING PATCH coroutines] Do not strip cleanup_point when promote temporaries out of current stmt

On Tue, Apr 7, 2020 at 11:45 PM Iain Sandoe wrote: > > Bin.Cheng wrote: > > > On Mon, Mar 16, 2020 at 5:34 PM Bin.Cheng wrote: > >> On Wed, Mar 11, 2020 at 5:07 PM Iain Sandoe wrote: > >>> Bin.Cheng wrote: > >>> > On Thu, Mar 5, 2020 at 10:18 PM Iain Sandoe wrote: > > >>> If it helps, I

Re: [RFC, Instruction Scheduler, Stage1] New hook/code to perform fusion of dependent instructions

On Tue, Apr 7, 2020 at 10:45 PM Pat Haugen via Gcc-patches wrote: > > The Power processor has the ability to fuse certain pairs of dependent > instructions to improve their performance if they appear back-to-back in > the instruction stream. In looking at the current support for > instruction fusi

Re: [C/C++, OpenACC] Reject vars of different scope in acc declare (PR94120)

I have now committed this patch as r10-7614-g13e41d8b9d3d7598c72c38acc86a3d97046c8373, reading "so we shall accept it" as approval … On 4/1/20 9:07 AM, Thomas Schwinge wrote: Even if the ICE is then fixed, should we still keep open (with a note) until

Re: [PATCH] Check DECL_CONTEXT of new/delete operators.

On Wed, 8 Apr 2020 at 08:35, Richard Biener wrote: > Ah, so __builtin_operator_new isn't a function but an alternate > new expression syntax? No, not a new-expression, because a new-expression calls operator new to obtain storage *and* begins the lifetime of one or more objects in that storage. __

Re: [PATCH, libphobos] Fix compilation dependencies on s390x-linux-musl

On 28/01/2020 05:00, Mathias Lang wrote: > Hi, > > This patch fixes GDC on s390x-linux-musl targets. It was specifically > tested under Alpine Linux (see > https://gitlab.alpinelinux.org/alpine/aports/commit/c123e0f14ab73976a36c651d47d134f249413f29 > ). > The patch fixes two issues: First, Musl al

[PATCH, testsuite] Fix PR94079 by respecting vect_hw_misalign

Hi, This is another vect case which requires special handling with vect_hw_misalign. The alignment of the second part requires misaligned vector access supports. This patch is to adjust the related guard condition and comments. Verified it on ppc64-redhat-linux (Power7 BE). Is it ok for trunk?

Re: [PATCH] sra: Fix sra_modify_expr handling of partial writes (PR 94482)

On Tue, 7 Apr 2020, Richard Biener wrote: > On April 7, 2020 6:25:26 PM GMT+02:00, Martin Jambor wrote: > >Hi, > > > >On Tue, Apr 07 2020, Richard Biener wrote: > >> On Tue, 7 Apr 2020, Martin Jambor wrote: > >> > >>> Hi, > >>> > >>> when sra_modify_expr is invoked on an expression that modifies

[Arm] Implement CDE intrinsics for MVE registers.

This patch updates the previous one by rebasing onto the recent MVE patches on trunk. Implement CDE intrinsics on MVE registers. Other than the basics required for adding intrinsics this patch consists of three changes. ** We separate out the MVE types and casts from the arm_mve.h header. This

Re: [PATCH, testsuite] Fix PR94079 by respecting vect_hw_misalign

On Wed, Apr 8, 2020 at 10:17 AM Kewen.Lin via Gcc-patches wrote: > > Hi, > > This is another vect case which requires special handling with > vect_hw_misalign. The alignment of the second part requires > misaligned vector access supports. This patch is to adjust > the related guard condition and

[Arm] Implement CDE predicated intrinsics for MVE registers

This is an update of the previous patch but rebased onto recent MVE patches. https://gcc.gnu.org/pipermail/gcc-patches/2020-April/543414.html These intrinsics are the predicated version of the intrinsics inroduced in https://gcc.gnu.org/pipermail/gcc-patches/2020-April/543527.html. These are not

[PATCH] [Arm] Implement scalar Custom Datapath Extension intrinsics

Patch rebased onto recent trunk. This patch introduces the scalar CDE (Custom Datapath Extension) intrinsics for the arm backend. There is nothing beyond the standard in this patch. We simply build upon what has been done by Dennis for the vector intrinsics. We do add `+cdecp6` to the default

Re: [PATCH] Allow new/delete operator deletion only for replaceable.

On Tue, Apr 7, 2020 at 5:01 PM Martin Liška wrote: > > Hi. > > The patch allows DCE to remove only replaceable operators new and delete. > That's achieved by proper mark up of all these operators in C++ FE. > The patch also brings all tests we've collected so far for the PR. > > Patch can bootstra

Re: [PATCH] Fix use of singleton in optinfo framework

Hi, since there has been some misunderstanding, I will explain. There are 4 possible options here: 1. FreeBSD 12.1-RELEASE, powerpc, GCC 4.2 2. FreeBSD 13.0-CURRENT (head branch), powerpc, LLVM 10.0.0 3. FreeBSD 12.1-RELEASE, powerpc64, GCC 4.2 4. FreeBSD 13.0-CURRENT (head branch), powerpc64, L

RE: [PATCH] [Arm] Implement scalar Custom Datapath Extension intrinsics

Hi Matthew, > -Original Message- > From: Matthew Malcomson > Sent: 08 April 2020 09:46 > To: gcc-patches@gcc.gnu.org > Cc: Kyrylo Tkachov ; Ramana Radhakrishnan > ; nd ; > ni...@redhat.com; Richard Earnshaw > Subject: [PATCH] [Arm] Implement scalar Custom Datapath Extension > intrinsics

Re: [PATCH v2 00/11] aarch64: Implement TImode comparisons

Segher Boessenkool writes: > On Mon, Apr 06, 2020 at 12:19:42PM +0100, Richard Sandiford wrote: >> The reason I'm not keen on using special modes for this case is that >> they'd describe one way in which the result can be used rather than >> describing what the instruction actually does. The inst

RE: [Arm] Implement CDE predicated intrinsics for MVE registers

Hi Matthew, > -Original Message- > From: Matthew Malcomson > Sent: 08 April 2020 09:32 > To: gcc-patches@gcc.gnu.org > Cc: Kyrylo Tkachov ; Richard Earnshaw > ; Ramana Radhakrishnan > ; ni...@redhat.com; nd > Subject: [Arm] Implement CDE predicated intrinsics for MVE registers > > This

Re: [PATCH v2 00/11] aarch64: Implement TImode comparisons

Richard Biener writes: > On Mon, Apr 6, 2020 at 1:20 PM Richard Sandiford > wrote: >> >> "Richard Earnshaw (lists)" writes: >> > On 03/04/2020 16:03, Richard Sandiford wrote: >> >> "Richard Earnshaw (lists)" writes: >> >>> On 03/04/2020 13:27, Richard Sandiford wrote: >> "Richard Earnshaw

[PATCH] [Arm] Implement scalar Custom Datapath Extension intrinsics

Suggestions applied. I didn't update the indentation since it seems good to me (with the `(unspec:SIDI ...)` indented one more level than the SET as suggested). I'm guessing it looked strange since that line is indented with a tab and that can look different in different text editors. This patc

RE: [Arm] Implement CDE intrinsics for MVE registers.

Hi Matthew, > -Original Message- > From: Matthew Malcomson > Sent: 08 April 2020 09:22 > To: gcc-patches@gcc.gnu.org > Cc: nd ; Kyrylo Tkachov ; > Ramana Radhakrishnan ; Richard > Earnshaw ; ni...@redhat.com > Subject: [Arm] Implement CDE intrinsics for MVE registers. > > > This patch u

Re: [testsuite][arm] Fix cmse-15.c expected output

Christophe Lyon via Gcc-patches writes: > Hi, > > While checking Martin's fix for PR ipa/94445, he made me realize that > the cmse-15.c testcase still fails at -Os because ICF means that we > generate > nonsecure2: > b nonsecure0 > > which is OK, but does not match the currently expe

[Patch][Fortran] Fix name conflict check for use-assoc (PR 92736)

Hi all, this issue only occurs when use associating a symbol in a module and then again use-associating the same symbol in its submodule. If one simply uses use m, only: i use m, only: i the issue does not occur as all symbols are only created after parsing all use statements. And for use

Re: [PATCH][Arm][1/3] Support for Arm Custom Datapath Extension (CDE): enable the feature

cdecp0, cdecp1, ..., cdecp7): New options. * config/arm/arm.c (arm_option_reconfigure_globals): Configure arm_arch_cde and arm_arch_cde_coproc to store the feature bits. * config/arm/arm.h (TARGET_CDE): New macro. * config/arm/arm_cde.h: New file. * doc/invoke.texi: Document CDE options +cdecp[0-7]. * doc/sourcebuild.texi (arm_v8m_main_cde_ok): Document new target supports option. (arm_v8m_main_cde_fp, arm_v8_1m_main_cde_mve): Likewise. gcc/testsuite/ChangeLog: 2020-04-08 Dennis Zhang * gcc.target/arm/pragma_cde.c: New test. * lib/target-supports.exp (arm_v8m_main_cde_ok): New target support option. (arm_v8m_main_cde_fp, arm_v8_1m_main_cde_mve): Likewise. arm-m-cde-cli-20200408.patch Description: arm-m-cde-cli-20200408.patch

[PATCH] rtl-optimization/93946 - fix TBAA for redundant store removal in CSE

It turns out RTL CSE tries to remove redundant stores but fails to do the usual validity check what such a change is TBAA neutral to later loads. This now triggers with the PR93946 testcases on nios2. Bootstrap & regtest running on x86_64-unknown-linux-gnu. Richard. 2020-04-08 Richard Biener

update polytypes.c -flax-vector-conversions msg

Since commit 2f6d557ff82876432be76b1892c6c3783c0095f4 AKA SVN-r269586, the inform() message suggesting the use of -flax-vector-conversions has had quotes around the option name, but the testcase still expected the message without the quotes. This patch adds to the expected compiler output the quot

RE: [PATCH][Arm][1/3] Support for Arm Custom Datapath Extension (CDE): enable the feature

> -Original Message- > From: Dennis Zhang > Sent: 08 April 2020 12:34 > To: Kyrylo Tkachov ; gcc-patches@gcc.gnu.org > Cc: nd ; Richard Earnshaw ; > Ramana Radhakrishnan > Subject: Re: [PATCH][Arm][1/3] Support for Arm Custom Datapath Extension > (CDE): enable the feature > > Hi Kyryl

Re: [PATCH] c++: ICE with defaulted comparison operator [PR94478]

On 4/7/20 3:14 PM, Marek Polacek wrote: Here we ICE because early_check_defaulted_comparison passed a null ctx to same_type_p. The attached test is ill-formed according to [class.compare.default]/1, so fixed by detecting this case early. Bootstrapped/regtested on x86_64-pc-linux-gnu, ok for tru

Re: [PATCH] c++: Fix ICE-on-invalid with lambda template [PR94507]

On 4/6/20 9:05 PM, Marek Polacek wrote: While reducing something else I noticed that we ICE on the following invalid code. In tsubst_lambda_expr, tsubst_template_decl has already reported an error and returned the error_mark_node, so make sure we don't ICE on that. I'm using a goto here because

Re: [PATCH] c++: Further fix for -fsanitize=vptr [PR94325]

On 4/6/20 6:43 PM, Jakub Jelinek wrote: Hi! For -fsanitize=vptr, we insert a NULL store into the vptr instead of just adding a CLOBBER of this. build_clobber_this makes the CLOBBER conditional on in_charge (implicit) parameter whenever CLASSTYPE_VBASECLASSES, but when adding this conditionaliza

Re: [PATCH] c++: Fix usage of CONSTRUCTOR_PLACEHOLDER_BOUNDARY inside array initializers [90996]

On 4/6/20 6:22 PM, Patrick Palka wrote: On Mon, 6 Apr 2020, Jason Merrill wrote: On 4/6/20 3:07 PM, Patrick Palka wrote: This PR reports that since the introduction of the CONSTRUCTOR_PLACEHOLDER_BOUNDARY flag, we are sometimes failing to resolve PLACEHOLDER_EXPRs inside array initializers tha

Re: [PATCH] Allow new/delete operator deletion only for replaceable.

On 4/8/20 4:47 AM, Richard Biener wrote: On Tue, Apr 7, 2020 at 5:01 PM Martin Liška wrote: Hi. The patch allows DCE to remove only replaceable operators new and delete. That's achieved by proper mark up of all these operators in C++ FE. The patch also brings all tests we've collected so far

Re: [PATCH] Allow new/delete operator deletion only for replaceable.

On Wed, Apr 08, 2020 at 09:20:07AM -0400, Jason Merrill via Gcc-patches wrote: > On 4/8/20 4:47 AM, Richard Biener wrote: > > On Tue, Apr 7, 2020 at 5:01 PM Martin Liška wrote: > > > > > > Hi. > > > > > > The patch allows DCE to remove only replaceable operators new and delete. > > > That's achi

Re: [PATCH] Allow new/delete operator deletion only for replaceable.

On 4/8/20 9:32 AM, Jakub Jelinek wrote: On Wed, Apr 08, 2020 at 09:20:07AM -0400, Jason Merrill via Gcc-patches wrote: On 4/8/20 4:47 AM, Richard Biener wrote: On Tue, Apr 7, 2020 at 5:01 PM Martin Liška wrote: Hi. The patch allows DCE to remove only replaceable operators new and delete. Th

Re: [PATCH v2] c++: Fix ICE-on-invalid with lambda template [PR94507]

On Wed, Apr 08, 2020 at 08:55:08AM -0400, Jason Merrill via Gcc-patches wrote: > On 4/6/20 9:05 PM, Marek Polacek wrote: > > While reducing something else I noticed that we ICE on the following > > invalid code. In tsubst_lambda_expr, tsubst_template_decl has already > > reported an error and retu

[PATCH] PR target/48240

Hi all, I'd like to submit this for PR48240. Bootstrapped on aarch64-unknown-linux-gnu. Okay for trunk when finished with regression? Andrea gcc/ChangeLog 2020-??-?? Andrea Corallo PR target/48240 * gcc/config/aarch64/falkor-tag-collision-avoidance.c (valid_src_p):

Re: [PATCH] c++: Fix ICE in tsubst_default_argument [PR92010]

On Tue, 7 Apr 2020, Jason Merrill wrote: > On 4/7/20 1:40 PM, Patrick Palka wrote: > > On Mon, 6 Apr 2020, Jason Merrill wrote: > > > On 4/6/20 11:45 AM, Patrick Palka wrote: > > > > On Wed, 1 Apr 2020, Jason Merrill wrote: > > > > > > > > > On 4/1/20 6:29 PM, Jason Merrill wrote: > > > > > > On 3

Re: PATCH -- Fix degree trignometric functions

FAIL: gfortran.dg/dec_math_5.f90 -O0 (test for excess errors) Excess errors: /opt/gcc/gcc-20200408/gcc/testsuite/gfortran.dg/dec_math_5.f90:132:9: Fatal Error: Cannot open module file 'ieee_arithmetic.mod' for reading at (1): No such file or directory compilation terminated

require tls_runtime for tls execution test

All TLS execution tests require tls_runtime, not just tls; pr78796.c is the only exception that is not otherwise limited to platforms known to support it. I suppose that's an oversight. On a platform whose linker is configured to disregard TLS relocations, this test compiles and assembles succe

Re: require tls_runtime for tls execution test

On Wed, Apr 08, 2020 at 12:04:16PM -0300, Alexandre Oliva wrote: > > All TLS execution tests require tls_runtime, not just tls; pr78796.c > is the only exception that is not otherwise limited to platforms known > to support it. I suppose that's an oversight. On a platform whose > linker is confi

[wwwdocs] [arm] Document CDE intrinsics from ACLE

New in GCC 10. ### Attachment also inlined for ease of reply### diff --git a/htdocs/gcc-10/changes.html b/htdocs/gcc-10/changes.html index 3d8e0ba989e860d307310378a2be99b32a27261f..389561d13c69b650528e8ed8859ebbb5760438c6 100644 --- a/htdocs/gcc-10/changes.html

Re: Disable aggregate walking in ipa code for optimize_debug

Hi, On Sat, Apr 04 2020, Jan Hubicka wrote: > Martin, > with optimize_debug or when FRE is disabled propagating aggregates is > useless since we are not going to use them. I think we should also > avoid working hard on the jump functions. > This patch disables it in ipa_load_from_param_agg, but I

Re: [PATCH] c++: Fix usage of CONSTRUCTOR_PLACEHOLDER_BOUNDARY inside array initializers [90996]

On Wed, 8 Apr 2020, Jason Merrill wrote: > On 4/6/20 6:22 PM, Patrick Palka wrote: > > On Mon, 6 Apr 2020, Jason Merrill wrote: > > > > > On 4/6/20 3:07 PM, Patrick Palka wrote: > > > > This PR reports that since the introduction of the > > > > CONSTRUCTOR_PLACEHOLDER_BOUNDARY flag, we are sometim

RE: [wwwdocs] [arm] Document CDE intrinsics from ACLE

> -Original Message- > From: Matthew Malcomson > Sent: 08 April 2020 16:09 > To: gcc-patches@gcc.gnu.org > Cc: ger...@pfeifer.com; Richard Earnshaw ; > ni...@redhat.com; Kyrylo Tkachov ; nd > ; Ramana Radhakrishnan > > Subject: [wwwdocs] [arm] Document CDE intrinsics from ACLE > > New

Re: [PATCH] Allow new/delete operator deletion only for replaceable.

On 4/8/20 3:34 PM, Jason Merrill wrote: On 4/8/20 9:32 AM, Jakub Jelinek wrote: On Wed, Apr 08, 2020 at 09:20:07AM -0400, Jason Merrill via Gcc-patches wrote: On 4/8/20 4:47 AM, Richard Biener wrote: On Tue, Apr 7, 2020 at 5:01 PM Martin Liška wrote: Hi. The patch allows DCE to remove only

[committed] libstdc++: Add comparison operators to result types

Some more C++20 changes from P1614R2, "The Mothership has Landed". * include/std/charconv (to_chars_result, from_chars_result): Add defaulted equality comparisons for C++20. * testsuite/20_util/from_chars/compare.cc: New test. * testsuite/20_util/to_chars/compare.cc

Re: [PATCH][Arm][1/3] Support for Arm Custom Datapath Extension (CDE): enable the feature

Hi Kyrylo, > > From: Kyrylo Tkachov > Sent: Wednesday, April 8, 2020 1:34 PM > To: Dennis Zhang; gcc-patches@gcc.gnu.org > Cc: nd; Richard Earnshaw; Ramana Radhakrishnan > Subject: RE: [PATCH][Arm][1/3] Support for Arm Custom Datapath Extension > (CDE): e

Re: [PATCH][Arm][2/4] Custom Datapath Extension intrinsics: instructions using FPU/MVE S/D registers

Hi kyrylo, > > From: Kyrylo Tkachov > Sent: Tuesday, April 7, 2020 3:07 PM > To: Dennis Zhang; gcc-patches@gcc.gnu.org > Cc: nd; Richard Earnshaw; Ramana Radhakrishnan > Subject: RE: [PATCH][Arm][2/4] Custom Datapath Extension intrinsics: > instructions

Re: unreliable/confusing dg-add-options arm_fp16_alternatives

On Mar 27, 2020, Alexandre Oliva wrote: > So, here are some potential fixes: > - install the patchlet for fp16-aapcs-3.c above, and be done with it > - add an arm_fp16_hw requirement to this test > - add to check_effective_target_arm_fp16_alternative_ok_nocache above a > check for arm_fp16,

Re: [PATCH] Allow new/delete operator deletion only for replaceable.

> From 2f8ba3418f10b41bb839aadb292447bd757238d0 Mon Sep 17 00:00:00 2001 > From: Martin Liska > Date: Tue, 7 Apr 2020 16:23:27 +0200 > Subject: [PATCH] Allow new/delete operator deletion only for replaceable. > > gcc/ChangeLog: > > 2020-04-07 Martin Liska > > PR c++/94314 > * g

[committed] libstdc++: Add comparison operators to types from Numerics clause

Some more C++20 changes from P1614R2, "The Mothership has Landed". * include/bits/slice_array.h (operator==(const slice&, const slice&)): Define for C++20. * include/std/complex (operator==(const T&, const complex&)) (operator!=(const complex&, const complex&)

Re: [PATCH][PPC64] [PR88877]

On Mon, 2020-04-06 at 14:58 +0530, kamlesh kumar via Gcc-patches wrote: > Hi Richard, > Here is a discussion we did some time ago > https://gcc.gnu.org/pipermail/gcc/2019-January/227834.html > please see https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88877 for more > info regarding the bug. > > We i

[patch,committed] Move gfortran.dg/dec_math_5.f90 to ./ieee/ (was: Re: PATCH -- Fix degree trignometric functions)

ess errors: /opt/gcc/gcc-20200408/gcc/testsuite/gfortran.dg/dec_math_5.f90:132:9: Fatal Error: Cannot open module file 'ieee_arithmetic.mod' for reading at (1): No such file or directory compilation terminated. Andreas. - Mentor Graphics (Deutschland) GmbH, Arnulfstraß

Re: [PATCH v2 1/2] RISC-V: Add shorten_memrefs pass

On Wed, Feb 19, 2020 at 3:40 AM Craig Blackmore wrote: > On 10/12/2019 18:28, Craig Blackmore wrote: > Thank you for your review. I have posted an updated patch below which I think > addresses your comments. > > Ping > > https://gcc.gnu.org/ml/gcc-patches/2019-12/msg00712.html This looks OK. The

Re: unreliable/confusing dg-add-options arm_fp16_alternatives

On Wed, 2020-04-08 at 12:27 -0300, Alexandre Oliva wrote: > On Mar 27, 2020, Alexandre Oliva wrote: > > > So, here are some potential fixes: > > - install the patchlet for fp16-aapcs-3.c above, and be done with it > > - add an arm_fp16_hw requirement to this test > > - add to check_effective_targ

Re: [PATCH] Allow new/delete operator deletion only for replaceable.

On Wed, Apr 08, 2020 at 05:46:52PM +0200, Jan Hubicka wrote: > I am not quite sure about the offloading done by openMP - I think that > one produces new decls. Yes, it does. It copies some FUNCTION_DECL flags over, but only selected ones (and all attributes but removes a few afterwards).

Re: Ping: [PATCH] testsuite: Tweak check-function-bodies interface

On Fri, 2020-04-03 at 17:37 +0100, Richard Sandiford wrote: > Ping for the doc/sourcebuild.texi and lib/scanasm.exp parts. > > Richard Sandiford writes: > > In g:2171a9207f51bc486ed9c502cb4da706f594615e I'd tried to fix > > various ILP32 testsuite failures by restricting some tests to LP64. > > U

Re: [PATCH] i386: Don't use AVX512F integral masks for V*TImode [PR94438]

On Fri, 2020-04-03 at 00:26 +0200, Jakub Jelinek wrote: > Hi! > > The ix86_get_mask_mode hook uses int mask for 512-bit vectors or 128/256-bit > vectors with AVX512VL (that is correct), and only for V*[SD][IF]mode if not > AVX512BW (also correct), but with AVX512BW it would stop checking the > ele

Re: [PATCH] x86: Insert ENDBR if function will be called indirectly

On Tue, 2020-03-31 at 08:11 -0700, H.J. Lu via Gcc-patches wrote: > Since constant_call_address_operand has > > ;; Test for a pc-relative call operand > (define_predicate "constant_call_address_operand" > (match_code "symbol_ref") > { > if (ix86_cmodel == CM_LARGE || ix86_cmodel == CM_LARGE_PI

Re: [PATCH] x86: Insert ENDBR if function will be called indirectly

On Wed, Apr 8, 2020 at 9:16 AM Jeff Law wrote: > > On Tue, 2020-03-31 at 08:11 -0700, H.J. Lu via Gcc-patches wrote: > > Since constant_call_address_operand has > > > > ;; Test for a pc-relative call operand > > (define_predicate "constant_call_address_operand" > > (match_code "symbol_ref") > >

Re: [PATCH] PR target/48240

Andrea Corallo writes: > Hi all, > > I'd like to submit this for PR48240. > > Bootstrapped on aarch64-unknown-linux-gnu. > Okay for trunk when finished with regression? OK, but the PR number looks like a typo. Also: > Andrea > > gcc/ChangeLog > > 2020-??-?? Andrea Corallo > > PR target

RE: [PATCH] PR target/48240

Hi Andrea > -Original Message- > From: Gcc-patches On Behalf Of Andrea > Corallo > Sent: 08 April 2020 15:19 > To: gcc-patches@gcc.gnu.org > Cc: nd > Subject: [PATCH] PR target/48240 > > Hi all, > > I'd like to submit this for PR48240. > > Bootstrapped on aarch64-unknown-linux-gnu. >

Re: [PATCH] x86: Insert ENDBR if function will be called indirectly

On Wed, 2020-04-08 at 09:23 -0700, H.J. Lu wrote: > On Wed, Apr 8, 2020 at 9:16 AM Jeff Law wrote: > > On Tue, 2020-03-31 at 08:11 -0700, H.J. Lu via Gcc-patches wrote: > > > Since constant_call_address_operand has > > > > > > ;; Test for a pc-relative call operand > > > (define_predicate "consta

Re: [RFC, Instruction Scheduler, Stage1] New hook/code to perform fusion of dependent instructions

On 4/8/20 2:46 AM, Richard Biener wrote: >> I have coded up a proof of concept that implements our needs via a new >> target hook. The hook is passed a pair of dependent insns and returns if >> they are a fusion candidate. It is called while removing the forward >> dependencies of the just schedule

Re: [patch,committed] Move gfortran.dg/dec_math_5.f90 to ./ieee/ (was: Re: PATCH -- Fix degree trignometric functions)

uld support IEEE or, if not, > UNSUPPORTED). > > See r10-7631-gfaa0817311f43e0d4d223d53c816b0c74ec35c4e > or attachment. > > Cheers, > > Tobias > > On 4/8/20 5:04 PM, Andreas Schwab wrote: > > > FAIL: gfortran.dg/dec_math_5.f90 -O0 (test for excess errors) > > Excess error

Re: [PATCH v3] Fix alignment for local variable [PR90811]

ping On Tue, Mar 31, 2020 at 4:33 PM Kito Cheng wrote: > > - The alignment for local variable was adjust during > estimate_stack_frame_size, >however it seems wrong spot to adjust that, expand phase will adjust that >but it little too late to some gimple optimization, which rely on cert

Re: [C/C++, OpenACC] Reject vars of different scope in acc declare (PR94120)

On Wed, Apr 8, 2020 at 12:55 AM Tobias Burnus wrote: > > I have now committed this patch > as r10-7614-g13e41d8b9d3d7598c72c38acc86a3d97046c8373, On Linux/x86, I got FAIL: g++.dg/declare-pr94120.C -std=c++14 (test for excess errors) FAIL: g++.dg/declare-pr94120.C -std=c++17 (test for excess er

Re: [PATCH] reject scalar array initialization with nullptr [PR94510]

On 4/7/20 3:36 PM, Marek Polacek wrote: On Tue, Apr 07, 2020 at 02:46:52PM -0600, Martin Sebor wrote: On 4/7/20 1:50 PM, Marek Polacek wrote: On Tue, Apr 07, 2020 at 12:50:48PM -0600, Martin Sebor via Gcc-patches wrote: Among the numerous regressions introduced by the change committed to GCC 9

Re: [PATCH v2] c++: Fix ICE-on-invalid with lambda template [PR94507]

On 4/8/20 9:44 AM, Marek Polacek wrote: On Wed, Apr 08, 2020 at 08:55:08AM -0400, Jason Merrill via Gcc-patches wrote: On 4/6/20 9:05 PM, Marek Polacek wrote: While reducing something else I noticed that we ICE on the following invalid code. In tsubst_lambda_expr, tsubst_template_decl has alre

[PATCH] RTEMS: Improve GCC specification

Add a start/end file specification if the -qrtems option is present. Allow targets to customize it. Support the standard -nodefaultlibs option. gcc/ * config/rtems.h (RTEMS_STARTFILE_SPEC): Define if undefined. (RTEMS_ENDFILE_SPEC): Likewise. (STARTFILE_SPEC): Update comm

[committed] openacc: Fix up declare-pr94120.C testcase [PR94533]

Hi! On Wed, Mar 11, 2020 at 02:28:44PM +0100, Tobias Burnus wrote: > gcc/testsuite/ > PR middle-end/94120 > * c-c++-common/goacc/declare-pr94120.c: New. > * g++.dg/declare-pr94120.C: New. This test has been put in a wrong directory, where OpenACC tests aren't tested with -

[PATCH] aarch64: Add TX3 machine model

Here is the patch introducing thunderxt311 maching model for the scheduler. A name for the new chip was added to the list of the names to be recognized as a valid parameter for mcpu and mtune flags. The TX2 cost model was reused for TX3. Bootstrapped on AArch64. 2020-04-08 Anton Youdkevitch

[PATCH] Some top-level configury syncing with gdb

Merge top-level configury changes from gdb We recently rearranged the gdb source tree to move a common library and gdbserver to the top-level. This made the build more uniform and also a bit faster (due to sharing of built objects). This patch re-syncs these changes the top-level configury back

[PATCH] vect: Fix up lowering of TRUNC_MOD_EXPR by negative constant [PR94524]

Hi! The first testcase below is miscompiled, because for the division part of the lowering we canonicalize negative divisors to their absolute value (similarly how expmed.c canonicalizes it), but when multiplying the division result back by the VECTOR_CST, we use the original constant, which can c

Re: [C/C++, OpenACC] Reject vars of different scope in acc declare (PR94120)

There was a glitch in the test case (location + dg-error), already fixed by Jakub (thanks!) in commit r10-7637-g08d1e7a5aabcf7eeac48bfd99deb80451b8f9974 Sorry, Tobias On 4/8/20 7:13 PM, H.J. Lu wrote: On Wed, Apr 8, 2020 at 12:55 AM Tobias Burnus wrote: I have now committed this patch as r1

[PATCH] cselib, reload: Fix cselib ICE on m68k/microblaze [PR94526]

Hi! The following testcase ICEs on m68k (and another one Jeff mailed me privately on microblaze). The problem is that reload creates two DEBUG_INSNs with the same value of (plus:P (reg:P sp) (const_int 0)), we compute correctly the same hash value for them, but then don't find them in the cselib h

Re: [testsuite][arm] Fix cmse-15.c expected output

On Wed, 8 Apr 2020 at 11:48, Richard Sandiford wrote: > > Christophe Lyon via Gcc-patches writes: > > Hi, > > > > While checking Martin's fix for PR ipa/94445, he made me realize that > > the cmse-15.c testcase still fails at -Os because ICF means that we > > generate > > nonsecure2: > >

Re: [PATCH] sra: Fix sra_modify_expr handling of partial writes (PR 94482)

Hi, On Wed, Apr 08 2020, Richard Biener wrote: > On Tue, 7 Apr 2020, Richard Biener wrote: > >> On April 7, 2020 6:25:26 PM GMT+02:00, Martin Jambor wrote: >> >Hi, >> > >> >On Tue, Apr 07 2020, Richard Biener wrote: >> >> On Tue, 7 Apr 2020, Martin Jambor wrote: >> >> >> >>> Hi, >> >>> >> >>> wh

Re: [PATCH] reject scalar array initialization with nullptr [PR94510]

On 4/7/20 2:50 PM, Martin Sebor wrote: Among the numerous regressions introduced by the change committed to GCC 9 to allow string literals as template arguments is a failure to recognize the C++ nullptr and GCC's __null constants as pointers. For one, I didn't realize that nullptr, being a null p

Re: [PATCH] vect: Fix up lowering of TRUNC_MOD_EXPR by negative constant [PR94524]

On April 8, 2020 8:14:56 PM GMT+02:00, Jakub Jelinek wrote: >Hi! > >The first testcase below is miscompiled, because for the division part >of the lowering we canonicalize negative divisors to their absolute >value >(similarly how expmed.c canonicalizes it), but when multiplying the >division >res

Re: [PATCH] sra: Fix sra_modify_expr handling of partial writes (PR 94482)

On April 8, 2020 8:34:08 PM GMT+02:00, Martin Jambor wrote: >Hi, > >On Wed, Apr 08 2020, Richard Biener wrote: >> On Tue, 7 Apr 2020, Richard Biener wrote: >> >>> On April 7, 2020 6:25:26 PM GMT+02:00, Martin Jambor > wrote: >>> >Hi, >>> > >>> >On Tue, Apr 07 2020, Richard Biener wrote: >>> >> On

Re: [PATCH] cselib, reload: Fix cselib ICE on m68k/microblaze [PR94526]

On Wed, 2020-04-08 at 20:23 +0200, Jakub Jelinek wrote: > Hi! > > The following testcase ICEs on m68k (and another one Jeff mailed me > privately on microblaze). > The problem is that reload creates two DEBUG_INSNs with the same > value of (plus:P (reg:P sp) (const_int 0)), we compute correctly th

Re: [PING PATCH coroutines] Do not strip cleanup_point when promote temporaries out of current stmt

On 4/7/20 11:44 AM, Iain Sandoe wrote: Bin.Cheng wrote: On Mon, Mar 16, 2020 at 5:34 PM Bin.Cheng wrote: On Wed, Mar 11, 2020 at 5:07 PM Iain Sandoe wrote: Bin.Cheng wrote: On Thu, Mar 5, 2020 at 10:18 PM Iain Sandoe wrote: If it helps, I could push a branch to users/iains/ on the F

PING [PATCH][gcc][PR94230]provide an option to change the size limitation for -Wmisleading-indent

Hi, Please take a look at the attached patch and let me know your comments. Thanks. Qing gcc/ChangeLog: 2020-04-03 qing zhao * common.opt: Add -flocation-ranges. * doc/invoke.texi: Document it. * toplev.c (process_options): set line_table->default_range_bits

Re: [PATCH] RTEMS: Improve GCC specification

On Wed, 2020-04-08 at 19:47 +0200, Sebastian Huber wrote: > Add a start/end file specification if the -qrtems option is present. > Allow targets to customize it. > > Support the standard -nodefaultlibs option. > > gcc/ > > * config/rtems.h (RTEMS_STARTFILE_SPEC): Define if undefined. >

Re: [PATCH] Some top-level configury syncing with gdb

On Wed, 2020-04-08 at 12:09 -0600, Tom Tromey wrote: > Merge top-level configury changes from gdb > > We recently rearranged the gdb source tree to move a common library > and gdbserver to the top-level. This made the build more uniform and > also a bit faster (due to sharing of built objects). >

Re: [PATCH] reject scalar array initialization with nullptr [PR94510]

On 4/8/20 12:43 PM, Jason Merrill wrote: On 4/7/20 2:50 PM, Martin Sebor wrote: Among the numerous regressions introduced by the change committed to GCC 9 to allow string literals as template arguments is a failure to recognize the C++ nullptr and GCC's __null constants as pointers. For one, I d

Re: [PATCH v2 00/11] aarch64: Implement TImode comparisons

On Tue, Apr 07, 2020 at 05:50:38PM -0700, Richard Henderson wrote: > On 4/7/20 4:58 PM, Segher Boessenkool wrote: > >> I wonder if it would be helpful to have > >> > >> (uoverflow_plus x y carry) > >> (soverflow_plus x y carry) > >> > >> etc. > > > > Those have three operands, which is nasty t

[committed] libphobos: Remove --enable-unix configure option.

Hi, This patch removes --enable-unix, an old option that predates the D2 library, and now is not useful on its own as all posix modules require the compiler to predefine version(Posix) anyway. So the option has been removed, and the logic moved into DRUNTIME_OS_SOURCES, where the conditional DRUN

Re: [PATCH] aarch64: Add TX3 machine model

On Wed, Apr 8, 2020 at 11:06 AM Anton Youdkevitch wrote: > > Here is the patch introducing thunderxt311 maching model > for the scheduler. A name for the new chip was added to the > list of the names to be recognized as a valid parameter for mcpu > and mtune flags. The TX2 cost model was reused f

[PATCH] c++: Fix wrong paren-init of aggregates interference [PR93790]

This PR points out that we are rejecting valid code in C++20. The problem is that we were surreptitiously transforming T& t(e) into T& t{e} which is wrong, because the type of e had a conversion function to T, while aggregate initialization of t from e doesn't work. Therefore, I was viola

[committed] libphobos: Remove --enable-thread-lib configure option.

Hi, This is another old option that doesn't make sense as a configurable. So the option has been removed, and the check for AC_SEARCH_LIBS moved into the main configure.ac file. Bootstrapped and regression tested on x86_64-linux-gnu, and committed to mainline. Regards Iain. --- libphobos/Change

[committed] libphobos: Add --enable-libphobos-checking configure option (PR94305)

Hi, As GDCFLAGS is overriden by the top-level make file with '-O2 -g', libphobos ends up always being built with all contracts, invariants, and asserts compiled in. This adds a new configurable that defaults to omit compiling any run-time checks into the library using '-frelease'. Other choices

Re: [PATCH] [ARC] Allow more ABIs in GLIBC_DYNAMIC_LINKER

Hi Claudiu, For glibc needs can this be backported to gcc-9 please ! Thx, -Vineet On 3/31/20 3:06 AM, Claudiu Zissulescu Ianculescu wrote: > Pushed. > > Thank you, > Claudiu > > On Sun, Mar 29, 2020 at 2:05 AM Vineet Gupta via Gcc-patches > wrote: >> Enable big-endian suffixed dynamic linker pe

[PATCH] c++: Stray RESULT_DECLs in result of constexpr function call [PR94034]

When evaluating the initializer of 'a' in the following example struct A { A *p = this; }; constexpr A foo() { return {}; } constexpr A a = foo(); the PLACEHOLDER_EXPR for 'this' in the aggregate initializer returned by foo gets resolved to the RESULT_DECL of foo. But due to guaranteed RVO

Re: [PATCH PR93674]Avoid introducing IV of enumeral type in case of -fstrict-enums

-- Sender:Richard Biener Sent At:2020 Mar. 20 (Fri.) 18:12 Recipient:bin.cheng Cc:Andrew Pinski ; GCC Patches Subject:Re: [PATCH PR93674]Avoid introducing IV of enumeral type in case of -fstrict-enums On Fri, Mar 20, 2020 at 10:2

[committed, obvious] Require pthread effective target for test case using -pthread option.

I noticed g++.dg/tree-ssa/pr93940.C was giving an error on nios2-elf due to lack of support for the -pthread option on this bare-metal target. I cut and pasted this fix from other existing testcases. Committed as obvious. -Sandra commit fe1837143f1bf1d6b072a3973b00576ee17c30a9 Author: Sandra

Re: [PATCH] c++: Fix wrong paren-init of aggregates interference [PR93790]

On 4/8/20 5:52 PM, Marek Polacek wrote: This PR points out that we are rejecting valid code in C++20. The problem is that we were surreptitiously transforming T& t(e) into T& t{e} which is wrong, because the type of e had a conversion function to T, while aggregate initialization of t

Re: [PATCH] Allow new/delete operator deletion only for replaceable.

Hi. We've got one another sneaky test-case (thank you Marc ;) ): $ cat pr94314-array.C #include #include int count = 0; __attribute__((malloc, noinline)) void* operator new[](unsigned long sz) { ++count; return ::operator new(sz); } void operator delete[](void* ptr) noexcept { --count

Re: [PATCH PR93674]Avoid introducing IV of enumeral type in case of -fstrict-enums

On Thu, Apr 9, 2020 at 4:23 AM bin.cheng wrote: > > -- > Sender:Richard Biener > Sent At:2020 Mar. 20 (Fri.) 18:12 > Recipient:bin.cheng > Cc:Andrew Pinski ; GCC Patches > Subject:Re: [PATCH PR93674]Avoid introducing IV of enumeral

  1   2   >