On Fri, Aug 28, 2020 at 6:29 AM Gary Oblock wrote:
>
> > If x_2 is a default def then the IL isn't correct in the first place. I
> > doubt
> > it is that way, btw. - we have verifiers that would blow up if it would.
>
> Richard,
>
> I'm just sharing this so you can tell me whether or not I'm
Pip Cet via Gcc writes:
> I may be missing an obvious workaround, but it seems we currently emit
> a #line directive when including lines from machine description files
> in C files, but never emit a second directive when switching back to
> the generated C file. This makes stepping through the
On Fri, 28 Aug 2020, Prathamesh Kulkarni via Gcc wrote:
> I wonder if that's (one of) the main factor(s) behind slowdown or it's
> not too relevant ?
Probably not. Some advice to make your search more directed:
Pass '-n' to 'perf report'. Relative sample ratios are hard to reason about
when
On Fri, Aug 28, 2020 at 1:24 PM Erick Ochoa
wrote:
>
> Hi,
>
> I'm testing the precision of IPA-PTA when compiling with -flto. I found
> this case when a global variable is marked as escaping even if the
> variable is a primitive type and no address is taken.
>
> This is the result of IPA-PTA
On Wed, 26 Aug 2020 at 16:50, Richard Biener wrote:
>
> On Wed, Aug 26, 2020 at 12:34 PM Prathamesh Kulkarni via Gcc
> wrote:
> >
> > Hi,
> > We're seeing a consistent regression >10% on calculix with -O2 -flto vs -O2
> > on aarch64 in our validation CI. I tried to investigate this issue a
> >
On Fri, Aug 28, 2020 at 1:17 PM Prathamesh Kulkarni
wrote:
>
> On Wed, 26 Aug 2020 at 16:50, Richard Biener
> wrote:
> >
> > On Wed, Aug 26, 2020 at 12:34 PM Prathamesh Kulkarni via Gcc
> > wrote:
> > >
> > > Hi,
> > > We're seeing a consistent regression >10% on calculix with -O2 -flto vs
>
Hi,
I'm testing the precision of IPA-PTA when compiling with -flto. I found
this case when a global variable is marked as escaping even if the
variable is a primitive type and no address is taken.
This is the result of IPA-PTA which I believe is wrong.
buff2 = { ESCAPED NONLOCAL }
buff1 = {
Hi,
This is the final report of the "Automatic Parallel Compilation
Viability" project. Please notice that this report is pretty
similar to the delivered from the 2nd evaluation, as this phase
consisted of mostly rebasing and bug fixing.
Please, reply this message for any question or
Snapshot gcc-9-20200828 is now available on
https://gcc.gnu.org/pub/gcc/snapshots/9-20200828/
and on various mirrors, see http://gcc.gnu.org/mirrors.html for details.
This snapshot has been generated from the GCC 9 git branch
with the following options: git://gcc.gnu.org/git/gcc.git branch
Hi,
On Thu, 27 Aug 2020 at 10:04, Richard Biener wrote:
>
> This makes sure to put special-ops expanded rhs left where
> expression rewrite expects it.
>
> Bootstrapped and tested on x86_64-unknown-linux-gnu, pushed.
>
> 2020-08-27 Richard Biener
>
> PR tree-optimization/96579
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96466
--- Comment #7 from Martin Liška ---
> As said you have to fake "regular" non-bool comparison operands,
> best based on the COND_EXPRs operand types (using integer types
> of the same size)
All right, do you mean something like what we do in
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96793
--- Comment #9 from Uroš Bizjak ---
(In reply to Paweł Bylica from comment #8)
> Did you consider fixing the __builtin_floor() implementation?
No, because you can use -msse4 to generate ROUNDxx instructions.
Re: [PATCH] lra: Canonicalize mult to shift in address reloads
Hi Christophe,
On 28/08/2020 10:16, Christophe Lyon wrote:
> Hi Alex,
>
>
> On Wed, 26 Aug 2020 at 17:15, Alex Coplan wrote:
> >
> > Thanks for the review, both.
> >
> > On 26/08/2020 09:19, Vladimir Makarov wrote:
> > >
> > > On
Thanks for doing this. In addition to what Segher said:
Jojo R writes:
> gcc/ChangeLog:
>
> * genemit.c (main): Print 'split line'.
> * Makefile.in (insn-emit.c): Define split count and file
>
> ---
> gcc/Makefile.in | 15 +
> gcc/genemit.c | 87
> So then you need to produce a changelog entry by hand.
I had this problem on some old Ubuntu 18.04. Anyway, here's new ChangeLog:
libstdc++-v3/ChangeLog:
* include/std/variant: Fix -Wignored-qualifiers
in system headers.
>That doesn't test this header at all.
It
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96624
--- Comment #3 from CVS Commits ---
The master branch has been updated by Paul Thomas :
https://gcc.gnu.org/g:9d463ce7f983f03f0d65da03cfa430e29a0840c2
commit r11-2919-g9d463ce7f983f03f0d65da03cfa430e29a0840c2
Author: Paul Thomas
Date: Fri
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96495
Paul Thomas changed:
What|Removed |Added
Assignee|unassigned at gcc dot gnu.org |pault at gcc dot gnu.org
Wei Wentao writes:
> Hi,
>
> This patch fix a typo in rtl.texi.
>
> Regards!
>
> Weiwt
>
> ---
> gcc/doc/rtl.texi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/gcc/doc/rtl.texi b/gcc/doc/rtl.texi
> index 501fa1a31da..f8e1f950823 100644
> --- a/gcc/doc/rtl.texi
> +++
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96829
Andrew Pinski changed:
What|Removed |Added
Resolution|--- |WONTFIX
Status|UNCONFIRMED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96827
--- Comment #1 from Andrew Pinski ---
Just in case, does adding -fno-strict-aliasing help?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96820
Richard Biener changed:
What|Removed |Added
Version|unknown |11.0
CC|
On Fri, Aug 28, 2020 at 10:48:43AM +0930, Alan Modra wrote:
> On Thu, Aug 27, 2020 at 03:17:45PM -0500, Segher Boessenkool wrote:
> > On Thu, Aug 27, 2020 at 01:51:25PM -0500, Bill Schmidt wrote:
> > It not the copy that is unnecessary: the preventing it *here*, manually,
> > is what is
On Mon, Aug 24, 2020 at 6:12 PM Jeff Law wrote:
> On Thu, 2020-08-06 at 12:42 +, Pip Cet via Gcc-patches wrote:
> > I've bootstrapped and run the test suite with the patch, without
> > differences.
> So it looks like Richard has given you some feedback and you've got some
> further
> work
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96818
--- Comment #8 from Richard Biener ---
I only see downsides forcing the same type in the IL, it makes propagation of
for example widening cast sources into switches much harder (need to convert
all case labels).
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96825
Martin Liška changed:
What|Removed |Added
Ever confirmed|0 |1
Last reconfirmed|
Actually ChangeLog also uses tabs, so it should be:
libstdc++-v3/ChangeLog:
* include/std/variant: Fix -Wignored-qualifiers
in system headers.
On Fri, 28 Aug 2020 at 08:55, Krystian Kuźniarek <
krystian.kuznia...@gmail.com> wrote:
> > So then you need to produce a changelog
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96827
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment #3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96832
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org,
> Sorry for the micromanagement, but I think this is easier to read if it flows
> as a
> single paragraph:
[snip...]
> I should have realised this would be the case, sorry, but now that there's
> only
> one rewrite, this should simply be:
>
> "&& reload_completed
>&& register_operand
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96832
Florian Weimer changed:
What|Removed |Added
CC||fw at gcc dot gnu.org
--- Comment #2
Hi,
attached patch fixes PR96418 where the code in the testsuite when compiled with
-fcoarray=single lead to an ICE. The reason was that the coarray object was
derefed as an array, but it was no array. Introducing the test for the
descriptor removes the ICE.
Regtests ok on x86_64-linux/FC31. Ok
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96820
--- Comment #5 from Martin Jambor ---
(In reply to Chengnian Sun from comment #1)
> Not sure if this is a dup to
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96730
No, this time it's build_user_friendly_ref_for_offset turning a nonsensical
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96827
--- Comment #2 from Richard Biener ---
Oddly enough with
> gcc-10 --version
gcc-10 (SUSE Linux) 10.2.1 20200805 [revision
dda1e9d08434def88ed86557d08b23251332c5aa]
Copyright (C) 2020 Free Software Foundation, Inc.
This is free software; see the
Ok, so here it is.
New diff:
diff --git a/libstdc++-v3/include/bits/atomic_base.h
b/libstdc++-v3/include/bits/atomic_base.h
index 015acef83c4..c4a763aae5c 100644
--- a/libstdc++-v3/include/bits/atomic_base.h
+++ b/libstdc++-v3/include/bits/atomic_base.h
@@ -231,7 +231,8 @@
gcc/ChangeLog:
* config.gcc (C-SKY): Add compatibility of elf target name.
libgcc/ChangeLog:
* config.host (C-SKY): Add compatibility of elf target name.
---
gcc/config.gcc | 2 +-
libgcc/config.host | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git
On 8/28/20 1:29 AM, Martin Sebor wrote:
With --enable-valgrind-annotations the change to the member function
signature in this patch triggers compilation errors during bootstrap:
I must confirm I didn't tested the configuration. Feel free to install
the patch, it's obvious.
Thank you,
Martin
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96793
Uroš Bizjak changed:
What|Removed |Added
CC|uros at gcc dot gnu.org|
Target Milestone|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96793
Uroš Bizjak changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned at
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96820
Martin Liška changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Ever confirmed|0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96624
Paul Thomas changed:
What|Removed |Added
Resolution|--- |FIXED
Status|NEW
Hi Alex,
On Wed, 26 Aug 2020 at 17:15, Alex Coplan wrote:
>
> Thanks for the review, both.
>
> On 26/08/2020 09:19, Vladimir Makarov wrote:
> >
> > On 2020-08-26 5:06 a.m., Richard Sandiford wrote:
> > > Alex Coplan writes:
> > >
> > > Minor nit, should be formatted as:
> > >
> > > static rtx
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96831
Bug ID: 96831
Summary: gcc.dg/tree-ssa/scev-[345].c FAIL on i?86 and arm
Product: gcc
Version: 11.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96832
Bug ID: 96832
Summary: Wrong assumption for -Werror=nonnull check
Product: gcc
Version: 10.2.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: c
Alex Coplan writes:
> Hello,
>
> Following on from the earlier patch to fix up the syntax for
> add/sub/adds/subs and friends with a sign/zero-extended operand [0],
> this patch removes the "mult" variants of these patterns which are
> all redundant.
>
> This patch removes the following patterns
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96834
Jakub Jelinek changed:
What|Removed |Added
CC||jakub at gcc dot gnu.org
--- Comment #1
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96798
--- Comment #3 from David Malcolm ---
Thanks. Unfortunately that didn't trigger the issue for me, sorry.
I'm wondering how best to debug this.
Could you attach the preprocessed source from the testcase(s) please?
Also, if you hack in a call
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96466
--- Comment #8 from rguenther at suse dot de ---
On Fri, 28 Aug 2020, marxin at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96466
>
> --- Comment #7 from Martin Liška ---
> > As said you have to fake "regular"
>Test suite confirmation:
>All tests pass. Tested on both Manjaro and some Ubuntu 18.04 with gcc10
and gcc8 respectively.
Jonathan, one more thing. I hope it's what you asked for cause all I did
was:
make bootstrap
make check
On Fri, 28 Aug 2020 at 08:32, Krystian Kuźniarek <
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96820
--- Comment #4 from Martin Liška ---
Where first bad value is 1840700270.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #12 from CVS Commits ---
The master branch has been updated by hongtao Liu :
https://gcc.gnu.org/g:58d6eea0e0754351b399a4b85562f81326a184ad
commit r11-2917-g58d6eea0e0754351b399a4b85562f81326a184ad
Author: liuhongt
Date: Wed Aug
On Aug 26, 2020, Qing Zhao wrote:
> There are two issues I can see with adding a default generator in middle end:
> 1. In order to determine where a target should not use the generic
> code to emit the zeroing sequence,
> a new target hook to determine this has to be added;
Yeah, a target hook
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96579
Christophe Lyon changed:
What|Removed |Added
CC||clyon at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96831
Richard Biener changed:
What|Removed |Added
CC||amker at gcc dot gnu.org,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96833
Bug ID: 96833
Summary: Functors within target offload code
Product: gcc
Version: 9.3.1
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: c++
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96793
--- Comment #11 from Uroš Bizjak ---
Created attachment 49146
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=49146=edit
Testcase, suitable for gcc testsuite
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96793
--- Comment #12 from Uroš Bizjak ---
(In reply to Richard Biener from comment #10)
> (In reply to Uroš Bizjak from comment #7)
> > Created attachment 49144 [details]
> > Proposed patch
> >
> > Patch in testing.
>
> OTOH we _do_ try to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96418
Andre Vehreschild changed:
What|Removed |Added
Status|ASSIGNED|WAITING
--- Comment #5 from Andre
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96834
Jakub Jelinek changed:
What|Removed |Added
Ever confirmed|0 |1
Summary|ICE： Segmentation
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96466
--- Comment #6 from rguenther at suse dot de ---
On Fri, 28 Aug 2020, marxin at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96466
>
> --- Comment #5 from Martin Liška ---
> Looking at the ICE, we actually ICE at the
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96793
--- Comment #8 from Paweł Bylica ---
Did you consider fixing the __builtin_floor() implementation?
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96834
Bug ID: 96834
Summary: ICE： Segmentation fault signal terminated program cc1
Product: gcc
Version: 11.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96831
--- Comment #2 from Richard Biener ---
I guess what is clearly missing is IVOPTs rewriting the ADDRESS uses with
the IV based on TARGET_MEM_REFs, thus [(int *) + _3 * 8]. Not sure
how hard that would be (do we even query targets for 'lea'
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96835
Bug ID: 96835
Summary: Constructor in offload template class
Product: gcc
Version: 10.1.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: target
Hu Jiangping writes:
> Hi,
>
> This patch add 'cd' command before 'make check-gcc' command
> when run the testsuite on selected tests.
>
> I think the implicit meaning of the original text is to
> execute the cd command to move to the gcc subdirectory of
> the object directory before executing
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96793
Richard Biener changed:
What|Removed |Added
CC||glisse at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96357
--- Comment #2 from CVS Commits ---
The master branch has been updated by Przemyslaw Wirkus :
https://gcc.gnu.org/g:b648814c02eb418aaf27897c480452172ee96303
commit r11-2922-gb648814c02eb418aaf27897c480452172ee96303
Author: Przemyslaw Wirkus
On Thu, Aug 27, 2020 at 3:20 PM Jakub Jelinek wrote:
>
> On Thu, Aug 27, 2020 at 03:07:59PM +0200, Richard Biener wrote:
> > > Also, isn't the pass also useful for TARGET_AVX and above (but in that
> > > case
> > > only if it is a simple memory load)? Or are avx/avx2 broadcast slower
> > >
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96744
--- Comment #13 from CVS Commits ---
The releases/gcc-10 branch has been updated by hongtao Liu
:
https://gcc.gnu.org/g:436ee9fec2751826bc48069d4dd320207fd9dfa4
commit r10-8684-g436ee9fec2751826bc48069d4dd320207fd9dfa4
Author: liuhongt
Date:
Hi,
On Thu, 27 Aug 2020 at 13:07, Richard Biener wrote:
>
> The following streamlines TARGET_MEM_REF dumping building
> on what we do for MEM_REF and thus dumping things like
> access type, TBAA type and base/clique. I've changed it
> to do semantic dumping aka base + offset + step * index
>
On Fri, Aug 28, 2020 at 10:52 AM Jakub Jelinek wrote:
>
> On Fri, Aug 28, 2020 at 08:47:06AM +0200, Richard Biener via Gcc-patches
> wrote:
> > IIRC elsewhere it was discussed to use ld to perform merging by
> > emitting separate rodata sections for constant sizes (4, 8, 16, 32, 64
> > byte
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96822
Richard Biener changed:
What|Removed |Added
Target Milestone|--- |11.0
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96825
Richard Biener changed:
What|Removed |Added
CC||marxin at gcc dot gnu.org
On Fri, Aug 28, 2020 at 3:26 AM Austin Morton via Gcc-patches
wrote:
>
> #pragma region is a feature introduced by Microsoft in order to allow
> manual grouping and folding of code within Visual Studio. It is
> entirely ignored by the compiler. Clang has supported this feature
> since 2012 when
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96466
--- Comment #5 from Martin Liška ---
Looking at the ICE, we actually ICE at the place where a fake comparison is
constructed:
if (TREE_CODE_CLASS (tcode) != tcc_comparison)
{
gcc_assert (VECTOR_BOOLEAN_TYPE_P (TREE_TYPE (op0)));
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96830
Bug ID: 96830
Summary: GCC does not complain template-head containing
requires clause
Product: gcc
Version: 10.2.0
Status: UNCONFIRMED
Severity: normal
On Fri, Aug 28, 2020 at 08:47:06AM +0200, Richard Biener via Gcc-patches wrote:
> IIRC elsewhere it was discussed to use ld to perform merging by
> emitting separate rodata sections for constant sizes (4, 8, 16, 32, 64
> byte sizes).
ld does that already, and gcc too.
> ld could always direct 8
On Fri, 28 Aug 2020, Christophe Lyon wrote:
> Hi,
>
>
> On Thu, 27 Aug 2020 at 13:07, Richard Biener wrote:
> >
> > The following streamlines TARGET_MEM_REF dumping building
> > on what we do for MEM_REF and thus dumping things like
> > access type, TBAA type and base/clique. I've changed it
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96830
--- Comment #1 from Jonathan Wakely ---
Reduced:
template concept C = requires { typename T::value_type; };
template
requires C
class Foo {
public:
void func();
};
template
void Foo::func()
{}
EDG compiles it without error too.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94672
--- Comment #13 from CVS Commits ---
The master branch has been updated by Tobias Burnus :
https://gcc.gnu.org/g:cb3c3d63315ceb4dc262e5efb83b42c73c43387d
commit r11-2923-gcb3c3d63315ceb4dc262e5efb83b42c73c43387d
Author: Tobias Burnus
Date:
On Fri, 28 Aug 2020 at 14:00, Richard Earnshaw
wrote:
>
> On 27/08/2020 14:27, Christophe Lyon via Gcc-patches wrote:
> > In comment 14 from PR94538, it was suggested to switch off jump tables
> > on thumb-1 cores when using -mpure-code, like we already do for thumb-2.
> >
> > This is what this
On 11/08/2020 12:36, Martin Liška wrote:
Hello.
All right, I did it in 3 steps:
1) - new exact argument is added (no default value) - I tested the on
x86_64-linux-gnu
and I build all cross targets.
2) set default value of exact = false
3) change places which calculate its own growth to use
On Fri, Aug 28, 2020 at 01:17:27AM -0500, Segher Boessenkool wrote:
> 1) Very many unnecessary moves are generated during expand *anyway*, a
> few more will not hurt;
> 2) In practice we always generate a move from a pseudo into r12 here,
> never a copy from r12 into r12;
> 3) Why generate dead
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96832
--- Comment #3 from Martin Sebor ---
I agree with recommending to use getcwd(0, 0).
At the same time, in https://sourceware.org/bugzilla/show_bug.cgi?id=25219#c6 I
suggested that "...perhaps the attribute should allow the pointer to be null
On 28/08/2020 15:27, Richard Earnshaw wrote:
> On 28/08/2020 14:24, Christophe Lyon via Gcc-patches wrote:
>> On Fri, 28 Aug 2020 at 14:00, Richard Earnshaw
>> wrote:
>>>
>>> On 27/08/2020 14:27, Christophe Lyon via Gcc-patches wrote:
In comment 14 from PR94538, it was suggested to switch
On 8/28/20 12:49 AM, Martin Liška wrote:
On 8/28/20 1:29 AM, Martin Sebor wrote:
With --enable-valgrind-annotations the change to the member function
signature in this patch triggers compilation errors during bootstrap:
I must confirm I didn't tested the configuration. Feel free to install
On Fri, Aug 28, 2020 at 12:36:00PM +0200, Richard Biener wrote:
> Guess this would work indeed. It's probably quite common to have
> both vector and non-vector constants because of vectorization
> and scalar epilogues. But note that elsewhere we're using
> the largest component mode to emit
On 27/08/2020 14:27, Christophe Lyon via Gcc-patches wrote:
> In comment 14 from PR94538, it was suggested to switch off jump tables
> on thumb-1 cores when using -mpure-code, like we already do for thumb-2.
>
> This is what this patch does, and also restores the previous value of
>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96793
--- Comment #14 from Marc Glisse ---
(In reply to Marc Glisse from comment #13)
> if (HONOR_SIGNED_ZEROS (mode))
> x2 = copysign (x2, x);
Hmm, I misread the comment, sorry. We already do that, for both floor and ceil.
But we
On Thu, Aug 27, 2020 at 10:00 AM Uros Bizjak wrote:
>
> On Thu, Aug 27, 2020 at 4:45 PM H.J. Lu wrote:
>
> > > > > > > How about target("baseline-isas-only")? All CPUID functions are
> > > > > > > inlined.
> > > > > >
> > > > > > No, I don't think this is a good idea. Now consider the situation
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96837
--- Comment #1 from Jakub Jelinek ---
That is not about if(false), but rather about nested parallelism.
libgomp has a thread cache only for the outermost parallel level, with nested
parallelism the threads are spawned and joined each time a
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96827
gcc at froghat dot ca changed:
What|Removed |Added
Attachment #49142|0 |1
is obsolete|
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96793
--- Comment #16 from Paweł Bylica ---
I have checked the glibc implementation of floorf().
Source here:
https://sourceware.org/git/?p=glibc.git;a=blob;f=sysdeps/ieee754/flt-32/s_floorf.c;h=da6c6dfa8ae86129e74d2e4391fac3a3c2ec;hb=HEAD
- It
On 28/08/2020 15:26, Martin Liška wrote:
On 8/28/20 4:24 PM, Andrew Stubbs wrote:
Should I just add "true" to fix it? That's enough to make it build.
Yes, please and push it as obvious.
I've committed the attached.
Andrew
amdgcn: Update vec_safe_grow_cleared usage
An API change broke the
Hi Tobias,
the patch looks ok to me.
Thanks for the patch.
Regards,
Andre
On Wed, 26 Aug 2020 18:29:40 +0200
Tobias Burnus wrote:
> This fixes an issue caused by the patch for PR 94672, which
> affects both GCC 10 and GCC 11.
>
> Only 'sVal' of 'subroutine foo' was affected, the
On 8/28/20 7:25 AM, Alan Modra wrote:
On Fri, Aug 28, 2020 at 01:17:27AM -0500, Segher Boessenkool wrote:
1) Very many unnecessary moves are generated during expand *anyway*, a
few more will not hurt;
2) In practice we always generate a move from a pseudo into r12 here,
never a copy from r12
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96827
--- Comment #5 from gcc at froghat dot ca ---
I was testing modifications to the testcase that caused it to compile correctly
and forgot to save my editor's buffer after undoing those changes.
I have attached the correct file. Sorry for being
On Fri, Aug 28, 2020 at 01:06:40PM +0200, Richard Biener via Gcc-patches wrote:
> > I don't see why it would break, it will not optimize { -1LL, -1LL }
> > vs. -1 scalar, sure, but it uses the hash and equality function the
> > rtl constant pool uses, which means it compares both the constants
> >
On Tue, 25 Aug 2020, Martin Liška wrote:
> I noticed your continual effort to change http:// links to https://.
> I've written a simple script that can do that in automatic way.
Thanks, Martin! My missions isn't actually changing http:// to https://
where possible, but checking for broken links
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96768
--- Comment #3 from Richard Earnshaw ---
Note that the switch table is in the .rodata section, so that's not a problem.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=96768
--- Comment #4 from Christophe Lyon ---
That's what I replied in the original PR94538, but Wilco said the best option
was to turn off switch tables:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=94538#c14
See also another comment from him:
1 - 100 of 178 matches
Mail list logo
