Re: GCC 6 backports

2018-04-25 Thread Martin Liška
Hi. Sending GCC 6 branch backports. Patches can bootstrap on ppc64le-redhat-linux and survives regression tests. I'm going to install the patches. Martin >From e0d32b1f9e0dd0486e63040e1ab8f5d8e9f0fbd5 Mon Sep 17 00:00:00 2001 From: marxin Date: Tue, 10 Apr 2018 07:24:59 + Subject: [PATCH 1/4

Re: [PATCH] Update nvptx newlib installation requirements

2018-04-25 Thread Richard Biener
On Tue, Apr 24, 2018 at 6:00 PM, Cesar Philippidis wrote: > On 04/24/2018 12:10 AM, Richard Biener wrote: > >> That's great news! Note that we usually keep copies of build dependences at >> ftp://gcc.gnu.org/pub/gcc/infrastructure/ and there's currently no nvptx >> newlib >> variant there. Mayb

[PATCH] Bump LTO_major_version on trunk

2018-04-25 Thread Richard Biener
This bumps the bytecode version. Committed to trunk. Richard. 2018-04-25 Richard Biener * lto-streamer.h (LTO_major_version): Bump to 8. Index: gcc/lto-streamer.h === --- gcc/lto-streamer.h (revision 259637) +++ gcc/l

[Ada] Regenerate gnat_ugn.texi

2018-04-25 Thread Eric Botcazou
Pierre-Marie just pointed out that I forgot to regenerate gnat_ugn.texi in my fix for PR ada/85007. Can I put this on the 8 branch too? 2018-04-25 Eric Botcazou PR ada/85007 * gnat_ugn.texi: Regenerate. -- Eric BotcazouIndex: gnat_ugn.texi =

Re: [Ada] Regenerate gnat_ugn.texi

2018-04-25 Thread Richard Biener
On Wed, Apr 25, 2018 at 10:52 AM, Eric Botcazou wrote: > Pierre-Marie just pointed out that I forgot to regenerate gnat_ugn.texi in my > fix for PR ada/85007. Can I put this on the 8 branch too? Yes. Richard. > > 2018-04-25 Eric Botcazou > > PR ada/85007 > * gnat_ugn.texi: R

Re: [PATCH] PR gcc/84923 - gcc.dg/attr-weakref-1.c failed on aarch64

2018-04-25 Thread Ramana Radhakrishnan
On Fri, Apr 13, 2018 at 7:08 AM, wrote: > From: Vladimir Mezentsev > > When weakref_targets is not empty a target cannot be removed from weak_decls. > A small example is below when 'wv12' is removed from the weak list on aarch64: > static vtype Wv12 __attribute__((weakref ("wv12"))); > exter

[PATCH] Try to use issetugid before falling back to insecure getenv

2018-04-25 Thread Matija Skala
--- libvtv/configure.ac | 3 +++ libvtv/vtv_utils.cc | 2 ++ 2 files changed, 5 insertions(+) diff --git a/libvtv/configure.ac b/libvtv/configure.ac index ba3009ee3fb..878ba02a2e2 100644 --- a/libvtv/configure.ac +++ b/libvtv/configure.ac @@ -115,6 +115,9 @@ AC_CHECK_FUNCS([__secure_getenv]) AC_

[nvptx, libgomp, testsuite, PR85519] Reduce recursion depth in declare_target-{1,2}.f90

2018-04-25 Thread Tom de Vries
Hi, when running the libgomp tests with nvptx accelerator on an Nvidia Titan V, we run into these failures: ... FAIL: libgomp.fortran/examples-4/declare_target-1.f90 -O1 execution test FAIL: libgomp.fortran/examples-4/declare_target-1.f90 -O2 execution test FAIL: libgomp.fortran/examples-

RE: [PATCH][i386] PR target/85473, Fix _movdir64b expansion with -mx32

2018-04-25 Thread Peryt, Sebastian
Hi, Patch has been updated and tested. Now I don't see any new regressions. Changelog stays the same. Is it ok for trunk? Thanks, Sebastian > -Original Message- > From: Peryt, Sebastian > Sent: Saturday, April 21, 2018 5:36 PM > To: gcc-patches@gcc.gnu.org > Cc: Uros Bizjak ; Kirill Y

Re: [nvptx, libgomp, testsuite, PR85519] Reduce recursion depth in declare_target-{1,2}.f90

