Re: Fix PR64878: do not jump thread across two loop iterations

2015-02-04 Thread Richard Biener
On February 4, 2015 10:20:06 PM CET, Sebastian Pop wrote: >Hi, > >The attached patch stops the recursion in the detection of FSM >jump-threads at >loop phi nodes after having visited a loop phi node. This avoids >jump-threading >two iterations forward that were possible due to a flip-flop operati

Re: Work around Firefox -fprofile-generate crash

2015-02-04 Thread Richard Biener
On February 5, 2015 12:28:14 AM CET, Jan Hubicka wrote: >Hi, >this patch avoids ICE in programs that uses multiple copies of libgcov >at once. >It is just a temporary workaround as discussed in the PR log. > >Tested on Firefox and comitted. I expect this will break on platforms with default -fno-

Re: Fix PR64876, regressions in powerpc64 Go testsuite

2015-02-04 Thread Jakub Jelinek
On Thu, Feb 05, 2015 at 02:09:54PM +1030, Alan Modra wrote: > Jakub, was your suggestion to use get_last_insn_anywhere() based on > not wanting to expose details that should be internal to > emit-rtl.[ch]? Yes. But if it doesn't work for what you want, either add a new accessor or use it directly

Fix ordering issue when removing varpool nodes

2015-02-04 Thread Jan Hubicka
Hi, this is second part of fix for PR ipa/61548. Before removing an variable, it is better to remove all its aliases so the constructors can be properly propagated. Bootstrapped/regtested x86_64-linux. Honza PR ipa/61548 * ipa.c (symbol_table::remove_unreachable_nodes): Fix orde

Fix ordering issue in varpol::remove_node

2015-02-04 Thread Jan Hubicka
Hi, this patch fixes ordering issue in varpool.c that, while removing alias it first removes link to its target and then tries to copy the constructor. Bootstrapped/regtested x86_64-linux, commited. PR ipa/61548 * varpool.c (varpool_node::remove): Fix order of variables. Index: var

Re: Fix PR64876, regressions in powerpc64 Go testsuite

2015-02-04 Thread Alan Modra
On Wed, Feb 04, 2015 at 09:06:02PM -0500, David Edelsohn wrote: > Testing only crtl->emit.sequence_stack->last > does not seem to be sufficient. It seems like it really needs to test > for more of the CALL sequence. Ugh. You're right, we do need to test more of the call sequence. Shame on me, I

Re: Fix PR64876, regressions in powerpc64 Go testsuite

2015-02-04 Thread David Edelsohn
On Tue, Feb 3, 2015 at 7:16 PM, Alan Modra wrote: > On Tue, Feb 03, 2015 at 11:14:49AM -0500, David Edelsohn wrote: >> On Tue, Feb 3, 2015 at 8:57 AM, Alan Modra wrote: >> > PR target/64876 >> > * config/rs6000/rs6000.c (chain_already_loaded): New function. >> > (rs6000_ca

Re: [PATCH] More fixes for update_web_docs_svn for jit docs (PR jit/64257)

2015-02-04 Thread Andrew Pinski
On Wed, Feb 4, 2015 at 10:32 AM, David Malcolm wrote: > To build the jit docs, we need to use sphinx 1.0 or later; the > 0.6.6 in EPEL 6 doesn't support all the directives we need. > > The alternate python-sphinx10 in EPEL 6 has 1.0.8: > python-sphinx10-1.0.8-1.el6.noarch > which is able to buil

Re: [PATCH] More fixes for update_web_docs_svn for jit docs (PR jit/64257)

2015-02-04 Thread David Malcolm
On Wed, 2015-02-04 at 17:26 -0500, David Malcolm wrote: > On Wed, 2015-02-04 at 22:21 +0100, Gerald Pfeifer wrote: > > On Wednesday 2015-02-04 14:10, David Malcolm wrote: > > > Gerald: Please can you do the "svn up" on the relevant machine again, > > > so that it gets the updated "update_web_docs_

Re: [debug-early] C++ clones and limbo DIEs

2015-02-04 Thread Aldy Hernandez
Howdy! Such simple requests. Such a pain in the ass to implement :). I know Richard will be tickled pink with the attached patch-- well, at least with the general approach, as generically handling DECLs has been his desire all along. I hope I got it right, as it was a royal pain to get the

Fix ordering issue with speculation reoslution

2015-02-04 Thread Jan Hubicka
Hi, this patch fixes issue with speculation - update is needed between every resolution and key insertion. Bootstrapped/regtested x86_64-linux, comitted. Honza PR ipa/64686 * ipa-inline.c (inline_small_functions): Fix ordering issue between speculation resolution and key

Re: [PATCH] Fix PR64068 and PR64559

2015-02-04 Thread Matthias Klose
On 02/05/2015 12:38 AM, Jakub Jelinek wrote: > On Thu, Feb 05, 2015 at 12:33:18AM +0100, Matthias Klose wrote: >> On 01/15/2015 06:23 PM, Martin Liška wrote: >>> Hello. >>> >>> This is Honsa's patch that I've just tested on x86_64-linux-pc. The patch is >>> preapproved by Honza >>> and is going to

Re: [PATCH] Fix PR64068 and PR64559

2015-02-04 Thread Jakub Jelinek
On Thu, Feb 05, 2015 at 12:33:18AM +0100, Matthias Klose wrote: > On 01/15/2015 06:23 PM, Martin Liška wrote: > > Hello. > > > > This is Honsa's patch that I've just tested on x86_64-linux-pc. The patch is > > preapproved by Honza > > and is going to be installed. > > backporting the fix for PR63

Re: [PATCH] Fix PR64068 and PR64559

