On 6/2/21 4:21 AM, Ilya Leoshkevich wrote:
> Bootstrapped and regtested on s390x-redhat-linux. Ok for master?
>
>
>
> Since commit dd1ef00c45ba ("Fix bug in the define_subst handling that
> made match_scratch unusable for multi-alternative patterns.") the
> workaround for that bug in *ashrdi3_3
On Tue, Jun 1, 2021 at 9:56 PM Martin Sebor wrote:
>
> On 5/27/21 2:53 PM, Jason Merrill wrote:
> > On 4/27/21 11:52 AM, Martin Sebor via Gcc-patches wrote:
> >> On 4/27/21 8:04 AM, Richard Biener wrote:
> >>> On Tue, Apr 27, 2021 at 3:59 PM Martin Sebor wrote:
>
> On 4/27/21 1:58 AM, R
Hi,
Indeed, the split condition needs to be populated. However, I doubt that the
pattern in question is used by the compiler. Do you have an example where it is
exercised?
Thanks,
Claudiu
From: Kewen Lin
Sent: Wednesday, June 2, 2021 8:04 AM
To: gcc-patches@gcc
On Tue, Jun 1, 2021 at 10:37 PM Pat Haugen via Gcc-patches
wrote:
>
> Make sure link reg save MEM has frame alias set, to match other link reg
> save/restore code.
>
> Bootstrap/regtest on powerpc64/powerpc64le with no new regressions. Ok for
> trunk?
>
> -Pat
>
>
> 2021-06-01 Pat Haugen
>
> gc
On Tue, Jun 1, 2021 at 9:07 PM apinski--- via Gcc-patches
wrote:
>
> From: Andrew Pinski
>
> This improves match_simplify_replace in phi-opt to handle the
> case where there is one cheap preparation statement in the
> middle basic block similar to xor_replacement and others.
> This allows to remo
On Wed, Jun 2, 2021 at 7:05 AM Kewen Lin wrote:
>
> gcc/ChangeLog:
>
> * config/i386/i386.md (*load_tp_x32_zext, *add_tp_x32_zext,
> *tls_dynamic_gnu2_combine_32): Fix empty split condition.
> * config/i386/sse.md (*_pmovmskb_lt,
> *_pmovmskb_zext_lt, *sse2_pmovmskb
Please discard this one, sorry for disturbing.
Obviously I'm new to git send-email.
On Wed, Jun 2, 2021 at 1:40 PM liuhongt via Gcc-patches
wrote:
>
> This is the updated patch.
>
>
--
BR,
Hongtao
For i386, it will enable below opt
from
notl%edi
vpbroadcastd%edi, %xmm0
vpand %xmm1, %xmm0, %xmm0
to
vpbroadcastd%edi, %xmm0
vpandn %xmm1, %xmm0, %xmm0
gcc/ChangeLog:
PR target/100711
* simplify-rtx.c (simplify_unary_operat
For i386, it will enable below opt
from
notl%edi
vpbroadcastd%edi, %xmm0
vpand %xmm1, %xmm0, %xmm0
to
vpbroadcastd%edi, %xmm0
vpandn %xmm1, %xmm0, %xmm0
gcc/ChangeLog:
PR target/100711
* simplify-rtx.c (simplify_unary_operat
This is the updated patch.
On 6/1/2021 8:21 PM, Ilya Leoshkevich via Gcc-patches wrote:
Bootstrapped and regtested on s390x-redhat-linux. Ok for master?
Since commit dd1ef00c45ba ("Fix bug in the define_subst handling that
made match_scratch unusable for multi-alternative patterns.") the
workaround for that bug in *
On Wed, 2021-06-02 at 00:05 -0500, Kewen Lin wrote:
> gcc/ChangeLog:
>
> * config/sh/sh.md (doloop_end_split): Fix empty split condition.
> ---
> gcc/config/sh/sh.md | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/config/sh/sh.md b/gcc/config/sh/sh.md
> index e
gcc/ChangeLog:
* config/cris/cris.md (*addi_reload): Fix empty split condition.
---
gcc/config/cris/cris.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/config/cris/cris.md b/gcc/config/cris/cris.md
index 7de0ec63fcf..d5a3c703a83 100644
--- a/gcc/config/cris/cri
gcc/ChangeLog:
* config/sparc/sparc.md (*snedi_zero_vis3,
*neg_snedi_zero_subxc, *plus_snedi_zero,
*plus_plus_snedi_zero, *minus_snedi_zero,
*minus_minus_snedi_zero): Fix empty split condition.
---
gcc/config/sparc/sparc.md | 12 ++--
1 file changed, 6 inse
gcc/ChangeLog:
* config/sh/sh.md (doloop_end_split): Fix empty split condition.
---
gcc/config/sh/sh.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/config/sh/sh.md b/gcc/config/sh/sh.md
index e3af9ae21c1..93ee7c9a7de 100644
--- a/gcc/config/sh/sh.md
+++ b/gcc/c
gcc/ChangeLog:
* config/or1k/or1k.md (*movdi): Fix empty split condition.
---
gcc/config/or1k/or1k.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/config/or1k/or1k.md b/gcc/config/or1k/or1k.md
index eb94efba0e4..495b3e277ba 100644
--- a/gcc/config/or1k/or1k.md
+
gcc/ChangeLog:
* config/mips/mips.md (, bswapsi2, bswapdi2): Fix empty
split condition.
---
gcc/config/mips/mips.md | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/gcc/config/mips/mips.md b/gcc/config/mips/mips.md
index eef3cfd50a8..455b9b802f6 100644
---
gcc/ChangeLog:
* config/h8300/combiner.md (*andsi3_lshiftrt_n_sb): Fix empty split
condition.
---
gcc/config/h8300/combiner.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/config/h8300/combiner.md b/gcc/config/h8300/combiner.md
index 20e19da0419..e31bd50
gcc/ChangeLog:
* config/arm/vfp.md (no_literal_pool_df_immediate,
no_literal_pool_sf_immediate): Fix empty split condition.
---
gcc/config/arm/vfp.md | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/gcc/config/arm/vfp.md b/gcc/config/arm/vfp.md
index f97af92
gcc/ChangeLog:
* config/i386/i386.md (*load_tp_x32_zext, *add_tp_x32_zext,
*tls_dynamic_gnu2_combine_32): Fix empty split condition.
* config/i386/sse.md (*_pmovmskb_lt,
*_pmovmskb_zext_lt, *sse2_pmovmskb_ext_lt,
*_pblendvb_lt): Likewise.
---
gcc/config/i38
gcc/ChangeLog:
* config/m68k/m68k.md (*zero_extend_inc, *zero_extend_dec,
*zero_extendsidi2): Fix empty split condition.
---
gcc/config/m68k/m68k.md | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/gcc/config/m68k/m68k.md b/gcc/config/m68k/m68k.md
index 59
gcc/ChangeLog:
* config/arc/arc.md (*bbit_di): Fix empty split condition.
---
gcc/config/arc/arc.md | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/config/arc/arc.md b/gcc/config/arc/arc.md
index 7a52551eef5..a03840c4c36 100644
--- a/gcc/config/arc/arc.md
+++ b/gcc
As Segher suggested, this patch is to emit the error message
if the split condition of define_insn_and_split is empty while
the insn condition isn't.
gcc/ChangeLog:
* gensupport.c (process_rtx): Emit error message for empty
split condition in define_insn_and_split while the insn
Hi all,
define_insn_and_split should avoid to use empty split condition
if the condition for define_insn isn't empty, otherwise it can
sometimes result in unexpected consequence, since the split
will always be done even if the insn condition doesn't hold.
To avoid forgetting to add "&& 1" onto sp
These are some minor changes to the H8 port to fix a latent bug and
prepare for having logical operations participate in redundant
test/compare elimination that I've been sitting on for a while.
These have been through my tester with testing timeouts dramatically
increased to give as much cove
Bootstrapped and regtested on s390x-redhat-linux. Ok for master?
Since commit dd1ef00c45ba ("Fix bug in the define_subst handling that
made match_scratch unusable for multi-alternative patterns.") the
workaround for that bug in *ashrdi3_31 is not only no
longer necessary, but actually breaks th
On Tue, Jun 1, 2021 at 6:17 PM Hongtao Liu wrote:
>
> On Wed, Jun 2, 2021 at 7:07 AM H.J. Lu via Gcc-patches
> wrote:
> >
> > On Tue, Jun 1, 2021 at 7:21 AM Jeff Law wrote:
> > >
> > >
> > >
> > > On 6/1/2021 7:29 AM, H.J. Lu via Gcc-patches wrote:
> > > > On Tue, Jun 1, 2021 at 6:25 AM Richard
On Wed, Jun 2, 2021 at 7:07 AM H.J. Lu via Gcc-patches
wrote:
>
> On Tue, Jun 1, 2021 at 7:21 AM Jeff Law wrote:
> >
> >
> >
> > On 6/1/2021 7:29 AM, H.J. Lu via Gcc-patches wrote:
> > > On Tue, Jun 1, 2021 at 6:25 AM Richard Biener
> > > wrote:
> > >> On Tue, Jun 1, 2021 at 3:05 PM H.J. Lu wro
On Tue, Jun 01, 2021 at 11:18:05AM -0500, Bill Schmidt wrote:
> Hi! PR100750 reports a failure on my part to require the ELFv2 ABI for
> one of the ROP tests. This fixes that.
It would be nice if we had a selector for when we can use -mrop-protect,
instead of assuming it is only for ELFv2.
Seg
Ping patch again.
Original patch (Change rs6000_const_f32_to_i32 return type)
| Date: Tue, 18 May 2021 16:39:28 -0400
| Subject: [PATCH] Change rs6000_const_f32_to_i32 return type.
| Message-ID: <20210518203928.ga15...@ibm-toto.the-meissners.org>
| https://gcc.gnu.org/pipermail/gcc-patches/2021-M
Ping patch again.
Original patch (Fix tests when running on power10, PR testsuite/100166).
| Date: Tue, 18 May 2021 16:59:12 -0400
| Subject: [PATCH 2/2] Fix tests when running on power10, PR testsuite/100166
| Message-ID: <20210518205912.gb18...@ibm-toto.the-meissners.org>
| https://gcc.gnu.org/
Ping patch again.
Original patch (Deal with prefixed loads/stores in tests, PR testsuite/100166):
| Date: Tue, 18 May 2021 16:57:59 -0400
| Subject: [PATCH 1/2] Deal with prefixed loads/stores in tests, PR
testsuite/100166
| Message-ID: <20210518205759.ga18...@ibm-toto.the-meissners.org>
| https
Ping patch again.
Original patch (Fix vec-splati-runnable.c test)
| Date: Tue, 18 May 2021 16:49:58 -0400
| Subject: [PATCH] Fix vec-splati-runnable.c test.
| Message-ID: <20210518204958.ga17...@ibm-toto.the-meissners.org>
| https://gcc.gnu.org/pipermail/gcc-patches/2021-May/570684.html
Note as
Ping patch again.
Original patch (fix xxevel predicates):
| Date: Tue, 18 May 2021 16:47:58 -0400
| Subject: [PATCH 2/2] Fix xxeval predicates.
| Message-ID: <20210518204758.gb16...@ibm-toto.the-meissners.org>
| https://gcc.gnu.org/pipermail/gcc-patches/2021-May/570683.html
--
Michael Meissner,
Ping patch again.
Original patch (move xx* builtins to vsx.md)
| Date: Tue, 18 May 2021 16:46:47 -0400
| Subject: [PATCH 1/2] Move xx* builtins to vsx.md.
| Message-ID: <20210518204647.ga16...@ibm-toto.the-meissners.org>
| https://gcc.gnu.org/pipermail/gcc-patches/2021-May/570682.html
--
Michae
Ping patch again:
Original patch (Allow __ibm128 on older PowerPC systems):
| Date: Tue, 18 May 2021 16:36:32 -0400
| Subject: [PATCH] Allow __ibm128 on older PowerPC systems.
| Message-ID: <20210518203632.ga15...@ibm-toto.the-meissners.org>
| https://gcc.gnu.org/pipermail/gcc-patches/2021-May/57
Ping patch again.
| Date: Tue, 18 May 2021 16:32:33 -0400
| Subject: [PATCH] Fix long double tests when default long double is not IBM.
| Message-ID: <20210518203233.ga15...@ibm-toto.the-meissners.org>
| https://gcc.gnu.org/pipermail/gcc-patches/2021-May/570678.html
--
Michael Meissner, IBM
IBM,
Ping patch again.
Original patch (Add IEEE 128-bit fp conditional move on PowerPC):
| Date: Tue, 18 May 2021 16:28:27 -0400
| Subject: [PATCH 2/2] Add IEEE 128-bit fp conditional move on PowerPC.
| Message-ID: <20210518202827.gb14...@ibm-toto.the-meissners.org>
--
Michael Meissner, IBM
IBM, M/S
Ping again.
Original patch (Add IEEE 128-bit min/max support on PowerPC):
| Date: Tue, 18 May 2021 16:26:06 -0400
| Subject: [PATCH 1/2] Add IEEE 128-bit min/max support on PowerPC.
| Message-ID: <20210518202606.ga14...@ibm-toto.the-meissners.org>
| https://gcc.gnu.org/pipermail/gcc-patches/2021-
On Tue, Jun 1, 2021 at 7:21 AM Jeff Law wrote:
>
>
>
> On 6/1/2021 7:29 AM, H.J. Lu via Gcc-patches wrote:
> > On Tue, Jun 1, 2021 at 6:25 AM Richard Biener
> > wrote:
> >> On Tue, Jun 1, 2021 at 3:05 PM H.J. Lu wrote:
> >>> On Mon, May 31, 2021 at 11:54:53PM -0600, Jeff Law wrote:
>
>
On Tue, Jun 01, 2021 at 04:35:53PM -0400, David Malcolm wrote:
> [...]
> Did this patch get reviewed/approved?
It's still under review, but I think it's close.
> Is the latest version still this one:
> https://gcc.gnu.org/pipermail/gcc-patches/2021-February/565581.html
> or is there a more rece
Brown paper bag time. The List-Id: should look like a hostname, not
like an email address. Somehow I put in an at-sign when changing my
gcc-patches example to the match-all example we have here.
Note that the "." in the procmail match are RE wildcards btw. This is
common practice in procmailrcs
On 6/1/21 3:56 PM, Martin Sebor wrote:
On 5/27/21 2:53 PM, Jason Merrill wrote:
On 4/27/21 11:52 AM, Martin Sebor via Gcc-patches wrote:
On 4/27/21 8:04 AM, Richard Biener wrote:
On Tue, Apr 27, 2021 at 3:59 PM Martin Sebor wrote:
On 4/27/21 1:58 AM, Richard Biener wrote:
On Tue, Apr 27, 2
On Tue, Jun 01, 2021 at 03:34:50PM -0500, Pat Haugen wrote:
> Make sure link reg save MEM has frame alias set, to match other link reg
> save/restore code.
Okay for trunk and any backports (please do at least GCC 11). Thanks!
Segher
On Tuesday, 1 June 2021 21:12:18 CEST Jason Merrill wrote:
> On 5/28/21 3:42 AM, Matthias Kretz wrote:
> > On Friday, 28 May 2021 05:05:52 CEST Jason Merrill wrote:
> >> I'd think you could get the same effect from a hypothetical
> >>
> >> namespace [[gnu::diagnose_as]] stdx = std::experimental;
>
On Mon, 2021-03-15 at 12:14 -0500, Qing Zhao via Gcc-patches wrote:
> (CC’ing gcc-patch alias).
>
> Hi, Kees,
>
>
> > On Mar 12, 2021, at 3:55 PM, Kees Cook wrote:
> >
> > On Fri, Mar 12, 2021 at 03:35:28PM -0600, Qing Zhao wrote:
> > > Hi, Kees,
> > >
> > > I am looking at the structure padd
Make sure link reg save MEM has frame alias set, to match other link reg
save/restore code.
Bootstrap/regtest on powerpc64/powerpc64le with no new regressions. Ok for
trunk?
-Pat
2021-06-01 Pat Haugen
gcc/ChangeLog:
* config/rs6000/rs6000-logue.c (rs6000_emit_prologue): Use
On 01/06/21 8:10 pm, Jonathan Wakely wrote:
On 01/06/21 18:47 +0100, Jonathan Wakely wrote:
On 01/06/21 18:45 +0100, Jonathan Wakely wrote:
On 22/05/21 18:35 +0200, François Dumont wrote:
diff --git
a/libstdc++-v3/testsuite/23_containers/unordered_set/96088.cc
b/libstdc++-v3/testsuite/23_cont
On 5/27/21 2:53 PM, Jason Merrill wrote:
On 4/27/21 11:52 AM, Martin Sebor via Gcc-patches wrote:
On 4/27/21 8:04 AM, Richard Biener wrote:
On Tue, Apr 27, 2021 at 3:59 PM Martin Sebor wrote:
On 4/27/21 1:58 AM, Richard Biener wrote:
On Tue, Apr 27, 2021 at 2:46 AM Martin Sebor via Gcc-patc
arc700 is legacy and there's no active development for it, so switch to
latest hs38_linux as default
Signed-off-by: Vineet Gupta
---
gcc/config/arc/arc.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/config/arc/arc.h b/gcc/config/arc/arc.h
index bd1fe0abd7af..252241a858
Since times immemorial there has been const_int_rtx for all values from
-64 to 64, but only constm1_rtx..const2_rtx have been available for
convenient use. Change this, so that we can use all values in
{-64,...,64} in RTL easily. This matters, because then we we just say
if (XEXP (x, 1) == cons
On 5/28/21 3:42 AM, Matthias Kretz wrote:
On Friday, 28 May 2021 05:05:52 CEST Jason Merrill wrote:
On 5/27/21 6:07 PM, Matthias Kretz wrote:
On Thursday, 27 May 2021 23:15:46 CEST Jason Merrill wrote:
On 5/27/21 2:54 PM, Matthias Kretz wrote:
namespace Vir {
inline namespace foo {
On 6/1/21 9:20 AM, Robin Dapp wrote:
As you say, the logic is convoluted. Let's simplify it rather than make
it more convoluted. One possibility would be to change || to | to avoid
the shortcut, and then
bool note = lastalign > curalign;
if (note)
curalign = lastalign;
I went with your s
On 6/1/21 1:37 PM, Patrick Palka wrote:
In the case of value-initializing an object of class type T,
[dcl.init.general]/8 says:
- if T has either no default constructor ([class.default.ctor]) or
a default constructor that is user-provided or deleted, then the
object is default-initi
From: Andrew Pinski
This improves match_simplify_replace in phi-opt to handle the
case where there is one cheap preparation statement in the
middle basic block similar to xor_replacement and others.
This allows to remove xor_replacement too.
OK? Bootstrapped and tested on x86_64-linux-gnu with
On 01/06/21 18:47 +0100, Jonathan Wakely wrote:
On 01/06/21 18:45 +0100, Jonathan Wakely wrote:
On 22/05/21 18:35 +0200, François Dumont wrote:
diff --git a/libstdc++-v3/testsuite/23_containers/unordered_set/96088.cc
b/libstdc++-v3/testsuite/23_containers/unordered_set/96088.cc
new file mode 1
On 17/05/21 18:14 +0100, Jonathan Wakely wrote:
On 12/05/21 17:16 +0100, Jonathan Wakely wrote:
On 12/05/21 18:51 +0300, Antony Polukhin via Libstdc++ wrote:
ср, 12 мая 2021 г. в 18:38, Antony Polukhin :
ср, 12 мая 2021 г. в 17:44, Jonathan Wakely :
On 12/05/21 12:58 +0300, Antony Polukhin
On 6/1/2021 8:28 AM, Martin Liška wrote:
Simple leak fix.
Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
Ready to be installed?
Thanks,
Martin
gcc/ChangeLog:
* ipa-icf.h: Use auto_vec for memory_access_types.
OK
jeff
On 01/06/21 18:45 +0100, Jonathan Wakely wrote:
On 22/05/21 18:35 +0200, François Dumont wrote:
diff --git a/libstdc++-v3/testsuite/23_containers/unordered_set/96088.cc
b/libstdc++-v3/testsuite/23_containers/unordered_set/96088.cc
new file mode 100644
index 000..53bb754dab6
--- /dev/nul
On 22/05/21 18:35 +0200, François Dumont wrote:
diff --git a/libstdc++-v3/testsuite/23_containers/unordered_set/96088.cc
b/libstdc++-v3/testsuite/23_containers/unordered_set/96088.cc
new file mode 100644
index 000..53bb754dab6
--- /dev/null
+++ b/libstdc++-v3/testsuite/23_containers/unor
In the case of value-initializing an object of class type T,
[dcl.init.general]/8 says:
- if T has either no default constructor ([class.default.ctor]) or
a default constructor that is user-provided or deleted, then the
object is default-initialized;
- otherwise, the object is zero-ini
Hi Richard,
> -Original Message-
> From: Richard Earnshaw
> Sent: 13 April 2021 14:55
> To: Srinath Parvathaneni ; gcc-
> patc...@gcc.gnu.org
> Cc: Richard Earnshaw
> Subject: Re: [GCC][Patch] arm: Fix the mve multilib for the broken cmse
> support (pr99939).
>
>
>
> On 12/04/2021 14:
Hi All,
On passing +cdecp[0-7] extension to the -march string in command line options,
multilib linking is failing as mentioned in PR100856. This patch fixes this
issue by generating a separate -march string only for multilib comparison.
Regression tested on arm-none-eabi and found no regressions
On Jun 01 2021, Segher Boessenkool wrote:
> @@ -226,7 +226,7 @@ our lists into a single folder named INLIST.gcc:
>
>
> :0
> -* ^Sender: .*-ow...@gcc.gnu.org
> +* ^List-Id: .*<.*@gcc.gnu.org>$
That will never match.
List-Id: Gcc-patches mailing list
Andreas.
--
Andreas Schwab, sch...@lin
Hi! PR100750 reports a failure on my part to require the ELFv2 ABI for
one of the ROP tests. This fixes that.
Tested on powerpc64-linux-gnu, committed as obvious.
Thanks!
Bill
PR100750: Require ELFv2 ABI for ROP test
2021-06-01 Bill Schmidt
gcc/testsuite/
PR testsuite/100750
The three-argument form of ranges::advance is supposed to return the
difference between the second argument and the distance the iterator was
advanced. When a non-random-access iterator is not advanced (because it
already equals the sentinel) we were returning 0 rather than n - 0.
libstdc++-v3/Cha
We haven't had Sender: for a while now. Use the standard List-Id:
header in its place.
Committed.
Segher
---
htdocs/lists.html | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/htdocs/lists.html b/htdocs/lists.html
index 42a44ab83d9b..4ac5d5a84cd1 100644
--- a/htdocs/
On 5/21/21 6:43 PM, Segher Boessenkool wrote:
Yes, wrappers is a no-go. But you could just have added the features
you need to the generic code? Was there a technical reason not to do
that? It sounds useful in many places, not just here.
I agree it would be nice if all the gen* tools had li
Before C++20 added destroying operator delete, by the time we called
operator delete for a pointer, the object would already be gone. But that
isn't true for destroying delete. Since the optimizers' assumptions about
operator delete are based on either DECL_IS_REPLACEABLE_OPERATOR (which
already
On 01/06/2021 15:05, Martin Liška wrote:
Hello.
The patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98636#c20
where
target option restore can be called and arm_fp16_format should be reset
to ARM_FP16_FORMAT_NONE.
It fixes the ICE in the PR.
Can please ARM folks test me the patch
On 6/1/21 4:54 PM, Andrew MacLeod wrote:
On 6/1/21 10:51 AM, Aldy Hernandez wrote:
Andrew's last set of changes fixes the bootstrap problem on i686 when
global ranges are exported from evrp. The specific patch that fixes the
problem is 715914d3:
Author: Andrew MacLeod
Date: Mon May
On Wed, 2021-05-26 at 10:00 -0400, Jason Merrill via Gcc-patches wrote:
> On 5/13/21 9:07 PM, Jason Merrill wrote:
> > On 5/13/21 7:38 PM, Martin Sebor wrote:
> > > On 5/13/21 1:28 PM, Jason Merrill via Gcc-patches wrote:
> > > > Ping.
> > > >
> > > > On 4/28/21 9:32 AM, Jason Merrill wrote:
> > >
On 6/1/21 10:51 AM, Aldy Hernandez wrote:
Andrew's last set of changes fixes the bootstrap problem on i686 when
global ranges are exported from evrp. The specific patch that fixes the
problem is 715914d3:
Author: Andrew MacLeod
Date: Mon May 31 12:13:50 2021 -0400
Andrew's last set of changes fixes the bootstrap problem on i686 when
global ranges are exported from evrp. The specific patch that fixes the
problem is 715914d3:
Author: Andrew MacLeod
Date: Mon May 31 12:13:50 2021 -0400
Do not calculate new values when evaluatin
Simple leak fix.
Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
Ready to be installed?
Thanks,
Martin
gcc/ChangeLog:
* ipa-icf.h: Use auto_vec for memory_access_types.
---
gcc/ipa-icf.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/ipa
On 6/1/21 3:34 AM, Richard Biener wrote:
On Tue, Jun 1, 2021 at 3:38 AM Andrew MacLeod via Gcc-patches
wrote:
An ongoing issue is the the order we evaluate things in can affect
decisions along the way. As ranger isn't a fully iterative pass, we can
sometimes come up with different results if b
On 6/1/2021 7:29 AM, H.J. Lu via Gcc-patches wrote:
On Tue, Jun 1, 2021 at 6:25 AM Richard Biener
wrote:
On Tue, Jun 1, 2021 at 3:05 PM H.J. Lu wrote:
On Mon, May 31, 2021 at 11:54:53PM -0600, Jeff Law wrote:
On 5/31/2021 11:50 PM, Richard Sandiford wrote:
"H.J. Lu via Gcc-patches" wri
Hello.
As seen in the PR, one can easily corrupt line number information and
we can end up with a function that ends before it starts ;)
I'm adding a new warning for that instead of the ICE.
Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
Ready to be installed?
Thanks,
Ma
Hello.
The patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98636#c20 where
target option restore can be called and arm_fp16_format should be reset
to ARM_FP16_FORMAT_NONE.
It fixes the ICE in the PR.
Can please ARM folks test me the patch on a Arm machine?
Thanks,
Martin
gcc/ChangeLog
> PR target/100711
> * simplify-rtx.c (simplify_unary_operation_1):
> Simplify (vec_duplicate (not op)) to (not (vec_duplicate op)).
This is not a simplification. If we want to do this we need to document
this canonicalisation (in md.texi, "Insn Canonicalizations").
> +/* P
Hello,
On Tue, 1 Jun 2021, Martin Liška wrote:
> On 5/31/21 5:49 PM, Michael Matz wrote:
> > Hello Martin,
> >
> > On Mon, 31 May 2021, Martin Liška wrote:
> >
> >> I've made quite some progress with the porting of the documentation and
> >> I would like to present it to the community now:
> >>
I have some old numbers from late April.
VRP vs ranger was more difficult to compare than evrp, since the
gimple is different (ASSERT_EXPRs). What I did was run a late evrp
pass before each VRP pass and compared branches that were folded, for
an estimate.
On it's own VRP1 could fold 5482 branche
On 5/19/21 11:48 PM, Joern Wolfgang Rennecke wrote:
We set default for some target options in TARGET_OPTION_OPTIMIZATION_TABLE,
but these can be overridden by specifying the corresponding explicit
-mXXX / -mno-XXX options.
When a function bears the attribue
__attribute__ ((optimize("02")))
the ta
On Tue, Jun 01, 2021 at 04:32:42PM +0800, Hongtao Liu wrote:
[ no attachment to reply to ]
Please send this with either the patch actually inline, or as
attachment with content-disposition inline, no encoding, and a valid
text mimetype. So that people can see it, also on the archives, and
actual
On Tue, Jun 1, 2021 at 1:17 PM Martin Liška wrote:
>
> On 5/28/21 2:46 PM, Richard Biener wrote:
> > On Fri, May 28, 2021 at 11:48 AM Martin Liška wrote:
> >>
> >> Hi.
> >>
> >> There's a fallout after my revision
> >> ebd5e86c0f41dc1d692f9b2b68a510b1f6835a3e. I would like to analyze
> >> all ca
On 6/1/21 6:01 AM, Richard Biener wrote:
On Mon, 31 May 2021, Andrew MacLeod wrote:
On 5/28/21 11:25 AM, Richard Biener wrote:
This makes sure to perform final value replacement of constants
when we also are sure to propagate those, like in VRP. This avoids
spurious diagnostics when doing so
On Mon, May 31, 2021 at 11:54:53PM -0600, Jeff Law wrote:
>
>
> On 5/31/2021 11:50 PM, Richard Sandiford wrote:
> > "H.J. Lu via Gcc-patches" writes:
> > > On Mon, May 31, 2021 at 06:32:04AM -0700, H.J. Lu wrote:
> > > > On Mon, May 31, 2021 at 6:26 AM Richard Biener
> > > > wrote:
> > > > > On
This is the revised testsuite change with v2 adding a check for no
separate note for
__attribute__((aligned (32), aligned (4)).
Regards
Robin
diff --git a/gcc/testsuite/c-c++-common/Wattributes.c b/gcc/testsuite/c-c++-common/Wattributes.c
index 4ad90441b4d..a97e5ad5f74 100644
--- a/gcc/testsu
On Tue, Jun 1, 2021 at 6:25 AM Richard Biener
wrote:
>
> On Tue, Jun 1, 2021 at 3:05 PM H.J. Lu wrote:
> >
> > On Mon, May 31, 2021 at 11:54:53PM -0600, Jeff Law wrote:
> > >
> > >
> > > On 5/31/2021 11:50 PM, Richard Sandiford wrote:
> > > > "H.J. Lu via Gcc-patches" writes:
> > > > > On Mon, M
On 01.06.21 12:02, Jakub Jelinek wrote:
* gfortran.dg/gomp/scan-1.f90: Likewise.
* gfortran.dg/gomp/pr99928-3.f90: New test.
* gfortran.dg/gomp/taskloop-1.f90: New test.
Otherwise LGTM.
Jakub
Thanks for the review! – I just saw that I missed to include another
testcase –
As you say, the logic is convoluted. Let's simplify it rather than make
it more convoluted. One possibility would be to change || to | to avoid
the shortcut, and then
bool note = lastalign > curalign;
if (note)
curalign = lastalign;
I went with your suggestion in the attached v2. Regtest
On Tue, Jun 01, 2021 at 11:53:12AM +0200, Tobias Burnus wrote:
> Fortran/OpenMP: Support (parallel) master taskloop (simd) [PR99928]
>
> PR middle-end/99928
>
> gcc/fortran/ChangeLog:
>
> * dump-parse-tree.c (show_omp_node, show_code_node): Handle
> (parallel) master taskloop (
On Tue, Jun 1, 2021 at 3:05 PM H.J. Lu wrote:
>
> On Mon, May 31, 2021 at 11:54:53PM -0600, Jeff Law wrote:
> >
> >
> > On 5/31/2021 11:50 PM, Richard Sandiford wrote:
> > > "H.J. Lu via Gcc-patches" writes:
> > > > On Mon, May 31, 2021 at 06:32:04AM -0700, H.J. Lu wrote:
> > > > > On Mon, May 31
On Tue, Jun 1, 2021 at 1:25 PM Martin Liška wrote:
>
> On 6/1/21 9:42 AM, Richard Biener wrote:
> > On Tue, Jun 1, 2021 at 9:33 AM Martin Liška wrote:
> >>
> >> @Richi: Can you please reply to this email?
> >
> > Not sure what I should add here? Honza suggested to mangle the
> > promoted symbol
On 6/1/21 3:11 PM, Richard Biener wrote:
On Tue, Jun 1, 2021 at 1:17 PM Martin Liška wrote:
On 5/28/21 2:46 PM, Richard Biener wrote:
On Fri, May 28, 2021 at 11:48 AM Martin Liška wrote:
Hi.
There's a fallout after my revision ebd5e86c0f41dc1d692f9b2b68a510b1f6835a3e. I
would like to ana
On 6/1/21 9:42 AM, Richard Biener wrote:
On Tue, Jun 1, 2021 at 9:33 AM Martin Liška wrote:
@Richi: Can you please reply to this email?
Not sure what I should add here? Honza suggested to mangle the
promoted symbol names.
Sure and I sent a patch for that.
I don't
really like the idea to
Pushed as obvious where I verified that it's really enabled with -O3.
Martin
gcc/ChangeLog:
PR other/100826
* doc/invoke.texi: Mention that -fgcse-after-reload
is enabled with -O3.
---
gcc/doc/invoke.texi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Hi,
As described in patch summary, this fixes the wrong code issue by adding
overflow-ness
check for iv1.step - iv2.step.
Bootstrap and test on x86_64. Any comments?
Thanks,
bin
pr100740-20210525.txt
Description: Binary data
On Mon, 31 May 2021, Andrew MacLeod wrote:
> On 5/28/21 11:25 AM, Richard Biener wrote:
> > This makes sure to perform final value replacement of constants
> > when we also are sure to propagate those, like in VRP. This avoids
> > spurious diagnostics when doing so only from within SCCP which can
(PR 99928 is mainly about handling clauses in combined constructs.
but it sees some related commits like this one. The PR is not
fully fixed for C/C++ and for Fortran a bit more is missing.)
This patch adds '(parallel) master taskloop (simd)' support to gfortran.
For full parsing support for the
1 - 100 of 118 matches
Mail list logo