Re: [PATCH] strlen: Return TODO_update_address_taken when memcmp has been optimized [PR96271]

On January 16, 2021 2:00:38 AM GMT+01:00, Jakub Jelinek wrote: >On Fri, Jan 15, 2021 at 10:02:41PM +0100, Jakub Jelinek via Gcc-patches >wrote: >> Like this? > >Bootstrapped/regtested successfully on x86_64-linux and i686-linux. OK. Thanks, Richard. >> 2021-01-15 Jakub Jelinek >> >>

Re: [PATCH] match.pd: Optimize ((cst << x) & 1) [PR96669]

On January 16, 2021 1:59:50 AM GMT+01:00, Jakub Jelinek wrote: >On Fri, Jan 15, 2021 at 09:21:07PM +0100, Richard Biener wrote: >> >BTW, I've tried to also fix what the PR wanted primarily by adding >> >/* Simplify (CST << x) & 1 to 0 if CST is even or to x == 0 if it is >> >odd. / >>

[PATCH] libstdc++-v3: Add -fcf-protection=none to -march=i486

-fcf-protection is automatically enabled in libstdc++ on Linux/x86. Starting from commit 77d372abec0fbf2cfe922e3140ee3410248f979e Author: H.J. Lu Date: Thu Jan 14 05:56:46 2021 -0800 x86: Error on -fcf-protection with incompatible target GCC issues an error on -fcf-protection with

Re: [PATCH] strlen: Return TODO_update_address_taken when memcmp has been optimized [PR96271]

On Fri, Jan 15, 2021 at 10:02:41PM +0100, Jakub Jelinek via Gcc-patches wrote: > Like this? Bootstrapped/regtested successfully on x86_64-linux and i686-linux. > 2021-01-15 Jakub Jelinek > > PR tree-optimization/96271 > * passes.def: Pass false argument to first two pass_cd_dce >

[PATCH] match.pd: Optimize ((cst << x) & 1) [PR96669]

On Fri, Jan 15, 2021 at 09:21:07PM +0100, Richard Biener wrote: > >BTW, I've tried to also fix what the PR wanted primarily by adding > >/* Simplify (CST << x) & 1 to 0 if CST is even or to x == 0 if it is > >odd. / > >(simplify > > (bit_and (lshift INTEGER_CST@1 @0) integer_onep) > > (if

[r11-6725 Regression] FAIL: g++.dg/modules/dep-2.C -std=c++2a scan-file \\n\\.PHONY: m\\\\:part\\.c\\+\\+m on Linux/x86_64

On Linux/x86_64, 1ba71fabb78b18884e9f479f45a257bab50e8959 is the first bad commit commit 1ba71fabb78b18884e9f479f45a257bab50e8959 Author: Nathan Sidwell Date: Fri Jan 15 06:47:13 2021 -0800 preprocessor: Make quoting : [PR 95253] caused FAIL: g++.dg/modules/dep-1_a.C -std=c++17

Re: [PATCH, rs6000] Update pr88233.c test (pr91799)

Hi! On Thu, Jan 14, 2021 at 03:25:46PM -0600, will schmidt wrote: > This is a follow-up fix to clean up pr91799. Per review of test results, > it appears that the combination of target and dg-require stanzas is > not sufficient to properly limit the test to 64-bit only on darwin. > > This adds

Re: [PATCH 1/2] nios2: Add -mcustom-fpu-cfg=fph2

On 1/15/21 2:27 PM, Sebastian Huber wrote: Hello Sandra, On 15/01/2021 20:12, Sandra Loosemore wrote: The following instructions supported by this component are not enabled by this option: * -mcustom-fmins * -mcustom-fmaxs * -mcustom-round The reason is that these instructions are only in

[patch] Reset force_source_line in final.c

Unlike the other global variables, it is not reset at the beginning of a function so can leak into the next one. Tested on x86-64/Linux, applied on the mainline as obvious. 2021-01-15 Eric Botcazou * final.c (final_start_function_1): Reset force_source_line. -- Eric Botcazoudiff

Re: [PATCH] PowerPC: Add float128/Decimal conversions.

On Thu, Jan 14, 2021 at 12:09:36PM -0500, Michael Meissner wrote: > [PATCH] PowerPC: Add float128/Decimal conversions. Same question here. Segher

Re: [PATCH] PowerPC: Map IEEE 128-bit long double built-ins.

Hi! On Thu, Jan 14, 2021 at 11:59:19AM -0500, Michael Meissner wrote: > >From 78435dee177447080434cdc08fc76b1029c7f576 Mon Sep 17 00:00:00 2001 > From: Michael Meissner > Date: Wed, 13 Jan 2021 21:47:03 -0500 > Subject: [PATCH] PowerPC: Map IEEE 128-bit long double built-ins. > > This patch