2015-02-04 Thread Matthias Klose
On 01/15/2015 06:23 PM, Martin Liška wrote: > Hello. > > This is Honsa's patch that I've just tested on x86_64-linux-pc. The patch is > preapproved by Honza > and is going to be installed. backporting the fix for PR63970 introduced PR64938. Backporting this patch as well fixes the ICE reported i

Re: [PATCH x86_64] Optimize access to globals in "-fpie -pie" builds with copy relocations

2015-02-04 Thread H.J. Lu
On Wed, Feb 4, 2015 at 3:10 PM, H.J. Lu wrote: > On Wed, Feb 4, 2015 at 2:47 PM, Bernhard Reutner-Fischer > wrote: >> On February 4, 2015 11:37:01 PM GMT+01:00, "H.J. Lu" >> wrote: >>>On Wed, Feb 4, 2015 at 1:53 PM, Sriraman Tallam >>>wrote: On Wed, Feb 4, 2015 at 10:57 AM, H.J. Lu wrote

Work around Firefox -fprofile-generate crash

2015-02-04 Thread Jan Hubicka
Hi, this patch avoids ICE in programs that uses multiple copies of libgcov at once. It is just a temporary workaround as discussed in the PR log. Tested on Firefox and comitted. Honza 2015-02-04 Jan Hubicka

Re: [PATCH x86_64] Optimize access to globals in "-fpie -pie" builds with copy relocations

2015-02-04 Thread H.J. Lu
On Wed, Feb 4, 2015 at 2:47 PM, Bernhard Reutner-Fischer wrote: > On February 4, 2015 11:37:01 PM GMT+01:00, "H.J. Lu" > wrote: >>On Wed, Feb 4, 2015 at 1:53 PM, Sriraman Tallam >>wrote: >>> On Wed, Feb 4, 2015 at 10:57 AM, H.J. Lu wrote: On Wed, Feb 4, 2015 at 10:51 AM, Sriraman Tallam >

Fix ipa-prop update of indirect edges that become speuclative

2015-02-04 Thread Jan Hubicka
Hi, this patch fixes quite nasty bug in update_indirect_edges_after_inlining that forets to update param_index of indirect edge when the call becomes speculative during inlining. Bootstrapped/regtested x86_64-linux, comitted. Jakub, do you think you can help with a testcase? I am not quite sure

Re: [PATCH x86_64] Optimize access to globals in "-fpie -pie" builds with copy relocations

2015-02-04 Thread Bernhard Reutner-Fischer
On February 4, 2015 11:37:01 PM GMT+01:00, "H.J. Lu" wrote: >On Wed, Feb 4, 2015 at 1:53 PM, Sriraman Tallam >wrote: >> On Wed, Feb 4, 2015 at 10:57 AM, H.J. Lu wrote: >>> On Wed, Feb 4, 2015 at 10:51 AM, Sriraman Tallam > wrote: On Wed, Feb 4, 2015 at 10:45 AM, H.J. Lu >wrote: > On W

Re: [RFC PATCH] Emit DW_LANG_Fortran{03,08}

2015-02-04 Thread Cary Coutant
>> did with -gdwarf-4). We're still putting a version number of 2 in the >> compilation unit header! But I guess even upgrading the CU header to > > We are not. On most targets we default to -gdwarf-4 and emit v. 4: Oops, sorry, you're right. I carelessly misread this: dw2_asm_output_data (2,

[SPARC] Fix off-by-one error in reg+off addressing mode

2015-02-04 Thread Eric Botcazou
It's probably there for a long time: the TARGET_LEGITIMATE_ADDRESS_P hook rejects the last allowed positive offset in the reg+off addressing mode. Tested on SPARC/Solaris, applied on the mainline. 2015-02-04 Eric Botcazou * config/sparc/sparc.c (RTX_OK_FOR_OFFSET_P): Fix off-by-one

Re: [PATCH x86_64] Optimize access to globals in "-fpie -pie" builds with copy relocations

2015-02-04 Thread H.J. Lu
On Wed, Feb 4, 2015 at 1:53 PM, Sriraman Tallam wrote: > On Wed, Feb 4, 2015 at 10:57 AM, H.J. Lu wrote: >> On Wed, Feb 4, 2015 at 10:51 AM, Sriraman Tallam wrote: >>> On Wed, Feb 4, 2015 at 10:45 AM, H.J. Lu wrote: On Wed, Feb 4, 2015 at 10:42 AM, Jakub Jelinek wrote: > On Wed, Feb 0

[committed] Fix #pragma omp atomic handling

2015-02-04 Thread Jakub Jelinek
Hi! As the testcases show, not using build_binary_op resulted in sometimes non-promoted arguments, and there were issues with division, where we initially want to use TRUNC_DIV_EXPR, but if the arguments are floating point, change it into RDIV_EXPR instead. Fixed thusly, bootstrapped/regtested on

Re: [PATCH] More fixes for update_web_docs_svn for jit docs (PR jit/64257)

2015-02-04 Thread David Malcolm
On Wed, 2015-02-04 at 22:21 +0100, Gerald Pfeifer wrote: > On Wednesday 2015-02-04 14:10, David Malcolm wrote: > > Gerald: Please can you do the "svn up" on the relevant machine again, > > so that it gets the updated "update_web_docs_svn"? > > That was my plan, yes. :-) I just did that and manua

Re: [RFC PATCH] Emit DW_LANG_Fortran{03,08}

2015-02-04 Thread Jakub Jelinek
ytes) .uleb128 0x1# (DIE (0xb) DW_TAG_compile_unit) .long .LASF0 # DW_AT_producer: "GNU C11 5.0.0 20150204 (experimental) -g -O2" It is only Darwin where because of broken toolchain we need to stick to DWARF2, because otherwise the buggy tools crash on it (and I think VxWorks). Jakub

Re: [RFC PATCH] Emit DW_LANG_Fortran{03,08}

