ram instead of
execute. That should simplify things slightly.
You should also most likely use concat here instead of
XNEWVEC/strcpy/strcat which will also simplify the code.
Like string = concat (DEFAULT_REAL_TARGET_MACHINE, "-", commands[0].prog);
I think this should be do
@0 @1)` directly the :c is not needed and genmatch will just generate
extra matching code that cannot be not get reached
Thanks,
Andrew.
>
>
>
> Pan
>
>
>
>
>
> From: Andrew Pinski
> Sent: Tuesday, May 21, 2024 7:40 PM
> To: Li, Pan2
> Cc: GCC Patches ;
On Wed, May 22, 2024 at 5:28 AM Richard Sandiford
wrote:
>
> Pengxuan Zheng writes:
> > This patch is a follow-up of r15-697-ga2e4fe5a53cf75 to also fold
> > vget_high_*
> > intrinsics to BIT_FILED_REF and remove the vget_high_* definitions from
> > arm_neon.h to use the new intrinsics framework
On Tue, May 14, 2024 at 10:27 AM Kyrill Tkachov
wrote:
>
> Hi Andrew,
>
> On Fri, May 3, 2024 at 8:50 PM Andrew Pinski wrote:
>>
>> This patch adds Qualcomm's new oryon-1 core; this is enough
>> to recongize the core and later on will add the tu
On Thu, May 23, 2024 at 8:01 AM Manolis Tsamis wrote:
>
> This pass detects cases of expensive store forwarding and tries to avoid them
> by reordering the stores and using suitable bit insertion sequences.
> For example it can transform this:
>
> strbw2, [x1, 1]
> ldr x0, [x1]
to it.
(propagate_with_phi): Add dce_ssa_names argument.
Update call to phiprop_insert_phi.
(pass_phiprop::execute): Update call to propagate_with_phi.
Call simple_dce_from_worklist if there was a change.
Signed-off-by: Andrew Pinski
---
gcc/tree-ssa-phiprop.cc
nu with no regressions.
gcc/ChangeLog:
* match.pd (bit_not_with_nop): Unconditionalize.
(maybe_cmp): Likewise.
(maybe_bit_not): New match pattern.
(`~X & X`): Use maybe_bit_not and add `:c` back.
(`~x ^ x`/`~x | x`): Likewise.
Signed-off-by: Andrew Pi
case.
gcc/testsuite/ChangeLog:
* gcc.dg/tree-ssa/bitops-8.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/generic-match-head.cc| 10 ++
gcc/gimple-match-head.cc | 13 +
gcc/match.pd | 4
gcc
:
* gcc.dg/tree-ssa/bitops-9.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/generic-match-head.cc| 6 --
gcc/gimple-match-head.cc | 6 --
gcc/testsuite/gcc.dg/tree-ssa/bitops-9.c | 15 +++
3 files changed, 23 insertions(+), 4 deletions
This just moves the tree scan earlier so we can detect the optimization and not
need to detect the vector splitting too.
Committed as obvious after a quick test.
gcc/testsuite/ChangeLog:
* gcc.dg/tree-ssa/bitops-9.c: Look at cdcde1 rather than optmization.
Signed-off-by: Andrew Pinski
On Mon, May 27, 2024 at 4:30 PM haochen.jiang
wrote:
>
> On Linux/x86_64,
>
> c5a7628470a7fb801ebeea82e16a4549db43bfa5 is the first bad commit
> commit c5a7628470a7fb801ebeea82e16a4549db43bfa5
> Author: Andrew Pinski
> Date: Sun May 26 17:59:21 2024 -07
d for aarch64-linux-gnu with no regressions.
gcc/ChangeLog:
PR sanitizer/115205
* asan.cc (asan_instrument): Call initialize_sanitizer_builtins
for hwasan.
Signed-off-by: Andrew Pinski
---
gcc/asan.cc | 1 +
1 file changed, 1 insertion(+)
diff --git a/gcc/asan.cc b/gcc/as
.
* config/or1k/or1k.opt.urls: Regenerate.
* config/riscv/riscv.opt.urls: Regenerate.
* config/rs6000/aix64.opt.urls: Regenerate.
* config/rs6000/linux64.opt.urls: Regenerate.
* config/sparc/sparc.opt.urls: Regenerate.
Signed-off-by: Andrew Pinski
---
gcc/common.opt.urls
On Thu, May 30, 2024 at 5:09 AM Filip Kastl wrote:
>
> Hi,
>
> This patch adds a transformation into the switch conversion pass --
> the "exponential index transform". This transformation can help switch
> conversion convert switches it otherwise could not. The transformation is
> intended for s
On Sun, Jun 2, 2024, 10:24 AM Andi Kleen wrote:
> Some of the cfg fixups in pro_and_epilogue for sibcalls were dependent on
> "optimize".
> Make them check cfun->tail_call_marked instead to handle the -O0 musttail
> case. This fixes the musttail test cases on arm targets.
>
> PR115255
>
>
On Mon, Jun 3, 2024 at 9:06 AM Peter0x44 wrote:
>
> 3 Jun 2024 4:14:28 pm Jonathan Wakely :
>
> > On Mon, 3 Jun 2024 at 14:36, Peter0x44 wrote:
> >>> +void
> >>> +std::breakpoint() noexcept
> >>> +{
> >>> +#if _GLIBCXX_HAVE_DEBUGAPI_H && defined(_WIN32) &&
> >>> !defined(__CYGWIN__)
> >>> + Debug
e -mabi=ilp32/-mabi=lp64 (riscv it
is more complex).
This most likely should be something more complex due to the above.
Maybe call it HOST_64_32 but even that seems wrong due to Aarch64
having ILP32 support and such.
What about HOST_64ABI_OPTS="-mabi=lp64"/HOST_32ABI_OPTS="-mabi=ilp32&q
On Fri, Aug 2, 2024 at 7:30 AM Jeff Law wrote:
>
>
>
> On 8/1/24 4:12 AM, Surya Kumari Jangala wrote:
> > lra: emit caller-save register spills before call insn [PR116028]
> >
> > LRA emits insns to save caller-save registers in the
> > inheritance/splitting pass. In this pass, LRA builds EBBs (Ex
On Fri, Aug 9, 2024 at 12:39 AM Jakub Jelinek wrote:
>
> Hi!
>
> These two spots were just non-standard, because they divided
> sizeof (omp_pragmas_simd) by sizeof (*omp_pragmas) and not
> the expected sizeof (*omp_pragmas_simd) and so weren't converted
> into ARRAY_SIZE. Both of the latter sizes
lier today.
Linaro CI's does `--disable-bootstrap` and there was no extra
testsuite failures with the patch.
So Linaro CI's is not catching all the bugs that a developer would
catch in the end. Because bootstrap is one of the normal requirements;
though usually only on one target.
Thanks,
Andrew
wasan.
(hwasan_finish_file): Likewise.
Signed-off-by: Andrew Pinski
---
gcc/asan.cc | 3 +++
1 file changed, 3 insertions(+)
diff --git a/gcc/asan.cc b/gcc/asan.cc
index 9e0f51b1477..5f262d54a3a 100644
--- a/gcc/asan.cc
+++ b/gcc/asan.cc
@@ -4276,6 +4276,7 @@ asan_instrument (void)
{
if (hwasan_sanit
/ChangeLog:
* g++.dg/tree-ssa/pr103660-0.C: New test.
* g++.dg/tree-ssa/pr103660-1.C: New test.
* gcc.dg/tree-ssa/pr103660-0.c: New test.
* gcc.dg/tree-ssa/pr103660-1.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/testsuite/g++.dg/tree-ssa/pr103660-0.C | 28
: New test.
* gcc.dg/tree-ssa/pr103660-4.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/match.pd | 10 +
gcc/testsuite/g++.dg/tree-ssa/pr103660-4.C | 35 ++
gcc/testsuite/gcc.dg/tree-ssa/pr103660-4.c | 43 ++
3 files
0-3.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/match.pd | 42 +++-
gcc/testsuite/g++.dg/tree-ssa/pr103660-2.C | 30 +++
gcc/testsuite/g++.dg/tree-ssa/pr103660-3.C | 30 +++
gcc/testsuite/gcc.dg/tree-ssa/pr103660-2.c
that is marked as Optimization that would might
also cause issues. We should check to make sure that also gets fixed
here too. Or error out for offloading targets can't have target
options with Optimization on them during the build.
Thanks,
Andrew Pinski
>
> Signed-off-by: Prathamesh Kulkarni
>
> Thanks,
> Prathamesh
t.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64.md | 37 ++
gcc/testsuite/gcc.target/aarch64/popcnt5.c | 19 +++
gcc/testsuite/gcc.target/aarch64/popcnt6.c | 19 +++
gcc/testsuite/gcc.target/aarch64/popcnt7.c | 18 +++
gcc
On Wed, Aug 14, 2024 at 2:21 PM Richard Sandiford
wrote:
>
> Andrew Pinski writes:
> > For popcount for bytes, we don't need the reduction addition
> > after the vector cnt instruction as we are only counting one
> > byte's popcount.
> > This changes the
From: Andrew Pinski
I didn't update the comment before factor_out_conditional_operation
correctly. this updates it to be correct and mentions unary operations
rather than just conversions.
Pushed as obvious.
gcc/ChangeLog:
* tree-ssa-phiopt.cc (factor_out_conditional_oper
(fold_builtin_bit_query): Don't expand double
`unsigned long long` typess if there is an optab entry for that
type.
Signed-off-by: Andrew Pinski
---
gcc/builtins.cc | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/gcc/builtins.cc b/gcc/builtins.cc
index 0b902896ddd..b4d51e
/ChangeLog:
* config/aarch64/aarch64.md (popcountti2): New define_expand.
gcc/testsuite/ChangeLog:
* gcc.target/aarch64/popcnt10.c: New test.
* gcc.target/aarch64/popcnt9.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64.md | 16
ch_op
instead of manually extracting from/creating the gimple.
gcc/testsuite/ChangeLog:
* gcc.target/i386/pr87007-5.c: Disable phi-opt.
Signed-off-by: Andrew Pinski
---
gcc/gimple-match-exports.cc | 14 +
gcc/gimple-match.h| 2 +
gcc/test
(pass_forwprop::execute): Update call to
fold_stmt.
Signed-off-by: Andrew Pinski
---
gcc/gimple-fold.cc | 43 +---
gcc/gimple-fold.h| 4 ++--
gcc/tree-ssa-forwprop.cc | 2 +-
3 files changed, 39 insertions(+), 10 deletions(-)
diff --git a/gcc/gimple
considering the binutils version is also mentioned as part of the
prerequisites for GCC with a newish version; I think mentioning it
also (which might get out of sync) in libstdc++ manual a little over
board.
See https://gcc.gnu.org/install/prerequisites.html .
Thanks,
Andrew Pinski
>
> > O
On Sun, Aug 18, 2024 at 3:42 PM Andrew Pinski wrote:
>
> On Sun, Aug 18, 2024 at 3:39 PM Eric Gallager wrote:
> >
> > On Sun, Aug 18, 2024 at 4:52 AM Gerald Pfeifer wrote:
> > >
> > > When I updated one of the links yesterday I noticed we have this obsol
On Sun, Aug 18, 2024 at 11:06 AM Jeff Law wrote:
>
>
>
> On 8/16/24 8:13 PM, Andrew Pinski wrote:
> > To start working on more with expressions with more than one operand,
> > converting
> > over to use gimple_match_op is needed.
> > The added side-effect here
functions).
gcc/testsuite/ChangeLog:
* gcc.dg/torture/pr116409-1.c: New test.
* gcc.dg/torture/pr116409-2.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/testsuite/gcc.dg/torture/pr116409-1.c | 7 +++
gcc/testsuite/gcc.dg/torture/pr116409-2.c | 7 +++
gcc/tree-ssa
.
PR tree-optimization/116412
gcc/ChangeLog:
* tree-ssa-phiopt.cc (factor_out_conditional_operation): Make sure
new_arg0
and new_arg1 are both gimple values.
gcc/testsuite/ChangeLog:
* gcc.dg/torture/pr116412-1.c: New test.
Signed-off-by: Andrew Pinski
---
gcc
-by: Andrew Pinski
---
gcc/testsuite/gcc.dg/torture/pr116409-1.c | 7 ++
gcc/testsuite/gcc.dg/torture/pr116409-2.c | 7 ++
gcc/tree-ssa-phiopt.cc| 30 +++
3 files changed, 34 insertions(+), 10 deletions(-)
create mode 100644 gcc/testsuite/gcc.dg
tract): Return false if op0
was not a SSA name nor a min invariant for
REALPART_EXPR/IMAGPART_EXPR/VCE
and BIT_FIELD_REF.
gcc/testsuite/ChangeLog:
* gcc.dg/torture/pr116412-1.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/gimple-match-exports.cc
-by: Andrew Pinski
---
gcc/config/aarch64/aarch64.md | 13 +++
gcc/testsuite/gcc.target/aarch64/popcnt10.c | 25 +
gcc/testsuite/gcc.target/aarch64/popcnt9.c | 25 +
3 files changed, 63 insertions(+)
create mode 100644 gcc/testsuite
On Sun, Aug 11, 2024 at 9:36 PM Andrew Pinski wrote:
>
> Sometimes initialize_sanitizer_builtins is not called before emitting
> the asan builtins with hwasan. In the case of the bug report, there
> was a path with the fortran front-end where it was not called.
> So l
On Tue, Aug 20, 2024 at 9:46 AM Richard Sandiford
wrote:
>
> Andrew Pinski writes:
> > On aarch64 (without !CSSC instructions), since popcount is implemented
> > using the SIMD instruction cnt,
> > instead of using two SIMD cnt (V8QI mode), it is better to use one 128
On Tue, Aug 20, 2024 at 11:18 AM Richard Sandiford
wrote:
>
> Richard Sandiford writes:
> > Andrew Pinski writes:
> >> When CSSC is not enabled, 128bit popcount can be implemented
> >> just via the vector (v16qi) cnt instruction followed by a reduction,
> >&
On Wed, Aug 21, 2024 at 1:56 AM Jonathan Wakely wrote:
>
> Tested x86_64-linux. Pushed to trunk.
>
> Probably worth backporting too. It could potentially cause new errors
> for people using arrays in std::variant, but that's forbidden by the
> standard.
It might be worth mentioning in porting_to
by wrapping arg0 with save_expr when it is not COMPLEX_EXPR.
gcc/testsuite/ChangeLog:
* gcc.dg/torture/pr116454-1.c: New test.
* gcc.dg/torture/pr116454-2.c: New test.
Signed-off-by: Andrew Pinski
Co-Authored-By: Richard Biener
---
gcc/fold-const.cc
On Thu, Aug 22, 2024 at 4:12 AM Richard Biener wrote:
>
> On Thu, 22 Aug 2024, Jennifer Schmitz wrote:
>
> > On 19 Aug 2024, at 21:02, Richard Sandiford
> > wrote:
> > >
> > > External email: Use caution opening links or attachments
> > >
> > >
> > > Jennifer Schmitz writes:
> > >> Thanks for t
ve library argument.
(add_to_obstack): Update call to is_directory.
(driver_handle_option): Likewise.
(spec_path): Likewise.
Signed-off-by: Andrew Pinski
---
gcc/gcc.cc | 24 ++--
1 file changed, 6 insertions(+), 18 deletions(-)
diff --git a/gcc/gcc.cc b/
.
PR bootstrap/105474
ChangeLog:
* configure: Regenerate.
* configure.ac: Error out if libstdc++ is not enabled
with bootstrapping.
Signed-off-by: Andrew Pinski
---
configure| 9 +
configure.ac | 9 +
2 files changed, 18 insertions(+)
diff
testsuite/116464
gcc/testsuite/ChangeLog:
* gcc.dg/torture/pr116420.c:
Signed-off-by: Andrew Pinski
---
gcc/testsuite/gcc.dg/torture/pr116420.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/testsuite/gcc.dg/torture/pr116420.c
b/gcc/testsuite/gcc.dg/torture
so that
`b[i]` is never 0. Use 1 in those places.
Signed-off-by: Andrew Pinski
---
gcc/testsuite/gcc.dg/vect/vect-mod-var.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/gcc/testsuite/gcc.dg/vect/vect-mod-var.c
b/gcc/testsuite/gcc.dg/vect/vect-mod-var.c
index eeed318c62b
On Thu, Aug 22, 2024 at 11:28 AM Andrew Pinski wrote:
>
> On Thu, Aug 22, 2024 at 4:12 AM Richard Biener wrote:
> >
> > On Thu, 22 Aug 2024, Jennifer Schmitz wrote:
> >
> > > On 19 Aug 2024, at 21:02, Richard Sandiford
> > > wrote:
> > > >
to the _BitInt lower.
Thanks,
Andrew Pinski
>
> Bootstrapped and tested on x86_64-unknown-linux-gnu, this will cause
> the following fallout which is similar to the aarch64 fallout in
> PR116463, complex SLP recognition being somewhat fragile. I'll track
> this there. Pushed.
&g
store flags.
gcc/testsuite/ChangeLog:
* gcc.dg/torture/pr116480-1.c: New test.
* gcc.dg/torture/pr116480-2.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/internal-fn.cc| 3 ++-
gcc/testsuite/gcc.dg/torture/pr116480-1.c | 8
gcc/testsuite/gcc.dg/to
e/libstdc++-prettyprinters/simple11.cc:31:
/home/apinski/src/upstream-gcc-isel/gcc/objdir/x86_64-pc-linux-gnu/libstdc++-v3/include/bits/stl_bvector.h:90:
note: declared private here
compiler exited with status 1
```
Noticed because of the new UNRESOLVED .
Thanks,
Andrew Pinski
>
> li
>> External email: Use caution opening links or attachments
> >>>>>>>
> >>>>>>>
> >>>>>>> On Fri, Aug 2, 2024 at 11:20 AM Kugan Vivekanandarajah
> >>>>>>> wrote:
> >>>>>>>>
While working on PR 114224, I found it would be useful to dump the
different costs of the expansion to make easier to understand why one
was chosen over the other.
Bootstrapped and tested on x86_64-linux-gnu.
Build and tested for aarch64-linux-gnu.
gcc/ChangeLog:
* internal-fn.cc (expand
xtra_costs, cortexa12_extra_costs,
cortexa15_extra_costs, v7m_extra_costs): Likewise.
gcc/testsuite/ChangeLog:
* gcc.target/aarch64/popcnt11.c: New test.
* gcc.target/aarch64/popcnt12.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64-cost-tables
(aarch64_rtx_addv_costs): New function.
(aarch64_rtx_costs): For unspec_addv, call aarch64_rtx_addv_costs.
For unspec_addv under a zero_extend, call aarch64_rtx_addv_costs.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64.cc | 35 +++
1 file
On Wed, Aug 28, 2024 at 12:26 AM Richard Biener
wrote:
>
> On Wed, Aug 28, 2024 at 6:34 AM Andrew Pinski
> wrote:
> >
> > While working on PR 114224, I found it would be useful to dump the
> > different costs of the expansion to make easier to understand why one
&g
those are ok
> > for trunk, thanks.
>
> Gah, sorry -- those got lost in the rebase. Is it OK to commit this
> together with the doc changes included as per the previous patch?
I am getting a new ERROR after this. Maybe you didn't notice this
since you were looking for new
internal-fn.cc (expand_POPCOUNT): Use OPTAB_WIDEN for PLUS and
XOR/AND expansion.
gcc/testsuite/ChangeLog:
* gcc.target/aarch64/popcnt-eq-1.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/internal-fn.cc| 4 +-
.../gcc.target/aarch64/popcn
t; > } else {
> > - set up_config(start) {{^([a-zA-Z_]\S+):$}}
> > + set up_config(start) {{^([a-zA-Z_]\S*):$}}
> > }
> >
> > # Regexp for the end of a function definition.
>
> This part is ok, thanks.
Note the issue with single function na
On Thu, Jun 6, 2024 at 6:02 AM Bert Wesarg wrote:
>
> Dear David,
>
> On Tue, May 28, 2024 at 10:07 PM David Malcolm wrote:
> >
> > No functional change intended.
> >
> > Successfully bootstrapped & regrtested on x86_64-pc-linux-gnu.
> > Pushed to trunk as r15-874-g9bda2c4c81b668.
> >
> > libcpp/
eader file is installed.
gcc/ChangeLog:
* Makefile.in (CPPLIB_H): Add label-text.h.
Signed-off-by: Andrew Pinski
---
gcc/Makefile.in | 1 +
1 file changed, 1 insertion(+)
diff --git a/gcc/Makefile.in b/gcc/Makefile.in
index c983b0c102a..f5adb647d3f 100644
--- a/gcc/Makefile.in
+++
On Thu, Jun 6, 2024 at 9:00 AM David Malcolm wrote:
>
> On Thu, 2024-06-06 at 08:40 -0700, Andrew Pinski wrote:
> > On Thu, Jun 6, 2024 at 6:02 AM Bert Wesarg
> > wrote:
> > >
> > > Dear David,
> > >
> > > On Tue, May 28, 2024 at 10:07 PM Dav
On Sun, Jun 9, 2024 at 2:22 PM Gerald Pfeifer wrote:
>
> On Fri, 13 Aug 2021, Andreas Tobler via Gcc-patches wrote:
> > I would like to commit the attached patch to trunk and after a settling
> > period also to all open branches.
> > Is this ok?
>
> Our MAINTAINERS file has the following entry:
>
/ChangeLog:
* config/aarch64/aarch64.md (popcountqi2): New pattern.
gcc/testsuite/ChangeLog:
* gcc.target/aarch64/popcnt5.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64.md | 26 ++
gcc/testsuite/gcc.target/aarch64/popc
/115388
* gcc.dg/torture/pr115388.c: Use `signed char` directly instead
of plain `char`.
Signed-off-by: Andrew Pinski
---
gcc/testsuite/gcc.dg/torture/pr115388.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/testsuite/gcc.dg/torture/pr115388.c
b/gcc/testsuite
/popcnt5.c: New test.
* gcc.target/aarch64/popcnt6.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64.md | 52 +++---
gcc/testsuite/gcc.target/aarch64/popcnt5.c | 19
gcc/testsuite/gcc.target/aarch64/popcnt6.c | 19
gcc
On Mon, Jun 10, 2024, 11:20 PM wrote:
> From: Pan Li
>
> The test cases of pr115387 are target independent, at least x86
> and riscv are able to reproduce. Thus, move these cases to
> the gcc.dg/torture.
>
> The below test suites are passed.
> 1. The rv64gcv fully regression test.
> 2. The x8
On Mon, May 20, 2024 at 11:08 PM Richard Biener
wrote:
>
> On Mon, May 20, 2024 at 11:37 PM Andrew Pinski (QUIC)
> wrote:
> >
> > > -Original Message-
> > > From: Richard Biener
> > > Sent: Sunday, May 19, 2024 11:55 AM
> > > To: Andr
d-off-by: Andrew Pinski
---
gcc/jit/jit-recording.cc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/jit/jit-recording.cc b/gcc/jit/jit-recording.cc
index 68a2e860c1f..70830e34965 100644
--- a/gcc/jit/jit-recording.cc
+++ b/gcc/jit/jit-recording.cc
@@ -19,6 +19,7 @@ along
On Tue, Jun 11, 2024 at 12:42 PM Andrew Pinski wrote:
>
> Just like r13-6662-g0e6f87835ccabf but this time for jit/jit-recording.cc.
>
> Pushed as obvious after a quick build to make sure jit still builds.
Backported also to GCC 14 and GCC 13.
Thanks,
Andrew
>
>
*/
(simplify
(bitop:c (rbitop:c @2 @1) @0)
(with { bool wascmp; }
(if (bitwise_inverted_equal_p (@0, @2, wascmp)
&& (!wascmp || element_precision (type) == 1))
(bitop @0 @1
The problem is bitwise_inverted_equal_p does not see that:
c.0_4 = (signed char) _1;
_5
t_op): Likewise.
(SVE_INT_UNARY): Remove UNSPEC_RBIT.
(optab): Likewise.
(sve_int_op): Likewise.
(min_elem_bits): Likewise.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64-simd.md | 3 +--
gcc/config/aarch64/aarch64-sve-builtins-base.cc | 2 +-
gcc/config/
-g948dbc5ee45f9f ). Since this is only working
around this one testcase this way.
Thanks,
Andrew Pinski
>
>
> for gcc/ChangeLog
>
> * doc/sourcebuild.texi (dg-additional-sources): Add linkonly.
>
> for gcc/testsuite/ChangeLog
>
>
_bitwise_equal_p.
* match.pd (maybe_truncate): New match pattern.
gcc/testsuite/ChangeLog:
* gcc.dg/tree-ssa/bitops-10.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/gimple-match-head.cc | 17 +---
gcc/match.pd | 7 +
gcc/testsuite
On Wed, Jun 12, 2024 at 1:32 PM Jason Merrill wrote:
>
> Tested x86_64-pc-linux-gnu, applying to trunk.
>
> -- 8< --
>
> A sample implementation of module std was breaking because the exports
> included 'using std::operator&' twice. Since Nathaniel's r15-964 for
> PR114867, the first using added
(expand_widen_pattern_expr): Likewise.
* optabs.h (expand_widen_pattern_expr): Likewise.
* expr.cc (expand_expr_real_2): Likewise
(do_store_flag): Likewise. Remove incorrect store to ops->code.
Signed-off-by: Andrew Pinski
---
gcc/expr.cc | 8
gcc/expr.h| 4 +++-
rays (but maybe that is not tracked).
Thanks,
Andrew Pinski
> This applies to OMP SIMD in-branch lowered
> functions result array which for the testcase otherwise prevents
> vectorization with SSE and for AVX and AVX512 ends up with spurious
> .MASK_STORE to the stack surviving.
>
ate can still use the built-in even though the
> name '__is_pointer' has been declared as an identifier.
>
> So Clang matches GCC and it works fine. Good!
Actually clang does not work the same as GCC in the end; it is much
more inconsistent in what it allows.
See https://gcc.gnu.org/b
gcc/ChangeLog:
* config/aarch64/aarch64.h (machine_function): Fix the size
of reg_is_wrapped_separately.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/config/aarch64/aarch64.h b/gcc/config/aarc
On Mon, Jun 17, 2024, 5:59 AM Tamar Christina
wrote:
> Hi,
>
> > -Original Message-
> > From: Pengxuan Zheng
> > Sent: Friday, June 14, 2024 12:57 AM
> > To: gcc-patches@gcc.gnu.org
> > Cc: Pengxuan Zheng
> > Subject: [PATCH v3] aarch64: Add vector popcount besides QImode
> [PR113859]
>
ending
on the name.
Thanks,
Andrew Pinski
>
> --
> Eric Botcazou
/testsuite/ChangeLog:
* gcc.target/aarch64/sve/pr97405-1.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/testsuite/gcc.target/aarch64/sve/pr97405-1.c | 13 +
1 file changed, 13 insertions(+)
create mode 100644 gcc/testsuite/gcc.target/aarch64/sve/pr97405-1.c
diff --git a/gcc
ne in this patch set.
Note there is a core which uses the variant selection so we can't
remove the code there.
Andrew Pinski (2):
aarch64: make thunderxt88p1 an alias of thunderxt88
aarch64: Add comment about thunderxt81/t83 being aliases
gcc/config/aarch64/aarch64-cores.def | 6 +++---
Since these were already aliases just make it clear on that.
gcc/ChangeLog:
* config/aarch64/aarch64-cores.def: Add comment
saying thunderxt81/t83 are aliases of octeontx81/83.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64-cores.def | 1 +
1 file changed, 1
rch64-cores.def (thunderxt88p1): Make an alias of
thunderxt88 and
move below thunderxt88.
* config/aarch64/aarch64-tune.md: Regenerate.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64-cores.def | 5 ++---
gcc/config/aarch64/aarch64-tune.md | 2 +-
2 files chang
On Mon, Jun 17, 2024 at 11:25 PM Pengxuan Zheng wrote:
>
> This patch improves GCC’s vectorization of __builtin_popcount for aarch64
> target
> by adding popcount patterns for vector modes besides QImode, i.e., HImode,
> SImode and DImode.
>
> With this patch, we now generate the following for V8
abled.
Note it is a (minor) missed optimization regression so the impact
looks to be small.
I am not sure if people have written code with this pattern, it
requires vectors and it fails only on targets where there is no vector
support enabled.
https://gcc.gnu.org/bugzilla/show_bu
ations_1): Handle PAREN_EXPR like
NEGATE_EXPR. And call expand_complex_paren for PAREN_EXPR.
gcc/testsuite/ChangeLog:
* gcc.dg/vect/pr68855.c: New test.
* gfortran.dg/vect/pr68855.f90: New test.
Signed-off-by: Andrew Pinski
---
gcc/testsuite/gcc.dg/vect/pr68855.c
&& target_supports_op_p (unsigned_type_for (type),
> +RSHIFT_EXPR, optab_scalar))
> +(with
> + {
> + unsigned int prec = element_precision (type);
> + tree utype = unsigned_type_for (type);
> + }
> +(view_convert:type
&
tps://gcc.gnu.org/bugzilla/show_bug.cgi?id=951 .
Thanks,
Andrew Pinski
>
> make pdf tested, pushed.
>
> * doc/passes.texi: Remove references to no longer existing
> passes.
> ---
> gcc/doc/passes.texi | 70 ++---
> 1 fi
On Fri, Jun 21, 2024 at 1:11 PM Jeff Law wrote:
>
>
>
> On 6/17/24 3:53 AM, Richard Sandiford wrote:
> > This patch makes target-independent code use force_subreg instead
> > of simplify_gen_subreg in some places. The criteria were:
> >
> > (1) The code is obviously specific to expand (where new
* value-range.h (class int_range): Mark as final.
(class prange): Likewise.
(class frange): Likewise.
Signed-off-by: Andrew Pinski
---
gcc/value-range.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/gcc/value-range.h b/gcc/value-range.h
index 4
t; > * match.pd: Optimization for above mentioned pattern.
Oh I just noticed the changelog should be improved too.
Eikansh,
The wording there needs to be independent from the commit message as
it gets added to ChangeLog and has no real reference back to the
commit message.
On Mon, Jun 24, 2024 at 7:20 PM Andrew MacLeod wrote:
>
>
> On 6/22/24 09:15, Richard Biener wrote:
> > On Fri, Jun 21, 2024 at 3:02 PM Andrew MacLeod wrote:
> >> This patch adds
> >>
> >> --param=vrp-block-limit=N
> >>
> >> When the basic block counter for a function exceeded 'N' , VRP is
On Mon, Jun 24, 2024 at 7:35 PM Andrew Pinski wrote:
>
> On Mon, Jun 24, 2024 at 7:20 PM Andrew MacLeod wrote:
> >
> >
> > On 6/22/24 09:15, Richard Biener wrote:
> > > On Fri, Jun 21, 2024 at 3:02 PM Andrew MacLeod
> > > wrote:
> > >>
: Andrew Pinski
---
gcc/c-family/c.opt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/c-family/c.opt b/gcc/c-family/c.opt
index b067369fa7e..864ef4e3b3d 100644
--- a/gcc/c-family/c.opt
+++ b/gcc/c-family/c.opt
@@ -1137,7 +1137,7 @@ C ObjC Var
: Copied from decomp22.C
and wrap tuple_size/tuple_element inside std namespace.
Signed-off-by: Andrew Pinski
---
gcc/cp/decl.cc| 16 +---
gcc/testsuite/g++.dg/cpp1z/decomp22.C | 2 +-
gcc/testsuite/g++.dg/cpp1z/decomp61.C | 53 +++
gcc
301 - 400 of 2416 matches
Mail list logo
