Re: [PATCH, testsuite] fix ggcplug.c test-case

2015-01-12 Thread Prathamesh Kulkarni
On 12 January 2015 at 14:19, Richard Biener rguent...@suse.de wrote: On Sun, 11 Jan 2015, Prathamesh Kulkarni wrote: Hi, The test-case plugin/ggcplug.c was failing due to flattening of tree.h and tree-core.h. Test-case was incorrect because it included gcc-plugin.h after tree.h whereas

[PATCH] IPA ICF: handle correctly indirect_calls

2015-01-12 Thread Martin Liška
Hello. Following patch is needed to pass LTO compilation for chromium. IPA ICF verifies polymorphic types for functions that have any function call. I forgot to handle indirect_calls. Patch can bootstrap on x86_64-linux-pc and new regression is seen. Ready for trunk? Thanks, Martin From

Re: [PATCH] Fix enum operands exchange in ipa-inline.c

2015-01-12 Thread Hurugalawadi, Naveen
Hi Richard, Thanks for the quick review and comments. Please find attached the modified patch as per your suggestion. Thanks, Naveen From: Richard Biener richard.guent...@gmail.com Sent: Monday, January 12, 2015 2:48 PM To: Hurugalawadi, Naveen Cc:

Re: [PATCH] IPA ICF: handle correctly indirect_calls

2015-01-12 Thread Richard Biener
On Mon, Jan 12, 2015 at 10:29 AM, Martin Liška mli...@suse.cz wrote: Hello. Following patch is needed to pass LTO compilation for chromium. IPA ICF verifies polymorphic types for functions that have any function call. I forgot to handle indirect_calls. Patch can bootstrap on

[gomp4] Merge trunk r219425 (2015-01-10) into gomp-4_0-branch

2015-01-12 Thread Thomas Schwinge
Hi! In r219453, I have committed a merge from trunk r219425 (2015-01-10) into gomp-4_0-branch. Grüße, Thomas pgphl0a_xYFFn.pgp Description: PGP signature

PR ipa/63470 (zero sized call_stmt)

2015-01-12 Thread Jan Hubicka
Hi, this ICE is caused by double updating in ipa-prop that reduces call stmt size once when it becomes speculative and again when it is turned to direct. Fixed by the following patch that makes updating to happen while duplication so ipa-prop needs to care only about case it turned real indirect

Re: [PATCH] Fix undefined label problem after crossjumping (PR rtl-optimization/64536)

2015-01-12 Thread Richard Biener
On Fri, 9 Jan 2015, Jakub Jelinek wrote: On Fri, Jan 09, 2015 at 03:10:16PM +0100, Richard Biener wrote: Well, you have until the end of next week ;) For GIMPLE this is a switch with all cases going to the same basic-block, right? I think we optimize that in cleanup_control_expr_graph via

[PATCH, autofdo] Some code cleanup

2015-01-12 Thread Yangfei (Felix)
Hi, The attached patch does some code cleanup for auto-profile.c: fix typos and remove some unnecessary MAX/MIN checks plus some else. OK for the trunk? Index: gcc/auto-profile.c === --- gcc/auto-profile.c (revision 219297)

Re: [PATCH] Fix enum operands exchange in ipa-inline.c

2015-01-12 Thread Richard Biener
On Mon, Jan 12, 2015 at 10:36 AM, Hurugalawadi, Naveen naveen.hurugalaw...@caviumnetworks.com wrote: Hi Richard, Thanks for the quick review and comments. Please find attached the modified patch as per your suggestion. Ok. Richard. Thanks, Naveen

Re: [PATCH] ipa-icf.c: Fix issues generated by original latest commit

2015-01-12 Thread Richard Biener
On Sat, Jan 10, 2015 at 10:03 AM, Chen Gang S gang.c...@sunrus.com.cn wrote: The related commit is 275e275 IPA ICF: target and optimization flags comparison.. For sem_function::equals_private(), fix the typo issue, and for target_opts_for_fn(), fix access NULL issue. For cross compiling

Re: [PATCH] Fix enum operands exchange in ipa-inline.c

2015-01-12 Thread Richard Biener
On Mon, Jan 12, 2015 at 7:58 AM, Hurugalawadi, Naveen naveen.hurugalaw...@caviumnetworks.com wrote: Hi, Sorry, Had forgot the ChangeLog. Ok, but please properly wrap the long lines, put '? gimple_...' on a new one. Thanks, Richard. ChangeLog 2015-01-12 Naveen H.S

Re: [testsuite] PATCH: Correct target selector in gcc.target/i386/nop-mcount.c

2015-01-12 Thread Uros Bizjak
On Mon, Jan 12, 2015 at 1:31 AM, H.J. Lu hjl.to...@gmail.com wrote: nonpic in target selector in gcc.target/i386/nop-mcount.c is ignored since {} is misplaced. This patch properly places {} in target selector. Tested on Linux/x86. OK for trunk? Thanks. H.J. ---

Re: [match-and-simplify] Remove printing for expression

2015-01-12 Thread Richard Biener
On Sat, 10 Jan 2015, Prathamesh Kulkarni wrote: On 8 January 2015 at 17:52, Richard Biener rguent...@suse.de wrote: On Sun, 21 Dec 2014, Prathamesh Kulkarni wrote: Hi, I removed printing for expression: from print_matches. I think it is out of place tvim here and we call print_matches

