Speedup inliner on Chromium

2015-03-08 Thread Jan Hubicka
Hi, compiling Chromium, ipa-icf introduces thousdands of aliases of one function from iostream traits (because it is duplicated so many times due to ipa-sra privatizing it to every compile unit). This triggers non-linear behaviour in will_be_removed_from_program_if_no_direct_calls_p. This patch m

Fix vectorizer alignment update wrt section anchors

2015-03-08 Thread Jan Hubicka
Hi, this patch avoids notice_global_symbol to create DECL_RTL prior IPA optimization. Consequently symtab_node::can_increase_alignment_p can check for presence of DECL_RTL with section anchor. This makes it possible to bump up alignment in tree-vect-data-refs.c in some cases (when DECL_RTL is not

Re: [patch, fortran] Bug 64432 - [5 Regression] SYSTEM_CLOCK(COUNT_RATE=rate) wrong result for integer(4)::rate

2015-03-08 Thread Jerry DeLisle
On 03/08/2015 04:58 PM, Steve Kargl wrote: On Mon, Mar 09, 2015 at 01:07:25AM +0200, Janne Blomqvist wrote: So I would prefer if we just hardcode the error values in the frontend (-HUGE, 0, 0), in case somebody tries to use the kind=1,2 versions, thus also removing the need for the new library f

[patch, doc] don't document old gcc behavior in current manual

2015-03-08 Thread Sandra Loosemore
In general, the GCC manual documents the version of the compiler that it goes with. New features are being added all the time, and it's not terribly useful to users to document that such-and-such a feature isn't supported in ancient GCC versions, or to suggest workarounds for old GCC versions

Fix ODR hash corrpution

2015-03-08 Thread Jan Hubicka
Hi, the odr_type_hash has two modes - if type names are availale it is hashing by type names, if they are not it is hashing by virtual tables (and workks only on polymorphic types). What I did not consider however was units where both are mixed. Here the hash does not play well with the equivalenc

Re: [patch, fortran] Bug 64432 - [5 Regression] SYSTEM_CLOCK(COUNT_RATE=rate) wrong result for integer(4)::rate

2015-03-08 Thread Steve Kargl
On Mon, Mar 09, 2015 at 01:07:25AM +0200, Janne Blomqvist wrote: > So I would prefer if we just hardcode the error values in the frontend > (-HUGE, 0, 0), in case somebody tries to use the kind=1,2 versions, > thus also removing the need for the new library functions, keeping the > existing simpler

[committed] Skip failing scan-assembler checks in g++.dg/abi/anon3.C and g++.dg/abi/rtti3.C

2015-03-08 Thread John David Anglin
Since r214177, DECL_COMDAT no longer triggers comdat_linkage. As a result, some types that were previously "weak" are no longer weak on hppa-hpux. Tested on hppa2.0w-hp-hpux11.11. Committed to trunk. Dave -- John David Anglin dave.ang...@bell.net 2015-03-08 John David Anglin

Re: [SH] Backport fix of PR 53988 to 4.9/4.8

2015-03-08 Thread Kaz Kojima
Oleg Endo wrote: > Backporting of the fix went into GCC 5 seems to intrusive for the > released branches. Hence I'd propose to remove the problematic patterns > altogether, as they can silently generate wrong code. The attached > patch does that. Tested briefly with 'make all' and with > make -

Re: Fix increase_alignment

2015-03-08 Thread Jan Hubicka
> > I unforutnately spot another problem. With presence of section anchors we > can not increase alignment of symbol that was already placed into the section. > I think this is reason why we still have pass_increase_alignments that runs > only on target with anchors. > > I added the following che

[committed] Add dg-require-effective-target named_sections to g++.dg/ipa/pr63621.C

2015-03-08 Thread John David Anglin
The test uses named sections. Tested on hppa2.0w-hp-hpux11.11. Committed to trunk. Dave -- John David Anglin dave.ang...@bell.net 2015-03-08 John David Anglin * g++.dg/ipa/pr63621.C: Add dg-require-effective-target named_sections. Index: g++.dg/ipa/pr63621.C

[committed] xfail gfortran.dg/quad_2.f90 on hppa*-*-hpux*

2015-03-08 Thread John David Anglin
The HP quad precision libcall has a 1 ULP error in computing the sqrt of 2. This causes the test to fail. Tested on hppa2.0w-hp-hpux11.11. Committed to trunk. Dave -- John David Anglin dave.ang...@bell.net 2015-03-08 John David Anglin PR target/62251 * gfortran.dg/q

Re: [patch, fortran] Bug 64432 - [5 Regression] SYSTEM_CLOCK(COUNT_RATE=rate) wrong result for integer(4)::rate

2015-03-08 Thread Janne Blomqvist
On Sun, Mar 8, 2015 at 6:43 AM, Jerry DeLisle wrote: > The attached patch fixes this regression Nice, thanks for working on this! Just out of curiosity, what was the reason for the powerpc failures you had with the previous version of the patch? > and implements KIND=1 and KIND=2 > compatible ca

[committed] Add dg-require-effective-target c99_runtime to gcc.dg/signbit-sa.c

2015-03-08 Thread John David Anglin
The signbit function was introduced in c99. Tested on hppa2.0w-hp-hpux11.11. Committed to trunk. Dave -- John David Anglin dave.ang...@bell.net 2015-03-08 John David Anglin * gcc.dg/signbit-sa.c: Add dg-require-effective-target c99_runtime Index: gcc.dg/signbit-sa.c

[committed] Add target c99_runtime to gcc.dg/ipa/pr64307.c

2015-03-08 Thread John David Anglin
gcc.dg/ipa/pr64307.c includes complex.h. It is only available on c99 targets and fails as a result on hppa*-*-hpux*. Tested on hppa64-hp-hpux11.11 and hppa2.0w-hp-hpux11.11. Committed to trunk. Dave -- John David Anglin dave.ang...@bell.net 2015-03-08 John David Anglin * gc

[committed] Add dg-require-alias to gcc.dg/ipa/ipa-icf-34.c

2015-03-08 Thread John David Anglin
This test requires alias support and fails on hppa2.0-hp-hpux11.11 as a result. Tested on hppa2.0-hp-hpux11.11. Committed to trunk. Dave -- John David Anglin dave.ang...@bell.net 2015-03-08 John David Anglin * gcc.dg/ipa/ipa-icf-34.c: Add dg-require-alias. Index: gcc.dg/ipa/

Re: PR c++/64626 - C++14 single quote should not always be a digit separator

2015-03-08 Thread Jakub Jelinek
On Sun, Mar 08, 2015 at 02:33:03PM -0400, Ed Smith-Rowland wrote: > I just look to see if a lexed number in libcpp ends in a '. > If so, back up one character and let string processing handle it. > > I get one warning and one error for each instance but I'd rather not error > when it should pass.

PR c++/64626 - C++14 single quote should not always be a digit separator

2015-03-08 Thread Ed Smith-Rowland
I just look to see if a lexed number in libcpp ends in a '. If so, back up one character and let string processing handle it. I get one warning and one error for each instance but I'd rather not error when it should pass. Built and tests clean on x86_64-linux. OK? libcpp/ 2015-03-09 Edwa

[SH] Backport fix of PR 53988 to 4.9/4.8

2015-03-08 Thread Oleg Endo
Hi, Backporting of the fix went into GCC 5 seems to intrusive for the released branches. Hence I'd propose to remove the problematic patterns altogether, as they can silently generate wrong code. The attached patch does that. Tested briefly with 'make all' and with make -k check-gcc RUNTESTFLAG

Re: [patch, build] Restore bootstrap in building libcc1 on darwin

2015-03-08 Thread Ian Lance Taylor
Iain Sandoe writes: > +Ian > + ping The patch is missing the change to maint-tool to generate the rules at the bottom of libiberty/Makefile.in. Right now the libiberty Makefile does not require GNU make, and it's not clear to me that we want to push that dependency onto all projects that use li

Re: [patch, build] Restore bootstrap in building libcc1 on darwin

2015-03-08 Thread Iain Sandoe
+Ian + ping On 11 Feb 2015, at 14:37, Iain Sandoe wrote: > Hi Jeff, > > On 9 Feb 2015, at 14:47, Jeff Law wrote: > >> On 02/01/15 09:42, Iain Sandoe wrote: >>> >>> This is a GCC5 bootstrap regression on 32bit Darwin hosts ( I can raise a >>> PR if that is considered necessary). >> Has this b

Re: sh-*-linux build failure

2015-03-08 Thread Kaz Kojima
Oleg Endo wrote: >> diff --git a/gcc/config/sh/t-linux b/gcc/config/sh/t-linux >> index d33c638..22b8e3e 100644 >> --- a/gcc/config/sh/t-linux >> +++ b/gcc/config/sh/t-linux >> @@ -1,2 +1,3 @@ >> MULTILIB_DIRNAMES= >> -MULTILIB_MATCHES = >> +MULTILIB_MATCHES = >> +MULTILIB_EXCEPTIONS = m2a m2a/

Re: sh-*-linux build failure

2015-03-08 Thread Oleg Endo
On Sun, 2015-03-08 at 17:14 +0900, Yoshinori Sato wrote: > Bootstrap failed in following configuration. > > configure --target=sh-unknown-linux --enable-languages=c > --disable-libatomic --disable-threads --disable-shared --disable-libssp > --disable-libquadmath --disable-libgomp > > xgcc: error:

Re: [Patch, fortran] PR65024 - [4.9/5 Regression] [OOP] ICE concerning unlimited polymorphic pointer

2015-03-08 Thread FX
>> This patch has something of a band aid flavour about it. However, the >> more I look at it the more I like it and it cannot do any harm. In any >> case, I spent a silly amount of time trying to understand why this >> component fails to get its backend_decl in the usual way and failed. Well, it

Re: [Patch, fortran] PR65024 - [4.9/5 Regression] [OOP] ICE concerning unlimited polymorphic pointer

2015-03-08 Thread Paul Richard Thomas
ping!!! On 26 February 2015 at 06:33, Paul Richard Thomas wrote: > Dear All, > > This patch has something of a band aid flavour about it. However, the > more I look at it the more I like it and it cannot do any harm. In any > case, I spent a silly amount of time trying to understand why this > co

Re: [PATCH] PR63175 - [4.9/5 regression] FAIL: gcc.dg/vect/costmodel/ppc/costmodel-bb-slp-9a.c scan-tree-dump-times slp2" basic block vectorized using SLP" 1

2015-03-08 Thread Richard Biener
On March 7, 2015 5:20:08 PM CET, Jeff Law wrote: >On 03/07/15 01:34, Richard Biener wrote: >> On March 6, 2015 9:22:05 PM CET, Martin Sebor >wrote: >>> On 03/06/2015 10:28 AM, Jeff Law wrote: On 03/02/15 09:28, Martin Sebor wrote: > On 03/02/2015 06:58 AM, Richard Biener wrote: >> On

sh-*-linux build failure

2015-03-08 Thread Yoshinori Sato
Bootstrap failed in following configuration. configure --target=sh-unknown-linux --enable-languages=c --disable-libatomic --disable-threads --disable-shared --disable-libssp --disable-libquadmath --disable-libgomp xgcc: error: SH2a does not support little-endian So add build exception for SH2A