[RFC][Patch, Fortran] Handle vector subscripts with CO_SUM/CO_MIN/CO_MAX

2014-05-25 Thread Tobias Burnus
The main purpose of this patch is to add support for vector subscripts for the coarray machinery - whose main use will be in the communication with coindexed variables. This patch applies this function to the CO_SUM/CO_MIN/CO_MAX collectives, which is admittedly only of limited use as one coul

Re: profile mode maintenance patch

2014-05-25 Thread Paolo Carlini
Hi, On 05/24/2014 11:39 PM, Jonathan Wakely wrote: On 24 May 2014 22:10, François Dumont wrote: Done but I forgot to fix the spelling. I will fix it in the future patch. No problem, it's clear what the comment means. I'm committing the below. Paolo. / 2014-05-25 Paolo C

Re: [C PATCH] Don't reject valid code with _Alignas (PR c/61053)

2014-05-25 Thread Marek Polacek
On Mon, May 05, 2014 at 10:27:03PM +0200, Marek Polacek wrote: > In this PR the issue is that we reject (valid) code such as > _Alignas (long long) long long foo; > with -m32, because we trip this condition: > >alignas_align = 1U << declspecs->align_log; >if (alignas_align < TYPE_ALIGN_UNI

[PATCH, PR58942, Cilk+] Fix ICE when pointer is used in array notation

2014-05-25 Thread Zamyatin, Igor
Hi! Following patch handles the case of pointers in Cilk+ builtins. Regtested in x86_64. Ok for trunk and 4.9? Thanks, Igor gcc/c/ChangeLog: 2014-05-23 Igor Zamyatin PR c/58942 * c-array-notation.c (fix_builtin_array_notation_fn): Handle the case with a pointer. gcc/cp/ChangeLog: 2014-05

Re: RFA: speeding up dg-extract-results.sh

2014-05-25 Thread Richard Sandiford
Bernd Schmidt writes: > On 02/13/2014 10:18 AM, Richard Sandiford wrote: >> contrib/ >> * dg-extract-results.py: New file. >> * dg-extract-results.sh: Use it if the environment seems suitable. > > I'm now seeing the following: > > Traceback (most recent call last): >File "../../git/g

Re: [PATCH, PR58942, Cilk+] Fix ICE when pointer is used in array notation

2014-05-25 Thread Marek Polacek
On Sun, May 25, 2014 at 09:31:50AM +, Zamyatin, Igor wrote: > +/* { dg-do compile } */ > +/* { dg-options "-fcilkplus" } */ > + > +int foo (int*p, int i) > +{ > + return __sec_reduce_max_ind(p[1:i]); > +} BTW, similar testcase seems to segfault too: int foo (int*p, int *i) { return __sec_r

Re: [PATCH][MIPS] P5600 scheduling

2014-05-25 Thread Richard Sandiford
Jaydeep Patil writes: > Hi Richard, > > Please refer to the attached patch files. > > gcc-p5600-noMSA.patch > The patch implements P5600 pipeline and scheduling for GP and FPU > instructions. This patch is OK, thanks. Generally (i.e. when I remember to check) we don't define TARGET_* options

Re: [Patch] Minor fixes for regtesting gfortran with -flto

2014-05-25 Thread Andreas Schwab
domi...@lps.ens.fr (Dominique Dhumieres) writes: > --- ../_clean/gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90 > 2013-01-25 09:57:48.0 +0100 > +++ gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90 2014-02-04 > 19:24:06.0 +0100 > @@ -16,7 +16,7 @@ integer :: aa(4

Re: [Patch] Minor fixes for regtesting gfortran with -flto

2014-05-25 Thread Dominique Dhumieres
> FAIL: gfortran.dg/bind_c_array_params_2.f90 -O scan-assembler-times > call[^\n\r]*myBindC 1 Sorry for that! Which target? and what is the pattern? Dominique

Re: [Patch] Minor fixes for regtesting gfortran with -flto

2014-05-25 Thread Andreas Schwab
domi...@lps.ens.fr (Dominique Dhumieres) writes: >> FAIL: gfortran.dg/bind_c_array_params_2.f90 -O scan-assembler-times >> call[^\n\r]*myBindC 1 > > Sorry for that! Which target? Pick any random from gcc-testresults. > and what is the pattern? Which pattern? Andreas. -- Andreas Schwab, s

Re: [Patch] Minor fixes for regtesting gfortran with -flto

2014-05-25 Thread Dominique Dhumieres
Could you try --- ../_clean/gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90 2014-05-24 16:17:53.0 +0200 +++ gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90 2014-05-25 18:52:16.0 +0200 @@ -16,7 +16,7 @@ integer :: aa(4,4) call test(aa) end -! { dg-final { scan-as

Re: another patch for PR60969

2014-05-25 Thread Christophe Lyon
Hi, Since this patch was committed, I can see aarch64_be-none-elf build fail in newlib with this error message: 0x8ba1fb check_rtl /tmp/5244922_15.tmpdir/aci-gcc-fsf/sources/gcc-fsf/trunk/gcc/lra.c:2083 0x8bd5b2 lra(_IO_FILE*) /tmp/5244922_15.tmpdir/aci-gcc-fsf/sources/gcc-fsf/trunk

Re: [Patch] Minor fixes for regtesting gfortran with -flto

2014-05-25 Thread Andreas Schwab
domi...@lps.ens.fr (Dominique Dhumieres) writes: > Could you try > > --- ../_clean/gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90 > 2014-05-24 16:17:53.0 +0200 > +++ gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90 2014-05-25 > 18:52:16.0 +0200 > @@ -16,7 +16,7

Re: [Patch] Minor fixes for regtesting gfortran with -flto

2014-05-25 Thread Dominique Dhumieres
> > Could you try > > > > --- ../_clean/gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90 > > 2014-05-24 16:17:53.0 +0200 > > +++ gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90 2014-05-25 > > 18:52:16.0 +0200 > > @@ -16,7 +16,7 @@ integer :: aa(4,4) > > call test(aa)

Re: [Patch] Minor fixes for regtesting gfortran with -flto

2014-05-25 Thread Andreas Schwab
domi...@lps.ens.fr (Dominique Dhumieres) writes: > Could you post the results (w/wo -flto) of > > grep "_myBindC" bind_c_array_params_2.s Empty. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something com

Re: [Patch] Minor fixes for regtesting gfortran with -flto

2014-05-25 Thread Dominique Dhumieres
> > grep "_myBindC" bind_c_array_params_2.s > > Empty. And grep "myBindC" bind_c_array_params_2.s ? Dominique

Re: [Patch] Minor fixes for regtesting gfortran with -flto

2014-05-25 Thread Andreas Schwab
domi...@lps.ens.fr (Dominique Dhumieres) writes: > grep "myBindC" bind_c_array_params_2.s bl myBindC jsr myBindC bl myBindC Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for someth

Re: [Patch] Minor fixes for regtesting gfortran with -flto

2014-05-25 Thread Dominique Dhumieres
> bl myBindC > jsr myBindC > bl myBindC Is this for the same target, or for 3 different ones? which one(s)? Does the following patch fixes the problem? --- ../_clean/gcc/testsuite/gfortran.dg/bind_c_array_params_2.f90 2014-05-24 16:17:53.0 +0200 +++ gcc

Re: [patch ping] libstdc++ testsuite cxxflags

2014-05-25 Thread David Edelsohn
On Sat, May 24, 2014 at 12:54 PM, Jonathan Wakely wrote: > On 24 May 2014 17:07, David Edelsohn wrote: >> This patch broke the ability to run the libstdc++ testsuite on AIX. >> >> I now see the following errors: >> >> bad switch "-O": must be -all, -about, -indices, -inline, -expanded, -line, >>

Re: [Patch] Minor fixes for regtesting gfortran with -flto

2014-05-25 Thread Andreas Schwab
domi...@lps.ens.fr (Dominique Dhumieres) writes: > Does the following patch fixes the problem? Worksforme. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something completely different."

Re: ipa-visibility TLC 2/n

2014-05-25 Thread Ramana Radhakrishnan
On Sun, May 25, 2014 at 6:54 AM, Jan Hubicka wrote: > Hi, > this patch adds code to rerite references in vtable initializers to local > aliases > when doing so is a win. > > Bootstrapped/regtested x86_64-linux, comitted. This is the most likely patch to have caused build failures on arm-linux-gn

[PATCH, libgfortran] PR 61310 CTIME intrinsic output incorrect on MinGW

2014-05-25 Thread Janne Blomqvist
Hi, GFortran currently uses strftime(...,"%c",...) to produce the result for the CTIME and FDATE intrinsics. Unfortunately, it seems that on MinGW this does not produce identical output to the C stdlib ctime(), even in the default locale. The attached patch implements an alternative approach, ori

Re: ipa-visibility TLC 2/n

2014-05-25 Thread Jan Hubicka
> On Sun, May 25, 2014 at 6:54 AM, Jan Hubicka wrote: > > Hi, > > this patch adds code to rerite references in vtable initializers to local > > aliases > > when doing so is a win. > > > > Bootstrapped/regtested x86_64-linux, comitted. > > This is the most likely patch to have caused build failur

Re: [PATCH, libgfortran] PR 61310 CTIME intrinsic output incorrect on MinGW

2014-05-25 Thread Steve Kargl
On Mon, May 26, 2014 at 12:21:21AM +0300, Janne Blomqvist wrote: > Hi, > > GFortran currently uses strftime(...,"%c",...) to produce the result > for the CTIME and FDATE intrinsics. Unfortunately, it seems that on > MinGW this does not produce identical output to the C stdlib ctime(), > even in th

Re: [PATCH] offline gcda profile processing tool

2014-05-25 Thread Jan Hubicka
> >> /* Size of the longest file name. */ > >> -static size_t gcov_max_filename = 0; > >> +/* We need to expose this static variable when compiling for gcov-tool. > >> */ > >> +#ifndef IN_GCOV_TOOL > >> +static > >> +#endif > >> +size_t gcov_max_filename = 0; > > > > > > Why max_filename needs t

[PATCH] PR 61249: fix documentation of __builtin_ia32_{vfrczss,vfrczsd,mpsadbw256}

2014-05-25 Thread Michael Tautschnig
Hi, After brief discussion it was concluded that PR 61249 really only is an issue of documentation of the named builtins not matching their definition given in config/i386/i386.c. The attached patch updates extend.texi accordingly. 2014-05-26 Michael Tautschnig PR target/61249

Re: [PATCH 4/7] Break up determine_known_aggregate_parts

2014-05-25 Thread Jan Hubicka
> Hi, > > the main purpose of this patch is to break up function > determine_known_aggregate_parts so that the next patch can use the > standalone bits and to make the changes slightly easier for review. > > However, this patch also removes some of the offset checks which Honza > correctly though

Re: [PATCH 7/7] Plug ipa-prop escape analysis into gimple_call_arg_flags

2014-05-25 Thread Jan Hubicka
> On Thu, May 22, 2014 at 2:49 PM, Martin Jambor wrote: > > Hi, > > > > On Wed, May 21, 2014 at 04:27:32PM +0200, Richard Biener wrote: > >> On Wed, May 21, 2014 at 3:16 PM, Martin Jambor wrote: > >> > Hi, > >> > > >> > this demonstrates how results of ipa-prop escape analysis from > >> > previou

Re: ipa-visibility TLC 2/n

2014-05-25 Thread Jan Hubicka
> > On Sun, May 25, 2014 at 6:54 AM, Jan Hubicka wrote: > > > Hi, > > > this patch adds code to rerite references in vtable initializers to local > > > aliases > > > when doing so is a win. > > > > > > Bootstrapped/regtested x86_64-linux, comitted. > > > > This is the most likely patch to have c

RE: [Patch: RL78] Add support for 64-bit doubles

2014-05-25 Thread Kaushik Phatak
Hi DJ, Please find below an updated patch which also adds the texi part. My earlier patch which addressed your comments is below, https://gcc.gnu.org/ml/gcc-patches/2014-05/msg01076.html The newlib and binutils part of this patch have been reviewed and committed: https://sourceware.org/ml/newlib/2

Re: libsanitizer merge from upstream r208536

2014-05-25 Thread Konstantin Serebryany
Hi Peter, Last time I tried, asan did not work on ppc32 for a large number of different reasons. In upstream build system ppc32 is simply disabled, so imho it should be also disabled in the GCC build. If there is enough interest in ppc32, please work with up on fixing upstream and enabling the bui

Re: ipa-visibility TLC 2/n

2014-05-25 Thread Ramana Radhakrishnan
On Mon, May 26, 2014 at 2:04 AM, Jan Hubicka wrote: >> > On Sun, May 25, 2014 at 6:54 AM, Jan Hubicka wrote: >> > > Hi, >> > > this patch adds code to rerite references in vtable initializers to >> > > local aliases >> > > when doing so is a win. >> > > >> > > Bootstrapped/regtested x86_64-linux

Re: [PATCH] Handle more COMDAT profiling issues

2014-05-25 Thread Jan Hubicka
> This patch attempts to address the lost profile issue for COMDATs in > more circumstances, exposed by function splitting. > > My earlier patch handled the case where the comdat had 0 counts since > the linker kept the copy in a different module. In that case we > prevent the guessed frequencies

Re: libsanitizer merge from upstream r208536

2014-05-25 Thread Jakub Jelinek
On Mon, May 26, 2014 at 08:57:11AM +0400, Konstantin Serebryany wrote: > Last time I tried, asan did not work on ppc32 for a large number of > different reasons. ??? Comparing my 4.9.0 ppc/ppc64 testresults, for 32-bit I see: FAIL: c-c++-common/asan/null-deref-1.c -O0 output pattern test, is A

Re: [patch, fortran] PR55117 Programs fails to read namelist (contains derived types objects)

2014-05-25 Thread Tobias Burnus
Jerry DeLisle wrote: Regression tested on x86-64. Test case attached with patch. OK for trunk? Looks good to me. Thanks for the patch. Tobias 2014-05-24 Tobias Burnus PR fortran/55117 * trans-io.c (nml_full_name, transfer_namelist_element): Insert a '+' rather t

Re: atomic update of profile counters (issue7000044)

2014-05-25 Thread Jan Hubicka
> 2013-11-19 Rong Xu > > * gcc/gcov-io.h: Add atomic function macros for compiler use. > * gcc/common.opt (fprofile-generate-atomic): New option. > * gcc/tree-profile.c (gimple_init_edge_profiler): Support for > atomic counter update. > (gimple_gen_edge_profiler)

Re: libsanitizer merge from upstream r208536

2014-05-25 Thread Konstantin Serebryany
On Mon, May 26, 2014 at 9:57 AM, Jakub Jelinek wrote: > On Mon, May 26, 2014 at 08:57:11AM +0400, Konstantin Serebryany wrote: >> Last time I tried, asan did not work on ppc32 for a large number of >> different reasons. > > ??? > Comparing my 4.9.0 ppc/ppc64 testresults, for 32-bit I see: > > FAIL

Re: [PATCH] Fix PR rtl-optimization/61278

2014-05-25 Thread Zhenqiang Chen
On 23 May 2014 19:56, Richard Biener wrote: > On Fri, May 23, 2014 at 12:33 PM, Zhenqiang Chen > wrote: >> On 23 May 2014 17:05, Richard Biener wrote: >>> On Fri, May 23, 2014 at 9:23 AM, Zhenqiang Chen >>> wrote: Hi, The patch fixes PR rtl-optimization/61278. Root cause for issu