Re: [PATCH] read-rtl.c: split out read_rtx_operand from read_rtx_code

2016-09-24 Thread Richard Sandiford
David Malcolm writes: > This patch moves the body of the RTL operand-parsing loop (about 180 lines of > code) from read_rtx_code out into a new function (read_rtx_operand). > > Hopefully this is a reasonable cleanup in itself, but it's also useful > for the RTL frontend I'm working on. > > Success

Re: [PATCH] Increase lto-min-partition

2016-09-24 Thread Markus Trippelsdorf
On 2016.09.23 at 15:29 +0200, Richard Biener wrote: > > > > So 5 looks too big to me. > > I think the issue is that the default number of partitions is too high > (32) which pessimizes 4-core machines if the units are too small. The more partitions are used the less memory is required at LTRA

Re: Implement -Wimplicit-fallthrough (version 9)

2016-09-24 Thread Marek Polacek
On Fri, Sep 23, 2016 at 10:12:00PM +, Joseph Myers wrote: > On Fri, 23 Sep 2016, Marek Polacek wrote: > > > On Fri, Sep 23, 2016 at 10:22:15AM -0400, Jason Merrill wrote: > > > On Thu, Sep 22, 2016 at 9:59 AM, Marek Polacek wrote: > > > >> This is very close, thanks. Let's give a more helpfu

Re: [accaf, fortran, 4/4] Allocatable components support in derived typed coarrays

2016-09-24 Thread Dominique d'Humières
Hi Andre, It seems that the committed test gfortran.dg/coarray_lib_comm_1.f90 is not right. I have to apply the patch --- ../_clean/gcc/testsuite/gfortran.dg/coarray_lib_comm_1.f90 2016-09-23 12:45:56.0 +0200 +++ gcc/testsuite/gfortran.dg/coarray_lib_comm_1.f902016-09-24 12:15:02.

Re: [PATCH] builtin expansion of memcmp for powerpc

2016-09-24 Thread David Edelsohn
This patch broke bootstrap on AIX. The ChangeLog entry was not committed to the gcc/ChangeLog file. TARGET_LITTLE_ENDIAN only is defined for PPC64 Linux. This patch should be using !BYTES_BIG_ENDIAN. Thanks, David

Re: [accaf, fortran, 4/4] Allocatable components support in derived typed coarrays

2016-09-24 Thread Andre Vehreschild
Hi Dominique, Yes of course. It makes sense to have the param a 4byte entity on 32 bit systems. Please commit. I am on a mobile only currently and can't do so before tomorrow afternoon. Thanks for that correction. Regards, Andre Am 24. September 2016 13:58:45 MESZ, schrieb "Dominique d'Humièr

Re: [accaf, fortran, 4/4] Allocatable components support in derived typed coarrays

2016-09-24 Thread Dominique d'Humières
Done as revision r240463. > Le 24 sept. 2016 à 16:32, Andre Vehreschild a écrit : > > Hi Dominique, > > Yes of course. It makes sense to have the param a 4byte entity on 32 bit > systems. Please commit. I am on a mobile only currently and can't do so > before tomorrow afternoon. > > Thanks f

Re: [PATCH] builtin expansion of memcmp for powerpc

2016-09-24 Thread Aaron Sawdey
On Sat, 2016-09-24 at 09:32 -0400, David Edelsohn wrote: > This patch broke bootstrap on AIX. > > The ChangeLog entry was not committed to the gcc/ChangeLog file. Changelog entry added. Forgot to write that emacs buffer before commit. > TARGET_LITTLE_ENDIAN only is defined for PPC64 Linux.  This

Re: [PATCH] - improve sprintf buffer overflow detection (middle-end/49905)

2016-09-24 Thread Andreas Schwab
/tree-ssa/builtin-sprintf-warn-1.c (test for warnings, line 1383) FAIL: gcc.dg/tree-ssa/builtin-sprintf-warn-1.c (test for excess errors) Excess errors: /daten/aranym/gcc/gcc-20160924/gcc/testsuite/gcc.dg/tree-ssa/builtin-sprintf-warn-1.c:1408:3: warning: specified size 4294967295 exceeds the size

Re: [v3 PATCH] Implement LWG 2729 for pair.

2016-09-24 Thread Ville Voutilainen
On 21 September 2016 at 13:33, Jonathan Wakely wrote: > This patch predates my testsuite changes, so this should now be > { dg-do compile { target c++11 } } > Otherwise OK for trunk, thanks. So, this has been applied to trunk, and also applies cleanly to the gcc-6 branch, and we need to backport

Re: [Fortran, accaf, patch, v1] Fixing compatibility with opencoarrays (mostly)

2016-09-24 Thread Andreas Schwab
FAIL: gfortran.dg/coarray_lib_comm_1.f90 -O0 scan-tree-dump-times original "_gfortran_caf_get \\(caf_token.0, \\(integer\\(kind=8\\)\\) parm.[0-9]+.data - \\(integer\\(kind=8\\)\\) a, 1, &parm.[0-9]+, 0B, &p, 4, 4, 1, 0B\\);" 1 FAIL: gfortran.dg/coarray_lib_comm_1.f90 -O0 scan-tree-dump-t

Re: [accaf, fortran, 4/4] Allocatable components support in derived typed coarrays

2016-09-24 Thread Dominique d'Humières
Really fixed by revision r240467. Sorry for the noise. Dominique > Le 24 sept. 2016 à 17:14, Dominique d'Humières a écrit : > > Done as revision r240463. > >> Le 24 sept. 2016 à 16:32, Andre Vehreschild a écrit : >> >> Hi Dominique, >> >> Yes of course. It makes sense to have the param a 4b

[PR77719] Fix ICE in pp_string, at pretty-print.c:955

2016-09-24 Thread kugan
Hi, In make_new_ssa_for_def (tree-reassoc.c) we should use gimple_get_lhs to get lhs instead of gimple_assign_lhs as stmt can be builtins too. Attached patch fixes this. Testcase from PR (attached) seems to fail. I dont any fortran so I didnt try fixing it. Any help here is appreciate. The l

[PR77719] Fix ICE in pp_string, at pretty-print.c:955

2016-09-24 Thread Dominique d'Humières
Hi, Two possibilities: (a) pass -w in the dg-options (b) replace INTEGER, PARAMETER :: dp=8 with INTEGER, PARAMETER :: dp=8, spdim=42 (or any number you like!-). Dominique