Hi
> -Original Message-
> From: Richard Sandiford [mailto:richard.sandif...@arm.com]
> Sent: Friday, April 17, 2020 9:38 PM
> To: duanbo (C)
> Cc: Wilco Dijkstra ; gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH PR94577] [AArch64] ：Add an error message in large
> code model for ilp32
>
> "
Since arm_acle.h includes stdint.h, its use requires the presence of
the right gnu/stub-*.h, so make sure to include arm_acle.h when
checking the effective targets that generally imply that the testcase
will include it: arm_dsp, arm_crc, arm_coproc[1-4]
This makes several tests unsupported rather
Since arm_cde.h includes stdint.h, its use requires the presence of
the right gnu/stub-*.h, so make sure to include it when checking the
arm_v8*m_main_cde* effective targets, otherwise we can decide CDE is
supported while it's not really (all tests that use arm_v8m_main_cde*
also include arm_cde.h
Hi,
You can disregard this patch, Wilco found problems with it, and I
think he is working on a more complete fix.
Thanks,
Christophe
On Thu, 16 Apr 2020 at 17:39, Christophe Lyon
wrote:
>
> Hi,
>
> In PR94538, Wilco mentioned that my patch to enable -mpure-code for
> v6m caused regressions in
On Fri, Apr 17, 2020 at 10:51 PM Segher Boessenkool
wrote:
>
> On Fri, Apr 17, 2020 at 08:53:08AM +0200, Richard Biener wrote:
> > On Thu, Apr 16, 2020 at 7:46 PM Segher Boessenkool
> > wrote:
> > >
> > > On Thu, Apr 16, 2020 at 10:33:45AM +0200, Richard Biener wrote:
> > > > On Wed, Apr 15, 2020
On Fri, Apr 17, 2020 at 10:51 PM Segher Boessenkool
wrote:
>
> On Fri, Apr 17, 2020 at 08:53:08AM +0200, Richard Biener wrote:
> > On Thu, Apr 16, 2020 at 7:46 PM Segher Boessenkool
> > wrote:
> > >
> > > On Thu, Apr 16, 2020 at 10:33:45AM +0200, Richard Biener wrote:
> > > > On Wed, Apr 15, 2020
On Fri, Apr 17, 2020 at 5:21 PM Richard Sandiford
wrote:
>
> This patch fixes a large lmbench performance regression with
> 128-bit SVE, compiled in length-agnostic mode.
>
> vect_better_loop_vinfo_p (new in GCC 10) tries to estimate whether
> a new loop_vinfo is cheaper than a previous one, with
> -Original Message-
> From: Andrea Corallo
> Sent: 16 April 2020 09:01
> To: Kyrylo Tkachov
> Cc: nd ; gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH V2]aarch64: falkor-tag-collision-avoidance.c fix
> valid_src_p for use of uninitialized value
>
> Hi all,
>
> I'd like to back-port th
Hi,
On 2020/4/18 00:32, Segher Boessenkool wrote:
> On Thu, Apr 16, 2020 at 08:21:40PM -0500, Segher Boessenkool wrote:
>> On Wed, Apr 15, 2020 at 10:18:16AM +0100, Richard Sandiford wrote:
>>> luoxhu--- via Gcc-patches writes:
-count = simplify_gen_binary (PLUS, mode, count, const1_rtx)
Hi Andre,
> -Original Message-
> From: Andre Vieira (lists)
> Sent: 16 April 2020 13:24
> To: gcc-patches@gcc.gnu.org
> Cc: Kyrylo Tkachov ; Richard Sandiford
> ; s...@amazon.com
> Subject: [PATCH 0/19][GCC-8] aarch64: Backport outline atomics
>
> Hi,
>
> This series backports all the p
Tiny update to accommodate unsigned int compare.
On 2020/4/20 16:21, luoxhu via Gcc-patches wrote:
Hi,
On 2020/4/18 00:32, Segher Boessenkool wrote:
On Thu, Apr 16, 2020 at 08:21:40PM -0500, Segher Boessenkool wrote:
On Wed, Apr 15, 2020 at 10:18:16AM +0100, Richard Sandiford wrote:
luoxhu--
On 20/04/20 07:01 +0200, Helmut Grohne wrote:
The and headers need their counter parts and
from the libc respectively, but libstdc++ wraps these
headers. Now and include these headers using
$ echo '#include ' | g++ -x c++ -E - -isystem /usr/include >/dev/null
In file included from :1:
/usr
On 4/3/20 12:32 PM, Martin Liška wrote:
Hi.
There's one more I've tested.
Martin
And one more.
Martin
>From 6db982d1cfccbde2a5812edf82ad330cf327bc23 Mon Sep 17 00:00:00 2001
From: Martin Liska
Date: Fri, 17 Apr 2020 09:33:05 +0200
Subject: [PATCH] Backport e9f799d25973fc38022c5ea71ed5a2bca5
Hi.
The patch prints total lines summary after all files are processed.
Moreover, I remove a check in generate_results where we should
be always given file_name != NULL.
Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
I'm planning to install the in next stage1 if there are
Hi.
The patch prevents a ifunc alias from removal in remove unreachable nodes.
Note that ifunc alias lives in a COMDAT section and so that
cgraph_node::can_remove_if_no_direct_calls_and_refs_p returned true for it.
Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
I was unab
Hi Harald,
The attached patch fixes that. OK for mainline / backport to 9-branch?
OK for both.
Thanks a lot for the patch!
Regards
Thomas
Am 19.04.20 um 20:03 schrieb José Rui Faustino de Sousa via Fortran:
Hi Thomas!
> ? In other words, maybe a check on the upper bound
> of the last dimension would be better?
>
You mean enforcing:
C928 (R921) The second subscript shall not be omitted from a
subscript-triplet in the last dim
Hi.
The patch is about a space printed in between multiple
entries in Referring and References in symtab_node dump.
Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
Ready to be installed?
Thanks,
Martin
gcc/ChangeLog:
2020-04-20 Martin Liska
* symtab.c (symta
Hi,
On Mon, Apr 20 2020, Martin Liška wrote:
> Hi.
>
> The patch is about a space printed in between multiple
> entries in Referring and References in symtab_node dump.
>
> Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
>
> Ready to be installed?
Given that "called by" and
Hello world,
I just committed as obvious (alternatively, reviewed and
committed) Steve's patch for PR 91800 from the PR,
after regression-testing.
Regards
Thomas
PR 91800 - reject Hollerith constants as type initializer.
2020-04-20 Steve Kargl
Thomas Koenig
PR fo
On 20/04/2020 08:47, Christophe Lyon via Gcc-patches wrote:
Since arm_cde.h includes stdint.h, its use requires the presence of
the right gnu/stub-*.h, so make sure to include it when checking the
arm_v8*m_main_cde* effective targets, otherwise we can decide CDE is
supported while it's not really
Some tests use --save-temps, but schedule-cleanups strictly matches
-save-temps, so we leave many temporary files after validation.
Instead of fixing every offending testcase, it's simpler and
future-proof to make schedule-cleanups handle both --save-temps and
-save-temps.
2020-04-20 Christophe L
On Mon, 20 Apr 2020 at 13:25, Matthew Malcomson
wrote:
>
> On 20/04/2020 08:47, Christophe Lyon via Gcc-patches wrote:
> > Since arm_cde.h includes stdint.h, its use requires the presence of
> > the right gnu/stub-*.h, so make sure to include it when checking the
> > arm_v8*m_main_cde* effective t
On 4/20/20 1:03 PM, Martin Jambor wrote:
Hi,
On Mon, Apr 20 2020, Martin Liška wrote:
Hi.
The patch is about a space printed in between multiple
entries in Referring and References in symtab_node dump.
Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
Ready to be install
Kyrylo Tkachov writes:
>> Bootstrapped and reg tested. Ok for release/gcc-9?
>
> Ok.
> Thanks,
> Kyrill
Pushed as 5e022e3b3f7b.
Thanks
Andrea
Fix a problem with the libffi testsuite using a method to determine the
compiler to use resulting in the tool being different from one the
library has been built with, and causing a catastrophic failure from the
inability to actually choose any compiler at all in a cross-compilation
configurati
Hi!
On Mon, Apr 20, 2020 at 09:56:34AM +0200, Richard Biener wrote:
> On Fri, Apr 17, 2020 at 10:51 PM Segher Boessenkool
> wrote:
> > > Yeah well, but RTL is not in SSA form
> >
> > "Webs" are not the *same* as SSA, in a few crucial ways; but they serve
> > similar purposes: they both make code
Hi,
Previously -fweb was disabled if only unroll small loops. After that
we find there is cases where it could help to rename pseudos and aovid
some anti-dependence which may occur after unroll.
Below is a patch to disable -fweb during loop unrolling.
Bootstrap and regtest pass on powerpc, spec
Hi Thomas,
I have now fixed it temporarily differently – without actually
testing that feature. – See attachment.
For the proper fix, it would help to get the semantic right in
OpenACC itself (→ ongoing discussion). I think the patch
for PR94120 is probably not completely right – and should be
f
On Mon, Apr 20, 2020 at 3:38 PM Segher Boessenkool
wrote:
>
> Hi!
>
> On Mon, Apr 20, 2020 at 09:56:34AM +0200, Richard Biener wrote:
> > On Fri, Apr 17, 2020 at 10:51 PM Segher Boessenkool
> > wrote:
> > > > Yeah well, but RTL is not in SSA form
> > >
> > > "Webs" are not the *same* as SSA, in a
On 16/04/20 15:21 +0200, Matthias Kretz wrote:
From: Matthias Kretz
If extra_tool_flags starts with a dash, an error like 'ERROR: verbose:
illegal argument: -march=native -O2 -std=c++17' is printed. This is
easily fixed by inserting a double dash before the variable.
* testsuite/lib/lib
"duanbo (C)" writes:
> Hi
>
>> -Original Message-
>> From: Richard Sandiford [mailto:richard.sandif...@arm.com]
>> Sent: Friday, April 17, 2020 9:38 PM
>> To: duanbo (C)
>> Cc: Wilco Dijkstra ; gcc-patches@gcc.gnu.org
>> Subject: Re: [PATCH PR94577] [AArch64] ：Add an error message in larg
This testcase triggered an ICE in rtx_vector_builder::step because
we were trying to use a stepped representation for floating-point
constants. The underlying problem was that the arguments to
rtx_vector_builder were the wrong way around, meaning that some
variations were likely to be incorrectly
Hi Frederik!
As you've been the last one to work on this code and get it into GCC
master branch (commit 5d183d1740d8d8b84991f186ce4d992ee799536f "Warn
about inconsistent OpenACC nested reduction clauses"), you get to look
into the following issue.
..., and yes, I'm aware that you have a follow-up
On 4/17/20 4:28 PM, Marek Polacek wrote:
As an extension (there should be a CWG about this though), we support
braced-init-list as a template argument, but convert_nontype_argument
had trouble digesting them. We ICEd because of the double coercion we
perform for template arguments: convert_nonty
Greeting from China,
Nice to meet you here,this is Bravo from Long Sail Logistics Co., Ltd.
We are a top leading logistics company in China,with 23 branch offices all over
China and overseas,over 1200 staffs work in Long Sail Logistics all over the
world.
If you have any inquiry for shipmen
This PR points out that we don't detect long double -> double narrowing
when long double happens to have the same precision as double; on x86_64
this can be achieved by -mlong-double-64.
[dcl.init.list]#7.2 specifically says "from long double to double or float,
or from double to float", but check
Hi,
This patch fixes up a previous commit that removed DRUNTIME_GC from
m4/druntime.m4
Committed to mainline.
Regards
Iain.
---
libphobos/ChangeLog:
* configure: Regenerate.
* configure.ac: Remove DRUNTIME_GC.
---
libphobos/configure| 1 -
libphobos/configure.ac | 1 -
2 f
Hi,
Normally, when we find a statement containing an await expression
this will be expanded to a statement list implementing the control
flow implied. The expansion process successively replaces each
await expression in a statement with the result of its await_resume().
In the case of conditiona
The last C++20 changes from P1614R2, "The Mothership has Landed"
* include/bits/stl_map.h (map): Define operator<=> and remove
operator< for C++20.
* include/bits/stl_multimap.h (multimap): Likewise.
* include/bits/stl_multiset.h (multiset): Likewise.
* incl
Hi,
This patch fixes the option --with-libphobos-druntime-only to match the
help text, as the prefixing 'libphobos' was intentional.
Tested on x86_64-linux-gnu, and committed to mainline.
Regards
Iain.
---
libphobos/ChangeLog:
* configure: Regenerate.
* configure.ac: Fix option
With P1614R2 fully implemented (except for the types which we
don't support at all) we can define the feature test macro to the new
value.
* include/std/version (__cpp_lib_three_way_comparison): Update value.
* libsupc++/compare (__cpp_lib_three_way_comparison): Likewise.
Tested
On 4/19/20 3:35 PM, Iain Sandoe wrote:
Hi,
Coroutine ramp functions have synthesised return values (the
user-authored function body cannot have an explicit 'return').
The current implementation attempts to optimise by building
the return in-place, in the manner of C++17 code. Clearly,
that was
Here is the patch introducing thunderxt311 maching model
for the scheduler. A name for the new chip was added to the
list of the names to be recognized as a valid parameter for mcpu
and mtune flags. The TX2 cost model was reused for TX3.
The previously used "cryptic" name for the command line
para
Hi Anton,
> -Original Message-
> From: Gcc-patches On Behalf Of Anton
> Youdkevitch
> Sent: 20 April 2020 18:05
> To: gcc-patches@gcc.gnu.org
> Cc: jo...@marvell.com
> Subject: [PATCH v2] aarch64: Add TX3 machine model
>
> Here is the patch introducing thunderxt311 maching model
> for th
Hello world,
after finding myself debug a PR where showing all the attributes
of a symbol would have helped enormously (which I realized only
afterwards), I went ahead and added most of the flags to show_attr,
in the hope that this will help all of us debugging some murky corners
of the compiler
The vector popcount expanders use a hardcoded subreg. This might lead
to double subregs being generated which then fail to match. With this
patch simplify_gen_subreg is used instead to fold the subregs.
Boostrapped and regression-tested on IBM z14.
Committed to mainline
gcc/ChangeLog:
2020-04
On Sat, 18 Apr 2020 at 03:35, kamlesh kumar via Libstdc++
wrote:
>
> On Fri, Apr 17, 2020, 10:59 PM kamlesh kumar
> wrote:
>
> > Fixes all this.
> > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92156
> > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91630
> > https://gcc.gnu.org/bugzilla/show_bug.
Hi Kyrylo,
On 20.4.2020 20:13 , Kyrylo Tkachov wrote:
Hi Anton,
-Original Message-
From: Gcc-patches On Behalf Of Anton
Youdkevitch
Sent: 20 April 2020 18:05
To: gcc-patches@gcc.gnu.org
Cc: jo...@marvell.com
Subject: [PATCH v2] aarch64: Add TX3 machine model
Here is the patch introdu
On Mon, 20 Apr 2020 at 21:09, Ville Voutilainen
wrote:
>
> On Sat, 18 Apr 2020 at 03:35, kamlesh kumar via Libstdc++
> wrote:
> >
> > On Fri, Apr 17, 2020, 10:59 PM kamlesh kumar
> > wrote:
> >
> > > Fixes all this.
> > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=92156
> > > https://gcc.gnu.o
parm = STRIP_NOPS (parm); is unnecessary and generates
warning: operation on 'parm' may be undefined [-Wsequence-point]
when cp/coroutines.cc is compiled with -std=c++11.
Bootstrapped/regtested on x86_64-pc-linux-gnu, ok for trunk?
* coroutines.cc (captures_temporary): Don't assign the re
Here is the patch introducing thunderxt311 maching model
for the scheduler. A name for the new chip was added to the
list of the names to be recognized as a valid parameter for mcpu
and mtune flags. The TX2 cost model was reused for TX3.
The previously used "cryptic" name for the command line
para
For future architecture with prefix instructions, always use plq
rather than lq for atomi load of quadword. Then we never have to
do the doubleword swap on little endian. Before this fix, -mno-pcrel
would generate lq with the doubleword swap (which was ok) and -mpcrel
would generate plq, also with
On 4/20/20 2:25 PM, Marek Polacek wrote:
parm = STRIP_NOPS (parm); is unnecessary and generates
warning: operation on 'parm' may be undefined [-Wsequence-point]
when cp/coroutines.cc is compiled with -std=c++11.
Bootstrapped/regtested on x86_64-pc-linux-gnu, ok for trunk?
yup, thanks for catch
On 4/18/20 10:49 AM, Marek Polacek wrote:
This is an ICE on invalid, because we're specializing S::foo in the
wrong namespace. cp_parser_class_specifier_1 parses S::foo in M
and then it tries to push the nested-name-specifier of foo, which is
S. By that, we're breaking the assumption of push_in
On 4/19/20 12:55 PM, Patrick Palka wrote:
This PR seems to be similar to PR c++/43382, except that the recursive call to
the variadic function with trailing return type in this testcase is additionally
given some explicit template arguments.
In the first testcase below, when resolving the recurs
On 4/19/20 12:59 PM, Marek Polacek wrote:
We issue bogus -Wparentheses warnings (3 of them!) for this fold expression:
((B && true) || ...)
Firstly, issuing a warning for a compiler-generated expression is wrong
and secondly, B && true must be wrapped in ( ) otherwise you'll get
error: binar
On 4/20/20 12:25 PM, Marek Polacek wrote:
This PR points out that we don't detect long double -> double narrowing
when long double happens to have the same precision as double; on x86_64
this can be achieved by -mlong-double-64.
[dcl.init.list]#7.2 specifically says "from long double to double o
A comment in satisfy_declaration constraints says
/* For inherited constructors, consider the original declaration;
it has the correct template information attached. */
d = strip_inheriting_ctors (d);
But this comment seems to be false when the inherited constructor points to an
instanti
On Mon, Apr 20, 2020 at 11:51:59AM -0400, Jason Merrill wrote:
> On 4/17/20 4:28 PM, Marek Polacek wrote:
> > As an extension (there should be a CWG about this though), we support
> > braced-init-list as a template argument, but convert_nontype_argument
> > had trouble digesting them. We ICEd beca
* testsuite/20_util/is_constructible/51185.cc: Make test class a
non-aggregate so that the test verifies the same thing in all -std
modes.
* testsuite/20_util/is_constructible/value-2.cc: Adjust expected
results for some types when paren-init for aggregates i
This test was supposed to be added two months ago as part of commit
120e873484f20d9a0b8400e2e464ac5b2088a747 but was omitted by mistake.
* testsuite/24_iterators/istreambuf_iterator/sentinel.cc: New test.
Tested x86_64-linux, committed to master.
commit 86119f14171a3e660e65004d9a168821c
On Mon, 20 Apr 2020, Jason Merrill wrote:
> On 4/19/20 12:55 PM, Patrick Palka wrote:
> > This PR seems to be similar to PR c++/43382, except that the recursive call
> > to
> > the variadic function with trailing return type in this testcase is
> > additionally
> > given some explicit template argu
The restrict pass computes the wrong lower bound of the size
of accesses to member arrays passed to strncpy as the source:
it uses the third argument to the function even though the size
of the read is limited by the length of the source when it is
a string. This results in false positive -Warray
Hi!
On 2020-04-17T17:57:06+0200, Tobias Burnus wrote:
> On 4/17/20 5:54 PM, Tobias Burnus wrote:
>> It turned out that doing
>> omp enter data map(alloc:FortranArray)
>> omp exit data map(delete:FortranArray)
>> left the array descriptor (fortranarray [as opposed to
>> fortranarray.data])
>>
Hi,
On Mon, 2020-04-20 at 14:00 -0500, Aaron Sawdey via Gcc-patches wrote:
> For future architecture with prefix instructions, always use plq
> rather than lq for atomi load of quadword. Then we never have to
atomic :-)
> do the doubleword swap on little endian. Before this fix, -mno-pcrel
> w
On 4/20/20 4:44 PM, Marek Polacek wrote:
On Mon, Apr 20, 2020 at 11:51:59AM -0400, Jason Merrill wrote:
On 4/17/20 4:28 PM, Marek Polacek wrote:
As an extension (there should be a CWG about this though), we support
braced-init-list as a template argument, but convert_nontype_argument
had troubl
Hi!
-Wpadded warnings aren't really useful for the artificial types that GCC
lays out for ubsan.
Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for
trunk?
2020-04-20 Jakub Jelinek
PR c/94641
* ubsan.c (ubsan_get_type_descriptor_type,
ubsan_get
Hi!
This patch addresses a missed optimization caused by the cselib changes.
Already in the past postreload could replace sp = sp + const_int with
sp = regxy if regxy already has the right value, but with the cselib
changes it happens several times more often. It can result in smaller
code, so it
Thank you for reviewing.
without _Decay to decay_t in the constructor which takes inplace_type_t,
cases like this fails
auto a = std::any(std::in_place_type, 5);
for these constructors, standard does not say anything about
not-sameness checks with any.
https://en.cppreference.com/w/cpp/utility/an
On Mon, Apr 20, 2020 at 06:37:08PM -0400, Jason Merrill wrote:
> On 4/20/20 4:44 PM, Marek Polacek wrote:
> > On Mon, Apr 20, 2020 at 11:51:59AM -0400, Jason Merrill wrote:
> > > On 4/17/20 4:28 PM, Marek Polacek wrote:
> > > > As an extension (there should be a CWG about this though), we support
>
For future architecture with prefix instructions, always use plq
rather than lq for atomic load of quadword. Then we never have to
do the doubleword swap on little endian. Before this fix, -mno-pcrel
would generate lq with the doubleword swap (which was ok) and -mpcrel
would generate plq, also with
This patch to the libgo by Benny Siegert fixes compiling the runtime
package on NetBSD. On NetBSD si_code in siginfo_t is a macro on
NetBSD, not a member of the struct itself, so add a C trampoline for
receiving its value. Also replace references to mos.waitsemacount
with the replacement and add
On Tue, 21 Apr 2020 at 04:10, kamlesh kumar wrote:
>
> Thank you for reviewing.
> without _Decay to decay_t in the constructor which takes inplace_type_t,
> cases like this fails
> auto a = std::any(std::in_place_type, 5);
>
> for these constructors, standard does not say anything about
> not-sam
On Tue, 21 Apr 2020, Jakub Jelinek wrote:
> Hi!
>
> -Wpadded warnings aren't really useful for the artificial types that GCC
> lays out for ubsan.
>
> Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for
> trunk?
I believe most of those warnings should simply not look at
On Tue, 21 Apr 2020 at 09:11, Ville Voutilainen
wrote:
>
> On Tue, 21 Apr 2020 at 04:10, kamlesh kumar wrote:
> >
> > Thank you for reviewing.
> > without _Decay to decay_t in the constructor which takes inplace_type_t,
> > cases like this fails
> > auto a = std::any(std::in_place_type, 5);
> >
76 matches
Mail list logo
