Re: Optimize lto location stremaing

2015-03-28 Thread Jan Hubicka
Hi, I actually got idea how to make partitioning safe for named labels w/o going the difficult route of makeing them part of symbol table. Will look into that tonight or tomorrow. > Also there's a juicy ICE with that worked around: > > /home/andi/lsrc/linux/drivers/scsi/bfa/bfa_ioc.c: In functi

Re: Optimize lto location stremaing

2015-03-28 Thread Jan Hubicka
> It doesn't build at all currently. The old problem with &&label jump table > initializers being spread incorrectly over partitions is back again, causing > the BPF interpret to throw a lot of > > .tmp_vmlinux1.ltrans31.ltrans.o:(.data+0x32268): undefined reference to > `.L3' > > I thought you

Re: Optimize lto location stremaing

2015-03-28 Thread Andi Kleen
On Sun, Mar 29, 2015 at 12:19:11AM +0100, Andi Kleen wrote: > > I think this was the last major offender for Chromoim/Libreoffice and > > Firefox. > > (Modulo the fact htat chromium needs 9GB for WPA. There seems not be much of > > low hanging fruit - chromium needs a lot of trees to be streamed i

Re: Optimize lto location stremaing

2015-03-28 Thread Andi Kleen
> I think this was the last major offender for Chromoim/Libreoffice and Firefox. > (Modulo the fact htat chromium needs 9GB for WPA. There seems not be much of > low hanging fruit - chromium needs a lot of trees to be streamed in that will > hopefully be tracked by early debug soon.) What is the st

[PATCH] libffi testsuite: Don't run testsuite/libffi.call/float2.c on hppa*-*-hpux*

2015-03-28 Thread John David Anglin
The libffi.call/float2.c test uses fabsl which was introduced in c99 and isn't available on hppa*-*-hpux*. In order to use the target selector with dg-run, I need to load target-supports-dg.exp in lib/libffi.exp. Tested on hppa2.0w-hp-hpux11.11. Okay for trunk? Dave -- John David Anglin

Re: [PR64164] drop copyrename, integrate into expand

2015-03-28 Thread Alexandre Oliva
On Mar 27, 2015, Alexandre Oliva wrote: > This patch reworks the out-of-ssa expander to enable coalescing of SSA > partitions that don't share the same base name. This is done only when > optimizing. > The test we use to tell whether two partitions can be merged no longer > demands them to have

[PATCH] libffi testsuite: Use split to ensure valid tcl list

2015-03-28 Thread John David Anglin
The attached change fixes tcl errors that occur running the complex.exp and go.exp test sets. See: . Tested on hppa2.0w-hp-hpux11.11. Okay for trunk? Dave -- John David Anglin dave.ang...@bell.net libffi-split.d Description: Binary d

[committed] hpux11_lwp_rwlock_valid fix

2015-03-28 Thread John David Anglin
In a recent update, libstdc++ started using the pthread_rwlock struct. This exposed a c++ error in the definition of __LWP_RWLOCK_VALID. It is an error to initialize a short field with the constant 0x8c91. The attached change changes the __LWP_RWLOCK_VALID value to -29551. Tested on hppa2.0w-

Re: [patch, libgfortran] [4.8/4.9/5 Regression] error reading (and writing) large text files in gfortran

2015-03-28 Thread Jerry DeLisle
On 03/28/2015 07:20 AM, H.J. Lu wrote: --- snip --- I think this caused: FAIL: gfortran.dg/io_constraints_8.f90 -O (test for errors, line 64) FAIL: gfortran.dg/io_constraints_8.f90 -O (test for errors, line 65) FAIL: gfortran.dg/io_constraints_8.f90 -O (test for errors, line 66) FAIL:

Re: [PATCH] fortran/65429 -- don't dereference a null pointer

2015-03-28 Thread Dominique d'Humières
> Le 28 mars 2015 à 15:50, Steve Kargl a > écrit : > > Can one do anything useful with a zero-sized array > of strings where the length of a non-existent > element of the array is nonzero? The only answer I can give is that the users’ imagination is unbounded! Dominique > -- > Steve

Re: [patch, libgfortran] [4.8/4.9/5 Regression] error reading (and writing) large text files in gfortran

2015-03-28 Thread Dominique d'Humières
On x86_64-apple-darwin14 with revision r221692, for a clean 4.8 tree configured with ../4.8_clean/configure --prefix=/opt/gcc/gcc4.8c --enable-languages=c,c++,fortran,objc,obj-c++,ada,java,lto --with-gmp=/opt/mp --with-system-zlib --with-isl=/opt/mp --enable-lto --enable-plugin I see in /opt/

Re: [patch, libgfortran] [4.8/4.9/5 Regression] error reading (and writing) large text files in gfortran

2015-03-28 Thread H.J. Lu
On Sat, Mar 28, 2015 at 7:43 AM, Dominique Dhumieres wrote: > Do you see "Fatal Error: Error count reached limit of ..." in your logs? > If yes, what is the value. > > Compiling the code with -fmax-errors=51, I get > Fatal Error: Error count reached limit of 51. > > This fatal error disappears wit

Re: [PATCH] fortran/65429 -- don't dereference a null pointer

2015-03-28 Thread Steve Kargl
>> 28 mars 2015, 01:33, Steve Kargl : >> >> On Sat, Mar 28, 2015 at 01:01:57AM +0100, Dominique Dhumieres wrote: >>> >>> AFAICT your test succeeds without your patch and does not test that the ICE >>> reported by FX is gone (indeed it is with your patch). >>> >> >> Yeah, I thought about that, b

Re: [patch, libgfortran] [4.8/4.9/5 Regression] error reading (and writing) large text files in gfortran

2015-03-28 Thread Dominique Dhumieres
Do you see "Fatal Error: Error count reached limit of ..." in your logs? If yes, what is the value. Compiling the code with -fmax-errors=51, I get Fatal Error: Error count reached limit of 51. This fatal error disappears with -fmax-errors=52, well below the limit in the test: -fmax-errors=100.

Re: [patch, libgfortran] [4.8/4.9/5 Regression] error reading (and writing) large text files in gfortran

2015-03-28 Thread Jerry DeLisle
On 03/28/2015 07:20 AM, H.J. Lu wrote: I think this caused: FAIL: gfortran.dg/io_constraints_8.f90 -O (test for errors, line 64) FAIL: gfortran.dg/io_constraints_8.f90 -O (test for errors, line 65) FAIL: gfortran.dg/io_constraints_8.f90 -O (test for errors, line 66) FAIL: gfortran.dg/i

Re: [patch, libgfortran] [4.8/4.9/5 Regression] error reading (and writing) large text files in gfortran

2015-03-28 Thread H.J. Lu
On Sat, Feb 7, 2015 at 6:52 AM, Paul Richard Thomas wrote: > Dear Jerry, > > This is OK for trunk. Maybe it is best to leave it for a week or two > before committing to the branches? > > Thanks for the patch. > > Paul > > On 7 February 2015 at 03:08, Jerry DeLisle wrote: >> With the attached patc

Re: [PATCH] fortran/65429 -- don't dereference a null pointer

2015-03-28 Thread Dominique d'Humières
See also my comment 2 in pr65429. Cheers, Dominique > Le 28 mars 2015 à 01:33, Steve Kargl a > écrit : > > On Sat, Mar 28, 2015 at 01:01:57AM +0100, Dominique Dhumieres wrote: >> >> AFAICT your test succeeds without your patch and does not test that the ICE >> reported by FX is gone (indeed

Re: [Patch, testsuite] Fixed comment in testcase

2015-03-28 Thread Andre Vehreschild
Hi all, fixed copyright statement (again) as r221752. - Andre On Fri, 27 Mar 2015 13:43:50 +0100 Andre Vehreschild wrote: > Hi all, > > commited as r221729 to meet the legal requirements and fix some typos. > > Regards, > Andre -- Andre Vehreschild * Email: vehre ad gmx dot de Inde

[PATCH] gcc/config/c6x/c6x.md (mvilc): Add "SI" for "unspec".

2015-03-28 Thread Chen Gang
The related warning: build/genrecog ../../gcc-c6x/gcc/common.md ../../gcc-c6x/gcc/config/c6x/c6x.md \ insn-conditions.md > tmp-recog.c [...] ../../gcc-c6x/gcc/config/c6x/c6x.md:1443: warning: source missing a mode? [...] 2015-03-28 Chen Gang * config/c6x/c6x.md (mvilc): A