Re: [PATCH][testsuite] (committed) Fix sed script errors in complex tests

On Jan 15, 2021, at 1:13 AM, Tamar Christina via Gcc-patches wrote: > I ran sed script late over the tests which accidentally > introduced a syntax error in the tests. > > This fixes it. > > Committed under the obvious rule. > > Ok for master? :-) Which is it? Anyway, Ok.

Re: [PATCH 1/2] nios2: Add -mcustom-fpu-cfg=fph2

Hello Sandra, On 15/01/2021 20:12, Sandra Loosemore wrote: The following instructions supported by this component are not enabled by this option: * -mcustom-fmins * -mcustom-fmaxs * -mcustom-round The reason is that these instructions are only in effect in combination with other options. If

Re: [PATCH] strlen: Return TODO_update_address_taken when memcmp has been optimized [PR96271]

On Fri, Jan 15, 2021 at 09:31:10PM +0100, Jakub Jelinek via Gcc-patches wrote: > On Fri, Jan 15, 2021 at 09:16:31PM +0100, Richard Biener wrote: > > >Apparently not. The passes after strlen1 are: > > >pr96271.c.191t.thread4 > > >pr96271.c.192t.vrp2 > > >pr96271.c.193t.copyprop5 > >

Re: [PATCH] strlen: Return TODO_update_address_taken when memcmp has been optimized [PR96271]

On Fri, Jan 15, 2021 at 09:16:31PM +0100, Richard Biener wrote: > >Apparently not. The passes after strlen1 are: > >pr96271.c.191t.thread4 > >pr96271.c.192t.vrp2 > >pr96271.c.193t.copyprop5 > >pr96271.c.194t.wrestrict > >pr96271.c.195t.dse4 > >pr96271.c.196t.cddce3 > >pr96271.c.197t.forwprop4 >

Re: [PATCH] match.pd: Generalize the PR64309 simplifications [PR96669]

On January 15, 2021 9:03:58 PM GMT+01:00, Jakub Jelinek wrote: >On Fri, Jan 15, 2021 at 08:50:20PM +0100, Richard Biener wrote: >> On January 15, 2021 7:38:35 PM GMT+01:00, Jakub Jelinek > wrote: >> >Hi! >> > >> >The following patch generalizes the PR64309 simplifications, so that >> >instead >>

Re: [PATCH] strlen: Return TODO_update_address_taken when memcmp has been optimized [PR96271]

On January 15, 2021 8:57:39 PM GMT+01:00, Jakub Jelinek wrote: >On Fri, Jan 15, 2021 at 08:48:32PM +0100, Richard Biener wrote: >> On January 15, 2021 7:32:35 PM GMT+01:00, Jakub Jelinek > wrote: >> >Hi! >> > >> >On the following testcase, handle_builtin_memcmp in the strlen pass >> >folds >>

Re: [PATCH] match.pd: Generalize the PR64309 simplifications [PR96669]

On Fri, Jan 15, 2021 at 08:50:20PM +0100, Richard Biener wrote: > On January 15, 2021 7:38:35 PM GMT+01:00, Jakub Jelinek > wrote: > >Hi! > > > >The following patch generalizes the PR64309 simplifications, so that > >instead > >of working only with constants 1 and 1 it works with any two power

Re: [PATCH] strlen: Return TODO_update_address_taken when memcmp has been optimized [PR96271]

On Fri, Jan 15, 2021 at 08:48:32PM +0100, Richard Biener wrote: > On January 15, 2021 7:32:35 PM GMT+01:00, Jakub Jelinek > wrote: > >Hi! > > > >On the following testcase, handle_builtin_memcmp in the strlen pass > >folds > >the memcmp into comparison of two MEM_REFs. But nothing triggers >

Re: [PATCH] match.pd: Generalize the PR64309 simplifications [PR96669]

