[PATCH] DSE: Use the constant source if possible

2022-05-20 Thread H.J. Lu via Gcc-patches
When recording store for RTL dead store elimination, check if the source register is set only once to a constant. If yes, record the constant as the store source. It eliminates unrolled zero stores after memset 0 in a loop where a vector register is used as the zero store source. gcc/ P

Re: [PATCH] Modula-2: merge proposal/review: 1/9 01.patch-set-01

2022-05-20 Thread Gaius Mulley via Gcc-patches
ome/gaius/opt/lib/gcc/x86_64-pc-linux-gnu/13.0.0/m2/m2pim -fplugin=m2rte -ftarget-ar=/usr/bin/ar -ftarget-ranlib=/usr/bin/ranlib -fobject-path=/home/gaius/opt/lib/gcc/x86_64-pc-linux-gnu/13.0.0/m2/m2pim -fplugin=m2rte -I/home/gaius/opt/lib/gcc/x86_64-pc-linux-gnu/13.0.0/m2/m2pim -o /tmp/cc8BoL3d

RE: [EXTERNAL] Re: [PATCH] Guard against applying scale with 0 denominator

2022-05-20 Thread Eugene Rozenfeld via Gcc-patches
Thank you for the feedback Richard. I attached a patch that saves/restores counts if the epilog doesn't use a scalar loop. Eugene -Original Message- From: Richard Biener Sent: Thursday, May 12, 2022 12:34 AM To: Eugene Rozenfeld Cc: Jan Hubicka ; gcc-patches@gcc.gnu.org Subject: Re: [

Re: [PATCH] Add divide by zero side effect.

2022-05-20 Thread Bernhard Reutner-Fischer via Gcc-patches
On 20 May 2022 16:39:20 CEST, Segher Boessenkool wrote: >On Fri, May 20, 2022 at 10:11:32AM +0200, Eric Botcazou wrote: >> > I suggest 'deduce', 'deduction', 'deducing a range'. What the code is >> > actually doing is deducing that 'b' in 'a / b' cannot be zero. Function in >> > GCC might be call

[PATCH] Support multilib-aware target lib flags self-specs overriding

2022-05-20 Thread Alexandre Oliva via Gcc-patches
This patch introduces -multiflags, short for multilib TFLAGS, as an option that does nothing by default, but that can be added to TFLAGS and mapped to useful options by driver self-specs. I realize -m is reserved for machine-specific flags, which this option sort-of isn't, but its intended use i

Re: [PATCH] Modula-2: merge proposal/review: 1/9 01.patch-set-01

2022-05-20 Thread Gaius Mulley via Gcc-patches
Richard Biener writes: > On Wed, May 18, 2022 at 3:47 PM Gaius Mulley via Gcc-patches > wrote: >> >> hello, >> >> this file is part-01 of the patch set for the gm2 review. Hi Richard, > I think you did a better job last time splitting the patches. ah many apologies I should have annotated the

RE: [x86 PATCH] Some additional ix86_rtx_costs clean-ups: NEG, AND and pandn.

2022-05-20 Thread Roger Sayle
Hi Uros, Many thanks for the review. As requested here is a revised version of the patch that's slightly more aggressive in its clean-ups (duplicating 6 lines of code allowed me to eliminate 9 lines of code), but most significantly also includes support for the andn for TARGET_BMI, and allows the

Re: [PATCH] libstdc++-v3: Set 26_numerics/random/n.b.d./op./values.cc timeout-factor to 3

2022-05-20 Thread Jonathan Wakely via Gcc-patches
On Fri, 20 May 2022 at 15:31, Hans-Peter Nilsson wrote: > > > From: Jonathan Wakely > > Date: Fri, 20 May 2022 11:03:40 +0200 > > > > Ok to commit (without renaming)? > > > > I'm OK with the timeout factor, but we could also solve it differently > > so that it doesn't take nearly 5 minutes, as in

Re: [PATCH] Add divide by zero side effect.

