[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

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

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_UNIT (type))

[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 igor.zamya...@intel.com PR c/58942 * c-array-notation.c (fix_builtin_array_notation_fn): Handle the case with a pointer.

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

2014-05-25 Thread Richard Sandiford
Bernd Schmidt bernds_...@t-online.de 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

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

Re: [PATCH][MIPS] P5600 scheduling

2014-05-25 Thread Richard Sandiford
Jaydeep Patil jaydeep.pa...@imgtec.com 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

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,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,

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 {

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*)

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) end

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

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 something

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 +++

Re: [patch ping] libstdc++ testsuite cxxflags

2014-05-25 Thread David Edelsohn
On Sat, May 24, 2014 at 12:54 PM, Jonathan Wakely jwakely@gmail.com 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,

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 hubi...@ucw.cz 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

[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,

Re: ipa-visibility TLC 2/n

2014-05-25 Thread Jan Hubicka
On Sun, May 25, 2014 at 6:54 AM, Jan Hubicka hubi...@ucw.cz 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

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 the

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 to be exported? For code

[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 m...@debian.org PR

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 thought

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 mjam...@suse.cz 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 mjam...@suse.cz wrote: Hi, this demonstrates how results of ipa-prop escape analysis from

Re: ipa-visibility TLC 2/n

2014-05-25 Thread Jan Hubicka
On Sun, May 25, 2014 at 6:54 AM, Jan Hubicka hubi...@ucw.cz 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

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:

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

Re: ipa-visibility TLC 2/n

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

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 on

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