On January 15, 2021 7:38:35 PM GMT+01:00, Jakub Jelinek wrote: >Hi! > >The following patch generalizes the PR64309 simplifications, so that >instead >of working only with constants 1 and 1 it works with any two power of >two >constants, and works also for right shift (in that case it rules out

Re: [PATCH] match.pd: Optimize (x < 0) ^ (y < 0) to (x ^ y) < 0 etc. [PR96681]

On January 15, 2021 7:25:33 PM GMT+01:00, Jakub Jelinek wrote: >Hi! > >This patch simplifies comparisons that test the sign bit xored >together. >If the comparisons are both < 0 or both >= 0, then we should xor the >operands >together and compare the result to < 0, if the comparisons are

Re: [PATCH] strlen: Return TODO_update_address_taken when memcmp has been optimized [PR96271]

On January 15, 2021 7:32:35 PM GMT+01:00, Jakub Jelinek wrote: >Hi! > >On the following testcase, handle_builtin_memcmp in the strlen pass >folds >the memcmp into comparison of two MEM_REFs. But nothing triggers >updating >of addressable vars afterwards, so even when the parameters are no

Re: Add dg-require-wchars to libstdc++ testsuite

On Fri, 15 Jan 2021, 16:19 Alexandre Oliva, wrote: > On Jan 15, 2021, Jonathan Wakely wrote: > > > On Thu, 14 Jan 2021, 22:22 Alexandre Oliva, wrote: > >> ... it is definitely the case that the target currently defines wchar_t, > >> and it even offers wchar.h and a lot of (maybe all?) wcs*

c++: Fix qualified array-type construction [PR 98538]

This was an assert that was too picky. The reason I had to alter array construction was that on stream in, we cannot dynamically determine a type's dependentness. Thus on stream out of the 'problematic' types, we save the dependentness for reconstruction. Fortunately the paths into

Re: preprocessor: Make quoting : [PR 95253]

I missed some testsuite fall out with my patch to fix mkdeps file mangling. PR preprocessor/95253 gcc/testsuite/ * g++.dg/modules/dep-1_a.C: Adjust expected output. * g++.dg/modules/dep-1_b.C: Likewise. * g++.dg/modules/dep-2.C: Likewise. --

Re: [patch] gcc.dg/analyzer tests: relax dependency on alloca.h

On Jan 15, 2021, David Malcolm wrote: > I think an issue here was that I assumed check_effective_target_alloca > checks that "alloca" is supported, whereas I now see that I was wrong; > it actually checks for "__builtin_alloca". Yeah, it's a little misleading. And it only tests

Re: [stage1][PATCH] Change semantics of -frecord-gcc-switches and add -frecord-gcc-switches-format.

Hello, Martin, Our testing detected unexpected -dumpbase-ext options making to the producer string. I tracked it down to something weird that happened in this patch: On Dec 4, 2020, Martin Liška wrote: > +++ b/gcc/dwarf2out.c > - case OPT_dumpbase: > - case OPT_dumpbase_ext: > -

Re: [PATCH 2/2] RTEMS: Add -mcustom-fpu-cfg=fph2 multilib

On 1/14/21 12:16 PM, Sebastian Huber wrote: This multilib supports Nios II configurations with the "Nios II Floating Point Hardware 2 Component". gcc/ * config/nios2/t-rtems: Reset all MULTILIB_* variables. Shorten multilib directory names. Use MULTILIB_REQUIRED instead of

Re: [PATCH 1/2] nios2: Add -mcustom-fpu-cfg=fph2

On 1/14/21 12:16 PM, Sebastian Huber wrote: The new -mcustom-fpu-cfg=fph2 option variant is useful to build a multilib for the "Nios II Floating Point Hardware 2 Component": https://www.intel.com/content/dam/www/programmable/us/en/pdfs/literature/ug/ug_nios2_custom_instruction.pdf Directly

Re: [PATCH] Add a new pattern in 4-insn combine

On Mon, Nov 30, 2020 at 11:08:22AM +0800, HAO CHEN GUI wrote: >   This patch adds a new pattern(combine 4 insns to 3 insns) in 4-insn > combine. In the patch, newpat is split twice. The newpat, newi2pat and As I said before, that has a lot of problems, and is only suitable for stage 1 (after

[pushed] c++: Improve copy elision for base subobjects [PR98642]

Three patches: 1) Rewrite a complete constructor call to call a base constructor if we're eliding a copy into a base subobject. 2) Elide the copy from a prvalue built for list-initialization into a base subobject. 3) Elide other copies from prvalues representing a constructor call into base

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

Hi! On Wed, Jan 13, 2021 at 02:15:04PM -0800, Carl Love wrote: > The patch was compiled and tested on: > >powerpc64le-unknown-linux-gnu (Power 8 BE) (I assume you mean powerpc64-linux instead?) > > Presumably it is safe (no side affects) when adding V4SI and V2DI here, > > with respect to

[pushed] c++: Fix list-init of array of no-copy type [PR63707]

build_vec_init_elt models initialization from some arbitrary object of the type, i.e. copy, but in the case of list-initialization we don't do a copy from the elements, we initialize them directly. Tested x86_64-pc-linux-gnu, applying to trunk. And 9/10, soon. gcc/cp/ChangeLog: PR

[PATCH] match.pd: Generalize the PR64309 simplifications [PR96669]

Hi! The following patch generalizes the PR64309 simplifications, so that instead of working only with constants 1 and 1 it works with any two power of two constants, and works also for right shift (in that case it rules out the first one being negative, as it is arithmetic shift then).

[PATCH] strlen: Return TODO_update_address_taken when memcmp has been optimized [PR96271]

Hi! On the following testcase, handle_builtin_memcmp in the strlen pass folds the memcmp into comparison of two MEM_REFs. But nothing triggers updating of addressable vars afterwards, so even when the parameters are no longer address taken, we force the parameters to stack and back anyway. The

[committed] bootstrap: fix failing diagnostic selftest on Windows [PR98696]

In one of the selftests in g:f10960558540636800cf5d3d6355969621fbc17e I didn't consider that paths can contain backslashes, which happens for the tempfiles on Windows hosts. Successfully bootstrapped & regrtested on x86_64-pc-linux-gnu. Confirmed by the reporter as fixing the issue on Windows.

[committed] testsuite: Add testcase coverage for already fixed [PR96671]

Hi! The fix for this PR didn't come with any test coverage, I've added tests that make sure we optimize it no matter what order of the x ^ y ^ z operands is used. Bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk. 2021-01-15 Jakub Jelinek PR

[PATCH] match.pd: Optimize (x < 0) ^ (y < 0) to (x ^ y) < 0 etc. [PR96681]

Hi! This patch simplifies comparisons that test the sign bit xored together. If the comparisons are both < 0 or both >= 0, then we should xor the operands together and compare the result to < 0, if the comparisons are different, we should compare to >= 0. Bootstrapped/regtested on x86_64-linux

[PATCH] aarch64: Implement vmlsl[_high]* intrinsics using builtins

Hi all, This patch reimplements some more intrinsics using RTL builtins in the straightforward way. Thankfully most of the RTL infrastructure is already in place for it. Bootstrapped and tested on aarch64-none-linux-gnu. Pushing to trunk. Thanks, Kyrill gcc/ *

Re: [PATCH 1/3] PowerPC: Add long double target-supports.

On Thu, 14 Jan 2021, Michael Meissner via Gcc-patches wrote: > +return [check_runtime_nocache ppc_long_double_ovveride_ibm128 { > +return [check_runtime_nocache ppc_long_double_ovveride_ieee128 { > +return [check_runtime_nocache ppc_long_double_ovveride_64bit { All these places

Re: [patch] gcc.dg/analyzer tests: relax dependency on alloca.h

On Fri, 2021-01-15 at 14:45 -0300, Alexandre Oliva wrote: > On Jan 15, 2021, Olivier Hainque wrote: > > > On 14 Jan 2021, at 22:13, Alexandre Oliva > > wrote: > > > Would you mind if I submitted an alternate patch to do so? > > Not at all, thanks for your feedback and for proposing > > an

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

On Fri, Jan 15, 2021 at 04:58:25PM +, Kwok Cheung Yeung wrote: > On 15/01/2021 3:07 pm, Kwok Cheung Yeung wrote: > > I have tested bootstrapping on x86_64 (no offloading) with no issues, > > and running the libgomp testsuite with Nvidia offloading shows no > > regressions. I have also tested

Re: The performance data for two different implementation of new security feature -ftrivial-auto-var-init

> On Jan 15, 2021, at 11:22 AM, Richard Biener wrote: > > On January 15, 2021 5:16:40 PM GMT+01:00, Qing Zhao > wrote: >> >> >>> On Jan 15, 2021, at 2:11 AM, Richard Biener >> wrote: >>> >>> >>> >>> On Thu, 14 Jan 2021, Qing Zhao wrote: >>> Hi,

Re: [patch] gcc.dg/analyzer tests: relax dependency on alloca.h

On Jan 15, 2021, Olivier Hainque wrote: > On 14 Jan 2021, at 22:13, Alexandre Oliva wrote: >> Would you mind if I submitted an alternate patch to do so? > Not at all, thanks for your feedback and for proposing > an alternative! Here's the modified patch. Regstrapped on x86_64-linux-gnu,

Re: [PATCH]AArch64: Add NEON, SVE and SVE2 RTL patterns for Multiply, FMS and FMA.

Tamar Christina writes: > Hi All, > > This adds implementation for the optabs for complex operations. With this the > following C code: > > void g (float complex a[restrict N], float complex b[restrict N], > float complex c[restrict N]) > { > for (int i=0; i < N; i++) >

Re: The performance data for two different implementation of new security feature -ftrivial-auto-var-init

On January 15, 2021 5:16:40 PM GMT+01:00, Qing Zhao wrote: > > >> On Jan 15, 2021, at 2:11 AM, Richard Biener >wrote: >> >> >> >> On Thu, 14 Jan 2021, Qing Zhao wrote: >> >>> Hi, >>> More data on code size and compilation time with CPU2017: >>> Compilation time data: the numbers

[PATCH] i386: Use cpp_define_formatted for __SIZEOF_FLOAT80__ definition

2021-01-15 Uroš Bizjak gcc/ * config/i386/i386-c.c (ix86_target_macros): Use cpp_define_formatted for __SIZEOF_FLOAT80__ definition. Bootstrapped and regression tested on x86_64-linux-gnu {,-m32}. Pushed to mainline. Uros. diff --git a/gcc/config/i386/i386-c.c

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

On 15/01/2021 3:07 pm, Kwok Cheung Yeung wrote: I have tested bootstrapping on x86_64 (no offloading) with no issues, and running the libgomp testsuite with Nvidia offloading shows no regressions. I have also tested all the gomp.exp tests in the main gcc testsuite, also with no issues. I am

preprocessor: Make quoting : [PR 95253]

I changed the quoting of ':', this restores it. Make doesn't need ':' quoting (in a filename). PR preprocessor/95253 libcpp/ * mkdeps.c (munge): Do not escape ':'. -- Nathan Sidwell diff --git i/libcpp/mkdeps.c w/libcpp/mkdeps.c index

c++: Fix langspecs with -fsyntax-only [PR98591]

-fsyntax-only is handled specially in the driver and causes it to add '-o /dev/null' (or a suitable OS-specific variant thereof). PCH is handled in the language driver. I'd not sufficiently protected the -fmodule-only action of adding a dummy assembler from the actions of -fsyntax-only, so

[pushed] aarch64: Add a minipass for fusing CC insns [PR88836]

This patch adds a small target-specific pass to remove redundant SVE PTEST instructions. There are two important uses of this: - Removing PTESTs after WHILELOs (PR88836). The original testcase no longer exhibits the problem due to more recent optimisations, but it can still be seen in

[pushed] recog: Fix insn_change_watermark destructor

Noticed while working on something else that the insn_change_watermark destructor could call cancel_changes for changes that no longer exist. The loop in cancel_changes is a nop in that case, but: num_changes = num; can mess things up. I think this would only affect nested uses of

[pushed] rtl-ssa: Fix a silly typo

s/ref/reg/ on a previously unused function name. Sorry for the blunder. Tested on aarch64-linux-gnu, aarch64_be-elf and x86_64-linux-gnu, pushed as obvious. Richard gcc/ * rtl-ssa/functions.h (function_info::ref_defs): Rename to... (function_info::reg_defs): ...this. *

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

On Mon, 11 Jan 2021, 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

[PATCH] c++: Fix up potential_constant_expression_1 FOR/WHILE_STMT handling [PR98672]

Hi! The following testcase is rejected even when it is valid. The problem is that potential_constant_expression_1 doesn't have the accurate *jump_target tracking cxx_eval_* has, and when the loop has a condition that isn't guaranteed to be always true, the body isn't walked at all. That is

Re: Add dg-require-wchars to libstdc++ testsuite

On Jan 15, 2021, Jonathan Wakely wrote: > On Thu, 14 Jan 2021, 22:22 Alexandre Oliva, wrote: >> ... it is definitely the case that the target currently defines wchar_t, >> and it even offers wchar.h and a lot of (maybe all?) wcs* functions. >> This was likely not the case when the patch was

Re: BoF DWARF5 patches (25% .debug section size reduction)

On Sun, Nov 15, 2020 at 11:41:24PM +0100, Mark Wielaard wrote: > On Tue, 2020-09-29 at 15:56 +0200, Mark Wielaard wrote: > > On Thu, 2020-09-10 at 13:16 +0200, Jakub Jelinek wrote: > > > On Wed, Sep 09, 2020 at 09:57:54PM +0200, Mark Wielaard wrote: > > > > --- a/gcc/doc/invoke.texi > > > > +++

Re: The performance data for two different implementation of new security feature -ftrivial-auto-var-init

> On Jan 15, 2021, at 2:11 AM, Richard Biener wrote: > > > > On Thu, 14 Jan 2021, Qing Zhao wrote: > >> Hi, >> More data on code size and compilation time with CPU2017: >> Compilation time data: the numbers are the slowdown against the >> default “no”: >> benchmarks A/no D/no

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

On 10/12/2020 2:38 pm, Jakub Jelinek wrote: On Wed, Dec 09, 2020 at 05:37:24PM +, Kwok Cheung Yeung wrote: --- a/gcc/c/c-typeck.c +++ b/gcc/c/c-typeck.c @@ -14942,6 +14942,11 @@ c_finish_omp_clauses (tree clauses, enum c_omp_region_type ort) pc = _CLAUSE_CHAIN (c);

[COMMITTED] IBM Z: Fix linking to libatomic in target test cases

Regtested on s390x-linux-gnu. Approved offline by Andreas Krebbel. Pushed. >8--->8--->8->8--->8--->8- One of the test cases failed to link because of missing paths to libatomic. Reuse procedures in lib/atomic-dg.exp to gather these paths. gcc/testsuite/ChangeLog:

[PATCH] tree-optimization/96376 - do check alignment for invariant loads

The testcases show that we fail to disregard alignment for invariant loads. The patch handles them like we handle gather and scatter. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. 2021-01-15 Richard Biener PR tree-optimization/96376 * tree-vect-stmts.c

Re: [PATCH] Add pytest for a GCOV test-case

On 1/15/21 1:28 PM, Rainer Orth wrote: Hi Martin, * If we now have an (even optional) dependency on python/pytest, this (with the exact versions and use) needs to be documented in install.texi. Done that. +be installed. Some optional tests also require Python3 and pytest module.

[PATCH] testsuite/96147 - scan for vectorized load

This changes gcc.dg/vect/bb-slp-9.c to scan for a vectorized load instead of a vectorized BB which then correctly captures the unaligned load we try to test and not some intermediate built from scalar vector. Pushed. 2021-01-15 Richard Biener PR testsuite/96147 *

[PATCH] testsuite/96147 - align vector access

This aligns p so that the testcase is meaningful for targets without a hw misaligned access. Pushed. 2021-01-15 Richard Biener PR testsuite/96147 * gcc.dg/vect/bb-slp-32.c: Align p. --- gcc/testsuite/gcc.dg/vect/bb-slp-32.c | 1 + 1 file changed, 1 insertion(+) diff --git

[PATCH] testsuite/96147 - key scanning on vect_hw_misalign

gcc.dg/vect/slp-45.c failed to key the vectorization capability scanning on vect_hw_misalign. Since the stores are strided they cannot be (all) analyzed to be aligned. Pushed. 2021-01-15 Richard Biener PR testsuite/96147 * gcc.dg/vect/slp-45.c: Key scanning on

[PATCH] testsuite/96147 - remove scanning for ! vect_hw_misalign

This removes scanning that's too difficult to get correct for all targets, leaving the correctness test for them and keeping the vectorization capability check to vect_hw_misalign targets. Pused. 2021-01-15 Richard Biener PR testsuite/96147 * gcc.dg/vect/slp-43.c: Remove !

Re: [PATCH] libatomic, libgomp, libitc: Fix bootstrap [PR70454]

On Fri, Jan 15, 2021 at 4:07 AM Richard Biener wrote: > > On Fri, 15 Jan 2021, Jakub Jelinek wrote: > > > On Thu, Jan 14, 2021 at 04:08:20PM -0800, H.J. Lu wrote: > > > Here is the updated patch. OK for master? > > > > Here is my version of the entire patch. > > > > Bootstrapped/regtested on

[PATCH] testsuite/96098 - remove redundant testcase

The testcase morphed in a way no longer testing what it was originally supposed to do and slightly altering it shows the original issue isn't fixed (anymore). The limit as set as result of PR91403 (and dups) prevents the issue for larger arrays but the testcase has double a[128][128]; which

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

On Fri, 15 Jan 2021 at 12:39, Daniel Engel wrote: > > Hi Christophe, > > On Mon, Jan 11, 2021, at 8:39 AM, Christophe Lyon wrote: > > 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,

Re: [PATCH] Add pytest for a GCOV test-case

Hi Martin, * If we now have an (even optional) dependency on python/pytest, this (with the exact versions and use) needs to be documented in install.texi. >>> >>> Done that. >> +be installed. Some optional tests also require Python3 and pytest >> module. >> It would be

[committed][OG10] DWARF address space for variables

This patch adds DWARF support for "local" variables that are actually located in a different address space. This situation occurs for variables shared between all the worker threads of an OpenACC gang. On AMD GCN the variables are allocated to the low-latency LDS memory associated with each

[committed][OG10] amdgcn: DWARF address spaces

This patch implements DWARF address spaces for pointers to LDS, etc., on AMD GCN. The address space mappings are defined by AMD in their DWARF proposals, and the LLVM implementation. ROCGDB does not actually support this feature yet, I don't believe, but will do so soonish. Committed to

Re: [PATCH] libatomic, libgomp, libitc: Fix bootstrap [PR70454]

On Fri, 15 Jan 2021, Jakub Jelinek wrote: > On Thu, Jan 14, 2021 at 04:08:20PM -0800, H.J. Lu wrote: > > Here is the updated patch. OK for master? > > Here is my version of the entire patch. > > Bootstrapped/regtested on x86_64-linux and i686-linux and additionally > tested with i686-linux

Re: [PATCH] vect: Use factored nloads for load cost modeling [PR82255]

On Fri, Jan 15, 2021 at 9:11 AM Kewen.Lin wrote: > > Hi, > > This patch follows Richard's suggestion in the thread discussion[1], > it's to factor out the nloads computation in vectorizable_load for > strided access, to ensure we can obtain the consistent information > when estimating the costs.

Re: [EXTERNAL] Re: [PATCH][tree-optimization]Optimize combination of comparisons to dec+compare

On Thu, Jan 14, 2021 at 10:04 PM Eugene Rozenfeld wrote: > > I got more feedback for the patch from Gabriel Ravier and Jakub Jelinek in > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96674 and re-worked it > accordingly. > > The changes from the previous patch are: > 1. Switched the tests to

[PATCH] libatomic, libgomp, libitc: Fix bootstrap [PR70454]

On Thu, Jan 14, 2021 at 04:08:20PM -0800, H.J. Lu wrote: > Here is the updated patch. OK for master? Here is my version of the entire patch. Bootstrapped/regtested on x86_64-linux and i686-linux and additionally tested with i686-linux --with-arch=i386 and x86_64-linux --with-arch_32=i386

[committed][OG10] amdgcn: Fix DWARF variables with alloca

This patch fixes DWARF frame calculations for functions that use alloca on AMD GCN. Like many other platforms, it achieves this by switching to frame-pointer mode for this function. The frame pointer is necessary for debugability only, so if the user specifies -fomit-frame-pointer then this

[committed][OG10] Fix offload dwarf info

This patch corrects a problem in which GDB ignores the debug info for offload kernel entry functions because they're represented as nested functions inside a function that does not exist on the accelerator device (only on the host). The fix is to add a notional code range to the non-existent

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

Hi Christophe, On Mon, Jan 11, 2021, at 8:39 AM, Christophe Lyon wrote: > 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

Re: [PATCH] x86: Error on -fcf-protection with incompatible target

On 1/14/21 4:18 PM, H.J. Lu via Gcc-patches wrote: > On Thu, Jan 14, 2021 at 6:51 AM Uros Bizjak wrote: >> >> On Thu, Jan 14, 2021 at 3:05 PM H.J. Lu wrote: >>> >>> -fcf-protection with CF_BRANCH inserts ENDBR32 at function entries. >>> ENDBR32 is NOP only on 64-bit processors and 32-bit

[PATCH v5 32/33] Import float<->__fp16 conversion from the CM0 library

gcc/libgcc/ChangeLog: 2021-01-13 Daniel Engel * config/arm/eabi/fcast.S (__aeabi_h2f, __aeabi_f2h): Added functions. * config/arm/fp16 (__gnu_f2h_ieee, __gnu_h2f_ieee, __gnu_f2h_alternative, __gnu_h2f_alternative): Disable build for v6m multilibs. *

[PATCH v5 30/33] Import float-to-integer conversion from the CM0 library

gcc/libgcc/ChangeLog: 2021-01-13 Daniel Engel * config/arm/bpabi-lib.h (muldi3): Removed duplicate. (fixunssfsi) Removed obsolete RENAME_LIBRARY directive. * config/arm/eabi/ffixed.S (__aeabi_f2iz, __aeabi_f2uiz, __aeabi_f2lz, __aeabi_f2ulz): New file. *

[PATCH v5 33/33] Drop single-precision Thumb-1 soft-float functions

With the complete CM0 library integrated, regression testing showed new failures with the message "compilation failed to produce executable": gcc.dg/fixed-point/convert-float-1.c gcc.dg/fixed-point/convert-float-3.c gcc.dg/fixed-point/convert-sat.c Investigating, this appears to be

[PATCH v5 29/33] Import integer-to-float conversion from the CM0 library

gcc/libgcc/ChangeLog: 2021-01-13 Daniel Engel * config/arm/bpabi-lib.h (__floatdisf, __floatundisf): Remove obsolete RENAME_LIBRARY directives. * config/arm/eabi/ffloat.S (__aeabi_i2f, __aeabi_l2f, __aeabi_ui2f, __aeabi_ul2f): New file. *

[PATCH v5 31/33] Import float<->double conversion from the CM0 library

gcc/libgcc/ChangeLog: 2021-01-13 Daniel Engel * config/arm/eabi/fcast.S (__aeabi_d2f, __aeabi_f2d): New file. * config/arm/lib1funcs.S: #include eabi/fcast.S (v6m only). * config/arm/t-elf (LIB1ASMFUNCS): Added _arm_d2f and _arm_f2d. --- libgcc/config/arm/eabi/fcast.S |

[PATCH v5 28/33] Import float division from the CM0 library

gcc/libgcc/ChangeLog: 2021-01-08 Daniel Engel * config/arm/eabi/fdiv.S (__divsf3, __fp_divloopf): New file. * config/arm/lib1funcs.S: #include eabi/fdiv.S (v6m only). * config/arm/t-elf (LIB1ASMFUNCS): Added _divsf3 and _fp_divloopf. --- libgcc/config/arm/eabi/fdiv.S |

[PATCH v5 27/33] Import float multiplication from the CM0 library

gcc/libgcc/ChangeLog: 2021-01-13 Daniel Engel * config/arm/eabi/fmul.S (__mulsf3): New file. * config/arm/lib1funcs.S: #include eabi/fmul.S (v6m only). * config/arm/t-elf (LIB1ASMFUNCS): Moved _mulsf3 to global scope (this object was previously blocked on v6m

[PATCH v5 26/33] Import float addition and subtraction from the CM0 library

Since this is the first import of single-precision functions, some common parsing and formatting routines are also included. These common rotines will be referenced by other functions in subsequent commits. However, even if the size penalty is accounted entirely to __addsf3(), the total compiled

[PATCH v5 25/33] Refactor Thumb-1 float subtraction into a new file

This will make it easier to isolate changes in subsequent patches. gcc/libgcc/ChangeLog: 2021-01-13 Daniel Engel * config/arm/bpabi-v6m.S (__aeabi_frsub): Moved to ... * config/arm/eabi/fadd.S: New file. * config/arm/lib1funcs.S: #include eabi/fadd.S (v6m only). ---

[PATCH v5 24/33] Import float comparison from the CM0 library

These functions are significantly smaller and faster than the wrapper functions and soft-float implementation they replace. Using the first comparison operator (e.g. '<=') in any program costs about 70 bytes initially, but every additional operator incrementally adds just 4 bytes. NOTE: It seems

[PATCH v5 23/33] Refactor Thumb-1 float comparison into a new file

gcc/libgcc/ChangeLog: 2021-01-13 Daniel Engel * config/arm/bpabi-v6m.S (__aeabi_cfcmpeq, __aeabi_cfcmple, __aeabi_cfrcmple, __aeabi_fcmpeq, __aeabi_fcmple, aeabi_fcmple, __aeabi_fcmpgt, aeabi_fcmpge): Moved to ... * config/arm/eabi/fcmp.S: New file. *

[PATCH v5 22/33] Import integer multiplication from the CM0 library

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 v5 21/33] Import 64-bit division from the CM0 library

gcc/libgcc/ChangeLog: 2021-01-13 Daniel Engel * config/arm/bpabi.c: Deleted unused file. * config/arm/eabi/ldiv.S (__aeabi_ldivmod, __aeabi_uldivmod): Replaced wrapper functions with a complete implementation. * config/arm/t-bpabi (LIB2ADD_ST): Removed bpabi.c.

[PATCH v5 20/33] Refactor Thumb-1 64-bit division into a new file

gcc/libgcc/ChangeLog: 2021-01-13 Daniel Engel * config/arm/bpabi-v6m.S (__aeabi_ldivmod/ldivmod): Moved to ... * config/arm/eabi/ldiv.S: New file. * config/arm/lib1funcs.S: #include eabi/ldiv.S (v6m only). --- libgcc/config/arm/bpabi-v6m.S | 81 -

[PATCH v5 19/33] Import 32-bit division from the CM0 library

gcc/libgcc/ChangeLog: 2021-01-07 Daniel Engel * config/arm/eabi/idiv.S: New file for __udivsi3() and __divsi3(). * config/arm/lib1funcs.S: #include eabi/idiv.S (v6m only). --- libgcc/config/arm/eabi/idiv.S | 299 ++ libgcc/config/arm/lib1funcs.S |

[PATCH v5 18/33] Merge Thumb-2 optimizations for 64-bit comparison

This effectively merges support for all architecture variants into a common function path with appropriate build conditions. ARM performance is 1-2 instructions faster; Thumb-2 is about 50% faster. gcc/libgcc/ChangeLog: 2021-01-13 Daniel Engel * config/arm/bpabi.S (__aeabi_lcmp,

[PATCH v5 17/33] Import 64-bit comparison from CM0 library

These are 2-5 instructions smaller and just as fast. Branches are minimized, which will allow easier adaptation to Thumb-2/ARM mode. gcc/libgcc/ChangeLog: 2021-01-13 Daniel Engel * config/arm/eabi/lcmp.S (__aeabi_lcmp, __aeabi_ulcmp): Replaced; add macro configuration to build

[PATCH v5 16/33] Refactor Thumb-1 64-bit comparison into a new file

This will make it easier to isolate changes in subsequent patches. gcc/libgcc/ChangeLog: 2021-01-13 Daniel Engel * config/arm/bpabi-v6m.S (__aeabi_lcmp, __aeabi_ulcmp): Moved to ... * config/arm/eabi/lcmp.S: New file. * config/arm/lib1funcs.S: #include eabi/lcmp.S. ---

[PATCH v5 15/33] Import 'popcnt' functions from the CM0 library

The functional overlap between the single- and double-word functions makes this implementation about 30% smaller than the C functions if both functions are linked together in the same appliation. gcc/libgcc/ChangeLog: 2021-01-13 Daniel Engel * config/arm/popcnt.S (__popcountsi,

[PATCH v5 14/33] Import 'parity' functions from the CM0 library

The functional overlap between the single- and double-word functions makes functions makes this implementation about half the size of the C functions if both functions are linked in the same application. gcc/libgcc/ChangeLog: 2021-01-13 Daniel Engel * config/arm/parity.S: New file for

  1   2   >