set = new_alias_set ();
return set;
}
```
And GOT_ALIAS_SET is used in more than COFF areas. Can you please fix
this definition?
Thanks,
Andrew Pinski
>
> Regards,
> Evgeny
>
> Evgeny Karpov (6):
> Move mingw_* declarations to the mingw folder
> Extract ix86 dllimp
iltin_isfinite but the patch only handles
> __builtin_isinf.
There is a patch for that; all 3 were pinged this morning:
isinf: https://gcc.gnu.org/pipermail/gcc-patches/2024-July/657879.html
isnormal: https://gcc.gnu.org/pipermail/gcc-patches/2024-July/657880.html
isfinite: https://gcc.gnu.org/pi
o be adding the cond version of them since at least for
aarch64 SVE, bic (andc) can be conditional.
Thanks,
Andrew Pinski
>
> BR,
> Kewen
> -
>
> PR tree-optimzation/115659
>
> gcc/ChangeLog:
>
> * config/rs6000/rs6000-builtins.def: Update some bif e
and others).
Other than that it looks good but I can't approve it.
Thanks,
Andrew Pinski
>
> Signed-off-by: Kugan Vivekanandarajah
>
> Bootstrapped and regression test on aarch64-linux-gnu. Is this OK for trunk?
> Thanks,
> Kugan
>
>
&g
fine_insn "*tf_to_fprx2_0"
[(set (subreg:DF (match_operand:FPRX2 0 "nonimmediate_operand" "+f") 0)
(subreg:DF (match_operand:TF1 "general_operand" "v") 0))]
...
(define_insn "*tf_to_fprx2_1"
[(set (subreg:DF
gt; + unsigned a : 4;
> +};
> +
> +static unsigned min_u (unsigned a, unsigned b)
> +{
> + return (b < a) ? b : a;
> +}
> +
> +__attribute__((noipa))
> +void bug (e * v, unsigned def, unsigned use) {
> + e & defE = *v;
> + defE.a = min_u (use + 1, 0xf);
> +}
> +
> +__a
tion.
```
int f1(int a)
{
return __builtin_ctz(__builtin_abs(a));
}
```
Should also be handled. Though this might be good to handle in the
backprop pass (gimple-ssa-backprop.cc) which handles sign changes like
this already.
Thanks,
Andrew Pinski
>
> gcc/testsuite/ChangeLog
> * gcc.dg/fold-ctz-1.c: New test case.
>
>
> Thanks in advance,
> Roger
> --
>
On Mon, Jul 22, 2024 at 7:41 PM Kewen.Lin wrote:
>
> Hi Andrew,
>
> on 2024/7/23 08:09, Andrew Pinski wrote:
> > On Sun, Jun 30, 2024 at 11:17 PM Kewen.Lin wrote:
> >>
> >> Hi,
> >>
> >> As PR115659 shows, assuming c = x CMP y, there are som
functions, ANDC/IORC to ANDN/IORN.
Signed-off-by: Andrew Pinski
---
gcc/config/rs6000/rs6000-builtins.def | 44 +--
gcc/config/rs6000/rs6000-string.cc| 2 +-
gcc/config/rs6000/rs6000.md | 4 +--
gcc/doc/md.texi | 8 ++---
gcc/gimple-isel.cc
-fdelayed-template-parsing . Does it make sense to add a similar
one for GCC? See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116064
and https://github.com/llvm/llvm-project/issues/96859 for reference.
Thanks,
Andrew Pinski
>
> -- >8 --
>
> When the scope of a qualified name is t
entry.
I have a patch which calls "string trim" on v3_additional_files before
calling split and that seems to fix the problem. I will post it once
my testsuite run is done.
Thanks,
Andrew Pinski
>
> libstdc++-v3/ChangeLog:
>
> * testsuite/lib/libstdc++.exp (libstdc++_init): Do not
7;t look at why it works for native
testing though).
Ran a full libstdc++ testsuite using a sim board for testing.
libstdc++-v3/ChangeLog:
* testsuite/lib/libstdc++.exp (v3_target_compile): Call
string trim on v3_target_compile before calling split.
Signed-off-by: Andrew Pinski
--
/testsuite/ChangeLog:
* g++.target/aarch64/vect_cmp-1.C: New test.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64-simd.md| 20 +-
gcc/testsuite/g++.target/aarch64/vect_cmp-1.C | 37 +++
2 files changed, 47 insertions(+), 10 deletions(-)
create
with no regressions.
gcc/ChangeLog:
* config/aarch64/aarch64-sve-builtins-base.cc (svbic_impl::expand):
Update
to use andn optab instead of using code_for_aarch64_bic.
* config/aarch64/aarch64-sve.md (@aarch64_bic): Rename to ...
(andn3): This.
Signed-off-by: Andrew
Likewise.
* gcc.dg/tree-ssa/pr96671-1.c: Likewise.
Signed-off-by: Andrew Pinski
---
gcc/match.pd | 17 ++
gcc/testsuite/gcc.dg/tree-ssa/bitops-1.c | 10 +++---
gcc/testsuite/gcc.dg/tree-ssa/bitops-6.c | 12 +++
gcc/testsuite/gcc.dg/
64.md (*_one_cmpl3): Rename
to ...
(n3): This.
(*_one_cmplsidi3_ze): Rename to ...
(*nsidi3_ze): this.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64.md | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/gcc/config/aarch64/aarch64.md
attern.
gcc/testsuite/ChangeLog:
* gcc.target/aarch64/bic-cst-2.c: New test.
* gcc.target/aarch64/bic_simd-2.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/match.pd | 10 ++
gcc/testsuite/gcc.target/aarch64/bic-cst-2.c | 31 +
g
On Wed, Jul 24, 2024 at 11:22 PM Kyrylo Tkachov wrote:
>
>
>
> > On 25 Jul 2024, at 04:14, Andrew Pinski wrote:
> >
> > External email: Use caution opening links or attachments
> >
> >
> > This renames the patterns orn3 to iorn3 so it
> >
.c: New test.
* gcc.target/aarch64/sve/dup-insr-2.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/match.pd | 17
.../gcc.target/aarch64/sve/dup-insr-1.c | 26 +++
.../gcc.target/aarch64/sve/dup-insr-2.c | 26
On Thu, Jul 25, 2024 at 12:01 PM Adhemerval Zanella Netto
wrote:
>
>
>
> On 17/07/24 14:00, Andrew Pinski wrote:
> > On Wed, Jul 17, 2024 at 5:24 AM Richard Biener
> > wrote:
> >>
> >> On Tue, Jul 16, 2024 at 3:36 PM Eikansh Gupta
> >> wrote
.
PR target/116065
gcc/ChangeLog:
* config/aarch64/aarch64.opt (mearly-ra=): Mark as Optimization rather
than Save.
gcc/testsuite/ChangeLog:
* gcc.target/aarch64/sve/target_optimization-1.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64.opt
On Thu, Jul 25, 2024 at 10:20 AM Richard Biener
wrote:
>
>
>
> > Am 25.07.2024 um 17:56 schrieb Andrew Pinski :
> >
> > It was noticed if we have `.VEC_SHL_INSERT ({ 0, ... }, 0)` it was not
> > being
> > simplified to just `{ 0, ... }`. This was gener
On Thu, Jul 25, 2024 at 5:16 AM Richard Biener
wrote:
>
> On Thu, Jul 25, 2024 at 4:16 AM Andrew Pinski
> wrote:
> >
> > To better create rtl directly from gimple, we can use
> > these already internal functions from the gimple.
> >
> > That is simplify
On Thu, Jul 25, 2024 at 4:09 PM Sam James wrote:
>
> Andi Kleen writes:
>
> > From: Andi Kleen
> >
> > - Run the target_effective tail_call checks without optimization to
> > match the actual test cases.
> > - Add an extra check for external tail calls to handle targets like
> > powerpc that can
el.cc (pass_gimple_isel::execute): Factor out
duplicate comparisons out to ...
(duplicate_comparison): New function.
Signed-off-by: Andrew Pinski
---
gcc/gimple-isel.cc | 66 --
1 file changed, 35 insertions(+), 31 deletions(-)
diff --
ff-by: Andrew Pinski
---
gcc/gimple-isel.cc| 5 +
.../gcc.dg/tree-ssa/dup_compare_cond-1.c | 19 +++
.../gcc.dg/tree-ssa/dup_compare_cond-2.c | 19 +++
.../gcc.dg/tree-ssa/dup_compare_cond-3.c | 19 +++
4
aller.
(pass_gimple_isel::execute): Don't check how many uses the
comparison had and call maybe_duplicate_comparison instead of
duplicate_comparison.
Signed-off-by: Andrew Pinski
---
gcc/gimple-isel.cc | 38 --
1 file changed, 24 inserti
y): Use prop_set_p.
(c_parser_gimple_compound_statement): Use prop_set_p.
(c_parser_gimple_goto_stmt): Use prop_set_p.
(c_parser_gimple_if_stmt): Use prop_set_p.
Signed-off-by: Andrew Pinski
---
gcc/c-family/c-gimplify.cc | 2 +-
gcc/c/gimple-parser.cc | 20 +--
On Sat, Jul 27, 2024 at 1:38 AM Lulu Cheng wrote:
>
> gcc/ChangeLog:
>
> * config/loongarch/loongarch.md (n): Rename to ...
> (n3): This.
Thanks for doing this for loongarch. Once I finish up my patch set;
loongarch should get benefit. Also it might be useful after my patch
set g
gt; * config/loongarch/lsx.md (vandn3): Rename to ...
> (andn3): This. .
> (vorn3): Rename to ...
> (iorn3): This.
You might want to add a testcase like I did for aarch64:
https://gcc.gnu.org/pipermail/gcc-patches/2024-July/658219.html .
Thanks
s provided.
* tree-ssa-sink.cc (pass_data_sink_code): Remove comment about
PROP_no_crit_edges.
Signed-off-by: Andrew Pinski
---
gcc/function.h | 2 +-
gcc/passes.cc| 2 --
gcc/tree-cfg.cc | 2 +-
gcc/tree-ssa-sink.cc | 2 --
4 files changed, 2 insertions(+), 6 deletions(-)
use an array.
* ir-prop.def: New file.
Signed-off-by: Andrew Pinski
---
gcc/function.h | 47 ---
gcc/ir-prop.def | 45 +
gcc/passes.cc | 45 +++--
3 fi
d Biener
> > > wrote:
> > > >
> > > > On Tue, Jul 23, 2024 at 10:27 AM Kugan Vivekanandarajah
> > > > wrote:
> > > > >
> > > > > On Tue, Jul 23, 2024 at 10:35 AM Andrew Pinski
> > > > > wrote:
> > &g
e-end/116134
gcc/ChangeLog:
* match.pd (`(a ? x : y) eq/ne (b ? x : y)`): Check that
a and b types match.
(`(a ? x : y) eq/ne (b ? y : x)`): Likewise.
gcc/testsuite/ChangeLog:
* gcc.dg/torture/pr116134-1.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/mat
* g++.dg/tree-ssa/pr50.C: Add extra checks on the test.
* gcc.dg/tree-ssa/pr50-1.c: Likewise.
* gcc.dg/tree-ssa/pr50.c: Likewise.
* g++.dg/torture/pr116120-1.c: New test.
* g++.dg/torture/pr116120-2.c: New test.
Signed-off-by: Andrew Pinsk
;& !defined(__AVX2__)
> /* A version of the fast scanner using SSE 4.2 vectorized string insns. */
>
> static const uchar *
> @@ -425,6 +438,71 @@ search_line_sse42 (const uchar *s, const uchar *end)
> #define search_line_sse42 search_line_sse2
> #endif
>
> +#ifdef H
If you need LP64E to please notify GCC project via
https://gcc.gnu.org/PR116152";);
}
```
Or something to that effect, that is use the note function. The if
around the warning function is to have the note only emit if the
warning is emitted.
Thanks,
Andrew Pinski
> +
>/* Zfi
On Wed, Jul 31, 2024 at 5:05 AM Richard Biener
wrote:
>
> On Tue, Jul 30, 2024 at 5:26 PM Andrew Pinski
> wrote:
> >
> > When this pattern was converted from being only dealing with 0/-1, we
> > missed that if `e == f` is true
> > then the optimization is w
s using the
full path of the object file. I am not sure this is a problem we need
to worry about right now but it might be one in the future and one
which we should document on coding conventions on naming things that
will be included in OBJS (which puts it into libbackend.a).
Thanks,
.dg/tree-ssa/pr50-1.c: Likewise.
* gcc.dg/tree-ssa/pr50.c: Likewise.
* g++.dg/torture/pr116120-1.C: New test.
* g++.dg/torture/pr116120-2.C: New test.
Signed-off-by: Andrew Pinski
---
gcc/match.pd | 23 ++
gcc/testsuite/g++.dg/t
f the statement was a debug statement.
gcc/testsuite/ChangeLog:
* c-c++-common/torture/pr116156-1.c: New test.
Signed-off-by: Andrew Pinski
---
.../c-c++-common/torture/pr116156-1.c | 30 +++
gcc/tree-ssa-forwprop.cc | 16 +-
2
ll to '(std::shared_ptr) (int)'
compiler exited with status 1
UNRESOLVED: libstdc++-prettyprinters/shared_ptr.cc compilation failed
to produce executable
Thanks,
Andrew Pinski
>
> Tested x86_64-linux. Pushed to trunk.
>
> -- >8 --
>
> We don't need to include
r_pat.
(output_add_clobbers): Update call to gen_exp for the changed
clobber_pat.
Signed-off-by: Andrew Pinski
---
gcc/genemit.cc | 38 +++---
1 file changed, 23 insertions(+), 15 deletions(-)
diff --git a/gcc/genemit.cc b/gcc/genemit.cc
index 98d0477424b..5d3d10f5061 1
On Thu, Aug 1, 2024 at 11:55 PM Richard Biener
wrote:
>
> On Thu, Aug 1, 2024 at 10:40 PM Andrew Pinski
> wrote:
> >
> > The problem here is that when forwprop does a copy prop, into a statement,
> > we mark the uses of that statement as possibly need to b
(ashift:DI
(reg:DI 0 ax [690]) (const_int 3 [0x3])) (label_ref:DI 1620))` as not
trapping and allowing it to be moved before the check of ax being in
the range [0..2] and we have eax being (unsigned long)(unsigned int)-9
in value. So we get a bogus address which will trap. I put my findings
in PR 1162
it_live_ranges_for_shrink_wrap): For the uses loop,
only look at non-debug insns.
gcc/testsuite/ChangeLog:
* g++.dg/torture/pr116179-1.C: New test.
Signed-off-by: Andrew Pinski
---
gcc/ira.cc| 4 +++-
gcc/testsuite/g++.dg/torture/pr116179-1.C |
On Sat, Aug 3, 2024 at 3:31 PM Jeff Law wrote:
>
>
>
> On 8/2/24 11:31 PM, Sam James wrote:
> > Andrew Pinski writes:
> >
> >> Late_combine exposed this latent bug in split_live_ranges_for_shrink_wrap.
> >> What it did was copy-prop regno 151 from reg
/ChangeLog:
* config/sh/sh.cc (sh_recog_treg_set_expr): Don't call make_insn_raw,
make the insn with a fake uid.
gcc/testsuite/ChangeLog:
* c-c++-common/torture/pr116189-1.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/config/sh/sh.cc
On Tue, May 28, 2024 at 8:28 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 Mon, Aug 5, 2024 at 3:23 AM Roger Sayle wrote:
>
>
> This patch refactors ashrv2di RTL expansion into a function so that it may
> be reused by a pre-reload splitter, such that DImode right shifts may be
> considered candidates during the Scalar-To-Vector (STV) pass. Currently
> DImode arithmet
eally be in an anonymous
namespaces but they were missing.
Note there is some unused classes/instances I noticed after putting
them in an anonymous namespace so I `#if 0`ed then. Should we just
remove them instead? They were the pointer related ops which was just
added too.
Andrew Pinski (5):
Since there will be only one instance of this class
alive at any time, it is better to declare this class as final.
Bootstrapped and tested on x86_64-linux-gnu.
gcc/ChangeLog:
* range-op.h (class range_op_table): Declare as a final class.
Signed-off-by: Andrew Pinski
---
gcc/range
): Make operator[] const.
Have singleton return const reference.
* range-op.cc (range_op_table::singleton): Update return type.
(operator_table): Make const.
Signed-off-by: Andrew Pinski
---
gcc/range-op.cc | 4 ++--
gcc/range-op.h | 4 ++--
2 files changed, 4 insertions
x86_64-linux-gnu with no regressions.
gcc/ChangeLog:
* range-op.h (class range_op_table): Value initialize m_range_tree.
Signed-off-by: Andrew Pinski
---
gcc/range-op.h | 7 ++-
1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/gcc/range-op.h b/gcc/range-op.h
index
leton.
Make most functions private (rather than protected).
Make ctor private.
Signed-off-by: Andrew Pinski
---
gcc/range-op.cc | 19 ++-
gcc/range-op.h | 5 +++--
2 files changed, 17 insertions(+), 7 deletions(-)
diff --git a/gcc/range-op.cc b/gcc/range-op.cc
index c576f6
(class pointer_min_max_operator): #if out and mark as final.
(class pointer_and_operator): Likewise.
(class pointer_or_operator): Likewise.
(class operator_pointer_diff): Wrap with anonymous
namespace and mark as final.
Signed-off-by: Andrew Pinski
---
gcc/range-op-flo
: Add -save-temps to the
options.
Signed-off-by: Andrew Pinski
---
gcc/testsuite/gcc.target/aarch64/simd/vmmla.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/testsuite/gcc.target/aarch64/simd/vmmla.c
b/gcc/testsuite/gcc.target/aarch64/simd/vmmla.c
index
arget/aarch64/if-compare_2.c: Support bit and bif for
both bar1 and bar2; add comment on why too.
Signed-off-by: Andrew Pinski
---
gcc/testsuite/gcc.target/aarch64/if-compare_2.c | 9 +++--
1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/gcc/testsuite/gcc.target/aarch
/116259
gcc/testsuite/ChangeLog:
* gcc.target/aarch64/pr116258.c: New test.
* gcc.target/aarch64/sve/pr116259-1.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/testsuite/gcc.target/aarch64/pr116258.c | 17 +
.../gcc.target/aarch64/sve/pr116259-1.c | 12
:
* tree-vect-patterns.cc (NUM_PATTERNS): Delete.
(vect_pattern_recog_1): Constify and change
recog_func to a reference.
(vect_pattern_recog): Use range-based loop over
vect_vect_recog_func_ptrs.
Signed-off-by: Andrew Pinski
---
gcc/tree-vect-patterns.cc | 12
signed types for the condition.
(`a?~t:t`): Likewise.
gcc/testsuite/ChangeLog:
* gcc.c-torture/execute/bitfld-signed1-1.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/match.pd| 6 +-
.../gcc.c-torture/execute/bitfld-signed1-1.c
Vector Instructions): Add that
the base_types could be a typedef of them.
Signed-off-by: Andrew Pinski
---
gcc/doc/extend.texi | 13 +++--
1 file changed, 7 insertions(+), 6 deletions(-)
diff --git a/gcc/doc/extend.texi b/gcc/doc/extend.texi
index 7b54a241a7b..e290265d68d 100644
On Mon, Apr 8, 2024 at 9:39 AM wrote:
>
> From: Pierre-Emmanuel Patry
>
> Hello,
>
> The rust frontend requires cargo to build some of it's components,
> it's presence was not checked during configuration.
WHY did this go in right before the release of GCC 14?
I don't get why this is considered
should document this option or not. On one hand it is
only supposed to be used by lto but on the other hand, someone could
use it on accident from the command line and we would get a bug report
saying the file passed to it is now 0.
Thanks,
Andrew Pinski
>
> Signed-off-by: Peter Damianov
Just fixes the link to the manual for the new -nostdlib++ option.
Signed-off-by: Andrew Pinski
---
htdocs/gcc-13/changes.html | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/htdocs/gcc-13/changes.html b/htdocs/gcc-13/changes.html
index 6930bd58..4384c329 100644
--- a/htdocs
gcc.c-torture/execute/eh_return-1.c: New test.
* gcc.c-torture/execute/eh_return-2.c: New test.
* gcc.c-torture/execute/eh_return-3.c: New test.
* gcc.c-torture/execute/eh_return-4.c: New test.
* gcc.target/aarch64/eh_return-4.c: New test.
Signed-off-by: Andrew P
sted on aarch64-linux-gnu with no regressions.
PR target/112304
gcc/ChangeLog:
* config/aarch64/aarch64.md (*csinc2_insn): Change the
predicate of the 1st operand to general_operand.
gcc/testsuite/ChangeLog:
* gcc.target/aarch64/cinc-2.c: New test.
Signed-off-by
a nontemporal store.
Bootstrapped and tested on x86_64-linux-gnu.
gcc/ChangeLog:
PR middle-end/112976
* cfgexpand.cc (expand_gimple_stmt_1): Remove
support for expanding nontemporal "moves" with
ssa names on the LHS.
Signed-off-by: Andrew Pinski
geLog:
PR middle-end/112976
* tree-cfg.cc (verify_gimple_assign): Verify that
nontmporal moves are stores.
* gimple.h (struct gimple): Note that only
nontemporal stores are supported.
Signed-off-by: Andrew Pinski
---
gcc/gimple.h| 3 ++-
gcc/tree-cfg.cc
This is a small cleanup of loop_versioning where m_nloops
is only used in the constructor so we can remove the whole
field.
Bootstrapped and tested on x86_64-linux-gnu.
gcc/ChangeLog:
* gimple-loop-versioning.cc (loop_versioning): Remove m_nloops field.
(loop_versioning::loop_ver
check actually
reduces the number of pointer walks needed.
Bootstrapped and tested on x86_64-linux-gnu with no regressions.
gcc/ChangeLog:
* tree-ssa-phiopt.cc (single_non_singleton_phi_for_edges):
Remove the special case of gimple_seq_singleton_p.
Signed-off-by: Andrew Pinski
rlier as calculating empty_or_with_defined_p
is an IR walk for a BB and that might be big.
Bootstrapped and tested on x86_64-linux-gnu with no regressions.
gcc/ChangeLog:
* tree-ssa-phiopt.cc (value_replacement): Move check for
NE/EQ earlier.
Signed-off-by: Andrew Pinski
---
gcc
linux-gnu with no regressions.
gcc/ChangeLog:
* tree-ssa-phiopt.cc (value_replacement): Reject undef variables
so they don't become unconditional used.
Signed-off-by: Andrew Pinski
---
gcc/tree-ssa-phiopt.cc | 7 +++
1 file changed, 7 insertions(+)
diff --git a/gc
emove quote
mark to shut up the warning.
Signed-off-by: Andrew Pinski
---
gcc/tree-cfg.cc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/tree-cfg.cc b/gcc/tree-cfg.cc
index 1c5b7df8541..b2d47b72084 100644
--- a/gcc/tree-cfg.cc
+++ b/gcc/tree-cfg.cc
@@ -4842,7 +4
On Tue, Apr 30, 2024 at 11:54 AM Jason Merrill wrote:
>
> On 2/20/24 19:06, Andrew Pinski wrote:
> > After r7-987-gf17a223de829cb, the access for the elements of a vector type
> > would lose the qualifiers.
> > So if we had `constvector[0]`, the type of the element of
* gcc.dg/tree-ssa/phi-opt-value-5.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/match.pd | 18 +
.../gcc.dg/tree-ssa/phi-opt-value-5.c | 39 +++
2 files changed, 57 insertions(+)
create mode 100644 gcc/testsuite/gcc.dg/tree-ssa
le GCC with? I suspect this is making a difference. It might also
be the sparc compiler that both of you two are using is causing wrong
code with some more complex C++ code even though it is at -O0.
The adding of the deconstructor to Value_Range might be causing the
structure to become a "non-POD" and different argument passing and it
was broken even at -O0 (this is just a guess).
Thanks,
Andrew Pinski
>
> Ian
_EXPR.
Bootstrapped and tested on x86_64-linux-gnu.
gcc/ChangeLog:
* tree-pretty-print.cc (print_declaration): Add parenthese
around DECL_INIT if it was a COMPOUND_EXPR.
Signed-off-by: Andrew Pinski
---
gcc/tree-pretty-print.cc | 9 -
1 file changed, 8 insertions(
e ): Fix
calls to dump_generic_node and also remove unreachable code that is
testing
`flags & TDF_SLIM`.
gcc/testsuite/ChangeLog:
* gfortran.dg/gomp/atomic-21.f90: Update testcase for the removal of
`;`.
Signed-off-by: Andrew Pinski
---
gcc/testsuite/gfortran.dg/gomp/a
ent is_stmt and don't print
a semicolon nor the initializer.
* tree-pretty-print.h (print_declaration): Add bool argument
and default it to true.
Signed-off-by: Andrew Pinski
---
gcc/tree-pretty-print.cc | 18 +++---
gcc/tree-pretty-print.h | 2 +-
2 f
atexit-3.C: New test.
* g++.dg/tree-ssa/cxa_atexit-4.C: New test.
* g++.dg/tree-ssa/cxa_atexit-5.C: New test.
Signed-off-by: Andrew Pinski
---
gcc/testsuite/g++.dg/tree-ssa/cxa_atexit-1.C | 20 +
gcc/testsuite/g++.dg/tree-ssa/cxa_atexit-2.C | 21 ++
gcc/testsuite/
failing on
sparc (32bit rather than 64bit).
Thanks,
Andrew Pinski
>
> If this is correct, OK for trunk?
>
> gcc/ChangeLog:
>
> * value-range.h (class Value_Range): Use a union.
> ---
> gcc/value-range.h | 24 +++-
> 1 file changed, 15 inserti
rch64 Options): Document oryon-1.
Signed-off-by: Andrew Pinski
Co-authored-by: Joel Jones
Co-authored-by: Wei Zhao
---
gcc/config/aarch64/aarch64-cores.def | 5 +
gcc/config/aarch64/aarch64-tune.md | 2 +-
gcc/doc/invoke.texi | 1 +
3 files changed, 7 insertions(+), 1 del
On Fri, May 3, 2024 at 8:08 AM Patrick Palka wrote:
>
> Hey Andrew,
>
> On Wed, 5 Apr 2023, Andrew Pinski wrote:
>
> > On Wed, Apr 5, 2023 at 10:32 AM Patrick Palka via Gcc-patches
> > wrote:
> > >
> > > On Wed, 5 Apr 2023, Patrick Palka wrote
On Fri, May 3, 2024 at 4:36 AM Richard Biener
wrote:
>
> On Thu, May 2, 2024 at 11:40 PM Andrew Pinski
> wrote:
> >
> > Right now we don't print that a DECL_EXPR and we get
> > basically double output of the decls and it looks confusing.
> > This
On Fri, May 3, 2024 at 4:41 AM Richard Biener
wrote:
>
> On Thu, May 2, 2024 at 11:40 PM Andrew Pinski
> wrote:
> >
> > When we have :
> > `void f (int y, int z) { int x = ( z++,y); }`
> >
> > This would have printed the decl's initializer withou
igned-off-by: Andrew Pinski
---
gcc/config/aarch64/driver-aarch64.cc | 14 ++
.../gcc.target/aarch64/cpunative/info_25 | 17
.../gcc.target/aarch64/cpunative/info_26 | 26 +++
.../aarch64/cpunative/native_cpu_25.c
atexit-6.C: New test.
Signed-off-by: Andrew Pinski
---
gcc/testsuite/g++.dg/tree-ssa/cxa_atexit-1.C | 20 +++
gcc/testsuite/g++.dg/tree-ssa/cxa_atexit-2.C | 21 +++
gcc/testsuite/g++.dg/tree-ssa/cxa_atexit-3.C | 19 +++
gcc/testsuite/g++.dg/tree-ssa/cxa_atexit-4.C | 20 +++
gcc/t
g++.dg/tree-ssa/cxa_atexit-5.C: New test.
* g++.dg/tree-ssa/cxa_atexit-6.C: New test.
Signed-off-by: Andrew Pinski
---
gcc/testsuite/g++.dg/tree-ssa/cxa_atexit-1.C | 20 +++
gcc/testsuite/g++.dg/tree-ssa/cxa_atexit-2.C | 21 +++
gcc/testsuite/g++.dg/tree-ssa/cxa_atexit-3.C |
* gcc.c-torture/execute/eh_return-5.c: New test.
* gcc.target/aarch64/eh_return-4.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64-protos.h | 1 +
gcc/config/aarch64/aarch64.cc | 78 ++-
gcc/config/aarch64
;r' alternative.
* config/aarch64/iterators.md (wv): New mode attr.
gcc/testsuite/ChangeLog:
* gcc.target/aarch64/csel_1.c: New test.
* gcc.target/aarch64/fcsel_2.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/config/aarch64/aarch64.md | 13 +++
obvious after a quick test on aarch64-linux-gnu.
gcc/testsuite/ChangeLog:
PR testsuite/114177
* gcc.target/aarch64/sve/loop_add_6.c: Use __SIZETYPE__ instead
of `unsigned long` for index and offset variables.
Signed-off-by: Andrew Pinski
---
gcc/testsuite/gcc.target/aarch64
h): Likewise and use less strong language.
Signed-off-by: Andrew Pinski
---
gcc/doc/invoke.texi | 41 ++---
1 file changed, 22 insertions(+), 19 deletions(-)
diff --git a/gcc/doc/invoke.texi b/gcc/doc/invoke.texi
index 9456ced468a..14ff4d25da7 100644
--- a/gcc/d
On Tue, May 7, 2024 at 1:45 PM Jeff Law wrote:
>
>
>
> On 4/30/24 9:21 PM, Andrew Pinski wrote:
> > This adds a few more of what is currently done in phiopt's value_replacement
> > to match. I noticed this when I was hooking up phiopt's value_replacement
>
where CMP is ==, > and >=.
(`x CMP nonnegative@y ? y : ABS`): New pattern.
gcc/testsuite/ChangeLog:
* gcc.dg/tree-ssa/phi-opt-41.c: New test.
Signed-off-by: Andrew Pinski
---
gcc/match.pd | 15 ++
gcc/testsuite/gcc.dg/tree
On Sat, Jul 22, 2023 at 8:36 PM Kito Cheng via Gcc-patches
wrote:
>
> OK for trunk, thanks:)
I have now backported it to 13 branch.
Thanks,
Andrew
>
> Andrew Pinski via Gcc-patches 於 2023年7月23日 週日
> 09:07 寫道：
>
> > The problem -fasynchronous-unwind-tables is on by
On Mon, Mar 11, 2024 at 11:41 PM Andrew Pinski (QUIC)
wrote:
>
> > -Original Message-
> > From: Andrew Pinski (QUIC)
> > Sent: Sunday, March 10, 2024 7:58 PM
> > To: gcc-patches@gcc.gnu.org
> > Cc: Andrew Pinski (QUIC)
>
On Sun, Oct 1, 2023 at 1:23 PM Andrew Pinski wrote:
>
> From: Andrew Pinski
>
> The problem here is after r6-7425-ga9fee7cdc3c62d0e51730,
> the comparison to see if the transformation could be done was using the
> wrong value. Instead of see if the inner was LE (for MIN and
On Thu, Feb 22, 2024 at 9:28 AM Andrew Pinski wrote:
>
> This looks like an oversight of handling DEMANGLE_COMPONENT_UNNAMED_TYPE.
> DEMANGLE_COMPONENT_UNNAMED_TYPE only has the u.s_number.number set while
> the code expected newc.u.s_binary.left would be valid.
>
On Wed, May 8, 2024 at 12:37 PM Iain Sandoe wrote:
>
> Hi Folks,
>
> I’d like to land a viable solution to this issue if possible, (it is a show-
> stopper for the aarch64-darwin development branch).
>
> > On 9 Apr 2024, at 14:55, Iain Sandoe wrote:
> >
> > So far, tested lightly on aarch64-darwi
401 - 500 of 2416 matches
Mail list logo
