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
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
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: [
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
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
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
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
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
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
> 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
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
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
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
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
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
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
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
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
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
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 :-)
> > >
> > >
> 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
> 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
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
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
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
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
26 matches
Mail list logo