2015-02-04 Thread Cary Coutant
> PS: Talking about DWARF5, do you know when it will be available as public > draft? I am especially looking forward to > http://dwarfstd.org/ShowIssue.php?issue=121221.1 (Allow DW_AT_type with > DW_TAG_string_type), which would be a low-hanging fruit in terms of > implementation. Contrary to the a

Re: [RFC PATCH] Emit DW_LANG_Fortran{03,08}

2015-02-04 Thread Cary Coutant
> DW_LANG_Fortran03 and DW_LANG_Fortran08 DW_AT_language values were recently > accepted into DWARF5. This patch changes GCC to handle those similarly to > how e.g. the -std=c++11, -std=c++14 or -std=c11 are handled. > > As it will take some time for consumers to catch up, I'm enabling that > only

Re: [PATCH x86_64] Optimize access to globals in "-fpie -pie" builds with copy relocations

2015-02-04 Thread Sriraman Tallam
On Wed, Feb 4, 2015 at 10:57 AM, H.J. Lu wrote: > On Wed, Feb 4, 2015 at 10:51 AM, Sriraman Tallam wrote: >> On Wed, Feb 4, 2015 at 10:45 AM, H.J. Lu wrote: >>> On Wed, Feb 4, 2015 at 10:42 AM, Jakub Jelinek wrote: On Wed, Feb 04, 2015 at 10:38:48AM -0800, H.J. Lu wrote: > Common symbo

RE: [PATCH,WWWDOCS] MIPS changes for GCC 5.0

2015-02-04 Thread Moore, Catherine
Hi Matthew, I made a few edits. I removed the markup in the process, so that will need to be added back. See the text at the end. Thanks, Catherine > -Original Message- > From: Matthew Fortune [mailto:matthew.fort...@imgtec.com] > Sent: Wednesday, February 04, 2015 11:46 AM > To: Moore

Re: [PATCH] More fixes for update_web_docs_svn for jit docs (PR jit/64257)

2015-02-04 Thread Gerald Pfeifer
On Wednesday 2015-02-04 14:10, David Malcolm wrote: > Gerald: Please can you do the "svn up" on the relevant machine again, > so that it gets the updated "update_web_docs_svn"? That was my plan, yes. :-) I just did that and manually ran the script, and it seems to work. Still, do you think you

Fix PR64878: do not jump thread across two loop iterations

2015-02-04 Thread Sebastian Pop
Hi, The attached patch stops the recursion in the detection of FSM jump-threads at loop phi nodes after having visited a loop phi node. This avoids jump-threading two iterations forward that were possible due to a flip-flop operation that exchange the value of the switch control variable as illus

Re: [Patch, fortran] PR 64757 - [5 Regression] ICE in fold_convert_loc, at fold-const.c:2353

2015-02-04 Thread Tobias Burnus
Dear Paul, dear all, Paul Richard Thomas wrote: Please find attached a reworked version of the patch for this PR. [...] In this version, the treatment of scalar and array components is cleanly separated. Bootstrapped and regtested on FC21/x86_64. OK for trunk? Looks good to me. Thanks for the

Re: Remove parameters after converting call to gcc_unreachable