Re: PATCH: PR bootstrap/64561: [5 Regression] HAVE_LD_PIE_COPYRELOC is defined to 1 for broken linker

2015-01-12 Thread Uros Bizjak
On Mon, Jan 12, 2015 at 2:48 AM, H.J. Lu hongjiu...@intel.com wrote: Hi, This patch updates Linux/x86-64 linker test for PIE with copy reloc. Tested with broken and working linkers on Linux/x86-64. OK to install? Thanks. H.J. --- 2015-01-12 H.J. Lu hongjiu...@intel.com PR

Re: [Patch, Fortran, OOP] PR 63733: [4.8/4.9/5 Regression] wrong resolution for OPERATOR generics

2015-01-12 Thread Paul Richard Thomas
Dear Janus, Since it is a regression, by all means update the branches. We usually, propose delaying a bit but I am not convinced that this is effective for this kind of bug fix - usually, further problems take a long time to emerge. Thus, I would recommend that you get on with it. Thanks Paul

Simplify badness metrics in inliner, take 2

2015-01-12 Thread Jan Hubicka
Hi, this is variant of my earlier patch I comited. It solves issues with -fprofile-use and various roundoff errors that triggered sanity checks (partly by disabling them). Bootstrapped/regtested x86_64-linux. Honza PR ipa/63967 PR ipa/64425 * ipa-inline.c

Re: [PATCH] ipa-icf.c: Fix issues generated by original latest commit

2015-01-12 Thread Martin Liška
On 01/12/2015 09:51 AM, Richard Biener wrote: On Sat, Jan 10, 2015 at 10:03 AM, Chen Gang S gang.c...@sunrus.com.cn wrote: The related commit is 275e275 IPA ICF: target and optimization flags comparison.. For sem_function::equals_private(), fix the typo issue, and for target_opts_for_fn(), fix

Re: [PATCH, testsuite] fix ggcplug.c test-case

2015-01-12 Thread Richard Biener
On Mon, 12 Jan 2015, Prathamesh Kulkarni wrote: On 12 January 2015 at 14:19, Richard Biener rguent...@suse.de wrote: On Sun, 11 Jan 2015, Prathamesh Kulkarni wrote: Hi, The test-case plugin/ggcplug.c was failing due to flattening of tree.h and tree-core.h. Test-case was incorrect

[match-and-simplify] Merge from trunk

2015-01-12 Thread Richard Biener
Committed. 2015-01-12 Richard Biener rguent...@suse.de Merge from trunk r218478 through r219383.

Re: [PATCH] fix visium build