2018-04-25 Thread Jakub Jelinek
On Wed, Apr 25, 2018 at 12:58:47PM +0200, Tom de Vries wrote: > Concluding, these tests run out thread stack on Nvidia Titan V because the > recursive functions have a larger frame size than we've seen for the Nvidia > architecture flavours that we've tested before. > > The patch fixes this by red

[RFH] split {generic,gimple}-match.c files

2018-04-25 Thread Richard Biener
The following patch^Whack splits $subject files into three, one for the predicates (due to an implementation detail) and two for the rest - for now into similar LOC size files. I'd like to get help on the makefile changes to make them less verbose, somehow globbing the -[12p] parts. Also you can

[wwwdocs] Correct notes about -Wreturn-type change for C++

2018-04-25 Thread Jonathan Wakely
It's always been ill-formed to say "return;" in a non-void function, the change in GCC 8 is when control flows of the end of a function. This corrects the release notes. Committed to CVS. Index: htdocs/gcc-8/changes.html === RCS file

Re: [PATCH 0/2] Require that constraints are used to reference global regs

2018-04-25 Thread Michael Matz
Hi, On Tue, 24 Apr 2018, Alexander Monakov wrote: > On Tue, 24 Apr 2018, Michael Matz wrote: > > What is lost here (it wasn't explicit before, but is the case and must > > continue to work) is that function calls and returns count as needing the > > observable value in the specified register (f

Re: [PATCH][i386] PR target/85473, Fix _movdir64b expansion with -mx32

2018-04-25 Thread Uros Bizjak
On Wed, Apr 25, 2018 at 1:03 PM, Peryt, Sebastian wrote: > Hi, > > Patch has been updated and tested. Now I don't see any new regressions. > > Changelog stays the same. > > Is it ok for trunk? OK for mainline and gcc-8 backport after a couple of days, the latter also needs approval from RM. >> >

Re: [RFH] split {generic,gimple}-match.c files

2018-04-25 Thread Richard Biener
On Wed, 25 Apr 2018, Richard Biener wrote: > > The following patch^Whack splits $subject files into three, one > for the predicates (due to an implementation detail) and two for > the rest - for now into similar LOC size files. > > I'd like to get help on the makefile changes to make them less >

Re: [RFH] split {generic,gimple}-match.c files

2018-04-25 Thread Richard Biener
On Wed, 25 Apr 2018, Richard Biener wrote: > On Wed, 25 Apr 2018, Richard Biener wrote: > > > > > The following patch^Whack splits $subject files into three, one > > for the predicates (due to an implementation detail) and two for > > the rest - for now into similar LOC size files. > > > > I'd

Re: [RFH] split {generic,gimple}-match.c files

2018-04-25 Thread Martin Liška
On 04/25/2018 01:42 PM, Richard Biener wrote: Hi. Thanks for working on that. > > The following patch^Whack splits $subject files into three, one > for the predicates (due to an implementation detail) and two for > the rest - for now into similar LOC size files. > > I'd like to get help on the

[PATCH] Improve bootstrap times

2018-04-25 Thread Richard Biener
The following patch improves bootstrap times in general, without resorting to file splitting. The observation that stage2 build of gimple-match.c improves by 50% when using -fno-checking suggests that we avoid doing redundant checking when bootstrapping and simply disable that when using the slow

Re: GCC 6 backports

2018-04-25 Thread Jan Hubicka
> Hi. > > Sending GCC 6 branch backports. > Patches can bootstrap on ppc64le-redhat-linux and survives regression tests. > I'm going to install the patches. > > Martin > >From e0d32b1f9e0dd0486e63040e1ab8f5d8e9f0fbd5 Mon Sep 17 00:00:00 2001 > From: marxin > Date: Tue, 10 Apr 2018 07:24:59 +000

Re: [PATCH] Improve bootstrap times

2018-04-25 Thread Richard Biener
On Wed, 25 Apr 2018, Richard Biener wrote: > > The following patch improves bootstrap times in general, without > resorting to file splitting. The observation that stage2 build > of gimple-match.c improves by 50% when using -fno-checking > suggests that we avoid doing redundant checking when boo

Re: PR85463 '[nvptx] "exit" in offloaded region doesn't terminate process' (was: [patch, libfortran, committed] Implement stop_numeric for minimal targets)

2018-04-25 Thread Martin Jambor
Hi, On Thu, Apr 19 2018, Thomas Schwinge wrote: > > Per PR85463 '[nvptx] "exit" in offloaded region doesn't terminate > process' that I just filed, we currently have to use "abort" instead of > "exit" for nvptx offloading, so I have applied the following in trunk > r259491, where I completed this

Re: [PATCH] Improve bootstrap times

2018-04-25 Thread Jakub Jelinek
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 profiledbootstrap > on a release build by training with -fchecking. Suggestions > welcome - not sure whether adjusting STA

Re: [PATCH] Improve bootstrap times

2018-04-25 Thread Richard Biener
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 profiledbootstrap > > on a release build by training with -fchecking. Sug

Re: [PATCH][i386] PR target/85473, Fix _movdir64b expansion with -mx32

2018-04-25 Thread H.J. Lu
On Wed, Apr 25, 2018 at 5:13 AM, Uros Bizjak wrote: > On Wed, Apr 25, 2018 at 1:03 PM, Peryt, Sebastian > wrote: >> Hi, >> >> Patch has been updated and tested. Now I don't see any new regressions. >> >> Changelog stays the same. >> >> Is it ok for trunk? > > OK for mainline and gcc-8 backport af

[PATCH] Don't offer suggestions for compiler-generated variables (PR c++/85515)

2018-04-25 Thread David Malcolm
Jason Turner's video C++ Weekly - Ep 112 - GCC's Leaky Abstractions shows two issues where g++ offers suggestions about implementation details: Example 1: int main () { auto lambda = [val = 2](){}; lambda.val; } : In function 'int main()': :5:10: error: 'struct main()::' has no member named

Re: [PATCH][i386] PR target/85473, Fix _movdir64b expansion with -mx32

2018-04-25 Thread Jakub Jelinek
On Wed, Apr 25, 2018 at 08:31:33AM -0700, H.J. Lu wrote: > On Wed, Apr 25, 2018 at 5:13 AM, Uros Bizjak wrote: > > On Wed, Apr 25, 2018 at 1:03 PM, Peryt, Sebastian > > wrote: > >> Hi, > >> > >> Patch has been updated and tested. Now I don't see any new regressions. > >> > >> Changelog stays the

Re: [PATCH] Don't offer suggestions for compiler-generated variables (PR c++/85515)

2018-04-25 Thread Jakub Jelinek
On Wed, Apr 25, 2018 at 11:41:51AM -0400, David Malcolm wrote: > @@ -1224,9 +1225,19 @@ lookup_field_fuzzy_info::fuzzy_lookup_field (tree type) > >for (tree field = TYPE_FIELDS (type); field; field = DECL_CHAIN (field)) > { > - if (!m_want_type_p || DECL_DECLARES_TYPE_P (field)) > -

[wwwdocs] Add notes on common C++ problems with GCC 8

2018-04-25 Thread Jonathan Wakely
Ville is going to prepare something for the new -Wclass-memacess and -Wcatch-value warnings. Suggestions for other gotchas to document are welcome. Committed to CVS. Index: htdocs/gcc-8/porting_to.html === RCS file: /cvs/gcc/wwwdoc

Re: [PATCH, rs6000] Improve Documentation of Built-In Functions Part 1

2018-04-25 Thread Kelvin Nilsen
Thank you for the prompt review and careful feedback. I didn't notice your message until this morning. At this point, I'll wait a few days before committing these changes as I understand we are still in the "RC phase of GCC 8". On 4/24/18 4:45 PM, Segher Boessenkool wrote: > Hi! > > On Tue, A

Re: [PATCH] Don't offer suggestions for compiler-generated variables (PR c++/85515)

2018-04-25 Thread Nathan Sidwell
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 are multiple members: $9 = These names have a space at the end, so

Re: [PATCH][arm] PR target/82518: Return false in ARRAY_MODE_SUPPORTED_P for BYTES_BIG_ENDIAN

2018-04-25 Thread Maxim Kuvyrkov
> On Mar 20, 2018, at 8:11 PM, Kyrill Tkachov > wrote: > > Hi all, > > This PR shows that we get the load/store_lanes logic wrong for arm big-endian. > It is tricky to get right. Aarch64 does it by adding the appropriate > lane-swapping > operations during expansion. > > I'd like to do the sa

Re: [PATCH] DWARF sort longer dirs before shorter ones in directory table.

2018-04-25 Thread Mark Wielaard
On Mon, 2018-04-16 at 09:41 +0200, Jakub Jelinek wrote: > On Mon, Apr 16, 2018 at 09:34:17AM +0200, Mark Wielaard wrote: > > gcc/ChangeLog: > > > > * dwarf2out.c (file_info_cmp): Sort longer dir prefixes before > > shorter ones. > > Ok, thanks. Pushed now that trunk is in stage 1 again.

Re: [PATCH] Try to use issetugid before falling back to insecure getenv

2018-04-25 Thread Andreas Schwab
On Apr 25 2018, Matija Skala wrote: > diff --git a/libvtv/configure.ac b/libvtv/configure.ac > index ba3009ee3fb..878ba02a2e2 100644 > --- a/libvtv/configure.ac > +++ b/libvtv/configure.ac > @@ -115,6 +115,9 @@ AC_CHECK_FUNCS([__secure_getenv]) > AC_GNU_SOURCE > AC_CHECK_FUNCS([secure_getenv])

[PATCH] Document that -Wreturn-type is enabled by default for C++

2018-04-25 Thread Jonathan Wakely
* doc/invoke.texi (-Wreturn-type): Document default status for C++. OK for trunk and gcc-8-branch? commit 20357a8ded851a3376b7e8978c77e7c56b1a273e Author: Jonathan Wakely Date: Wed Apr 25 19:51:10 2018 +0100 Document that -Wreturn-type is enabled by default for C++

[PATCH] Minor documentation correction in aarch64-simd.md

2018-04-25 Thread Indu Bhagat
In function minmax_replacement in tree-ssa-phiopt.c, MIN_EXPR/MAX_EXPR are substituted for when the following condition is false - (HONOR_NANS (type) || HONOR_SIGNED_ZEROS (type)). So for FP mode, this is false when _both_ of the following conditions are fulfilled : 1. flag_signed_zeros is zero

[PATCH][arm][FreeBSD] PR libgcc/84292

2018-04-25 Thread Andreas Tobler
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. Thanks, Andreas 2018-04-25 Andreas Tobler Maryse Levavasseur PR libgcc/84292 * config/arm/freebsd-atomic.c (SYNC_OP_

[Committed] Remove myself as MIPS maintainer

2018-04-25 Thread Moore, Catherine
2018-04-25 Catherine Moore * MAINTAINERS (mips): Remove myself as MIPS maintainer.

Re: RFC: bash completion

2018-04-25 Thread David Malcolm
[moving from gcc to gcc-patches mailing list] On Wed, 2018-04-25 at 15:12 +0200, Martin Liška wrote: > On 04/24/2018 06:27 PM, David Malcolm wrote: > > On Tue, 2018-04-24 at 16:45 +0200, Martin Liška wrote: > > > Hi. > > > > > > Some time ago, I investigated quite new feature of clang which > > >

Re: [PATCH 0/4] ASAN for MIPS (o32)

2018-04-25 Thread Hans-Peter Nilsson
> Date: Fri, 23 Mar 2018 03:49:01 +0100 > From: Hans-Peter Nilsson The patches enabling ASAN for MIPS (32-bit) have now been committed, on trunk for gcc-9. I didn't persist pinging release maintainers for gcc-8, but I'd certainly like to backport them, if so allowed. brgds, H-P

GCC 8.1 RC1 Bootstrap comparison failure on AIX

2018-04-25 Thread David Edelsohn
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 random timestamp. 1949c1949 < [1936]m 0x0060 1

[PATCH] C/C++: Add -Waddress-of-packed-member

2018-04-25 Thread H.J. Lu
When address of packed member of struct or union is taken, it may result in an unaligned pointer value. This patch adds -Waddress-of-packed-member to check alignment at pointer assignment and warn unaligned address as well as unaligned pointer: $ cat x.i struct pair_t { char c; int i; } __att

PING: [PATCH] Don't mark IFUNC resolver as only called directly

2018-04-25 Thread H.J. Lu
On Thu, Apr 12, 2018 at 3:50 PM, H.J. Lu wrote: > On Thu, Apr 12, 2018 at 6:39 AM, H.J. Lu wrote: >> On Thu, Apr 12, 2018 at 5:17 AM, Jan Hubicka wrote: On Thu, Apr 12, 2018 at 1:29 PM, H.J. Lu wrote: > Since IFUNC resolver is called indirectly, don't mark IFUNC resolver as > onl

Re: [PATCH] Try to use issetugid before falling back to insecure getenv

2018-04-25 Thread Matija Skala
Dne sreda, 25. april 2018 ob 19:53:08 CEST je Andreas Schwab napisal(a): > On Apr 25 2018, Matija Skala wrote: > > diff --git a/libvtv/configure.ac b/libvtv/configure.ac > > index ba3009ee3fb..878ba02a2e2 100644 > > --- a/libvtv/configure.ac > > +++ b/libvtv/configure.ac > > @@ -115,6 +115,9 @@ AC