2022-05-20 Thread Segher Boessenkool
On Fri, May 20, 2022 at 10:11:32AM +0200, Eric Botcazou wrote: > > I suggest 'deduce', 'deduction', 'deducing a range'. What the code is > > actually doing is deducing that 'b' in 'a / b' cannot be zero. Function in > > GCC might be called like 'deduce_ranges_from_stmt'. > > Or "infer", "inference

Re: [PATCH] libstdc++-v3: Set 26_numerics/random/n.b.d./op./values.cc timeout-factor to 3

2022-05-20 Thread Hans-Peter Nilsson via Gcc-patches
> From: Jonathan Wakely > Date: Fri, 20 May 2022 11:03:40 +0200 > > Ok to commit (without renaming)? > > I'm OK with the timeout factor, but we could also solve it differently > so that it doesn't take nearly 5 minutes, as in the attached patch. > The testDiscreteDist function can be parametrize

Re: [PATCH 1/8] Move FINAL and OVERRIDE from ansidecl.h to gcc-plugin.h

2022-05-20 Thread David Malcolm via Gcc-patches
On Thu, 2022-05-19 at 14:45 +0200, Richard Biener wrote: > On Thu, May 19, 2022 at 2:43 PM David Malcolm via Gcc-patches > wrote: > > > > On Thu, 2022-05-19 at 08:32 -0400, David Malcolm wrote: > > > As of GCC 11 onwards we have required a C++11 compiler, such as > > > GCC > > > 4.8 > > > or late

Re: [PATCH] gdc 9, 10 and 11 bug fix

2022-05-20 Thread Iain Buclaw via Gcc-patches
Excerpts from Marc Aurèle La France's message of Mai 20, 2022 6:56 am: > On Tue, 17 May 2022, Marc Aurèle La France wrote: >> On Tue, 17 May 2022, Iain Buclaw wrote: >>> Excerpts from Marc Aurèle La France's message of Mai 17, 2022 5:31 pm: On Tue, 17 May 2022, Iain Buclaw wrote: > Excerpt

Re: [PATCH] configure: add --disable-fix-includes

2022-05-20 Thread Alexandre Oliva via Gcc-patches
On May 11, 2022, Martin Liška wrote: > Ready to be installed? Hmm... I don't like that --disable-fixincludes would still configure, build and even install fixincludes. This would be surprising, given that the semantics of disabling a component is to not even configure it. How about leaving th

[PATCH] Modula-2: merge proposal/review: 8/9 08.patch-set-06 v2

2022-05-20 Thread Gaius Mulley via Gcc-patches
Hello, for completeness here is v2 of the patch set: (m2pp.c has been renamed m2pp.cc and a missing dependency added to m2rte). 6. gcc/m2/Make-lang.in == - New file: gcc/m2/Make-lang.in - # To

Re: [PATCH] Add divide by zero side effect.

2022-05-20 Thread Alexandre Oliva via Gcc-patches
On May 18, 2022, Andrew MacLeod via Gcc-patches wrote: > I'm open to suggestions for a better term! How about inference? -- Alexandre Oliva, happy hackerhttps://FSFLA.org/blogs/lxo/ Free Software Activist GNU Toolchain Engineer Disinformation flourishes

Re: [PATCH] Add divide by zero side effect.

2022-05-20 Thread Richard Biener via Gcc-patches
On Fri, May 20, 2022 at 10:17 AM Alexander Monakov wrote: > > On Fri, 20 May 2022, Richard Biener wrote: > > > On Fri, May 20, 2022 at 8:38 AM Alexander Monakov > > wrote: > > > > > > On Fri, 20 May 2022, Richard Biener via Gcc-patches wrote: > > > > > > > > Still waiting for a suggestion, since

Re: [PATCH] Modula-2: merge proposal/review: 1/9 01.patch-set-01

2022-05-20 Thread Richard Biener via Gcc-patches
On Wed, May 18, 2022 at 3:47 PM Gaius Mulley via Gcc-patches wrote: > > hello, > > this file is part-01 of the patch set for the gm2 review. I think you did a better job last time splitting the patches. This one contains the driver changes for which I have a hard time remembering the reason they

