Re: [PATCH 6/n] OpenMP 4.0 offloading infrastructure: option handling

2015-02-19 Thread Thomas Schwinge
Hi! On Wed, 18 Feb 2015 18:04:21 +0100, I wrote: > On Mon, 13 Oct 2014 14:33:11 +0400, Ilya Verbin wrote: > > On 13 Oct 12:19, Jakub Jelinek wrote: > > > On Sat, Oct 11, 2014 at 06:49:00PM +0400, Ilya Verbin wrote: > > > > 2. -foffload-abi=[lp64|ilp32] > > > >This option is supposed to tell m

Re: nvptx offloading patches [3/n], RFD

2015-02-19 Thread Bernd Schmidt
On 02/17/2015 05:40 PM, Jakub Jelinek wrote: On Tue, Feb 17, 2015 at 04:21:06PM +, Joseph Myers wrote: On Tue, 17 Feb 2015, Jakub Jelinek wrote: Third attempt failed with: ../../../libgcc/config/nvptx/realloc.c:24:20: fatal error: stdlib.h: No such file or directory compilation terminated

Re: [PATCH] Fix for PR c++/60269

2015-02-19 Thread Jakub Jelinek
On Thu, Feb 19, 2015 at 03:39:19AM +, Iyer, Balaji V wrote: > Attached, please find a patch that is a fix for PR c++/60269. Tested on > x86_64 and have no regression issues. Is this OK for trunk? > +2015-02-18  Balaji V. Iyer  > + > +   PR c++/60269 > +   * parser.c (cp_parser_cilk_s

[stage1] Postpone expanding va_arg until pass_stdarg

2015-02-19 Thread Tom de Vries
Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002-Add-gimple_find_sub_bbs.patch - 0003-Factor-optimize_va_list_gpr_fpr_size-out-of-pass_std.patch - 0004-Handle-internal_fn-in-operand_equal_p.patch - 0005-Postpone-expanding-va_arg-

If we're building an offloading compiler, always enable the LTO front end (was: nvptx offloading patches [3/n], RFD)

2015-02-19 Thread Thomas Schwinge
Hi! On Wed, 18 Feb 2015 13:35:18 +0100, Jakub Jelinek wrote: > On Wed, Feb 18, 2015 at 01:09:53PM +0100, Thomas Schwinge wrote: > > On Wed, 18 Feb 2015 12:34:38 +0100, Jakub Jelinek wrote: > > > offloading fails: > > > > > > /usr/src/gcc/objnvptxinst/usr/local/bin/../libexec/gcc/x86_64-pc-linux

[PATCH][1/5] Disable lang_hooks.gimplify_expr in free_lang_data

2015-02-19 Thread Tom de Vries
On 19-02-15 11:29, Tom de Vries wrote: Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002-Add-gimple_find_sub_bbs.patch - 0003-Factor-optimize_va_list_gpr_fpr_size-out-of-pass_std.patch - 0004-Handle-internal_fn-in-operand_equal_p

Re: If we're building an offloading compiler, always enable the LTO front end (was: nvptx offloading patches [3/n], RFD)

2015-02-19 Thread Jakub Jelinek
On Thu, Feb 19, 2015 at 11:48:17AM +0100, Thomas Schwinge wrote: > Like this? Yes. > commit 56c0312469f583ba3fa9fa2777981742ab6d6c75 > Author: Thomas Schwinge > Date: Thu Feb 19 11:41:23 2015 +0100 > > If we're building an offloading compiler, always enable the LTO front end. > >

[PATCH] Fix ICE in early inliner (PR ipa/65008)

2015-02-19 Thread Marek Polacek
The problem exposed by this PR is (IIUC) that we hadn't gotten around to recomputing the inline parameters in the case when optimize_inline_calls introduces new statements. That results in ICEing later on because in estimate_edge_growth we assert that estimated size of a statement is not 0. This

[PATCH][2/5] Add gimple_find_sub_bbs

2015-02-19 Thread Tom de Vries
On 19-02-15 11:29, Tom de Vries wrote: Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002-Add-gimple_find_sub_bbs.patch - 0003-Factor-optimize_va_list_gpr_fpr_size-out-of-pass_std.patch - 0004-Handle-internal_fn-in-operand_equal_p

[PATCH][3/5] Factor optimize_va_list_gpr_fpr_size out of pass_stdarg::execute

2015-02-19 Thread Tom de Vries
On 19-02-15 11:29, Tom de Vries wrote: Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002-Add-gimple_find_sub_bbs.patch - 0003-Factor-optimize_va_list_gpr_fpr_size-out-of-pass_std.patch - 0004-Handle-internal_fn-in-operand_equal_p

Re: [PATCH] Fix ICE in early inliner (PR ipa/65008)

2015-02-19 Thread Richard Biener
On Thu, 19 Feb 2015, Marek Polacek wrote: > The problem exposed by this PR is (IIUC) that we hadn't gotten around to > recomputing the inline parameters in the case when optimize_inline_calls > introduces new statements. That results in ICEing later on because in > estimate_edge_growth we assert

[PATCH][4/5] Handle internal_fn in operand_equal_p

2015-02-19 Thread Tom de Vries
On 19-02-15 11:29, Tom de Vries wrote: Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002-Add-gimple_find_sub_bbs.patch - 0003-Factor-optimize_va_list_gpr_fpr_size-out-of-pass_std.patch - 0004-Handle-internal_fn-in-operand_equal_p

Re: [PATCH 1/n] OpenMP 4.0 offloading infrastructure

2015-02-19 Thread Thomas Schwinge
Hi! On Fri, 26 Sep 2014 16:36:21 +0400, Ilya Verbin wrote: > The patch has been updated: > if a compiler is configured as accelerator, > it installs *-accel-*-g++, and other drivers. But only *-accel-*-gcc is > needed. > Therefore I suppressed their installation in corresponding Makefiles. >

Offloading compilers' support libraries (was: nvptx offloading patches [3/n], RFD)

2015-02-19 Thread Thomas Schwinge
Hi! On Thu, 19 Feb 2015 10:28:46 +0100, Bernd Schmidt wrote: > On 02/17/2015 05:40 PM, Jakub Jelinek wrote: > > On Tue, Feb 17, 2015 at 04:21:06PM +, Joseph Myers wrote: > >> On Tue, 17 Feb 2015, Jakub Jelinek wrote: > >> > >>> Third attempt failed with: > >>> ../../../libgcc/config/nvptx/re

[PATCH][5/5] Postpone expanding va_arg until pass_stdarg

2015-02-19 Thread Tom de Vries
On 19-02-15 11:29, Tom de Vries wrote: Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002-Add-gimple_find_sub_bbs.patch - 0003-Factor-optimize_va_list_gpr_fpr_size-out-of-pass_std.patch - 0004-Handle-internal_fn-in-operand_equal_p

Re: Offloading compilers' support libraries

2015-02-19 Thread Bernd Schmidt
On 02/19/2015 12:42 PM, Thomas Schwinge wrote: This specific buglet aside (that the handling of intelmic and nvptx offloading is inconsistent) -- will we have to add such handling to each and every library that is built for the offloading compilers? (Including libraries that aren't part of the G

[PATCH] Fix LTO -g ICE (PR lto/65012)

2015-02-19 Thread Jakub Jelinek
Hi! When building systemd with LTO and -g, we ICE, because we stream in __PRETTY_FUNCTION__ VAR_DECL, which isn't referenced in the IL except for BLOCK_VARS and thus doesn't get a varpool node streamed with it. Later on when dwarf2out.c calls make_decl_rtl_for_debug on it, it creates the varpool_n

Re: Offloading compilers' support libraries

2015-02-19 Thread Thomas Schwinge
Hi! On Thu, 19 Feb 2015 13:08:20 +0100, Bernd Schmidt wrote: > On 02/19/2015 12:42 PM, Thomas Schwinge wrote: > > This specific buglet aside (that the handling of intelmic and nvptx > > offloading is inconsistent) -- will we have to add such handling to each > > and every library that is built f

[PATCH][ARM] Remove an unused reload hook.

2015-02-19 Thread Matthew Wahab
The LEGITIMIZE_RELOAD_ADDRESS macro is only needed for reload. Since the ARM backend no longer supports reload, this macro is not needed and this patch removes it. Tested arm-none-linux-gnueabihf with gcc-check. Ok for trunk? now or in stage 1? Matthew gcc/ 2015-02-19 Matthew Wahab

Re: [PATCH][1/5] Disable lang_hooks.gimplify_expr in free_lang_data

2015-02-19 Thread Richard Biener
On Thu, 19 Feb 2015, Tom de Vries wrote: > On 19-02-15 11:29, Tom de Vries wrote: > > Hi, > > > > I'm posting this patch series for stage1: > > - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch > > - 0002-Add-gimple_find_sub_bbs.patch > > - 0003-Factor-optimize_va_list_gpr_fpr_size-

Re: [PATCH][2/5] Add gimple_find_sub_bbs

2015-02-19 Thread Richard Biener
On Thu, 19 Feb 2015, Tom de Vries wrote: > On 19-02-15 11:29, Tom de Vries wrote: > > Hi, > > > > I'm posting this patch series for stage1: > > - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch > > - 0002-Add-gimple_find_sub_bbs.patch > > - 0003-Factor-optimize_va_list_gpr_fpr_size-

Re: [PATCH][2/5] Add gimple_find_sub_bbs

2015-02-19 Thread Jakub Jelinek
On Thu, Feb 19, 2015 at 01:41:05PM +0100, Richard Biener wrote: > I hope we can get rid of this again (which needs re-writing of all > targets va-arg gimplification hooks... - we are in need of some > generic diamond/triangle CFG pattern builders anyway). We already have some in asan.c - create_co

Re: [PATCH][4/5] Handle internal_fn in operand_equal_p

2015-02-19 Thread Jakub Jelinek
On Thu, Feb 19, 2015 at 01:44:46PM +0100, Richard Biener wrote: > I'd call it a bug though, and we do have internal fns in > generic already thus the issue is latent (with ubsan at least). > > Which means ok for trunk now. But the patch should better handle the internal calls right. I.e. return 0

Re: [PATCH][2/5] Add gimple_find_sub_bbs

2015-02-19 Thread Marek Polacek
On Thu, Feb 19, 2015 at 01:48:58PM +0100, Jakub Jelinek wrote: > On Thu, Feb 19, 2015 at 01:41:05PM +0100, Richard Biener wrote: > > I hope we can get rid of this again (which needs re-writing of all > > targets va-arg gimplification hooks... - we are in need of some > > generic diamond/triangle CF

Re: [PATCH][3/5] Factor optimize_va_list_gpr_fpr_size out of pass_stdarg::execute

2015-02-19 Thread Richard Biener
On Thu, 19 Feb 2015, Tom de Vries wrote: > On 19-02-15 11:29, Tom de Vries wrote: > > Hi, > > > > I'm posting this patch series for stage1: > > - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch > > - 0002-Add-gimple_find_sub_bbs.patch > > - 0003-Factor-optimize_va_list_gpr_fpr_size-

Re: [PATCH][5/5] Postpone expanding va_arg until pass_stdarg

2015-02-19 Thread Richard Biener
On Thu, 19 Feb 2015, Tom de Vries wrote: > On 19-02-15 11:29, Tom de Vries wrote: > > Hi, > > > > I'm posting this patch series for stage1: > > - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch > > - 0002-Add-gimple_find_sub_bbs.patch > > - 0003-Factor-optimize_va_list_gpr_fpr_size-

Re: [PATCH][4/5] Handle internal_fn in operand_equal_p

2015-02-19 Thread Richard Biener
On Thu, 19 Feb 2015, Tom de Vries wrote: > On 19-02-15 11:29, Tom de Vries wrote: > > Hi, > > > > I'm posting this patch series for stage1: > > - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch > > - 0002-Add-gimple_find_sub_bbs.patch > > - 0003-Factor-optimize_va_list_gpr_fpr_size-

Re: [PATCH][2/5] Add gimple_find_sub_bbs

2015-02-19 Thread Richard Biener
On Thu, 19 Feb 2015, Jakub Jelinek wrote: > On Thu, Feb 19, 2015 at 01:41:05PM +0100, Richard Biener wrote: > > I hope we can get rid of this again (which needs re-writing of all > > targets va-arg gimplification hooks... - we are in need of some > > generic diamond/triangle CFG pattern builders a

Re: [PATCH][4/5] Handle internal_fn in operand_equal_p

2015-02-19 Thread Richard Biener
On Thu, 19 Feb 2015, Jakub Jelinek wrote: > On Thu, Feb 19, 2015 at 01:44:46PM +0100, Richard Biener wrote: > > I'd call it a bug though, and we do have internal fns in > > generic already thus the issue is latent (with ubsan at least). > > > > Which means ok for trunk now. > > But the patch sho

Re: [PATCH] PR rtl-optimization/32219: optimizer causees wrong code in pic/hidden/weak symbol checking

2015-02-19 Thread H.J. Lu
On Wed, Feb 18, 2015 at 6:17 AM, Alex Velenko wrote: > On 13/02/15 05:11, Richard Henderson wrote: >> >> On 02/12/2015 08:14 PM, H.J. Lu wrote: >>> >>> I tried the second patch. Results look good on Linux/x86-64. >> >> >> Thanks. My results concurr. I went ahead and installed the patch as >> po

Re: nvptx offloading patches [2/n]

2015-02-19 Thread Jakub Jelinek
On Tue, Feb 17, 2015 at 09:55:32PM +0100, Bernd Schmidt wrote: > On 02/17/2015 06:10 PM, Jakub Jelinek wrote: > > > >What exact testcase are you trying to fix with this patch, and how do you > >think offloading of code using va_list can work? > > The exact testcase is any offloaded program - strea

Re: [PATCH][2/5] Add gimple_find_sub_bbs

2015-02-19 Thread Richard Biener
On Thu, 19 Feb 2015, Marek Polacek wrote: > On Thu, Feb 19, 2015 at 01:48:58PM +0100, Jakub Jelinek wrote: > > On Thu, Feb 19, 2015 at 01:41:05PM +0100, Richard Biener wrote: > > > I hope we can get rid of this again (which needs re-writing of all > > > targets va-arg gimplification hooks... - we

Re: [PATCH] Fix testsuite race on additional_sources

2015-02-19 Thread H.J. Lu
On Wed, Feb 18, 2015 at 7:05 AM, Maxim Kuvyrkov wrote: > Hi, > > This testsuite patch fixes race on additional_source testsuite variable. > When a test has both dg-additional-sources and "dg-do run { target FOO }" > directives, it may occur that the FOO test will attempt to use > additional_so

Re: [PATCH] Fix testsuite race on additional_sources

2015-02-19 Thread Maxim Kuvyrkov
> On Feb 19, 2015, at 4:31 PM, H.J. Lu wrote: > > On Wed, Feb 18, 2015 at 7:05 AM, Maxim Kuvyrkov > wrote: >> Hi, >> >> This testsuite patch fixes race on additional_source testsuite variable. >> When a test has both dg-additional-sources and "dg-do run { target FOO }" >> directives, it may o

Re: [PATCH] PR64959: SFINAE in UDLs

2015-02-19 Thread Alex Velenko
On 18/02/15 18:30, Jakub Jelinek wrote: On Wed, Feb 18, 2015 at 06:29:34PM +, Alex Velenko wrote: this patch also fixes issues for arm-none-eabi. Could someone add this patch? ENOPATCH Jakub Hi Jakub, I meant Andrea Azzarone's patch in the previous e-mail. It has not been added

Re: nvptx offloading patches [2/n]

2015-02-19 Thread Richard Biener
On Thu, Feb 19, 2015 at 2:09 PM, Jakub Jelinek wrote: > On Tue, Feb 17, 2015 at 09:55:32PM +0100, Bernd Schmidt wrote: >> On 02/17/2015 06:10 PM, Jakub Jelinek wrote: >> > >> >What exact testcase are you trying to fix with this patch, and how do you >> >think offloading of code using va_list can w

Re: [PATCH] PR64959: SFINAE in UDLs

2015-02-19 Thread Jakub Jelinek
On Fri, Feb 13, 2015 at 11:21:26PM +0100, Andrea Azzarone wrote: > We can use the same trick used in the other tests. Patch attached. > Sorry about that! Thanks. I wrote a ChangeLog entry for this and committed. Jakub

Re: ping #3: [RFA] Add --with-libz-prefix option in config/zlib.m4

2015-02-19 Thread H.J. Lu
On Wed, Feb 18, 2015 at 11:17 PM, Mark Wielaard wrote: > On Wed, Feb 18, 2015 at 01:54:17PM -0800, H.J. Lu wrote: >> On Wed, Feb 18, 2015 at 1:40 PM, Mark Wielaard wrote: >> > On Wed, 2015-02-18 at 12:53 -0800, H.J. Lu wrote: >> >> On Wed, Feb 18, 2015 at 12:32 PM, Mark Wielaard wrote: >> >> > T

Re: [PATCH] PR rtl-optimization/32219: optimizer causees wrong code in pic/hidden/weak symbol checking

2015-02-19 Thread Richard Henderson
On 02/18/2015 06:17 AM, Alex Velenko wrote: > By changing behaviour of varasm.c:default_binds_local_p, this patch changes > behaviour of gcc/config/arm/arm.c:arm_function_in_section_p and through it > breaks gcc/config/arm/arm.c:arm_is_long_call_p for weak symbols. > > As a result, I get regressio

Re: [PATCH][4/5] Handle internal_fn in operand_equal_p

2015-02-19 Thread Tom de Vries
On 19-02-15 14:07, Richard Biener wrote: On Thu, 19 Feb 2015, Jakub Jelinek wrote: On Thu, Feb 19, 2015 at 01:44:46PM +0100, Richard Biener wrote: I'd call it a bug though, and we do have internal fns in generic already thus the issue is latent (with ubsan at least). Which means ok for trunk

Fix PR65093, timeouts with 26_numerics/random/binomial_distribution/operators/values.cc

2015-02-19 Thread Hans-Peter Nilsson
Lately, 26_numerics/random/binomial_distribution/operators/values.cc has started to FAIL on trunk with a timeout for my autotester for cris-elf, a soft-float simulator target running a cgen-generated simulator on a six-year-old x86_64-linux-gnu host. The reason it's started to fail in the last few

Re: PATCH for c++/64948

2015-02-19 Thread Andrea Azzarone
Ping? 2015-02-13 15:51 GMT+01:00 Andrea Azzarone : > Hi all, > > this patch try to fix PR c++/64948 (Lambda reference capture > initialization in template function creates segmentation fault). > > 2015-2-13 Andrea Azzarone > PR c++/64948 > * lambda.c (add_capture) Do not consider as rvalues a

[doc, committed] copy-edit x86 transactional memory intrinsics section

2015-02-19 Thread Sandra Loosemore
A while ago when I was doing some other work on extend.texi, I noticed that the section on x86 transactional memory intrinsics was in need of cleanup. I've checked in this patch to do some content-free copy-editing -- fixing typos and markup, rephrasing some awkward sentences, etc. To tell t

[PATCH, 4.8] Backport alias patches to 4.8 branch

2015-02-19 Thread Uros Bizjak
Hello! I would like to port following patches to 4.8 branch: 2014-11-05 Uros Bizjak Backport from mainline 2015-01-22 Wei Mi PR rtl-optimization/64557 * dse.c (record_store): Call get_addr for mem_addr. (check_mem_read_rtx): Likewise. Backport from mainline 201

Re: [doc, committed] copy-edit x86 transactional memory intrinsics section

2015-02-19 Thread Patrick Marlier
Thanks Sandra. Just a minor comment. -Valid abort status bits (when the value is not @code{_XBEGIN_STARTED}) are: +If the transaction aborts, the return value is one of: Here it is really bits. So maybe something like that: If the transaction aborts, the return value is a combination of the follo

[patch] PR debug/46102 Disable -feliminate-dwarf2-dups when reading a PCH

2015-02-19 Thread Aldy Hernandez
[And this time, actually CCing the list :)]. Gentlemen! Reading in the compiler state for pch (gt_pch_restore) obliterates the DIE table, and consequently the DW_TAG_GNU_[BE]INCL* DIEs that may have been in it. This causes inconsistencies when reading in _any_ pre-compiled header into a source

Re: [patch] PR debug/46102 Disable -feliminate-dwarf2-dups when reading a PCH

2015-02-19 Thread Jakub Jelinek
On Thu, Feb 19, 2015 at 08:45:08AM -0800, Aldy Hernandez wrote: > [And this time, actually CCing the list :)]. > > Gentlemen! > > Reading in the compiler state for pch (gt_pch_restore) obliterates the > DIE table, and consequently the DW_TAG_GNU_[BE]INCL* DIEs that may have > been in it. This ca

Re: [PATCH] PR rtl-optimization/32219: optimizer causees wrong code in pic/hidden/weak symbol checking

2015-02-19 Thread Alex Velenko
On 19/02/15 14:16, Richard Henderson wrote: On 02/18/2015 06:17 AM, Alex Velenko wrote: By changing behaviour of varasm.c:default_binds_local_p, this patch changes behaviour of gcc/config/arm/arm.c:arm_function_in_section_p and through it breaks gcc/config/arm/arm.c:arm_is_long_call_p for weak s

[PATCH] S/390: Cleanup for the hotpatch testcases

2015-02-19 Thread Andreas Krebbel
Hi, the attached patch fixes some minor issues with the recently applied hotpatch testcases. Bye, -Andreas- 2015-02-19 Andreas Krebbel * gcc.target/s390/hotpatch-8.c: Add -march=g5. * gcc.target/s390/hotpatch-9.c: Add -march=g5. * gcc.target/s390/hotpatch-compile-1.

[PATCH] S/390: Remove --save-temps from testcases not requiring it

2015-02-19 Thread Andreas Krebbel
Hi, the attached patch does some cleanup in the S/390 specific testcases. - The scan-assembler testcases do not need --save-temps. - One of them needs it so with the patch we make sure to cleanup the files. Bye, -Andreas- 2015-02-19 Andreas Krebbel * hotpatch-1.c: Remove --save-t

[PATCH] S/390: Suppress implicit function declaration warning in some testcases

2015-02-19 Thread Andreas Krebbel
Hi, the attached patch add the -Wno-implicit-function-declaration option to the S/390 specific testcases which require it. Bye, -Andreas- 2015-02-19 Andreas Krebbel * gcc.target/s390/20090223-1.c: Add -Wno-implicit-function-declaration option. * gcc.target/s390/pr42

Re: [PATCH] PR rtl-optimization/32219: optimizer causees wrong code in pic/hidden/weak symbol checking

2015-02-19 Thread Richard Henderson
On 02/19/2015 09:08 AM, Alex Velenko wrote: > Your suggestion seem to fix gcc.target/arm/long-calls-1.c, but has to be > thoroughly tested. Before you do complete testing, please also delete the TREE_STATIC test. That bit should never be relevant to functions, as it indicates not that it is in the

[PATCH] Fix hoist-register-pressure testcases for -m31 on s390x

2015-02-19 Thread Andreas Krebbel
Hi, the attached patch makes the hoist-register-pressure testcases to work with -m31 on 64 bit. Ok to apply? Bye, -Andreas- 2015-02-19 Andreas Krebbel * gcc.dg/hoist-register-pressure-1.c: Make S/390 target check work with -m31 on 64 bit. * gcc.dg/hoist-register-pr

[patch] PR debug/58123: Set correct location for TRY blocks

2015-02-19 Thread Aldy Hernandez
[Ughh...I'm apparently incapable of CCing gcc-patches on the first try. My apologies again.] Please see the long explanation in the PR: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=58123 But the short explanation for this patch is that, at the very least, we should fix the location of the GIM

Re: [patch] PR debug/46102 Disable -feliminate-dwarf2-dups when reading a PCH

2015-02-19 Thread Aldy Hernandez
On 02/19/2015 08:50 AM, Jakub Jelinek wrote: On Thu, Feb 19, 2015 at 08:45:08AM -0800, Aldy Hernandez wrote: [And this time, actually CCing the list :)]. Gentlemen! Reading in the compiler state for pch (gt_pch_restore) obliterates the DIE table, and consequently the DW_TAG_GNU_[BE]INCL* DIEs

Re: [patch] PR debug/46102 Disable -feliminate-dwarf2-dups when reading a PCH

2015-02-19 Thread Jakub Jelinek
On Thu, Feb 19, 2015 at 10:33:20AM -0800, Aldy Hernandez wrote: > Well, any PCH file we generate will have some sort of early DIE in it (at > the very least the compilation unit DIE) and we will read these in at PCH > read-in time, obliterating whatever was already there. But most > importantly, w

[GOOGLE] Disable overflow warning of the offset

2015-02-19 Thread Dehao Chen
The offset overflow warning would cause build fails when function's start line is missing(0). Until the start line issues is fixed, we will suppress this warning. Testing on-going. OK for google-4_9? Thanks, Dehao Index: gcc/auto-profile.c ===

Re: [GOOGLE] Disable overflow warning of the offset

2015-02-19 Thread Xinliang David Li
ok david On Thu, Feb 19, 2015 at 11:07 AM, Dehao Chen wrote: > The offset overflow warning would cause build fails when function's > start line is missing(0). Until the start line issues is fixed, we > will suppress this warning. > > Testing on-going. OK for google-4_9? > > Thanks, > Dehao > > I

Re: Fix PR65093, timeouts with 26_numerics/random/binomial_distribution/operators/values.cc

2015-02-19 Thread Jonathan Wakely
On 19/02/15 16:31 +0100, Hans-Peter Nilsson wrote: Lately, 26_numerics/random/binomial_distribution/operators/values.cc has started to FAIL on trunk with a timeout for my autotester for cris-elf, a soft-float simulator target running a cgen-generated simulator on a six-year-old x86_64-linux-gnu h

Re: [doc, committed] copy-edit x86 transactional memory intrinsics section

2015-02-19 Thread Sandra Loosemore
On 02/19/2015 09:38 AM, Patrick Marlier wrote: Thanks Sandra. Just a minor comment. -Valid abort status bits (when the value is not @code{_XBEGIN_STARTED}) are: +If the transaction aborts, the return value is one of: Here it is really bits. So maybe something like that: If the transaction abort

[PATCH][committed, trivial] Fix reference to additional_sources in testsuite

2015-02-19 Thread Maxim Kuvyrkov
Hi, I've just committed obvious fix to PR65116 after testing it with a fresh bootstrap and regression test with all languages. I have also confirmed that it still fixes the race on additional_sources. The fix is trivial, and I don't think I could have goofed up with a testsuite change more an

Re: [PATCH] Fix ICE in early inliner (PR ipa/65008)

2015-02-19 Thread Marek Polacek
On Thu, Feb 19, 2015 at 12:06:40PM +0100, Richard Biener wrote: > On Thu, 19 Feb 2015, Marek Polacek wrote: > > > The problem exposed by this PR is (IIUC) that we hadn't gotten around to > > recomputing the inline parameters in the case when optimize_inline_calls > > introduces new statements. Th

[v850] fix branch limits

2015-02-19 Thread DJ Delorie
The branch limits are a bit too far, resulting in reloc errors in rare cases. Ok? * config/v850/v850.md (branch_normal): Adjust branch limits. (branch_invert): Likewise. (branch_z_normal): Likewise. (branch_z_invert): Likewise. (branch_nz_normal): Likewise

[patch] libstdc++/58357 change std::rotate to std::_V2::rotate

2015-02-19 Thread Jonathan Wakely
DR 488 changed std:rotate to return an iterator, implemented on trunk in r219793. To avoid problems where code expecting a non-void return links to an instantiation of the old void-returning definition I'm putting the new definition in the inline namespace _V2 (already used for other entities that

Re: [PATCH] PR rtl-optimization/32219: optimizer causees wrong code in pic/hidden/weak symbol checking

2015-02-19 Thread Uros Bizjak
On Mon, Feb 16, 2015 at 5:30 PM, Richard Henderson wrote: 2015-02-12 H.J. Lu Richard Henderson PR rtl/32219 * cgraphunit.c (cgraph_node::finalize_function): Set definition before notice_global_symbol. (varpool_nod

Re: [PATCH] PR rtl-optimization/32219: optimizer causees wrong code in pic/hidden/weak symbol checking

2015-02-19 Thread H.J. Lu
On Thu, Feb 19, 2015 at 1:04 PM, Uros Bizjak wrote: > On Mon, Feb 16, 2015 at 5:30 PM, Richard Henderson wrote: > > 2015-02-12 H.J. Lu > Richard Henderson > > PR rtl/32219 > * cgraphunit.c (cgraph_node::finalize_function): Set definition >

Re: [PATCH] PR rtl-optimization/32219: optimizer causees wrong code in pic/hidden/weak symbol checking

2015-02-19 Thread H.J. Lu
On Thu, Feb 19, 2015 at 1:07 PM, H.J. Lu wrote: > On Thu, Feb 19, 2015 at 1:04 PM, Uros Bizjak wrote: >> On Mon, Feb 16, 2015 at 5:30 PM, Richard Henderson wrote: >> >> 2015-02-12 H.J. Lu >> Richard Henderson >> >> PR rtl/32219 >> * cgraphunit

Re: [PATCH] PR rtl-optimization/32219: optimizer causees wrong code in pic/hidden/weak symbol checking

2015-02-19 Thread Richard Henderson
On 02/19/2015 01:04 PM, Uros Bizjak wrote: > 2015-02-19 Uros Bizjak > > * config/alpha/alpha.c (alpha_in_small_data_p): Reject common symbols. Ok. Thanks. r~

Re: [PATCH] PR rtl-optimization/32219: optimizer causees wrong code in pic/hidden/weak symbol checking

2015-02-19 Thread Richard Henderson
On 02/19/2015 01:07 PM, H.J. Lu wrote: >> > The attached patch fixes all alpha-linux-gnu failures. >> > >> > 2015-02-19 Uros Bizjak >> > >> > * config/alpha/alpha.c (alpha_in_small_data_p): Reject common symbols. >> > >> > Patch was bootstrapped and regression tested on alphaev68-linux-gnu.

Re: [PATCH] PR rtl-optimization/32219: optimizer causees wrong code in pic/hidden/weak symbol checking

2015-02-19 Thread H.J. Lu
On Thu, Feb 19, 2015 at 1:16 PM, Richard Henderson wrote: > On 02/19/2015 01:04 PM, Uros Bizjak wrote: >> 2015-02-19 Uros Bizjak >> >> * config/alpha/alpha.c (alpha_in_small_data_p): Reject common symbols. > > Ok. Thanks. We tried it on ia64 and failed with Ada build. -- H.J.

Re: [PATCH] PR rtl-optimization/32219: optimizer causees wrong code in pic/hidden/weak symbol checking

2015-02-19 Thread Uros Bizjak
On Thu, Feb 19, 2015 at 10:35 PM, H.J. Lu wrote: > On Thu, Feb 19, 2015 at 1:16 PM, Richard Henderson wrote: >> On 02/19/2015 01:04 PM, Uros Bizjak wrote: >>> 2015-02-19 Uros Bizjak >>> >>> * config/alpha/alpha.c (alpha_in_small_data_p): Reject common symbols. >> >> Ok. Thanks. > > We tri

Fix alignment propagation

2015-02-19 Thread Jan Hubicka
Hi, this patch fixes alignment propagation that causes wrong code on solex and firefox. Patch is by Martin, I just added the obvous MINUS_EXPR fix (the offset would be wrong, but I see no reason for MINUX_ExPR appearing there with constant parameter), went ahead and commited the fix. Tested on

Re: Fix alignment propagation

2015-02-19 Thread Marek Polacek
On Fri, Feb 20, 2015 at 12:39:29AM +0100, Jan Hubicka wrote: > Hi, > this patch fixes alignment propagation that causes wrong code on solex and > firefox. > Patch is by Martin, I just added the obvous MINUS_EXPR fix (the offset would > be wrong, > but I see no reason for MINUX_ExPR appearing ther

Re: [committed] Change "Q" and "T" constraints to memory constraints

2015-02-19 Thread John David Anglin
On 2015-02-16, at 1:47 PM, John David Anglin wrote: > On 2015-02-16, at 11:38 AM, Richard Henderson wrote: > >>> >>> Possibly the constant can somehow be forced into the data section where the >>> relocations >>> aren't a problem? >> >> Hmm. It looks like we might already do that. See >> de

[C/C++ PATCH] -Wlogical-not-parentheses tweaks (PR c/65120)

2015-02-19 Thread Jakub Jelinek
Hi! As reported, !!x == y is quite common in the Linux kernel and unlike the !x == y case it usually doesn't mean mistyped !(x == y) or x != y. clang++ apparently doesn't warn about that either, and it doesn't warn even about the case where ! is applied to a bool. As the argument is already fold

[PATCH testsuite committed] Fix a few testcases for SH

2015-02-19 Thread Kaz Kojima
I've committed the attached patch to fix two failures of tree-ssa tests on SH. Tested on sh4-unknown-linux-gnu and i686-pc-linux-gnu. Regards, kaz -- 2015-02-19 Kaz Kojima * gcc.dg/tree-ssa/20040204-1.c: Don't XFAIL on sh*-*-*. * gcc.dg/tree-ssa/sra-12.c: Skip for sh*-

[committed] Always force (const (plus (symbol) (const_int))) to const mem

2015-02-19 Thread John David Anglin
In spite of the relocation, this sequence is shorter than loading the constant and symbol_ref independently when the constant doesn't fit in 14 bits. So, I believe this is a win. I've also added a REG_EQUAL note to improve optimization. Tested on hppa-unknown-linux-gnu, hppa2.0w-hp-hpux11.11 an

Re: [PATCH, 4.8] Backport "Fix debug-insn sensitivity in RA" patch to 4.8

2015-02-19 Thread Vladimir Makarov
On 2015-02-18 12:25 PM, Uros Bizjak wrote: Hello! Richard's patch at [1] is needed to fix a bootstrap failure on alpha-linux-gnu on 4.8 branch. Without the patch, IRA creates different sequences, depending on the presence of -g option. One of the many comparison failures is in expr.c, where -f

Re: [PATCH, AArch64] [4.9] Handle SYMBOL_SMALL_TPREL appropriately

2015-02-19 Thread Hurugalawadi, Naveen
Hi Marcus, >> The handling of SYMBOL_SMALL_TPREL is present in 4.9 and very clearly >> has exactly the same issue. Please find attached the patch ported for gcc-4.9. Please review the patch and let us know if its okay? Regression tested on aarch64-elf. Thanks, Naveen 2015-02-20 Andrew Pinski

Re: Fix alignment propagation

2015-02-19 Thread Jan Hubicka
> On Fri, Feb 20, 2015 at 12:39:29AM +0100, Jan Hubicka wrote: > > Hi, > > this patch fixes alignment propagation that causes wrong code on solex and > > firefox. > > Patch is by Martin, I just added the obvous MINUS_EXPR fix (the offset > > would be wrong, > > but I see no reason for MINUX_ExPR

[PATCH] Fix for PR c++/60198

2015-02-19 Thread Iyer, Balaji V
Hello Everyone, Attached, please find a patch that is a fix for PR c++/60198. Is this OK for trunk? Here are the changelog entries: Cp/ChangeLog +2015-02-19 Balaji V. Iyer + + PR c++/60198 + * pt.c (tsubst_copy_and_build): Added CILK_SPAWN_STMT case. + Testsuite/ChangeLo