2015-02-04 Thread Jakub Jelinek
On Wed, Feb 04, 2015 at 08:47:20PM +0100, Jan Hubicka wrote: > Bootstrapped/regtested x86_64-linux, OK? > PR middle-end/64922 > * gimple.c: Include gimple-ssa.h Missing dot at the end of line. > (maybe_remove_unused_call_args): New function. > * gimple.h (maybe_remove_unus

Remove parameters after converting call to gcc_unreachable

2015-02-04 Thread Jan Hubicka
Hi, this is second attempt to fix the issue with arguments being passed to builtlin_unreachable after https://gcc.gnu.org/ml/gcc-patches/2014-12/msg01025.html As suggested, I fixup the IL right after redirection instead waiting for tree-ssa-dce. Bootstrapped/regtested x86_64-linux, OK?

Re: [PATCH] PR rtl-optimization/64905: Clear hard frame pointer alignment if not needed

2015-02-04 Thread H.J. Lu
On Wed, Feb 4, 2015 at 11:20 AM, Vladimir Makarov wrote: > > On 2015-02-04 11:16 AM, H.J. Lu wrote: >> >> When hard frame pointer isn't needed, the register for hard frame pointer >> may be reused. This patch clears alignment on hard frame pointer if hard >> frame pointer isn't needed. OK for tr

Patch ping

2015-02-04 Thread Jakub Jelinek
Hi! I'd like to ping 2 patches: http://gcc.gnu.org/ml/gcc-patches/2015-01/msg02530.html - P2 - PR61925 - fix x86 #pragma GCC target handling http://gcc.gnu.org/ml/gcc-patches/2015-01/msg02432.html - emit DW_LANG_Fortran{03,08} for -gdwarf-5 Jakub

Re: [PATCH] PR rtl-optimization/64905: Clear hard frame pointer alignment if not needed

2015-02-04 Thread Vladimir Makarov
On 2015-02-04 11:16 AM, H.J. Lu wrote: When hard frame pointer isn't needed, the register for hard frame pointer may be reused. This patch clears alignment on hard frame pointer if hard frame pointer isn't needed. OK for trunk after bootstrap and test on Linux/x86-64? LRA can set up frame_poin

Re: [PATCH] More fixes for update_web_docs_svn for jit docs (PR jit/64257)

2015-02-04 Thread David Malcolm
On Wed, 2015-02-04 at 19:47 +0100, Gerald Pfeifer wrote: > On Wednesday 2015-02-04 13:32, David Malcolm wrote: > > OK for trunk? > > > > gcc/jit/ChangeLog: > > PR jit/64257 > > * docs/conf.py (html_theme): Change from 'pyramid' > > to 'sphinxdoc'. > > > > maintainer-scripts/ChangeLog:

Re: [PATCH x86_64] Optimize access to globals in "-fpie -pie" builds with copy relocations

2015-02-04 Thread H.J. Lu
On Wed, Feb 4, 2015 at 10:51 AM, Sriraman Tallam wrote: > On Wed, Feb 4, 2015 at 10:45 AM, H.J. Lu wrote: >> On Wed, Feb 4, 2015 at 10:42 AM, Jakub Jelinek wrote: >>> On Wed, Feb 04, 2015 at 10:38:48AM -0800, H.J. Lu wrote: Common symbol should be resolved locally for PIE. >>> >>> binds_loc

Re: [PATCH x86_64] Optimize access to globals in "-fpie -pie" builds with copy relocations

2015-02-04 Thread Sriraman Tallam
On Wed, Feb 4, 2015 at 10:45 AM, H.J. Lu wrote: > On Wed, Feb 4, 2015 at 10:42 AM, Jakub Jelinek wrote: >> On Wed, Feb 04, 2015 at 10:38:48AM -0800, H.J. Lu wrote: >>> Common symbol should be resolved locally for PIE. >> >> binds_local_p yes, binds_to_current_def_p no. >> > > Is SYMBOL_REF_LOCAL_

Re: [PATCH] More fixes for update_web_docs_svn for jit docs (PR jit/64257)

2015-02-04 Thread Gerald Pfeifer
On Wednesday 2015-02-04 13:32, David Malcolm wrote: > OK for trunk? > > gcc/jit/ChangeLog: > PR jit/64257 > * docs/conf.py (html_theme): Change from 'pyramid' > to 'sphinxdoc'. > > maintainer-scripts/ChangeLog: > PR jit/64257 > * update_web_docs_svn: Update build of

Re: Re: PR39570 (gfortran) cabs and cabsf are named differently on NetBSD 5

2015-02-04 Thread Krister Walfridsson
Apologies for the slow response. On Mon, 26 Jan 2015, Kai-Uwe Eckhardt wrote: according to gcc/MAINTAINERS Jason and Krister are NetBSD maintainers for GCC and can approve patches like yours, so let me copy them. (Should this be applied now, at least the copyright years need to be adjusted to

Re: [PATCH x86_64] Optimize access to globals in "-fpie -pie" builds with copy relocations

2015-02-04 Thread H.J. Lu
On Wed, Feb 4, 2015 at 10:42 AM, Jakub Jelinek wrote: > On Wed, Feb 04, 2015 at 10:38:48AM -0800, H.J. Lu wrote: >> Common symbol should be resolved locally for PIE. > > binds_local_p yes, binds_to_current_def_p no. > Is SYMBOL_REF_LOCAL_P set to binds_local_p or binds_to_current_def_p? -- H.J

Re: [PATCH x86_64] Optimize access to globals in "-fpie -pie" builds with copy relocations

2015-02-04 Thread Jakub Jelinek
On Wed, Feb 04, 2015 at 10:38:48AM -0800, H.J. Lu wrote: > Common symbol should be resolved locally for PIE. binds_local_p yes, binds_to_current_def_p no. Jakub

Re: [PATCH x86_64] Optimize access to globals in "-fpie -pie" builds with copy relocations

2015-02-04 Thread H.J. Lu
On Wed, Feb 4, 2015 at 10:31 AM, Jakub Jelinek wrote: > On Wed, Feb 04, 2015 at 10:27:34AM -0800, Sriraman Tallam wrote: >> On Tue, Feb 3, 2015 at 5:16 PM, H.J. Lu wrote: >> > On Tue, Feb 3, 2015 at 2:19 PM, Jakub Jelinek wrote: >> >> On Tue, Feb 03, 2015 at 02:03:14PM -0800, H.J. Lu wrote: >> >

Re: [PATCH x86_64] Optimize access to globals in "-fpie -pie" builds with copy relocations

2015-02-04 Thread Jakub Jelinek
On Wed, Feb 04, 2015 at 10:27:34AM -0800, Sriraman Tallam wrote: > On Tue, Feb 3, 2015 at 5:16 PM, H.J. Lu wrote: > > On Tue, Feb 3, 2015 at 2:19 PM, Jakub Jelinek wrote: > >> On Tue, Feb 03, 2015 at 02:03:14PM -0800, H.J. Lu wrote: > >>> So we aren't SYMBOL_REF_EXTERNAL_P nor > >>> SYMBOL_REF_LO

Re: [PATCH x86_64] Optimize access to globals in "-fpie -pie" builds with copy relocations

2015-02-04 Thread Sriraman Tallam
On Tue, Feb 3, 2015 at 5:16 PM, H.J. Lu wrote: > On Tue, Feb 3, 2015 at 2:19 PM, Jakub Jelinek wrote: >> On Tue, Feb 03, 2015 at 02:03:14PM -0800, H.J. Lu wrote: >>> So we aren't SYMBOL_REF_EXTERNAL_P nor >>> SYMBOL_REF_LOCAL_P. What do we reference? >> >> That is reasonable. There is no guaran

[PATCH] More fixes for update_web_docs_svn for jit docs (PR jit/64257)

2015-02-04 Thread David Malcolm
To build the jit docs, we need to use sphinx 1.0 or later; the 0.6.6 in EPEL 6 doesn't support all the directives we need. The alternate python-sphinx10 in EPEL 6 has 1.0.8: python-sphinx10-1.0.8-1.el6.noarch which is able to build the jit docs, apart from not having the "pyramid" theme (this th

Re: [patch+7.9] compile: Filter out -fpreprocessed

2015-02-04 Thread Doug Evans
On Tue, Feb 3, 2015 at 11:10 AM, Mark Wielaard wrote: > On Tue, 2015-02-03 at 19:59 +0100, Jan Kratochvil wrote: >> On Tue, 03 Feb 2015 19:50:40 +0100, Doug Evans wrote: >> > On Fri, Jan 16, 2015 at 2:42 PM, Jan Kratochvil >> > wrote: >> > > [...] >> > > It is wrong that gcc puts -fpreprocessed i

Re: [PATCH 2/4] Add liboffloadmic

2015-02-04 Thread Jakub Jelinek
On Wed, Feb 04, 2015 at 08:44:42PM +0300, Ilya Verbin wrote: > contrib/ > * gcc_update (files_and_dependencies): Add rules for liboffloadmic and > liboffloadmic/plugin. Ok, thanks. Jakub

Re: [PATCH 2/4] Add liboffloadmic

2015-02-04 Thread Ilya Verbin
Hi, On 12 Dec 11:46, Thomas Schwinge wrote: > On Tue, 21 Oct 2014 21:20:34 +0400, Ilya Verbin wrote: > > This patch contains liboffloadmic library. > > > liboffloadmic/ > > Initial commit. Imported from upstream: > > https://www.openmprtl.org/sites/default/files/liboffload_oss.tgz > >

Re: [libobjc] Fix failures on AIX (PR libobjc/63765)

2015-02-04 Thread Andrew Pinski
On Wed, Feb 4, 2015 at 9:07 AM, Mike Stump wrote: > On Feb 4, 2015, at 2:28 AM, Rainer Orth wrote: >> Rainer Orth writes: On Jan 28, 2015, Mike Stump wrote: > On Jan 28, 2015, at 2:27 AM, Rainer Orth > wrote: >> >> * Remove the definition of _XOPEN_SOURCE completely. > >>

Re: [PATCH][wwwdocs] Document support for Cortex-A72

2015-02-04 Thread Gerald Pfeifer
On Wednesday 2015-02-04 16:19, Matthew Wahab wrote: This patch documents in gcc-5/changes.html the addition of support for the Cortex-A72 to the ARM and the AArch64 backends. Looks good to me, but you may want to wait a bit for ARMers to chime in. Or go ahead and thing can always be tweaked l

Re: [libobjc] Fix failures on AIX (PR libobjc/63765)

2015-02-04 Thread Mike Stump
On Feb 4, 2015, at 2:28 AM, Rainer Orth wrote: > Rainer Orth writes: >>> On Jan 28, 2015, Mike Stump wrote: On Jan 28, 2015, at 2:27 AM, Rainer Orth wrote: > > * Remove the definition of _XOPEN_SOURCE completely. I think I prefer this one… and there is no hint wh

RE: [PATCH,WWWDOCS] MIPS changes for GCC 5.0

2015-02-04 Thread Matthew Fortune
Andrew Pinski writes: > On Wed, Feb 4, 2015 at 8:46 AM, Matthew Fortune > wrote: > > Hi Catherine, > > > > I've made a first pass at writing up the MIPS changes for GCC 5.0. > > Could you take a read and see what needs some more work? > > One comment below. > > >> Support for the Cavium Net

Re: [PATCH,WWWDOCS] MIPS changes for GCC 5.0

2015-02-04 Thread Andrew Pinski
On Wed, Feb 4, 2015 at 8:46 AM, Matthew Fortune wrote: > Hi Catherine, > > I've made a first pass at writing up the MIPS changes for GCC 5.0. > Could you take a read and see what needs some more work? One comment below. > > Thanks, > Matthew > > Index: htdocs/gcc-5/changes.html > ==

[PATCH,WWWDOCS] MIPS changes for GCC 5.0

2015-02-04 Thread Matthew Fortune
Hi Catherine, I've made a first pass at writing up the MIPS changes for GCC 5.0. Could you take a read and see what needs some more work? Thanks, Matthew Index: htdocs/gcc-5/changes.html ===

[PATCH][wwwdocs] Document support for Cortex-A72

2015-02-04 Thread Matthew Wahab
Hello, This patch documents in gcc-5/changes.html the addition of support for the Cortex-A72 to the ARM and the AArch64 backends. Tested by checking the updated webpage in firefox. Matthew Index: htdocs/gcc-5/changes.html === RCS

Re: [PATCH] testsuite: Do not xfail guality/pr{41447,41616}-1.c

2015-02-04 Thread Jakub Jelinek
On Wed, Feb 04, 2015 at 08:10:35AM -0600, Segher Boessenkool wrote: > On a native i686 there are only four fails (the one with linker plugin works). Yeah, I wrap linker because it is 64-bit and thus doesn't support 32-bit plugins. > I can xfail 41447 for ia32 if you want? The change for 41616 is

[PATCH] PR rtl-optimization/64905: Clear hard frame pointer alignment if not needed

2015-02-04 Thread H.J. Lu
When hard frame pointer isn't needed, the register for hard frame pointer may be reused. This patch clears alignment on hard frame pointer if hard frame pointer isn't needed. OK for trunk after bootstrap and test on Linux/x86-64? Thanks. H.J. -- gcc/ PR rtl-optimization/64905

Re: PR lto/64837: lto plugin doesn't call ld_plugin_release_input_file

2015-02-04 Thread H.J. Lu
On Wed, Feb 4, 2015 at 7:35 AM, Cary Coutant wrote: >>>If you're going to insist on calling the release_input_file API from >>>the claim_file handler, I'm going to have to fix gold to ignore the >>>call to avoid a premature unlock of the object file. >> >> What's the proper solution for not leakin

Re: PR lto/64837: lto plugin doesn't call ld_plugin_release_input_file

2015-02-04 Thread Cary Coutant
>>If you're going to insist on calling the release_input_file API from >>the claim_file handler, I'm going to have to fix gold to ignore the >>call to avoid a premature unlock of the object file. > > What's the proper solution for not leaking those filedescriptors? There was a bug in gold where it

Re: RFA: RL78: Fix gcc testsuite failures

2015-02-04 Thread DJ Delorie
Ok.

Re: Merge current set of OpenACC changes from gomp-4_0-branch

2015-02-04 Thread Julian Brown
On Tue, 3 Feb 2015 23:01:04 +0300 Ilya Verbin wrote: > On 03 Feb 13:00, Julian Brown wrote: > > On Tue, 3 Feb 2015 14:28:44 +0300 > > Ilya Verbin wrote: > > > On 27 Jan 14:07, Julian Brown wrote: > > > > On Mon, 26 Jan 2015 17:34:26 +0300 > > > > Ilya Verbin wrote: > > > > > Here is my current

[PATCH, committed] jit: fix typo in docs

2015-02-04 Thread David Malcolm
Fix for a trivial but confusing typo in the jit docs. Committed to trunk as r220408. gcc/jit/ChangeLog: * docs/topics/contexts.rst (gcc_jit_context_acquire): Fix typo. * docs/_build/texinfo/libgccjit.texi: Regenerate. --- gcc/jit/docs/topics/contexts.rst | 2 +- 1 file c

Re: [Patch, fortran] PR 64757 - [5 Regression] ICE in fold_convert_loc, at fold-const.c:2353

2015-02-04 Thread Paul Richard Thomas
Dear All, Please find attached a reworked version of the patch for this PR. I have no idea at all, why the original version worked for array components on my laptop. In this version, the treatment of scalar and array components is cleanly separated. Bootstrapped and regtested on FC21/x86_64. OK f

Re: [PATCH] testsuite: Do not xfail guality/pr{41447,41616}-1.c

2015-02-04 Thread Segher Boessenkool
On Wed, Feb 04, 2015 at 12:55:55PM +0100, Jakub Jelinek wrote: > On Wed, Feb 04, 2015 at 03:48:38AM -0800, Segher Boessenkool wrote: > > These testcases currently XPASS on most targets and configs, so let's > > reduce the noise. Okay for mainline? > > Doesn't look like that is the case. obj69 is

Re: PATCH: PR target/64905: unsigned short is loaded with 4-byte load (movl)

2015-02-04 Thread Uros Bizjak
On Wed, Feb 4, 2015 at 2:21 PM, Uros Bizjak wrote: > On Mon, Feb 2, 2015 at 10:39 PM, H.J. Lu wrote: >> This patch fixes a long standing bug where aligned_operand ignores >> alignment of memory operand less than 32 bits. It drops address >> decomposition and returns false if alignment of memory

Re: [PATCH][ARM] Add support for -mcpu=cortex-a72

2015-02-04 Thread Kyrill Tkachov
On 04/02/15 13:37, James Greenhalgh wrote: On Wed, Feb 04, 2015 at 01:23:06PM +, Ramana Radhakrishnan wrote: On Wed, Feb 4, 2015 at 10:36 AM, Matthew Wahab wrote: Hello, The Cortex-A72 is an ARMv8 core with the same architectural features as the Cortex-A57. This patch adds support for th

Commit: FR30: Fix di_operand and nonimmediate_di_operand predicates

2015-02-04 Thread Nick Clifton
Hi Guys, I am checking in the patch below to add SUBREG to the list of rtx codes accepted by the di_operand and nonimmediate_di_operand predicates in the FR30 backend. This should resolve PR 64408. Cheers Nick gcc/ChangeLog 2015-02-04 Nick Clifton PR target/64408 * c

Re: [PATCH][AArch64] Add support for -mcpu=cortex-a72

2015-02-04 Thread James Greenhalgh
On Wed, Feb 04, 2015 at 12:57:11PM +, Marcus Shawcroft wrote: > On 4 February 2015 at 10:35, Matthew Wahab wrote: > > Hello, > > > > The Cortex-A72 is an ARMv8 core with the same architectural features as the > > Cortex-A57. This patch adds support for the command line option > > -mcpu=cortex-

Re: [PATCH][ARM] Add support for -mcpu=cortex-a72

2015-02-04 Thread James Greenhalgh
On Wed, Feb 04, 2015 at 01:23:06PM +, Ramana Radhakrishnan wrote: > On Wed, Feb 4, 2015 at 10:36 AM, Matthew Wahab wrote: > > Hello, > > > > The Cortex-A72 is an ARMv8 core with the same architectural features as the > > Cortex-A57. This patch adds support for the command line option > > -mcpu

Re: [PATCH][ARM] Add support for -mcpu=cortex-a72

2015-02-04 Thread Ramana Radhakrishnan
On Wed, Feb 4, 2015 at 10:36 AM, Matthew Wahab wrote: > Hello, > > The Cortex-A72 is an ARMv8 core with the same architectural features as the > Cortex-A57. This patch adds support for the command line option > -mcpu=cortex-a72 with the same effect as the -mcpu=cortex-a57 option, with > only the n

[PATCH, x86] Add TARGET_OVERRIDE_OPTIONS_AFTER_CHANGE hook

2015-02-04 Thread Christian Bruel
While trying to reduce the PR64835 case for ARM and x86, I noticed that the alignment flags are cleared for x86 when attribute optimized is used. With the attached testcases, the visible effects are twofold : 1) Functions compiled in with attribute optimize (-O2) are not aligned as if they wer

Re: PATCH: PR target/64905: unsigned short is loaded with 4-byte load (movl)

2015-02-04 Thread Uros Bizjak
On Mon, Feb 2, 2015 at 10:39 PM, H.J. Lu wrote: > This patch fixes a long standing bug where aligned_operand ignores > alignment of memory operand less than 32 bits. It drops address > decomposition and returns false if alignment of memory operand less > is than 32 bits. Tested on Linux/x86-64.

Re: [PATCH][AArch64] Add support for -mcpu=cortex-a72

2015-02-04 Thread Ramana Radhakrishnan
On Wed, Feb 4, 2015 at 12:57 PM, Marcus Shawcroft wrote: > On 4 February 2015 at 10:35, Matthew Wahab wrote: >> Hello, >> >> The Cortex-A72 is an ARMv8 core with the same architectural features as the >> Cortex-A57. This patch adds support for the command line option >> -mcpu=cortex-a72 with the

Re: [PATCH] PR64387

2015-02-04 Thread Uros Bizjak
On Wed, Feb 4, 2015 at 12:05 PM, Ilya Tocar wrote: > I think that fix for avx2 part should be backported to 4.8/4.9 > What do you think? OK also for branches. Thanks, Uros.

Re: [PATCH][AArch64] Add support for -mcpu=cortex-a72

2015-02-04 Thread Marcus Shawcroft
On 4 February 2015 at 10:35, Matthew Wahab wrote: > Hello, > > The Cortex-A72 is an ARMv8 core with the same architectural features as the > Cortex-A57. This patch adds support for the command line option > -mcpu=cortex-a72 with the same effect as the -mcpu=cortex-a57 option, only > the name being

Re: [PATCH][AArch64] Use std::swap instead of manually swapping in aarch64-ldpstp.md

2015-02-04 Thread Marcus Shawcroft
On 4 February 2015 at 12:18, Kyrill Tkachov wrote: > Hi all, > > This patch makes use of std::swap in every peephole2 of aarch64-ldp-stp.md > instead of manually swapping rtxen. > No functional change, just a cleanup. > Bootstrapped and tested on aarch64. > > I'm proposing this for next stage1 tog

Re: [PATCH, ivopt] Try aligned offset when get_address_cost

2015-02-04 Thread Richard Biener
On February 4, 2015 11:32:54 AM CET, Eric Botcazou wrote: >> For some TARGET, like ARM THUMB1, the offset in load/store should be >nature >> aligned. But in function get_address_cost, when computing max_offset, >it >> only tries byte-aligned offsets: >> >> ((unsigned HOST_WIDE_INT) 1 << i) - 1

Re: [AArch64] Rid the world of NAMED_PARAM

2015-02-04 Thread Marcus Shawcroft
On 4 February 2015 at 12:06, James Greenhalgh wrote: > > Hi, > > HAVE_DESIGNATED_INITIALIZERS is not set for C++, so the NAMED_PARAM macros > using it provide false security when we compile aarch64.c. Removing this > is an obvious cleanup and gets rid of some confusing dead code. > > This patch re

Re: [Patch ARM] Improve guality tests - part 1 - pr36728-1.c

2015-02-04 Thread Ramana Radhakrishnan
On 04/02/2015 11:40, Jakub Jelinek wrote: On Wed, Feb 04, 2015 at 11:33:19AM +, Ramana Radhakrishnan wrote: --- a/gcc/testsuite/gcc.dg/guality/pr36728-1.c +++ b/gcc/testsuite/gcc.dg/guality/pr36728-1.c @@ -49,5 +49,6 @@ main () int l = 0; asm ("" : "=r" (l) : "0" (l)); a = foo

Re: [PATCH][AArch64] Use std::swap instead of manually swapping in aarch64-ldpstp.md

2015-02-04 Thread James Greenhalgh
On Wed, Feb 04, 2015 at 12:18:29PM +, Kyrill Tkachov wrote: > Hi all, > > This patch makes use of std::swap in every peephole2 of > aarch64-ldp-stp.md instead of manually swapping rtxen. > No functional change, just a cleanup. > Bootstrapped and tested on aarch64. > > I'm proposing this for

[PATCH][AArch64] Use std::swap instead of manually swapping in aarch64-ldpstp.md

2015-02-04 Thread Kyrill Tkachov
Hi all, This patch makes use of std::swap in every peephole2 of aarch64-ldp-stp.md instead of manually swapping rtxen. No functional change, just a cleanup. Bootstrapped and tested on aarch64. I'm proposing this for next stage1 together with the other AArch64 patch that moves a couple of pla

[PATCH][ARM] Rewrite vc NEON patterns to use RTL operations rather than UNSPECs

2015-02-04 Thread Kyrill Tkachov
Hi all, This patch improves the vc patterns in neon.md to use proper RTL operations rather than UNSPECS. It is done in a similar way to the analogous aarch64 operations i.e. vceq is expressed as (neg (eq (...) (...))) since we want to write all 1s to the result element when 'eq' holds and 0s

[AArch64] Rid the world of NAMED_PARAM

2015-02-04 Thread James Greenhalgh
Hi, HAVE_DESIGNATED_INITIALIZERS is not set for C++, so the NAMED_PARAM macros using it provide false security when we compile aarch64.c. Removing this is an obvious cleanup and gets rid of some confusing dead code. This patch removes all the magic macros. Bootstrapped and tested on aarch64-non

Re: [PATCH] testsuite: Do not xfail guality/pr{41447,41616}-1.c

2015-02-04 Thread Jakub Jelinek
On Wed, Feb 04, 2015 at 03:48:38AM -0800, Segher Boessenkool wrote: > These testcases currently XPASS on most targets and configs, so let's > reduce the noise. Okay for mainline? Doesn't look like that is the case. obj69 is x86_64, obj70 is i686. grep '\(41447\|41616\).*execution' obj{69,70}/gcc

Commit: MSP430: Extend msp430_use_f5_series_hwmult()

2015-02-04 Thread Nick Clifton
Hi Guys, I am applying the patch below to extend the MSP430 port of gcc's auto-recognition of F5 multiply hardware enabled MCUs. Cheers Nick gcc/ChangeLog 2015-02-04 Nick Clifton * config/msp430/msp430.c (msp430_use_f5_series_hwmult): Add more prefixes of known F5 using

[PATCH] testsuite: Do not xfail guality/pr{41447,41616}-1.c

2015-02-04 Thread Segher Boessenkool
These testcases currently XPASS on most targets and configs, so let's reduce the noise. Okay for mainline? Segher 2015-02-04 Segher Boessenkool gcc/testsuite/ * gcc.dg/guality/pr41447-1.c: Remove xfail. * gcc.dg/guality/pr41616-1.c: Ditto. --- gcc/testsuite/gcc.dg/guality

Re: [Patch ARM] Improve guality tests - part 1 - pr36728-1.c

2015-02-04 Thread Jakub Jelinek
On Wed, Feb 04, 2015 at 11:33:19AM +, Ramana Radhakrishnan wrote: > --- a/gcc/testsuite/gcc.dg/guality/pr36728-1.c > +++ b/gcc/testsuite/gcc.dg/guality/pr36728-1.c > @@ -49,5 +49,6 @@ main () >int l = 0; >asm ("" : "=r" (l) : "0" (l)); >a = foo (l + 1, l + 2, l + 3, l + 4, l + 5, l

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

2015-02-04 Thread Jakub Jelinek
On Sat, Nov 01, 2014 at 12:57:45PM +0100, Bernd Schmidt wrote: > This is not against current trunk; it applies to gomp-4_0-branch where it is > one of the necessary parts to make offloading x86->nvptx work. The issue is > that the LTO file format depends on the machine_modes enum, it needs to > mat

Re: [PATCH][AArch64] Use target builtin instead of __builtin_sqrt for vsqrt_f64

2015-02-04 Thread Marcus Shawcroft
On 12 January 2015 at 15:52, Kyrill Tkachov wrote: > Hi all, > > As raised in https://gcc.gnu.org/ml/gcc-patches/2014-12/msg01237.html and > discussed in that thread, using __builtin_sqrt for vsqrt_f64 may end up in a > call to the library sqrt at -O0. To avoid that this patch uses a target > buil

RE: [PATCH MIPS RFA] Regression cleanup for nan2008 toolchain

2015-02-04 Thread Robert Suchanek
> > 2015-02-02 Robert Suchanek > > > >* gcc.target/mips/loongson-simd.c: Update comment to clarify the > need > >for mips_nanlegacy target. > > > > diff --git a/gcc/testsuite/gcc.target/mips/loongson-simd.c > > b/gcc/testsuite/gcc.target/mips/loongson-simd.c > > index 949632e..9c

Re: [Patch ARM] Improve guality tests - part 1 - pr36728-1.c

2015-02-04 Thread Ramana Radhakrishnan
On 04/02/2015 11:10, Jakub Jelinek wrote: On Wed, Feb 04, 2015 at 11:03:29AM +, Ramana Radhakrishnan wrote: Changelog: * gcc.dg/guality/pr36728-1.c: Skip some tests for arm. Segher and I discussed an alternative approach - marking these as "m" (arg1) , "m" (arg2) etc in the asm b

RFA: RL78: Fix gcc testsuite failures

2015-02-04 Thread Nick Clifton
Hi DJ, Please can I apply the patch below to fix some RL78 gcc testsuite failures ? The patch does two things: Firstly in the RL78 assembler version of the __addsf3 function it fixes a corner case where rounding up the fraction results in an overflow into an unused bit. The problem h

Re: [Patch ARM] Improve guality tests - part 1 - pr36728-1.c

2015-02-04 Thread Jakub Jelinek
On Wed, Feb 04, 2015 at 11:03:29AM +, Ramana Radhakrishnan wrote: > > > > Changelog: > > > > * gcc.dg/guality/pr36728-1.c: Skip some tests for arm. > > > > Segher and I discussed an alternative approach - marking these as "m" > (arg1) , "m" (arg2) etc in the asm blocks also gives us the s

Re: [PATCH][AArch64] Use target builtin instead of __builtin_sqrt for vsqrt_f64

2015-02-04 Thread Kyrill Tkachov
Ping. https://gcc.gnu.org/ml/gcc-patches/2015-01/msg00710.html Thanks, Kyrill On 27/01/15 09:45, Kyrill Tkachov wrote: On 19/01/15 15:46, Kyrill Tkachov wrote: On 19/01/15 15:44, James Greenhalgh wrote: On Mon, Jan 12, 2015 at 05:30:46PM +, Andrew Pinski wrote: On Mon, Jan 12, 2015 at

Re: [PATCH] PR64387

2015-02-04 Thread Ilya Tocar
I think that fix for avx2 part should be backported to 4.8/4.9 What do you think? On 14 Jan 14:18, Ilya Tocar wrote: > Hi, > > This patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64387 > Which was caused by different in predicates between vec_unpacks_hi > and vec_extract_hi. > Ok for tr

Re: [Patch ARM] Improve guality tests - part 1 - pr36728-1.c

2015-02-04 Thread Ramana Radhakrishnan
> > Changelog: > > * gcc.dg/guality/pr36728-1.c: Skip some tests for arm. > Segher and I discussed an alternative approach - marking these as "m" (arg1) , "m" (arg2) etc in the asm blocks also gives us the same effect and then probably removes the need to rely on such target markers. I've ju

Re: nvptx offloading patches [2/n]

2015-02-04 Thread Jakub Jelinek
On Wed, Feb 04, 2015 at 11:55:54AM +0100, Jakub Jelinek wrote: > On Sat, Nov 01, 2014 at 12:51:32PM +0100, Bernd Schmidt wrote: > > LTO has a mechanism not to stream out common nodes that are expected to be > > identical on each run. When using LTO to communicate between compilers for > > different

  1   2   >