David,
Seems there is something wrong with my email box. I lost your email. I
reconfigured the box and it should be OK now.
Could you inform me how to exclude AIX from the condition testing? By
the ABI? Thanks a lot.
Haochen Gui
On 7/21/20 3:07 PM, Patrick Palka wrote:
In the testcase from the PR we are seeing excessive memory use (> 5GB)
during constexpr evaluation, almost all of which is due to the call to
decl_constant_value in the VAR_DECL/CONST_DECL branch of
cxx_eval_constant_expression. We reach here every time
gcc/
* config/csky/t-csky-linux: Delete big endian CPUs' multilib.
---
gcc/config/csky/t-csky-linux | 8 +---
1 file changed, 1 insertion(+), 7 deletions(-)
diff --git a/gcc/config/csky/t-csky-linux b/gcc/config/csky/t-csky-linux
index 16656c3..df471ed 100644
---
> -Original Message-
> From: Richard Sandiford [mailto:richard.sandif...@arm.com]
> Sent: Friday, July 17, 2020 5:04 PM
> To: xiezhiheng
> Cc: Richard Biener ; gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH PR94442] [AArch64] Redundant ldp/stp instructions
> emitted at -O3
>
Cut...
>
>
On 7/20/20 7:28 PM, Marek Polacek wrote:
P1975R0 tweaks the static_cast wording: it says that "An expression e can be
explicitly converted to a type T if [...] T is an aggregate type having a first
element x and there is an implicit conversion sequence from e to the type of
x." This already
I've created a much more rudimentary setup for myself to deal
with the same problem. I copy tests from Bugzilla, sometimes
with tweaks, and compile them from time to time as I revisit
unresolved bugs. I've also thought about adding those to
the test suite and marking them XFAIL but I don't
On 7/20/20 4:18 PM, Patrick Palka wrote:
In the below testcase, duplicate_decls wasn't merging the tsubsted
friend declaration for 'void add(auto)' with its definition, because
reduce_template_parm_level (during tsubst_friend_function) lost the
DECL_VIRTUAL_P flag on the invented 'auto' template
On 7/16/20 11:06 AM, Marek Polacek wrote:
This is DR 2032 which says that the restrictions regarding template
parameter packs and default arguments apply to variable templates as
well, but we weren't detecting that.
Bootstrapped/regtested on x86_64-pc-linux-gnu, ok for trunk?
gcc/cp/ChangeLog:
On 7/12/20 9:38 PM, Roger Sayle wrote:
>
> The following patch addds support for PTX's rcp.rn.f32 and rcp.rn.f64
> instructions. Note that the "rcp.rn" forms of this instruction
> calculate the fully IEEE compliant result for the reciprocal, unlike
> the rcp.approx variants that just provide
On Wed, 29 Jul 2020, Jason Merrill wrote:
> On 7/24/20 9:49 AM, Patrick Palka wrote:
> > In the below testcase, we're ICEing from alias_ctad_tweaks ultimately
> > because the implied deduction guide for X's user-defined constructor
> > already has constraints associated with it. We then carry
This patch introduces an attribute to add extra aliases to a symbol
when its definition is output. The main goal is to ease interfacing
C++ with Ada, as C++ mangled names have to be named, and in some cases
(e.g. when using stdint.h typedefs in function arguments) the symbol
names may vary
On Tue, Jul 28, 2020 at 5:45 PM Marek Polacek via Gcc-patches <
gcc-patches@gcc.gnu.org> wrote:
> In Bugzilla, for the c++ component, we currently have over 3200 open
> bugs. In
> my experience, a good amount of them have already been fixed; my periodical
> sweeps always turn up a bunch of PRs
On 7/23/20 3:08 PM, Martin Sebor wrote:
Another test case added to the bug since I posted the patch shows
that the no-warning bit set by the C++ front end is easily lost
during early folding, triggering the -Wnonull again despite
the suppression. The attached revision tweaks the folder in just
On 7/24/20 9:49 AM, Patrick Palka wrote:
In the below testcase, we're ICEing from alias_ctad_tweaks ultimately
because the implied deduction guide for X's user-defined constructor
already has constraints associated with it. We then carry over these
constraints to 'fprime', the overlying
On 7/29/20 3:23 PM, Patrick Palka wrote:
On Wed, 29 Jul 2020, Patrick Palka wrote:
In the testcase below, template argument deduction for the call
g(id) goes wrong because the functions in the overload set id
each have a yet-undeduced auto return type, and this undeduced return
type makes
On Wed, 29 Jul 2020, Richard Biener wrote:
> Note that any workable solution is fine with me, I just
> don't feel comfortable approving the solution involving
> ../curr_stage and friends. Joseph, would HJs latest
> patch be OK technically?
Yes, I think that's OK.
--
Joseph S. Myers
On 7/14/20 4:50 AM, Jakub Jelinek wrote:
Hi!
For C++11 we already emit an error if a constexpr function doesn't contain
a return statement, because in C++11 that is the only thing it needs to
contain, but for C++14 we would normally issue a -Wreturn-type warning.
As mentioned by Jonathan, such
On Wed, 29 Jul 2020, Patrick Palka wrote:
> In the testcase below, template argument deduction for the call
> g(id) goes wrong because the functions in the overload set id
> each have a yet-undeduced auto return type, and this undeduced return
> type makes try_one_overload fail to match up any of
In the testcase below, template argument deduction for the call
g(id) goes wrong because the functions in the overload set id
each have a yet-undeduced auto return type, and this undeduced return
type makes try_one_overload fail to match up any of these functions with
g's parameter type, leading
On 7/13/20 5:19 PM, Patrick Palka wrote:
When considering to instantiate a member of a class template as part of an
explicit instantiation of the class template, we need to first check the
member's constraints before proceeding with the instantiation of the member.
Bootstrapped and regtested on
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'gcc' has been submitted
by the Swedish team of translators. The file is available at:
https://translationproject.org/latest/gcc/sv.po
(This file, 'gcc-10.2.0.sv.po', has
Hi Will,
On Wed, Jul 29, 2020 at 12:40:08PM -0500, will schmidt wrote:
> V2 has completed tests on powerpc64le-unknown-linux-gnu Power8LE, with
> other regression tests still in progress on some other powerpc platforms.
Thanks for that :-)
> +;; xvtlsbb BF,XB
> +;; Set the CR field BF to
P1825R0 extends the C++11 implicit move on return by removing the
constraints on the called constructor: previously, it needed to take an
rvalue reference to the type of the returned variable. The paper also
allows move on throw of parameters and implicit move of rvalue references.
Discussion on
Our implementation of C++11 implicit move was wrong for return; we didn't
actually hit the check for the type of the first parameter of the selected
constructor, because we didn't see LOOKUP_PREFER_RVALUE set properly.
Fixing that to look at the right flags fixed the issue for this testcase,
but
On Tue, 28 Jul 2020, Martin Sebor via Gcc-patches wrote:
> + /* A list of VLA variable bounds or null if not specified. */
> + tree vbchain = NULL_TREE;
> + if (parm->declarator->kind == cdk_array)
> + if (pd->kind != cdk_array)
> + break;
> + /* Skip all constant
[PATCH] RS6000 Add testlsbb by Byte operations
Hi,
Add support for new instructions to test LSB by Byte.
[v2] Additional updates per feedback. Including adding _all
to the internal name, typos and cosmetic fixups throughout, extraneous
-mvsx removed from tests.
V2 has completed
On Wed, Jul 29, 2020 at 06:30:16PM +0200, Tobias Burnus wrote:
> Adds 'order(concurrent)'. OpenMP 5.0 also permits it
> for 'loop' but gfortran does not yet support 'loop'.
>
> (That the argument is passed on to the ME can be
> seen by the testcases as the errors are emitted
> by the ME.)
>
>
Adds 'order(concurrent)'. OpenMP 5.0 also permits it
for 'loop' but gfortran does not yet support 'loop'.
(That the argument is passed on to the ME can be
seen by the testcases as the errors are emitted
by the ME.)
OK?
Tobias
-
Mentor Graphics (Deutschland) GmbH, Arnulfstraße
> Based on the collected numbers in PR95435, I suggest the following
> tuning changes:
>
> gcc/ChangeLog:
>
> PR target/95435
> * config/i386/x86-tune-costs.h: Use libcall for large sizes for
> -m32. Start using libcall from 128+ bytes.
OK,
thanks!
Honza
> ---
>
> The patch improves readability of the memcpy and memset
> expansion strategies.
>
> gcc/ChangeLog:
>
> * config/i386/x86-tune-costs.h: Change code formatting.
OK,
thanks!
Honza
> ---
> gcc/config/i386/x86-tune-costs.h | 38 +++-
> 1 file changed, 28
On Tue, Jul 28, 2020, 10:24 PM Fāng-ruì Sòng wrote:
> On Thu, May 14, 2020 at 12:16 AM Richard Biener
> wrote:
> >
> > On Wed, May 13, 2020 at 5:50 PM Fangrui Song wrote:
> > >
> > > On 2020-05-13, Eric Botcazou wrote:
> > > >> Did I mention I dislike -fsplit-dwarf? ;)
> > > >
> > > >Seconded,
Hi!
On Wed, Jul 29, 2020 at 05:02:04PM +0800, Jojo R wrote:
> 在 2020年7月24日 +0800 PM9:19，Segher Boessenkool ，写道：
> > On Fri, Jul 24, 2020 at 12:03:16PM +0200, Richard Biener via Gcc-patches
> > wrote:
> > > It will also produce different build results depending on the build host
> > > which is
This patch adds support for zero-length arrays in OpenACC data
transfers. Previously, trying to use an array section with zero length
would cause a fatal error at runtime.
This patch requires that my other patch "OpenACC: Separate enter/exit
data APIs" is already applied.
Unfortunately,
This patch does not implement anything new, but simply separates OpenACC
'enter data' and 'exit data' into two libgomp API functions. The
original API name is kept for backward compatibility, but no longer
referenced by the compiler.
The previous implementation assumed that it would always
On Tue, Jul 28, 2020 at 12:23 PM Martin Liška wrote:
>
> On 7/27/20 1:31 PM, Richard Biener wrote:
> > No, add gro*_exact variants and replace existing ones with this, then switch
> > to exact = false for the non-_exact variants. Or add a exact=false argument
> > to all of them and make all
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'gcc' has been submitted
by the Japanese team of translators. The file is available at:
https://translationproject.org/latest/gcc/ja.po
(This file, 'gcc-10.2.0.ja.po', has
This mentions that -gsplit-dwarf now requires -g to enable debug info
generation.
Pushed.
* gcc-11/changes.html: Mention -gsplit-dwarf behavioral change.
---
htdocs/gcc-11/changes.html | 3 +++
1 file changed, 3 insertions(+)
diff --git a/htdocs/gcc-11/changes.html
On Wed, Jul 29, 2020 at 7:24 AM Fāng-ruì Sòng wrote:
>
> On Thu, May 14, 2020 at 12:16 AM Richard Biener
> wrote:
> >
> > On Wed, May 13, 2020 at 5:50 PM Fangrui Song wrote:
> > >
> > > On 2020-05-13, Eric Botcazou wrote:
> > > >> Did I mention I dislike -fsplit-dwarf? ;)
> > > >
> > >
On Wed, 29 Jul 2020, Richard Biener wrote:
> On Wed, 29 Jul 2020, H.J. Lu wrote:
>
> > On Wed, Jul 29, 2020 at 4:01 AM Richard Biener
> > wrote:
> > >
> > > On Fri, Jul 17, 2020 at 5:32 PM H.J. Lu via Gcc-patches
> > > wrote:
> > > >
> > > > On Fri, Jul 17, 2020 at 6:27 AM Richard Biener
> >
Hi Nathan,
On 29.07.20 14:52, Nathan Sidwell wrote:
On 7/28/20 3:28 AM, Tiziano Müller wrote:
Hi there,
this patch intends to fix the regression introduced in gcc 10 with
__has_include,
see also https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95889
Thanks, do you have write access? I don't
Hi Joe,
> -Original Message-
> From: Gcc-patches On Behalf Of Joe
> Ramsay
> Sent: 29 July 2020 11:45
> To: François Dumont via Gcc-patches
> Subject: [PATCH v2][GCC] arm: Enable no-writeback vldr.16/vstr.16.
>
> Hi,
>
> There was previously no way to specify that a register operand
On Wed, Jul 29, 2020 at 02:57:25PM +0200, Tobias Burnus wrote:
> I have missed this OpenMP 5 feature already a couple of
> times myself ...
> (The other related OpenMP 5 feature is that 'map(x) use_device_ptr(x)'
> is permitted on the same directive; but that already works and
> is tested for in
I have missed this OpenMP 5 feature already a couple of
times myself ...
(The other related OpenMP 5 feature is that 'map(x) use_device_ptr(x)'
is permitted on the same directive; but that already works and
is tested for in the large use_device*.f90 libgomp tests.)
OK?
Tobias
-
On 7/28/20 3:28 AM, Tiziano Müller wrote:
Hi there,
this patch intends to fix the regression introduced in gcc 10 with
__has_include,
see also https://gcc.gnu.org/bugzilla/show_bug.cgi?id=95889
Thanks, do you have write access? I don't see your name in MAINTAINERS.
[for avoidance of doubt,
Sorry for later replay, I will add testcases on a following patch if the
patch is accepted.
Regards,
Cooper
On 2020/7/28 上午9:23, Kito Cheng wrote:
Add testcase later is OK to me.
On Tue, Jul 28, 2020 at 6:55 AM Jim Wilson wrote:
On Sun, Jul 19, 2020 at 7:04 PM cooper wrote:
Ping
On
One obvious fix we've just hit with Honza.
Martin
contrib/ChangeLog:
2020-07-29 Martin Liska
* git-backport.py: fix how are ChangeLog paths combined.
---
contrib/git-backport.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/contrib/git-backport.py
On Wed, 29 Jul 2020, H.J. Lu wrote:
> On Wed, Jul 29, 2020 at 4:01 AM Richard Biener
> wrote:
> >
> > On Fri, Jul 17, 2020 at 5:32 PM H.J. Lu via Gcc-patches
> > wrote:
> > >
> > > On Fri, Jul 17, 2020 at 6:27 AM Richard Biener wrote:
> > > >
> > > > On Fri, 17 Jul 2020, H.J. Lu wrote:
> > > >
On Wed, Jul 29, 2020 at 4:01 AM Richard Biener
wrote:
>
> On Fri, Jul 17, 2020 at 5:32 PM H.J. Lu via Gcc-patches
> wrote:
> >
> > On Fri, Jul 17, 2020 at 6:27 AM Richard Biener wrote:
> > >
> > > On Fri, 17 Jul 2020, H.J. Lu wrote:
> > >
> > > > On Fri, Jul 17, 2020 at 12:08 AM Richard Biener
On Wed, 29 Jul 2020, Richard Biener wrote:
> On Fri, Jul 17, 2020 at 5:32 PM H.J. Lu via Gcc-patches
> wrote:
> >
> > On Fri, Jul 17, 2020 at 6:27 AM Richard Biener wrote:
> > >
> > > On Fri, 17 Jul 2020, H.J. Lu wrote:
> > >
> > > > On Fri, Jul 17, 2020 at 12:08 AM Richard Biener
> > > >
On Fri, Jul 17, 2020 at 5:32 PM H.J. Lu via Gcc-patches
wrote:
>
> On Fri, Jul 17, 2020 at 6:27 AM Richard Biener wrote:
> >
> > On Fri, 17 Jul 2020, H.J. Lu wrote:
> >
> > > On Fri, Jul 17, 2020 at 12:08 AM Richard Biener wrote:
> > > >
> > > > On Wed, 15 Jul 2020, Joseph Myers wrote:
> > > >
Hi,
There was previously no way to specify that a register operand cannot
have any writeback modifiers, and as a result the argument to vldr.16
and vstr.16 could be erroneously output with post-increment. This
change adds a constraint which forbids all writeback, and
selects it in the relevant
This adds verification for the SCEV cache to avoid stale entries
that when picked up will lead to ICEs or other surprises.
Bootstrapped / tested on x86_64-unknown-linux-gnu with the two
previous fixes.
Posted for reference and archival purposes, I'm not going to push
this at this point.
This fixes two more places, in loop interchange and in the
vectorizer where the SCEV verifier sees stale entries.
Boostrapped/tested on x86_64-unknown-linux-gnu, pushed.
2020-07-29 Richard Biener
* tree-vectorizer.c (vectorize_loops): Reset the SCEV
cache if we removed any
This restores a lost setting of something_changed with the
recent refactoring of the substitute and fold engine. The
reported ICE in the PR was meanwhile mitigated in other ways
but the issue can still result in missed optimizations via
failed runs of CFG cleanup.
Bootstrapped/tested on
Committed as r11-2395,
https://gcc.gnu.org/g:269322ece17202632bc354e9c510e4a5bd6ad84b
with review comments applied plus a follow-up commit for an indentation issue I
missed:
r11-2399, https://gcc.gnu.org/g:6de5600a8bd1ef0ad3d57670efdcc68bb3484276
Tobias
On 7/29/20 10:23 AM, Jakub Jelinek
On 17/07/20 12:11 pm, Jonathan Wakely wrote:
N.B. the 0/6 entry of a patch series is supposed to be a cover letter
describing the changes in the series, not one of the patches.
If you have patches 0/6, 1/6, 2/6 ... 6/6 then you actually have seven
patches, not six!
Anyway ...
On 19/12/19
I eventually committed the attached patch which consider all your
remarks and moreover put back the move constructor implementation where
it used to be (not inline). It limits the size of the patch.
I also added comments on true_type/false_type new usages like you
advised on [Hashtable 0/6]
Jonathan Wakely via Gcc-patches writes:
>>diff --git a/libstdc++-v3/testsuite/lib/dg-options.exp
>>b/libstdc++-v3/testsuite/lib/dg-options.exp
>>index 9bfae71adf3..bf6219c3042 100644
>>--- a/libstdc++-v3/testsuite/lib/dg-options.exp
>>+++ b/libstdc++-v3/testsuite/lib/dg-options.exp
>>@@ -260,7
On 28/07/20 21:44 -0400, David Edelsohn via Libstdc++ wrote:
atomic_float/value_init.cc requires libatomic on some targets, i.e.,
when it tries to perform an atomic operation with a 64 bit floating
point double type on a 32 bit target. This patch adds AIX to the list
of targets that require the
在 2020年7月24日 +0800 PM9:19，Segher Boessenkool ，写道：
> On Fri, Jul 24, 2020 at 12:03:16PM +0200, Richard Biener via Gcc-patches
> wrote:
> > On Fri, Jul 24, 2020 at 10:13 AM Rainer Orth
> > wrote:
> > > Jojo R writes:
> > > > +insn-generated-split-num = $(shell grep -c ^processor /proc/cpuinfo)
>
Thanks for doing this. +1 for the best fix being to add XFAILing tests
to the main testsute, enabled by default. I don't see any other realistic
way of ensuring that fixes are matched with PRs at the time that the fix
is made (rather than some time after the fact).
Marek Polacek via Gcc-patches
在 2020年7月24日 +0800 PM9:57，Joseph Myers ，写道：
> On Fri, 24 Jul 2020, Jojo R wrote:
>
> > + -csplit insn-$*.c /parallel\ compilation/ -k -s
> > {$(insn-generated-split-num)} -f insn-$* -b "%d.c"
> > + -( [ ! -s insn-$*0.c ] && for i in {1..$(insn-generated-split-num)}; do
> > touch insn-$*$$i.c;
On Wed, Jul 29, 2020 at 10:10:35AM +0200, Tobias Burnus wrote:
> + case AB_OMP_REQ_REVERSE_OFFLOAD:
> +gfc_omp_requires_add_clause (OMP_REQ_REVERSE_OFFLOAD,
> + "reverse_offload",
> +
在 2020年7月24日 +0800 PM6:03，Richard Biener ，写道：
> On Fri, Jul 24, 2020 at 10:13 AM Rainer Orth
> wrote:
> >
> > Jojo R writes:
> >
> > > gcc/ChangeLog:
> > >
> > > * genemit.c (main): Print 'split line'.
> > > * Makefile.in (insn-emit.c): Define split count and file
> > [...]
> > > diff --git
On 7/29/20 10:01 AM, Jojo R wrote:
Thank you and it’s fixed in the next version patch
Hey.
One small note: can you please link the next v4 email thread to this one?
It's much easier to follow all the discussion related to your patch set.
Thanks,
Martin
And another update; I removed the global variable, which
was only used for checking at the end – and
reconstruct its value in a local variable just before calling
gfc_check_omp_requires.
(Following a suggestion by Jakub and removing a kind-of left
over from the first implementation.)
Tobias
On
On Wed, Jul 29, 2020 at 9:49 AM Stefan Schulze Frielinghaus
wrote:
>
> On Wed, Jul 29, 2020 at 09:11:12AM +0200, Richard Biener wrote:
> > On Tue, Jul 28, 2020 at 5:36 PM Stefan Schulze Frielinghaus
> > wrote:
> > >
> > > On Tue, Jul 28, 2020 at 08:55:57AM +0200, Richard Biener wrote:
> > > > On
在 2020年7月24日 +0800 PM4:12，Rainer Orth ，写道：
> Jojo R writes:
>
> > gcc/ChangeLog:
> >
> > * genemit.c (main): Print 'split line'.
> > * Makefile.in (insn-emit.c): Define split count and file
> [...]
> > diff --git a/gcc/Makefile.in b/gcc/Makefile.in
> > index 2ba76656dbf..75841e49127 100644
> >
On Wed, Jul 29, 2020 at 09:11:12AM +0200, Richard Biener wrote:
> On Tue, Jul 28, 2020 at 5:36 PM Stefan Schulze Frielinghaus
> wrote:
> >
> > On Tue, Jul 28, 2020 at 08:55:57AM +0200, Richard Biener wrote:
> > > On Mon, Jul 27, 2020 at 4:20 PM Stefan Schulze Frielinghaus
> > > wrote:
> > > >
>
On Tue, Jul 28, 2020 at 5:36 PM Stefan Schulze Frielinghaus
wrote:
>
> On Tue, Jul 28, 2020 at 08:55:57AM +0200, Richard Biener wrote:
> > On Mon, Jul 27, 2020 at 4:20 PM Stefan Schulze Frielinghaus
> > wrote:
> > >
> > > On Mon, Jul 27, 2020 at 12:29:11PM +0200, Richard Biener wrote:
> > > > On
70 matches
Mail list logo