2015-01-12 Thread Richard Biener
On Fri, 9 Jan 2015, Prathamesh Kulkarni wrote: Hi, The tree.h and tree-core.h flattening patch: (https://gcc.gnu.org/ml/gcc-patches/2015-01/msg00467.html broke visium build. The attached patch fixes that. Built on visium-elf. OK to commit ? Ok. Thanks, Richard.

Re: [PATCH, testsuite] fix ggcplug.c test-case

2015-01-12 Thread Richard Biener
On Sun, 11 Jan 2015, Prathamesh Kulkarni wrote: Hi, The test-case plugin/ggcplug.c was failing due to flattening of tree.h and tree-core.h. Test-case was incorrect because it included gcc-plugin.h after tree.h whereas gcc-plugin.h should be the first header to be included by plugins. No, it

Re: [PATCH] Enable experimental TSAN support for Ada

2015-01-12 Thread Richard Biener
On Sun, Jan 11, 2015 at 1:39 PM, Bernd Edlinger bernd.edlin...@hotmail.de wrote: Hi Richard, On Fri, 9 Jan 2015 17:19:57, Richard Biener wrote: Yes. As said, you generally need to run folding results through force_gimple_operand. Richard. I have now used force_gimple_operand instead

Re: Simplify badness metrics in inliner, take 2

2015-01-12 Thread Markus Trippelsdorf
On 2015.01.12 at 10:59 +0100, Markus Trippelsdorf wrote: On 2015.01.12 at 10:30 +0100, Jan Hubicka wrote: this is variant of my earlier patch I comited. It solves issues with -fprofile-use and various roundoff errors that triggered sanity checks (partly by disabling them). The new

[PATCH][ARM] Fix PR target/64460: Set 'shift' attr properly on some patterns

2015-01-12 Thread Kyrill Tkachov
Hi all, In this PR we ICE when compiling with -mtune=xscale. The ICE is a segfault in xscale_sched_adjust_cost. The root cause is that xscale_sched_adjust_cost uses the value of the 'shift' insn attribute to index the recog operands. In GCC 5 the form and number of operands in those patterns

Re: [PATCH 7/10] OpenACC 2.0 support for libgomp - OpenACC runtime, NVidia PTX/CUDA plugin

2015-01-12 Thread Thomas Schwinge
Hi! On Tue, 23 Sep 2014 19:19:31 +0100, Julian Brown jul...@codesourcery.com wrote: This patch contains the bulk of the OpenACC 2.0 runtime support, [...] --- /dev/null +++ b/libgomp/libgomp-plugin.c @@ -0,0 +1,106 @@ +/* Exported (non-hidden) functions exposing libgomp interface for

[PATCH] Fix PR64535 - increase emergency EH buffers via a new allocator

2015-01-12 Thread Richard Biener
This fixes PR64535 by changing the fixed object size emergency pool to a variable EH object size (but fixed arena size) allocator. Via combining the dependent and non-dependent EH arenas this should allow around 600 bad_alloc throws in OOM situations on x86_64-linux compared to the current 64

[PATCH] Fix PR64404

2015-01-12 Thread Richard Biener
I am testing the following patch to fix a latent bug in the vectorizer dealing with redundant DRs. Bootstrap and regtest pending on x86_64-unknown-linux-gnu. Richard. 2015-01-12 Richard Biener rguent...@suse.de PR tree-optimization/64404 * tree-vect-stmts.c

Re: [PATCH 7/10] OpenACC 2.0 support for libgomp - OpenACC runtime, NVidia PTX/CUDA plugin

2015-01-12 Thread Thomas Schwinge
Hi! On Mon, 12 Jan 2015 15:37:46 +0100, I wrote: On Tue, 23 Sep 2014 19:19:31 +0100, Julian Brown jul...@codesourcery.com wrote: This patch contains the bulk of the OpenACC 2.0 runtime support, [...] --- /dev/null +++ b/libgomp/plugin-nvptx.c @@ -0,0 +1,1854 @@ +/* Plugin for NVPTX

Re: [x86, PATCH] operand reordering for commutative operations

2015-01-12 Thread Yuri Rumyantsev
Hi All, Thanks a lot for your comments. I've re-written reorder_operands as you proposed, but I'd like to know if we should apply this reordering at -O0? I will re-send the patch after testing completion. Thanks. Yuri. 2015-01-09 13:13 GMT+03:00 Richard Biener richard.guent...@gmail.com: On

Re: [arm][patch] fix arm_neon_ok check on !arm_arch7

2015-01-12 Thread Ramana Radhakrishnan
Sorry about the slow response- have been on holiday and still catching up on email. On 12/01/15 13:16, Andrew Stubbs wrote: Ping. On 23/12/14 16:46, Andrew Stubbs wrote: On 03/12/14 15:03, Andrew Stubbs wrote: The tools have always allowed us to drop down the arch to march=armv5te along

Re: [x86, PATCH] operand reordering for commutative operations

2015-01-12 Thread Richard Biener
On Mon, Jan 12, 2015 at 4:00 PM, Yuri Rumyantsev ysrum...@gmail.com wrote: Hi All, Thanks a lot for your comments. I've re-written reorder_operands as you proposed, but I'd like to know if we should apply this reordering at -O0? No, I think we can spare those cycles there. Richard. I will

[PATCH][Aarch64] PR64149: Remove -mlra/-mno-lra option for Aarch64.

2015-01-12 Thread Matthew Wahab
Hello, The LRA register is enabled by default for the Aarch64 backend and -mno-lra should no longer be used. This patch removes the -mlra/-mno-lra option for AArch64. Tested aarch64-none-linux-gnu with gcc-check. Matthew 2015-01-08 Matthew Wahab matthew.wa...@arm.com PR

[PATCH] Fix PR64436: broken logic to process bitwise ORs in bswap pass

2015-01-12 Thread Thomas Preud'homme
Hi all, To identify if a set of loads, shift, cast, mask (bitwise and) and bitwise OR is equivalent to a load or byteswap, the bswap pass assign a number to each byte loaded according to its significance (1 for lsb, 2 for next least significant byte, etc.) and form a symbolic number such as

Re: [arm][patch] fix arm_neon_ok check on !arm_arch7

2015-01-12 Thread Andrew Stubbs
Ping. On 23/12/14 16:46, Andrew Stubbs wrote: On 03/12/14 15:03, Andrew Stubbs wrote: The tools have always allowed us to drop down the arch to march=armv5te along with using -mfpu=neon. We are now changing command line behaviour, so an inform in terms of diagnostics to the user would be

C++ PATCH for c++/64547 (constexpr fn returning void)

2015-01-12 Thread Jason Merrill
In C++14 a constexpr function doesn't need to return a value. Tested x86_64-pc-linux-gnu, applying to trunk. commit 9675a7bde41b5430197854d8c1822c8f4d95b95e Author: Jason Merrill ja...@redhat.com Date: Fri Jan 9 01:46:16 2015 -0500 PR c++/64547 * constexpr.c

Re: [PATCH][ARM] Implement TARGET_SCHED_MACRO_FUSION_PAIR_P

2015-01-12 Thread Ramana Radhakrishnan
On Thu, Dec 4, 2014 at 9:19 AM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: On 02/12/14 22:58, Ramana Radhakrishnan wrote: On Tue, Nov 11, 2014 at 11:55 AM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, This is the arm implementation of the macro fusion hook. It tries to fuse

Re: [PATCH][ARM] Fix PR target/64460: Set 'shift' attr properly on some patterns

2015-01-12 Thread Kyrill Tkachov
Now with patch attached Kyrill On 12/01/15 14:27, Kyrill Tkachov wrote: Hi all, In this PR we ICE when compiling with -mtune=xscale. The ICE is a segfault in xscale_sched_adjust_cost. The root cause is that xscale_sched_adjust_cost uses the value of the 'shift' insn attribute to index the

Re: [PATCH 7/10] OpenACC 2.0 support for libgomp - OpenACC runtime, NVidia PTX/CUDA plugin

2015-01-12 Thread Thomas Schwinge
Hi! On Tue, 23 Sep 2014 19:19:31 +0100, Julian Brown jul...@codesourcery.com wrote: This patch contains the bulk of the OpenACC 2.0 runtime support, [...] --- /dev/null +++ b/libgomp/libgomp-plugin.h @@ -0,0 +1,57 @@ +/* An interface to various libgomp-internal functions for use by

Re: [PATCH]: Fix for PR ipa/64550

2015-01-12 Thread Richard Biener
On Mon, 12 Jan 2015, Martin Liška wrote: Hello. Following patch is fix for PR ipa/64550 which can bootstrap on x86_64-linux-pc. Explanation for the patch is described here: [1]. I hope this is correct fix for such cases? Ah, using TREE_THIS_VOLATILE on the result of ao_ref_base is wrong

Re: [PATCH, testsuite] fix ggcplug.c test-case

2015-01-12 Thread Prathamesh Kulkarni
On 12 January 2015 at 15:49, Richard Biener rguent...@suse.de wrote: On Mon, 12 Jan 2015, Prathamesh Kulkarni wrote: On 12 January 2015 at 14:36, Richard Biener rguent...@suse.de wrote: On Mon, 12 Jan 2015, Prathamesh Kulkarni wrote: On 12 January 2015 at 14:19, Richard Biener

Re: [PATCH] Flatten tree.h and tree-core.h (Version 3)

2015-01-12 Thread Prathamesh Kulkarni
On 12 January 2015 at 16:24, Andreas Schwab sch...@suse.de wrote: I'm getting this testsuite regression: FAIL: gcc.dg/plugin/ggcplug.c compilation Fixed with r219458. Thanks, Prathamesh In file included from /usr/local/gcc/gcc-20150112/gcc/testsuite/../../gcc/tree.h:23:0

Re: [PATCH, testsuite] fix ggcplug.c test-case

2015-01-12 Thread Richard Biener
On Mon, 12 Jan 2015, Prathamesh Kulkarni wrote: On 12 January 2015 at 14:36, Richard Biener rguent...@suse.de wrote: On Mon, 12 Jan 2015, Prathamesh Kulkarni wrote: On 12 January 2015 at 14:19, Richard Biener rguent...@suse.de wrote: On Sun, 11 Jan 2015, Prathamesh Kulkarni wrote:

Re: [PATCH, i386] Remove EBX usage from asm code

2015-01-12 Thread Jakub Jelinek
On Mon, Jan 12, 2015 at 01:36:05PM +0300, Evgeny Stupachenko wrote: frame_dummy does not use EBX in allocation now as there are enough other registers (that we don't need to save/restore). So if we do not modify frame_dummy EBX should stay unchanged. frame_dummy does not initialize EBX

Re: [PATCH][ARM][cleanup] Use R0_REGNUM and R1_REGNUM instead of 0 and 1 where appropriate

2015-01-12 Thread Ramana Radhakrishnan
On Thu, Dec 11, 2014 at 9:34 AM, Kyrill Tkachov kyrylo.tkac...@arm.com wrote: Hi all, While looking in this area on other business I noticed we could be using the names R0_REGNUM and R1_REGNUM when creating those REG rtxs since it's a bit more descriptive that just 0 and 1. Tested

Re: [PATCH, i386] Remove EBX usage from asm code

2015-01-12 Thread Evgeny Stupachenko
Agree, I've missed the usage of the function __register_frame_info_bases (frame_dummy assembly had only indirect call when I miss -pie in compilation). There is no reference on glibc that way. Sorry for the confusion. So that is potentially buggy right now. On Mon, Jan 12, 2015 at 1:50 PM, Jakub

[PATCH] Fix PR64357

2015-01-12 Thread Richard Biener
The following patch fixes PR64357 (or papers over some latent issue). We were not protecting a certain aspect of simple latches properly (a simple latch should belong to its loop). Bootstrap and regtest running on x86_64-unknown-linux-gnu. Richard. 2015-01-12 Richard Biener rguent...@suse.de

Re: Simplify badness metrics in inliner, take 2

2015-01-12 Thread Markus Trippelsdorf
On 2015.01.12 at 10:30 +0100, Jan Hubicka wrote: this is variant of my earlier patch I comited. It solves issues with -fprofile-use and various roundoff errors that triggered sanity checks (partly by disabling them). The new assert triggers during Firefox LTO build on ppc64: (final libxul

Re: [PATCH, testsuite] fix ggcplug.c test-case

2015-01-12 Thread Prathamesh Kulkarni
On 12 January 2015 at 14:36, Richard Biener rguent...@suse.de wrote: On Mon, 12 Jan 2015, Prathamesh Kulkarni wrote: On 12 January 2015 at 14:19, Richard Biener rguent...@suse.de wrote: On Sun, 11 Jan 2015, Prathamesh Kulkarni wrote: Hi, The test-case plugin/ggcplug.c was failing due to

[wwwdocs, patch] Update Fortran part of gcc-5/changes.html

2015-01-12 Thread Tobias Burnus
Hi all, hi Gerald, sync the changes from https://gcc.gnu.org/wiki/GFortran/News#GCC5 for the today's added compatibilty section and Janne's locale addition. If there are no objects or comments, I will commit it this evening Tobias, who is really behind reading fortran@gcc emails. Index:

[PATCH]: Fix for PR ipa/64550

2015-01-12 Thread Martin Liška
Hello. Following patch is fix for PR ipa/64550 which can bootstrap on x86_64-linux-pc. Explanation for the patch is described here: [1]. I hope this is correct fix for such cases? Thanks, Martin [1] https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64550 From

Re: [PATCH] Flatten tree.h and tree-core.h (Version 3)

2015-01-12 Thread Andreas Schwab
I'm getting this testsuite regression: FAIL: gcc.dg/plugin/ggcplug.c compilation In file included from /usr/local/gcc/gcc-20150112/gcc/testsuite/../../gcc/tree.h:23:0, from /usr/local/gcc/gcc-20150112/gcc/testsuite/gcc.dg/plugin/ggcplug.c:8: /usr/local/gcc/gcc-20150112/gcc

Re: [PATCH 0/2] Offloading from dlopened libraries

2015-01-12 Thread Ilya Verbin
Hi! How about this patch? It adds a new symbol into GOMP_4.0.1 symver, so it would be nice to include it into GCC 5 release. On 14 Nov 02:53, Ilya Verbin wrote: This patch fixes offloading from dlopened libraries, part 1 is for libgomp and part 2 is for intelmic plugin. How it works: When

Re: [PATCH, i386] Remove EBX usage from asm code

2015-01-12 Thread Evgeny Stupachenko
frame_dummy does not use EBX in allocation now as there are enough other registers (that we don't need to save/restore). So if we do not modify frame_dummy EBX should stay unchanged. frame_dummy does not initialize EBX register at the beginning it expects that EBX is pic from glibc frame_dummy is

[PATCH] Fix PR64530

2015-01-12 Thread Richard Biener
This fixes PR64530 by fixing a mistake (oops) in the iteration over all data-ref pairs in pg_add_dependence_edges. Bootstrap and regtest pending on x86_64-unknown-linux-gnu. Richard. 2015-01-12 Richard Biener rguent...@suse.de PR tree-optimization/64530 *

[gomp4] Replace enum omp_clause_map_kind with enum gomp_map_kind (was: Including a file from include/ in gcc/*.h)

2015-01-12 Thread Thomas Schwinge
Hi! On Mon, 22 Dec 2014 16:13:01 +0100, I wrote: I'm sending this again with some more people copied -- because I see you're working on tree.h/tree-core.h flattening, or know you're familiar with GCC plugins. ;-) Here is a question concerning both of that, where I'd appreciate your input.

[PATCH,MIPS] Add support for the R6 LSA and DLSA instructions

2015-01-12 Thread Matthew Fortune
This patch adds support for the R6 [D]LSA instructions. The support has been structured to allow MSA (when implemented) to turn on the same instructions as they are also added by the MSA ASE. I have continued to use the idea of 'ghost' options in the testsuite to indicate what features are

[PATCH,MIPS] Only pass floating-point options to the assembler then

2015-01-12 Thread Matthew Fortune
The new behaviour of the GCC driver passing floating point options like -msoft-float to the assembler is essential for the new o32 ABI extensions but is a change in behaviour. In particular GCC 5 used with binutils 2.24 would require a user to fix any hand-crafted code that made use of

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

2015-01-12 Thread Kyrill Tkachov
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 builtin for sqrt on DF mode and uses that to implement the

Re: [Patch, Fortran, OOP] PR 63733: [4.8/4.9/5 Regression] wrong resolution for OPERATOR generics

2015-01-12 Thread Janus Weil
Good, I fully agree. Fortunately the patch applies cleanly to the 4.9 branch and regtests without errors. Thus I have applied it as r219475. Will do 4.8 soon. Cheers, Janus 2015-01-12 9:30 GMT+01:00 Paul Richard Thomas paul.richard.tho...@gmail.com: Dear Janus, Since it is a regression, by

[PATCH] Fix PR64568

2015-01-12 Thread Richard Biener
The following avoids splitting TARGET_MEM_REFs by attaching REAL/IMAGPART_EXPRs around it which isn't allowed. Bootstrap and regtest running on x86_64-unknown-linux-gnu. Richard. 2014-01-12 Richard Biener rguent...@suse.de PR tree-optimization/64568 * tree-ssa-forwprop.c

[PATCH][test] Gate gcc.dg/aru-2.c test on profiling support

2015-01-12 Thread Kyrill Tkachov
Hi all, This recently added test adds -pg to its dg-options but not all targets support this and fail at link-time with bin/ld: cannot find -lc_p. Looking around I see that all tests that use -pg also do a dg-require-profiling. This patch adds that. With this patch the test doesn't FAIL

[PATCH] Fix REE for vector modes (PR rtl-optimization/64286)

2015-01-12 Thread Jakub Jelinek
Hi! As mentioned in the PR, giving up for all vector mode extensions is unnecessary, but unlike scalar integer extensions, where the low part of the extended value is the original value, for vectors this is not true, thus the old value is lost. Which means we can perform REE, but only if all

[PATCH] Use ldexp instead of scalbln for portability (PR other/64370)

2015-01-12 Thread Jakub Jelinek
Hi! As mentioned in the PR, HPUX doesn't have scalbln, but does have ldexp and that function is already used in gcj-dump, so supposedly it is more portable to use ldexp. Also in glibc it is defined in libc in addition to libm. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk?

Re: [testsuite] PATCH: Add check_effective_target_pie

2015-01-12 Thread H.J. Lu
On Mon, Jan 12, 2015 at 12:03 PM, Jeff Law l...@redhat.com wrote: On 01/12/15 12:59, H.J. Lu wrote: I don't know if -pg will work PIE on any targets. For Linux/x86 the choices of crt1.o are %{!shared: %{pg|p|profile:gcrt1.o%s;pie:Scrt1.o%s;:crt1.o%s}} -shared, -pg and -pie are mutually

[PATCH 2/4] Pipeline model for APM XGene-1.

2015-01-12 Thread Philipp Tomsich
--- gcc/config/aarch64/aarch64.md | 1 + gcc/config/arm/xgene1.md | 531 ++ 2 files changed, 532 insertions(+) create mode 100644 gcc/config/arm/xgene1.md diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md index

[PATCH 0/4, AArch64, v4] APM X-Gene 1 cost-table and pipeline model

2015-01-12 Thread Philipp Tomsich
Marcus Ramana, Attached is the updated---and hopefully final---revision of the changes to get XGene-1 properly wired up in the AArch64 and AArch64 backends. On the AArch64 side, we've only removed the URL from the credits of the xgene1.md file and the remaining content is unchanged (safe for

[PATCH 1/4] Core definition for APM XGene-1 and associated cost-table.

2015-01-12 Thread Philipp Tomsich
To keep this change separately buildable from the pipeline model, this patch directs the APM XGene-1 to use the generic scheduling model. --- gcc/ChangeLog-2014 | 8 +++ gcc/config/aarch64/aarch64-cores.def | 1 + gcc/config/aarch64/aarch64-tune.md | 2 +-

[PATCH 3/4] Change the type of the prefetch-instructions to 'prefetch'.

2015-01-12 Thread Philipp Tomsich
--- gcc/config/aarch64/aarch64.md | 2 +- gcc/config/arm/types.md | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md index 1f6b1b6..98f4f30 100644 --- a/gcc/config/aarch64/aarch64.md +++

Re: [RFC PATCH] Handle sequence in reg_set_p

2015-01-12 Thread Jeff Law
On 01/11/15 04:40, Oleg Endo wrote: Any particular reason why the SEQUENCE handling isn't done first, then the REG_INC and CALL insn handling? I'd probably explicitly return false if we had a sequence and none of its elements returned true. There's no need to check anything on the toplevel

[PATCH, committed] jit-playback.c: fix missing fclose

2015-01-12 Thread David Malcolm
Reported by David Binderman within discussion of PR jit/63854. Before/after jit.sum has 7272 passes. Committed to trunk as r219487. gcc/jit/ChangeLog: * jit-playback.c (gcc::jit::playback::context::read_dump_file): Add missing fclose on error-handling path. ---

Re: [PATCH, committed] Fix build of jit (was Re: [PATCH] Flatten tree.h and tree-core.h (Version 3))

2015-01-12 Thread Prathamesh Kulkarni
On 13 January 2015 at 00:01, Mike Stump mikest...@comcast.net wrote: On Jan 11, 2015, at 2:33 PM, Prathamesh Kulkarni prathamesh.kulka...@linaro.org wrote: oops, sorry about this. We will build further flattening patches with --enable-languages=all,go,jit,ada. Shall that cover all the

Re: [testsuite] PATCH: Add check_effective_target_pie

2015-01-12 Thread H.J. Lu
On Mon, Jan 12, 2015 at 10:09 AM, Jeff Law l...@redhat.com wrote: On 01/11/15 16:58, H.J. Lu wrote: Hi, This patch adds check_effective_target_pie to check if the current multilib generatse PIE by default. I will submit other patches to use it. OK for trunk? Thanks. H.J. ---

[PATCH] Install libgcj.pc as libgcj-5.pc rather than libgcj-5.0.pc (PR libgcj/64219)

2015-01-12 Thread Jakub Jelinek
Hi! This patch changes the libgcj*.pc installed filename to match the new GCC versioning scheme. Bootstrapped/regtested on x86_64-linux and i686-linux, tested make install. -rw-r--r--. 1 jakub jakub 192 Jan 12 21:02 /tmp/blah/usr/local/lib64/pkgconfig/libgcj-5.pc -rw-r--r--. 1 jakub jakub 192

[PATCH 4/4] Wire X-Gene 1 up in the ARM (32bit) backend as a AArch32-capable core.

2015-01-12 Thread Philipp Tomsich
--- gcc/ChangeLog-2014| 10 ++ gcc/config/arm/arm-cores.def | 1 + gcc/config/arm/arm-tables.opt | 3 +++ gcc/config/arm/arm-tune.md| 3 ++- gcc/config/arm/arm.c | 22 ++ gcc/config/arm/arm.md | 11 +--

Re: [COMMITTED] Merge libffi with upstream

2015-01-12 Thread Uros Bizjak
Hello! Upstream libffi has added support for Go closures (using the static chain), and support for complex numbers. Perhaps less relevant is new support for arc, microblaze, moxie, nios, and or1k targets. Without additional changes for Go, this merge has little effect. Within the gcc tree

Re: [testsuite] PATCH: Add check_effective_target_pie

2015-01-12 Thread Jeff Law
On 01/12/15 12:29, H.J. Lu wrote: Is this an inherent restriction of -fPIE, or is it merely an implementation detail? If the latter, is that implementation detail a target issue? ie, could we have a target that supports profiling in conjunction with -fPIE? If so, then this test seems too

[PATCH] Fix VRP ICE with -Wtype-limits (PR tree-optimization/64563)

2015-01-12 Thread Jakub Jelinek
Hi! On the following testcase we ICE with -Os -Wtype-limits, as VR_UNDEFINED has NULL vr0-min and vr0-max. From what the code does I believe the code only means to handle VR_RANGE and not anything else. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2015-01-12 Jakub

Re: LTO streaming of TARGET_OPTIMIZE_NODE

2015-01-12 Thread Jan Hubicka
On 09 Jan 12:45, Jakub Jelinek wrote: --- gcc/cgraphunit.c.jj 2015-01-09 12:01:33.0 +0100 +++ gcc/cgraphunit.c2015-01-09 12:22:27.742692667 +0100 @@ -2108,11 +2108,14 @@ ipa_passes (void) if (g-have_offload) { section_name_prefix =

[PATCH] Fix up some gcc.dg/vect/ testcases with -fpic (PR testsuite/64028)

2015-01-12 Thread Jakub Jelinek
Hi! Various gcc.dg/vect/ testcases now fail on the trunk with -fpic. The problem is that they expect that the global vars bind locally and vectorizer can increase their alignment, but with -fpic that does not work, as one can interpose them. Fixed by adding dg-add-options bind_pic_locally.

[PATCH] Fix up computed goto on POINTERS_EXTEND_UNSIGNED targets (PR middle-end/63974)

2015-01-12 Thread Jakub Jelinek
Hi! The 991213-3.c testcase ICEs on aarch64-linux with -mabi=ilp32 since wide-int merge. The problem is that x = convert_memory_address (Pmode, x) is used twice on a VOIDmode CONST_INT, which is wrong. For non-VOIDmode rtl the second convert_memory_address is a NOP, but for VOIDmode the second

[PATCH] Fix PR64461, Incorrect code on coldfire targets

2015-01-12 Thread Jeff Law
As suggested by Andreas in the PR, the simplest fix for this problem is to disable the various trunc* patterns for TARGET_COLDFIRE. That's precisely what this patch does. Built cross compilers with and without the m68k.md hunk. Verified the test failed without the m68k.mk hunk and passed

[PATCH] Optimize (x % 5) % 5 in VRP (PR tree-optimization/64454)

2015-01-12 Thread Jakub Jelinek
Hi! This patch optimizes away TRUNC_MOD_EXPR by constant second argument (if not 0 and not type's minimum) if the range of the first argument is already known to be [-op1 + 1, op1 - 1] or its subset. Bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2015-01-12 Jakub Jelinek

Re: [PATCH] Fix up some gcc.dg/vect/ testcases with -fpic (PR testsuite/64028)

2015-01-12 Thread Jeff Law
On 01/12/15 13:08, Jakub Jelinek wrote: Hi! Various gcc.dg/vect/ testcases now fail on the trunk with -fpic. The problem is that they expect that the global vars bind locally and vectorizer can increase their alignment, but with -fpic that does not work, as one can interpose them. Fixed by

Re: [PATCH] Fix up computed goto on POINTERS_EXTEND_UNSIGNED targets (PR middle-end/63974)

2015-01-12 Thread Jeff Law
On 01/12/15 13:19, Jakub Jelinek wrote: Hi! The 991213-3.c testcase ICEs on aarch64-linux with -mabi=ilp32 since wide-int merge. The problem is that x = convert_memory_address (Pmode, x) is used twice on a VOIDmode CONST_INT, which is wrong. For non-VOIDmode rtl the second

Open Issues in the TSAN Runtime

2015-01-12 Thread Bernd Edlinger
Hi Jakub, I am asking if we plan to merge the TSAN runtime from the LLVM tree soon, or if it is better to cherry pick specific changes from there. I am especially interested in fixing these two issues, but there may be other important improvements too:

Re: [Fortran, Patch] PR60334 - Segmentation fault on character pointer assignments

2015-01-12 Thread Paul Richard Thomas
Hi Andre, + if (INDIRECT_REF_P (parmse.string_length)) +/* In chains of functions/procedure calls the string_length already + is a pointer to the variable holding the length. Therefore + remove the deref on call. */ +parmse.string_length = TREE_OPERAND

Re: [PATCH] Use ldexp instead of scalbln for portability (PR other/64370)

2015-01-12 Thread Jeff Law
On 01/12/15 13:10, Jakub Jelinek wrote: Hi! As mentioned in the PR, HPUX doesn't have scalbln, but does have ldexp and that function is already used in gcj-dump, so supposedly it is more portable to use ldexp. Also in glibc it is defined in libc in addition to libm. Bootstrapped/regtested on

Re: [PATCH] Fix -mstack-arg-probe (PR target/64513)

2015-01-12 Thread Jeff Law
On 01/12/15 13:26, Jakub Jelinek wrote: Hi! For -mstack-arg-probe we push %rax and/or %r10 in the prologue, and mark that insn as RTX_FRAME_RELATED_P. But that means that the dwarf2 pass also considers that the %rax/%r10 registers, which are call used, to be saved in the unwind info, but they

Re: Open Issues in the TSAN Runtime

2015-01-12 Thread Jakub Jelinek
On Mon, Jan 12, 2015 at 09:53:16PM +0100, Bernd Edlinger wrote: I am asking if we plan to merge the TSAN runtime from the LLVM tree soon, or if it is better to cherry pick No. specific changes from there. Yes, I'll try to cherry-pick those tomorrow. I am especially interested in fixing

[PATCH][4.9] PR 64569 - Backport support for MIPS binutils 2.25

2015-01-12 Thread Matthew Fortune
This is a minimal backport of features added to GCC 5 to enable use of binutils 2.25 with GCC 4.9 for MIPS soft-float builds. Further details in the PR: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=64569 The commits which are being backported are listed below (the last one is posted but not

[PATCH] Fix -mstack-arg-probe (PR target/64513)

2015-01-12 Thread Jakub Jelinek
Hi! For -mstack-arg-probe we push %rax and/or %r10 in the prologue, and mark that insn as RTX_FRAME_RELATED_P. But that means that the dwarf2 pass also considers that the %rax/%r10 registers, which are call used, to be saved in the unwind info, but they are never restored, which makes the dwarf2

Re: [PATCH] Fix VRP ICE with -Wtype-limits (PR tree-optimization/64563)

2015-01-12 Thread Jeff Law
On 01/12/15 13:01, Jakub Jelinek wrote: Hi! On the following testcase we ICE with -Os -Wtype-limits, as VR_UNDEFINED has NULL vr0-min and vr0-max. From what the code does I believe the code only means to handle VR_RANGE and not anything else. Bootstrapped/regtested on x86_64-linux and

Re: [PATCH] Fix VRP ICE with -Wtype-limits (PR tree-optimization/64563)

2015-01-12 Thread Jakub Jelinek
On Mon, Jan 12, 2015 at 01:37:47PM -0700, Jeff Law wrote: On 01/12/15 13:01, Jakub Jelinek wrote: On the following testcase we ICE with -Os -Wtype-limits, as VR_UNDEFINED has NULL vr0-min and vr0-max. From what the code does I believe the code only means to handle VR_RANGE and not anything

Re: [PATCH] Optimize (x % 5) % 5 in VRP (PR tree-optimization/64454)

2015-01-12 Thread Jeff Law
On 01/12/15 13:28, Jakub Jelinek wrote: Hi! This patch optimizes away TRUNC_MOD_EXPR by constant second argument (if not 0 and not type's minimum) if the range of the first argument is already known to be [-op1 + 1, op1 - 1] or its subset. Bootstrapped/regtested on x86_64-linux and i686-linux,

Re: [PATCH] testsuite/lib/target-supports.exp: Fix check_effective_target_lto

2015-01-12 Thread Jeff Law
On 01/11/15 12:26, Ilya Verbin wrote: On 09 Jan 10:29, Thomas Schwinge wrote: As this was the only use of ENABLE_LTO in the testsuite, I suggest to also remove it from the gcc/Makefile.in:site.exp rule. Done. Here is an updated and retested patch. OK for trunk? gcc/ * Makefile.in

Re: [PATCH] Fix REE for vector modes (PR rtl-optimization/64286)

2015-01-12 Thread Jeff Law
On 01/12/15 12:59, Jakub Jelinek wrote: Hi! As mentioned in the PR, giving up for all vector mode extensions is unnecessary, but unlike scalar integer extensions, where the low part of the extended value is the original value, for vectors this is not true, thus the old value is lost. Which

Re: [Patch docs 4/5] Update Output Template/Statement from md.texi

2015-01-12 Thread Jeff Law
On 01/06/15 04:21, James Greenhalgh wrote: Hi, This patch updates the text in the Output Template and Output Statement sections of md.texi. I was aiming to: * Remove outdated details of the compiler. * Remove long or obscure words that, while accurate, only served to obfuscate a

[doc, committed] fix -Wbad-function-cast example

2015-01-12 Thread Sandra Loosemore
I was confused by the description of -Wbad-function-cast. It talks about function calls, but the malloc example looked more like a declaration, and IIRC it's not valid to redeclare functions from the standard C library with the wrong return type (or at the very least we shouldn't encourage

[patch] libstdc++/64553 and libstdc++/64560 facet shims without RTTI or wchar_t

2015-01-12 Thread Jonathan Wakely
Two patches to make the new cxx11-shim_facets.cc file compile when RTTI and wchar_t are disabled. Tested x86_64-linux, commited to trunk. commit d2cbfa8426fae046eea01630e24d4d15c9aa1e61 Author: Jonathan Wakely jwak...@redhat.com Date: Mon Jan 12 11:46:57 2015 + PR libstdc++/64553

Re: [testsuite] PATCH: Add check_effective_target_pie

2015-01-12 Thread Magnus Granberg
måndag 12 januari 2015 12.11.17 skrev H.J. Lu: On Mon, Jan 12, 2015 at 12:03 PM, Jeff Law l...@redhat.com wrote: On 01/12/15 12:59, H.J. Lu wrote: I don't know if -pg will work PIE on any targets. For Linux/x86 the choices of crt1.o are %{!shared:

  1   2   >