Re: [PATCH] libstdc++-v3: Set 26_numerics/random/n.b.d./op./values.cc timeout-factor to 3

2022-05-20 Thread Jonathan Wakely via Gcc-patches
On Fri, 20 May 2022 at 10:03, Jonathan Wakely wrote: > > On Fri, 20 May 2022 at 03:20, Hans-Peter Nilsson via Libstdc++ > wrote: > > > > Also, how about shortening those test-suite file-paths? > > They're path sort-of overlong for any git (and ChangeLog) > > commit-line limit. 1/2 :-) > > Yes, th

Re: [PATCH] libstdc++-v3: Set 26_numerics/random/n.b.d./op./values.cc timeout-factor to 3

2022-05-20 Thread Jonathan Wakely via Gcc-patches
On Fri, 20 May 2022 at 03:20, Hans-Peter Nilsson via Libstdc++ wrote: > > Also, how about shortening those test-suite file-paths? > They're path sort-of overlong for any git (and ChangeLog) > commit-line limit. 1/2 :-) Yes, they're silly. I like what libc++ does, i.e. name the testsuite directori

Re: [PATCH] Add divide by zero side effect.

2022-05-20 Thread Alexander Monakov via Gcc-patches
On Fri, 20 May 2022, Richard Biener wrote: > On Fri, May 20, 2022 at 8:38 AM Alexander Monakov wrote: > > > > On Fri, 20 May 2022, Richard Biener via Gcc-patches wrote: > > > > > > Still waiting for a suggestion, since "side effect" is the description > > > > that made sense to me :-) > > > > > >

Re: [PATCH] Add divide by zero side effect.

2022-05-20 Thread Eric Botcazou via Gcc-patches
> I think side-effect captures it quite well even if it overlaps with a term > used in language standards. IMO it's very confusing, see the subject: "Add divide by zero side effect". The only side effect of dividing by zero is (possibly) raising a trap. -- Eric Botcazou

Re: [PATCH] Add divide by zero side effect.

2022-05-20 Thread Eric Botcazou via Gcc-patches
> I suggest 'deduce', 'deduction', 'deducing a range'. What the code is > actually doing is deducing that 'b' in 'a / b' cannot be zero. Function in > GCC might be called like 'deduce_ranges_from_stmt'. Or "infer", "inference", "inferring a range". > Please don't overload 'side effect' if possibl

Re: [PATCH] Add divide by zero side effect.

2022-05-20 Thread Richard Biener via Gcc-patches
On Fri, May 20, 2022 at 8:38 AM Alexander Monakov wrote: > > On Fri, 20 May 2022, Richard Biener via Gcc-patches wrote: > > > > Still waiting for a suggestion, since "side effect" is the description > > > that made sense to me :-) > > > > I think side-effect captures it quite well even if it overl

Re: [PATCH v3 09/10] libgcc: Add support for HF mode (aka _Float16) in libbid

2022-05-20 Thread Christophe Lyon via Gcc-patches
On 5/19/22 21:35, Joseph Myers wrote: On Thu, 19 May 2022, Christophe Lyon via Gcc-patches wrote: Hi Joseph, May I ping you on this version? (I've moved the tests to gcc.dg/torture/ and checked they work on aarch64 and x86_64. This version is OK, given a bug report filed in Bugzilla for t

Re: [PATCH] ipa-icf: skip variables with body_removed

2022-05-20 Thread Martin Liška
On 5/19/22 17:02, Jan Hubicka wrote: >> Similarly to cgraph_nodes, it may happen that body_removed is set >> during merging of symbols. >> >> PR ipa/105600 >> >> Patch can bootstrap on x86_64-linux-gnu and survives regression tests. >> >> Ready to be installed? >> Thanks, >> Martin >> >> gcc/C

GCC 9.4.1 Status Report (2022-05-20)

2022-05-20 Thread Richard Biener via Gcc-patches
Status == The GCC 9 branch is now frozen for its final release, GCC 9.5. The branch will be closed after that. Quality Data Priority # Change from last report --- --- P1 P2 369 - 58 P3