On Sat, Jul 6, 2019 at 1:26 AM Katsuhiro Suzuki wrote:
> This patch fixes ambiguous mode of some compare insns of RISC-V.
> Only sge, slt and sle are using but other compare insns use
> . It seems first group mode settings are ambiguous.
Richard Sandiford submitted a patch that fixes every
On Fri, Jul 5, 2019 at 11:19 PM Richard Sandiford
wrote:
> gcc/
> * config/riscv/pic.md (*local_pic_load_s)
> (*local_pic_load_u): Explicitly specify the mode iterator
> referenced by , giving...
> (*local_pic_load_s, *local_pic_load_u):
> ...these.
> *
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=80518
--- Comment #6 from Eric Gallager ---
(In reply to Arnaud Desitter from comment #2)
> Interesting. Shame that there is no rationale.
>
> I suppose that "-Wsuggest-override=2" could warn about "override" missing
> for destructor.
I'd just like
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87274
--- Comment #5 from Eric Gallager ---
(In reply to Manuel López-Ibáñez from comment #4)
> (In reply to Jonathan Wakely from comment #2)
> > (In reply to Patrick J. LoPresti from comment #0)
> > > Note that my code does not use any quad-precision
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77510
Eric Gallager changed:
What|Removed |Added
CC||mfortune at gmail dot com
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=44566
Eric Gallager changed:
What|Removed |Added
CC||froydnj at gcc dot gnu.org,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61577
--- Comment #59 from EML ---
Created attachment 46575
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46575=edit
islower 32 bit test assembler
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61577
--- Comment #58 from EML ---
32 Bit:
Second Test Program (derived from a crashing conftest)
#include
int
main ()
{
int j = 0;
j = islower(0);
printf("j is: %d\n", j);
return 0;
}
[.LCFI2:]
mov r35 = r1
.body
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84877
--- Comment #16 from Hans-Peter Nilsson ---
(In reply to Jorn Wolfgang Rennecke from comment #13)
> I tried if I could reproduce this with a cross-compiler built for
> --target=hppa-linux-gnu;
My target of concern is cris-elf, just as in my
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84877
--- Comment #15 from Jorn Wolfgang Rennecke ---
Created attachment 46574
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46574=edit
patch for the case that the stack is sufficiently aligned
(In reply to dave.anglin from comment #11)
> $sp
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84877
--- Comment #14 from John David Anglin ---
Created attachment 46573
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=46573=edit
Preproccessed source for pr84877.c
I don't believe there are any new failures or ICEs on hppa-linux with your
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84877
--- Comment #13 from Jorn Wolfgang Rennecke ---
(In reply to Hans-Peter Nilsson from comment #12)
> (In reply to Jorn Wolfgang Rennecke from comment #10)
> > Created attachment 46567 [details]
> > Fix for targets that pass the argument by
Snapshot gcc-10-20190707 is now available on
ftp://gcc.gnu.org/pub/gcc/snapshots/10-20190707/
and on various mirrors, see http://gcc.gnu.org/mirrors.html for details.
This snapshot has been generated from the GCC 10 SVN branch
with the following options: svn://gcc.gnu.org/svn/gcc/trunk revision
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91102
--- Comment #5 from Vladimir Makarov ---
(In reply to Jakub Jelinek from comment #1)
>
> Vlad, could you please have a look?
The culprit patch is actually
https://gcc.gnu.org/viewcvs/gcc?view=revision=265942
Prohibiting reload for any user
On Sun, Jul 7, 2019 at 9:48 PM Akshat Garg wrote:
> On Sun, Jul 7, 2019 at 7:49 PM Paul E. McKenney
> wrote:
>
>> On Sat, Jul 06, 2019 at 12:39:45PM +0530, Akshat Garg wrote:
>> > On Sat, Jul 6, 2019 at 1:09 AM Akshat Garg wrote:
>> >
>> > > On Fri, Jul 5, 2019 at 7:18 AM Paul E. McKenney <
>>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61577
--- Comment #57 from dave.anglin at bell dot net ---
On 2019-07-05 7:57 p.m., elowe at elowe dot com wrote:
> Doing some more testing on my "gprel unfixed fix" 32-bit gcc, I found out that
> it seems to be missing the 32-bit pointer swizzling
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91030
--- Comment #37 from Janne Blomqvist ---
One thing we could do would be to switch to pread and pwrite instead of using
lseek. That would avoid a few syscalls when updating the record length marker.
Though I guess the issue with GPFS isn't
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91030
Janne Blomqvist changed:
What|Removed |Added
CC||jb at gcc dot gnu.org
--- Comment #36
Hello world,
the attached patch sets the I/O block size for unformatted files to
2**17 and makes this, and the block size for formatted files,
adjustable via environment variables.
The main reason is that -fconvert=big-endian was quite slow on
some HPC systems. A bigger buffer should eliminate
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=61339
Martin Sebor changed:
What|Removed |Added
Status|NEW |ASSIGNED
Assignee|unassigned
On 7/7/19 3:45 AM, Richard Sandiford wrote:
> DCE tries to delete dead stores to local data and also tries to insert
> debug binds for simple cases:
>
> /* If this is a store into a variable that is being optimized away,
> add a debug bind stmt if possible. */
> if
On Sun, Jul 07, 2019 at 10:41:43AM +0100, Richard Sandiford wrote:
> gcc/testsuite/
> * c-c++-common/guality/Og-static-wo-1.c: New test.
> * g++.dg/guality/guality.exp: Separate the c-c++-common tests into
> "Og" and "general" tests. Run the latter at -O0 and -Og only.
Do we
On 7/7/19 3:43 AM, Richard Sandiford wrote:
> This patch stops gimple and rtl DSE from running by default at -Og.
> The idea is both to improve compile time and to stop us from deleting
> stores that we can't track in debug info.
>
> We could rein this back in future for stores to local variables
On 7/7/19 3:41 AM, Richard Sandiford wrote:
> This patch prevents -Og from deleting stores to write-only variables,
> so that the values are still available when debugging. This seems
> more convenient than forcing users to use __attribute__((used))
> (probably conditionally, if it's not
On 7/7/19 3:40 AM, Richard Sandiford wrote:
> Although BOOT_CFLAGS can be used to bootstrap with -Og, having a
> dedicated build config is sometimes more convenient.
>
> Tested on aarch64-linux-gnu and x86_64-linux-gnu. OK to install?
>
> Richard
>
>
> 2019-07-07 Richard Sandiford
>
>
On 7/7/19 3:39 AM, Richard Sandiford wrote:
> guality.exp is silently skipped on Linux systems with
> kernel.yama.ptrace_scope=1 because gdb fails to attach to the
> sanity check test. This patch uses PR_SET_PTRACER (where available)
> to avoid this.
>
> prctl was apparently added in Linux
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91090
Jeffrey A. Law changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
This code was copied and reindented from tree-vrp.c. It looks like in
that process the second conditional moved inside the true arm of the
first conditional, which was wrong.
It won't cause incorrect code, just missed optimizations. I also
verified that the code is still live -- it triggers
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91090
--- Comment #3 from Jeffrey A. Law ---
Author: law
Date: Sun Jul 7 18:42:45 2019
New Revision: 273184
URL: https://gcc.gnu.org/viewcvs?rev=273184=gcc=rev
Log:
PR tree-optimization/91090
* tree-ssa-dom.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84877
--- Comment #12 from Hans-Peter Nilsson ---
(In reply to Jorn Wolfgang Rennecke from comment #10)
> Created attachment 46567 [details]
> Fix for targets that pass the argument by invisible reference
Thanks for your efforts. This *may* have
Currently the second lower-subreg pass is run right before RA. This
is much too late to be very useful. At least for targets that do not
have RTL patterns for operations on multi-register modes it is a lot
better to split patterns earlier, before combine and all related
passes.
This adds an
In some cases the cascaded settings for pic/pie/PIC/PIE can cause a confusing
scenario for Darwin where "fPIC fno-PIE” produces a wrong code generation
setting.
So we make a final check on PIC options; for Darwin these are not dependent on
the PIE ones, although PIE does require PIC to support
For PPC Darwin, we need the JBSR long jump code to be enabled when generating
kernel code. Now we have that handled in rs6000.c, we can drop the conflated
setting in the common code. The gcc-4.2.1 compilers don’t generate symbol stubs
for any X86 case.
tested on x86_64-darwin16,
applied to
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91107
Bug ID: 91107
Summary: __attribute__((pure)) to function with non-const
pointers
Product: gcc
Version: 8.3.0
Status: UNCONFIRMED
Severity: normal
It is unused. Committing.
2019-07-07 Segher Boessenkool
* config/rs6000/rs6000.md (Ffre): Delete.
---
gcc/config/rs6000/rs6000.md | 1 -
1 file changed, 1 deletion(-)
diff --git a/gcc/config/rs6000/rs6000.md b/gcc/config/rs6000/rs6000.md
index 9e81df9..3cc42cb6 100644
---
Thanks, Steve.
Fixed with revisions 273176, 7 & 8.
Paul
On Sat, 6 Jul 2019 at 18:05, Steve Kargl
wrote:
>
> On Sat, Jul 06, 2019 at 02:29:06PM +0100, Paul Richard Thomas wrote:
> > As anticipated, 8-branch required a different patch but the difference
> > was much smaller than anticipated.
> >
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91077
Paul Thomas changed:
What|Removed |Added
Status|NEW |RESOLVED
Resolution|---
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=84877
--- Comment #11 from dave.anglin at bell dot net ---
On 2019-07-06 3:39 p.m., amylaar at gcc dot gnu.org wrote:
> Since the esirisc port is not in the FSF tree, it doesn't really count for
> testing; also, the behaviour will vary depending on
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91077
--- Comment #10 from Paul Thomas ---
Author: pault
Date: Sun Jul 7 14:50:53 2019
New Revision: 273178
URL: https://gcc.gnu.org/viewcvs?rev=273178=gcc=rev
Log:
2019-07-07 Paul Thomas
PR fortran/91077
* trans-array.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90813
--- Comment #28 from rguenther at suse dot de ---
On July 7, 2019 2:59:29 PM GMT+02:00, "tkoenig at gcc dot gnu.org"
wrote:
>https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90813
>
>--- Comment #27 from Thomas Koenig ---
>(In reply to Richard
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91077
--- Comment #9 from Paul Thomas ---
Author: pault
Date: Sun Jul 7 14:32:53 2019
New Revision: 273177
URL: https://gcc.gnu.org/viewcvs?rev=273177=gcc=rev
Log:
2019-07-07 Paul Thomas
PR fortran/91077
* trans-array.c
O compression algorithms: zlib
gcc version 10.0.0 20190707 (experimental) (GCC)
-COMMAND LINE USED FOR COMPILATION-
~/pull-martin-compiler/build/gcc/xgcc -B ~/pull-martin-compiler/build/gcc/ -O0
crash2.i -w
during RTL pass: final
crash2.c: In function ‘func_1’:
c
-pc-linux-gnu
Configured with: ../gcc/configure --enable-languages=c,c++ --enable-lto
--disable-bootstrap : (reconfigured) ../gcc/configure --enable-lto
--disable-bootstrap --enable-languages=c,c++,lto --no-create --no-recursion
Thread model: posix
Supported LTO compression algorithms: zlib
gcc version 10
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91100
--- Comment #2 from dave.anglin at bell dot net ---
On 2019-07-07 4:10 a.m., ebotcazou at gcc dot gnu.org wrote:
> Let's just disable the test on HP-UX as it is on Solaris.
Okay. The pass/fail seems inconsistent.
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90813
--- Comment #27 from Thomas Koenig ---
(In reply to Richard Biener from comment #26)
> The odd thing is that for the Fortran example even the tree oracle claims
> the store doesn't alias the load...
>
> Because they are different variables!
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91104
Bug ID: 91104
Summary: auto... in lambda expression reorders parameters
Product: gcc
Version: 9.1.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91060
--- Comment #6 from Richard Biener ---
hmm, armeb-linux doesn't use newlib. awkward to set up a cross :/
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91060
Richard Biener changed:
What|Removed |Added
CC||rsandifo at gcc dot gnu.org
---
From: "Dragan Mladjenovic"
This patch prevents merging of CALL instructions that that have different
REG_CALL_DECL notes attached to them.
On most architectures this is not an important distinction. Usually instruction
patterns
for calls to different functions reference different SYMBOL_REF-s,
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91077
--- Comment #8 from Paul Thomas ---
Author: pault
Date: Sun Jul 7 10:53:37 2019
New Revision: 273176
URL: https://gcc.gnu.org/viewcvs?rev=273176=gcc=rev
Log:
2019-07-07 Paul Thomas
PR fortran/91077
* trans-array.c
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85746
Marc Glisse changed:
What|Removed |Added
CC||rsandifo at gcc dot gnu.org
--- Comment
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91068
rsandifo at gcc dot gnu.org changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91068
--- Comment #2 from rsandifo at gcc dot gnu.org
---
Author: rsandifo
Date: Sun Jul 7 09:49:24 2019
New Revision: 273175
URL: https://gcc.gnu.org/viewcvs?rev=273175=gcc=rev
Log:
PR91068: Fix MIPS fallout from IRA matched operand changes
DCE tries to delete dead stores to local data and also tries to insert
debug binds for simple cases:
/* If this is a store into a variable that is being optimized away,
add a debug bind stmt if possible. */
if (MAY_HAVE_DEBUG_BIND_STMTS
&& gimple_assign_single_p (stmt)
&&
This patch stops gimple and rtl DSE from running by default at -Og.
The idea is both to improve compile time and to stop us from deleting
stores that we can't track in debug info.
We could rein this back in future for stores to local variables
with is_gimple_reg_type, but at the moment we don't
This patch prevents -Og from deleting stores to write-only variables,
so that the values are still available when debugging. This seems
more convenient than forcing users to use __attribute__((used))
(probably conditionally, if it's not something they want in release
builds).
Tested on
Although BOOT_CFLAGS can be used to bootstrap with -Og, having a
dedicated build config is sometimes more convenient.
Tested on aarch64-linux-gnu and x86_64-linux-gnu. OK to install?
Richard
2019-07-07 Richard Sandiford
config/
* bootstrap-Og.mk: New file.
gcc/
*
guality.exp is silently skipped on Linux systems with
kernel.yama.ptrace_scope=1 because gdb fails to attach to the
sanity check test. This patch uses PR_SET_PTRACER (where available)
to avoid this.
prctl was apparently added in Linux 2.1.57, so I don't think we
need any tests other than __linux
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90813
--- Comment #26 from Richard Biener ---
(In reply to Segher Boessenkool from comment #25)
> At expand time, the assignment is
>
>
> ;; c_ = c;
>
> (insn 35 34 36 (set (reg/f:DI 140)
> (unspec:DI [
> (symbol_ref:DI
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91102
--- Comment #4 from Jakub Jelinek ---
(In reply to Segher Boessenkool from comment #2)
> 91102.c: In function 'foo':
> 91102.c:7:1: warning: control reaches end of non-void function
> [-Wreturn-type]
> 7 | }
> | ^
>
> so
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85746
--- Comment #6 from Marc Glisse ---
The problematic file seems to be tree-dfa.o, and the only difference I see in
the .original dump is the use of __builtin_constant_p in wi::lshift (3
occurrences because of templates)
- if (precision
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=91100
Eric Botcazou changed:
What|Removed |Added
Status|UNCONFIRMED |NEW
Last reconfirmed|
Good morning.
I have some code that looks like
typedef unsigned long long uint64;
typedef unsigned int uint32;
typedef struct { uint64 x[8]; } __attribute__((aligned(64))) v_t;
inline v_t xor(v_t a, v_t b)
{
v_t Q;
for (int i=0; i<8; i++) Q.x[i] = a.x[i] ^ b.x[i];
return Q;
}
void
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90756
--- Comment #19 from Mike Hommey ---
Applying the patch from PR90139 and this one works fixes the ICEs on this file
with both GCC 7 and 8.
64 matches
Mail list logo
