On 25 April 2018 16:11:23 CEST, Richard Biener wrote:
>On Wed, 25 Apr 2018, Jakub Jelinek wrote:
>
>> On Wed, Apr 25, 2018 at 03:52:28PM +0200, Richard Biener wrote:
>> > Forcefully setting STAGE3_[CT]FLAGS doesn't have any effect on
>> > a checking enabled build but it will disrupt profiledbootst
The following patch restores checking of integer<->pointer conversions
which I broke with a cut&paste error when introducing ptrofftype_p
and friends. It also contains a fix from Jakub for the fallout in
omp expansion.
Bootstrapped and tested on x86_64-unknown-linux-gnu with all languages,
appli
On Thu, 26 Apr 2018, Bernhard Reutner-Fischer wrote:
> On 25 April 2018 16:11:23 CEST, Richard Biener wrote:
> >On Wed, 25 Apr 2018, Jakub Jelinek wrote:
> >
> >> On Wed, Apr 25, 2018 at 03:52:28PM +0200, Richard Biener wrote:
> >> > Forcefully setting STAGE3_[CT]FLAGS doesn't have any effect on
On Wed, Apr 25, 2018 at 7:10 PM, Nathan Sidwell wrote:
> On 04/25/2018 11:41 AM, David Malcolm wrote:
>>
>> Jason Turner's video C++ Weekly - Ep 112 - GCC's Leaky Abstractions shows
>> two issues where g++ offers suggestions about implementation details:
>
>
>> For the lambda capture case, there a
On Wed, Apr 25, 2018 at 8:54 PM, Jonathan Wakely wrote:
> * doc/invoke.texi (-Wreturn-type): Document default status for C++.
>
> OK for trunk and gcc-8-branch?
OK.
>
On Wed, Apr 25, 2018 at 10:41 PM, Andreas Tobler
wrote:
> Hi all,
>
> I'm going to commit this patch to all active branches as soon as the branch
> status permits.
> Built and tested on native armv5 FreeBSD12.
If you think it is safe then it's fine for gcc-8-branch now.
Richard.
> Thanks,
> And
On Thu, Apr 26, 2018 at 3:40 AM, David Edelsohn wrote:
> Jakub and Richi,
>
> GCC 8.1 is experiencing the same bootstrap failure with GCC 8.1 RC1 as
> we saw previously.
>
> Bootstrap comparison failure!
> gcc/function-tests.o differs
>
> And the same reason: unique, static symbol that includes a
On Thu, Apr 26, 2018 at 09:56:30AM +0200, Richard Biener wrote:
> On Thu, Apr 26, 2018 at 3:40 AM, David Edelsohn wrote:
> > Jakub and Richi,
> >
> > GCC 8.1 is experiencing the same bootstrap failure with GCC 8.1 RC1 as
> > we saw previously.
> >
> > Bootstrap comparison failure!
> > gcc/function
On Thu, 26 Apr 2018, Richard Biener wrote:
> On Thu, 26 Apr 2018, Bernhard Reutner-Fischer wrote:
>
> > On 25 April 2018 16:11:23 CEST, Richard Biener wrote:
> > >On Wed, 25 Apr 2018, Jakub Jelinek wrote:
> > >
> > >> On Wed, Apr 25, 2018 at 03:52:28PM +0200, Richard Biener wrote:
> > >> > Force
On Thu, Apr 26, 2018 at 09:56:30AM +0200, Richard Biener wrote:
> --- gcc/cgraph.h(revision 259668)
> +++ gcc/cgraph.h(working copy)
> @@ -,7 +,7 @@ public:
>void dump (FILE *f);
>
>/* Dump symbol table to stderr. */
> - inline DEBUG_FUNCTION void debug (void)
> +
On Thu, Apr 26, 2018 at 10:08:09AM +0200, Richard Biener wrote:
> Ok, so the following passed bootstrap & profiledbootstrap (and I checked
> we only get -fno-checking at the appropriate places and not -fchecking)
> on trunk, a bootstrap with all languages and release checking is running
> (just to
In PR85116 there's a loop which isn't optimized well, first because
it is not header copied. The reason is the do_while_loop_p predicate is
"strange". The following adds the natural test - whether the single
latch predecessor exits the loop. It also removes the check that
claims that a loop hea
On Thu, Apr 26, 2018 at 10:28:52AM +0200, Jakub Jelinek wrote:
> On Thu, Apr 26, 2018 at 09:56:30AM +0200, Richard Biener wrote:
> > --- gcc/cgraph.h(revision 259668)
> > +++ gcc/cgraph.h(working copy)
> > @@ -,7 +,7 @@ public:
> >void dump (FILE *f);
> >
> >/* Dump
Hi Martin!
On Wed, 25 Apr 2018 15:56:26 +0200, Martin Jambor wrote:
> On Thu, Apr 19 2018, Thomas Schwinge wrote:
> > * testsuite/libgomp.oacc-fortran/error_stop-1.f: New file.
> > * testsuite/libgomp.oacc-fortran/error_stop-2.f: Likewise.
> > * testsuite/libgo
Hi,
response files were added to the GNU toolchain to work around length limits
put on the command line by some OSes, typically Windows. For the compiler,
the implementation is as follows: if a response file is passed to the driver
at link time, typically containing the list of object files, t
I wanted to add '-' as a synonym for stdout in the dumping machinery. But this
bit of cleanup's needed first. We check the special stdout/stderr names in 3
different places -- that's crying out for a worker function. We also check
those streams by name before determining whether to fclose. I
Seen by Christophe Lyon, verified with a cross that this fixes the
issue.
aarch64 people, can you please test & commit this?
Thanks,
Richard.
2018-04-26 Richard Biener
* config/aarch64/aarch64.c: Simplify ap.__stack advance and
fix for ILP32.
Index: gcc/config/aarch64/aarc
On Thu, 26 Apr 2018, Richard Biener wrote:
>
> In PR85116 there's a loop which isn't optimized well, first because
> it is not header copied. The reason is the do_while_loop_p predicate is
> "strange". The following adds the natural test - whether the single
> latch predecessor exits the loop.
The following makes loop-header copying stop after the first exit test
it copied. The reports rightfully complain about too much peeling.
If some cases pop up which show we should peel up to a specific test
we need to improve this heuristic which simply errs on the easy side.
Bootstrap & regtest
Hi,
On Thu, Apr 26 2018, Thomas Schwinge wrote:
> Hi Martin!
>
> On Wed, 25 Apr 2018 15:56:26 +0200, Martin Jambor wrote:
>> On Thu, Apr 19 2018, Thomas Schwinge wrote:
>> > * testsuite/libgomp.oacc-fortran/error_stop-1.f: New file.
>> > * testsuite/libgomp.oacc-fortran/er
On 26 April 2018 at 14:09, Richard Biener wrote:
>
> Seen by Christophe Lyon, verified with a cross that this fixes the
> issue.
>
> aarch64 people, can you please test & commit this?
>
As I have just written in bugzilla, this patch avoids an ICE when
compiling newlib's sysopen.i,
but I still se
Kyrill Tkachov writes:
> Hi all,
>
> I totally botched up this sve test file in 259437.
> It needs C++, so move it to g++.dg/other and make it a .C file.
> Also adds the target guards to prevent it from running on non-aarch64 targets.
>
> Tested that it passes on aarch64-none-elf and doesn't get
On Thu, 26 Apr 2018, Christophe Lyon wrote:
> On 26 April 2018 at 14:09, Richard Biener wrote:
> >
> > Seen by Christophe Lyon, verified with a cross that this fixes the
> > issue.
> >
> > aarch64 people, can you please test & commit this?
> >
>
> As I have just written in bugzilla, this patch
Hi,
for the attached testcase compiled/linked with -shared -fPIC -O, the Solaris
linker chokes on SPARC because the generated code uses the local-dynamic TLS
model for a global symbol:
ld: fatal: relocation error: R_SPARC_TLS_LDM_HI22: file /var/tmp//ccKG6GQC.o:
symbol Test::blah(int)::mything
Kyrill Tkachov writes:
> On 24/04/18 17:41, Jakub Jelinek wrote:
>> On Tue, Apr 24, 2018 at 05:22:15PM +0100, Kyrill Tkachov wrote:
>>> I've cleaned up the testcase a bit to leave only the function that
>>> generates the invalid instruction,
>>> making it shorter.
>>>
>>> Jakub, is the patch ok t
On 26 April 2018 at 15:41, Richard Biener wrote:
> On Thu, 26 Apr 2018, Christophe Lyon wrote:
>
>> On 26 April 2018 at 14:09, Richard Biener wrote:
>> >
>> > Seen by Christophe Lyon, verified with a cross that this fixes the
>> > issue.
>> >
>> > aarch64 people, can you please test & commit thi
On Thu, Apr 26, 2018 at 2:00 PM, Nathan Sidwell wrote:
> I wanted to add '-' as a synonym for stdout in the dumping machinery. But
> this bit of cleanup's needed first. We check the special stdout/stderr
> names in 3 different places -- that's crying out for a worker function. We
> also check t
Hi,
when stack checking is enabled in Ada, it is supposed to be able to handle the
case of a recursive function that does essentially nothing. But in this case
the IPA machinery will compute that the function is nothrow, which means that
the Storage_Error exception cannot be caught by the call
This fixes a wrong loop-invariant motion applied to an adjustment of the frame
pointer generated by expand_builtin_setjmp_receiver, e.g. for a target using
the SJLJ exception handling scheme.
Tested on x86-64/Linux, applied on the mainline.
2018-04-26 Eric Botcazou
* loop-invariant
Here's the patch to allow '-' as a synonym for 'stdout'. It's easier to type,
and a convention used elsewhere.
Also document the existing stdout/stderr selection along with the new behaviour?
ok for trunk?
nathan
--
Nathan Sidwell
2018-04-26 Nathan Sidwell
* dumpfile.c (dump_open): Allow
On 04/26/2018 11:12 AM, Nathan Sidwell wrote:
Index: doc/invoke.texi
===
--- doc/invoke.texi (revision 259680)
+++ doc/invoke.texi (working copy)
@@ -13368,9 +13368,10 @@ Says to make debugging dumps during comp
@var{letter
On Thu, Apr 26, 2018 at 9:42 AM, Eric Botcazou wrote:
> Hi,
>
> for the attached testcase compiled/linked with -shared -fPIC -O, the Solaris
> linker chokes on SPARC because the generated code uses the local-dynamic TLS
> model for a global symbol:
>
> ld: fatal: relocation error: R_SPARC_TLS_LDM_
On Thu, Apr 26, 2018 at 3:45 AM, Richard Biener
wrote:
> On Wed, Apr 25, 2018 at 7:10 PM, Nathan Sidwell wrote:
>> On 04/25/2018 11:41 AM, David Malcolm wrote:
>>>
>>> Jason Turner's video C++ Weekly - Ep 112 - GCC's Leaky Abstractions shows
>>> two issues where g++ offers suggestions about imple
Hi!
ICC apparently has these two intrinsics (why it doesn't have a maskz_ one
is unclear to me) which are like _mm512_{,mask_}mullo_epi64, except they are
available in AVX512F rather than just AVX512DQ and if AVX512DQ is not
enabled they expand to 3 vpmuludq instructions + 3 shifts + 2 adds; for
A
Hi!
As explained in the comment below, when doing the inter-bb range test
optimization and are trying to optimize the
a >= 0 && a < b
where b is known to be >= 0 into
(unsigned) a < (unsigned) b, we need to be careful if the a >= 0 condition
is in a different bb from the a < b one and the a >= 0 b
On Wed, 18 Apr 2018, Matthew Fortune wrote:
> OK to apply but given the release date I've added RMs to approve for
> trunk right now.
Applied to trunk, now that GCC 8 has branched. Thanks for your review.
Maciej
On Wed, 18 Apr 2018, Matthew Fortune wrote:
> Apologies as before on slowness. This looks trivially OK to me but that's
> thanks to the description, much appreciated.
You are welcome, that just means I did my homework right.
> OK for commit but adding RMs given the imminent release.
Applied t
On Thu, 2018-04-26 at 15:53 -0400, Jason Merrill wrote:
> On Thu, Apr 26, 2018 at 3:45 AM, Richard Biener
> wrote:
> > On Wed, Apr 25, 2018 at 7:10 PM, Nathan Sidwell
> > wrote:
> > > On 04/25/2018 11:41 AM, David Malcolm wrote:
> > > >
> > > > Jason Turner's video C++ Weekly - Ep 112 - GCC's Le
Hi, Richi
This patches causes a boostrap failure on AIX. Everything miscompares.
The code itself is the same, but the DWARF debug information contains many
differences.
- David
This patch is aimed at removing bogus .eh_frame info emitted after the
zero terminator in crtend.o, which will cause a ld warning and slow
exception handling. The right fix for the PR is probably to change
libgcc/Makefile.in to always supply -fno-asynchronous-unwind-tables
when building crtbegin.o
On 04/19/2018 03:43 AM, Andrew Sadek wrote:
On Wed, Apr 18, 2018 at 6:57 PM, Michael Eager wrote:
Hi Andrew --
Check indents in the following files:
(Make sure that your tabs are set at 8 chars.)
--- gcc/config/microblaze/microblaze.c
--- gcc/config/microblaze/microblaze.md
I have re-run ch
This patch adds ATTRIBUTE_NONSTRING, which will be used to curb
-Wstringop-truncation warnings in binutils. OK to apply?
* ansidecl.h (ATTRIBUTE_NONSTRING): Define.
diff --git a/include/ansidecl.h b/include/ansidecl.h
index c11daff..ec5f34d 100644
--- a/include/ansidecl.h
+++ b/include/a
I'm thinking of going back on my choice not to fix special function bugs
on 7.
These build and test clean on gcc-7.
OK?
2018-04-30 Edward Smith-Rowland <3dw...@verizon.net>
PR libstdc++/pr66689 - comp_ellint_3 and ellint_3 return garbage values
* include/tr1/ell_integral.tc
Hi, all,
For nds32 inline assembly code output, a newline character was missing.
The following commit is to fix the issue.
Committed as Rev.259642: https://gcc.gnu.org/r259642
gcc/
* config/nds32/nds32.h (ASM_APP_ON): Add missing newline character.
Best regards,
jasonwucj
Hi, all,
There was an error in nds32_print_operand() to output
cctl register. It would be better to set value earlier.
The following commit is to fix such issue.
Committed as Rev.259643: https://gcc.gnu.org/r259643
gcc/
* config/nds32/nds32.c (nds32_print_operand): Set op_value earlier
Hi, all,
For nds32 NDS32_ABI_2FP_PLUS abi design,
some settings and compilation options are required to be adjusted
so that we can build correct libgcc.a library.
Committed as Rev.259645: https://gcc.gnu.org/r259645
libgcc/
* config/nds32/sfp-machine.h: Fix settings for NDS32_ABI_2FP_PL
Hi, all,
We have to split move_di/df pattern when hard register is odd
so that the wrong assembly code can be avoided.
Committed as Rev.259646: https://gcc.gnu.org/r259646
gcc/
* config/nds32/nds32-doubleword.md: New define_split pattern for
illegal register number.
Best regar
Hi, all,
In the implementation of nds32_can_use_bclr_p(), nds32_can_use_bset_p(),
and nds32_can_use_btgl_p(), we better use HOST_WIDE_INT type and GET_MODE_MASK()
so that the mask would be good for the bit-operation checking.
Committed as Rev.259647: https://gcc.gnu.org/r259647
gcc/
*
48 matches
Mail list logo
