Re: [PATCH] New attribute to create target clones

2015-10-12 Thread Evgeny Stupachenko
Hi All, Here is a new version of patch (attached). Bootstrap and make check are in progress (all new tests passed). New test case g++.dg/ext/mvc4.C fails with ICE, when options lower than "-mavx" are passed. However it has the same behavior if "target_clones" attribute is replaced by 2 correspond

Re: [PATCH 8/9] Add TARGET_ADDR_SPACE_ZERO_ADDRESS_VALID

2015-10-12 Thread Richard Henderson
On 10/12/2015 09:10 PM, Richard Biener wrote: The check_loadstore change should instead have adjusted the flag_delete_null_pointer_checks guard in infer_nonnull_range_by_dereference. Nope, that doesn't work. You have to wait until you see the actual MEM being dereferenced before you can look

Re: [PATCH] gcc/ira.c: Check !HAVE_FP_INSTEAD_INSNS when frame pointer is needed and as global register

2015-10-12 Thread Mike Stump
On Oct 12, 2015, at 3:32 PM, Chen Gang wrote: > > OK, thanks. If we really need to fix it, which target hook should I use? > (or do we need a new target hook?) So, the first discussion would be if it is, or is not a bug. If it isn’t, then there is no fix. No fix, no target hook. So far, Bern

Re: Test for __cxa_thread_atexit_impl when cross-compiling libstdc++ for GNU targets

2015-10-12 Thread Joseph Myers
On Mon, 12 Oct 2015, Bernd Schmidt wrote: > A similar sequence of tests also occurs for *-aix*. I don't suppose the > function is likely to exist there or on other non-glibc targets? Given that the case there has "# We don't yet support AIX's TLS ABI." and GCC_CHECK_TLS commented out, I don't th

Re: [PATCH] PR66870 PowerPC64 Enable gold linker with split stack

2015-10-12 Thread Alan Modra
On Mon, Oct 12, 2015 at 10:15:04AM -0500, Lynn A. Boger wrote: > Thanks for doing this Alan. I agree this looks better to me. > > I assume by "etc" you mean you did biarch builds for your bootstraps on BE? By "etc" I meant "and regression tested". I built four configurations, powerpc-linux 32-b

Re: [PATCH] gcc/fold-const.c: Correct the report warning position.

2015-10-12 Thread Chen Gang
Hello all: Is this patch OK? If it still needs to do anything, please let me know, I shall try. Thanks. On 9/1/15 21:42, Chen Gang wrote: > On 8/31/15 19:12, Richard Biener wrote: >> On Sat, Aug 29, 2015 at 2:57 PM, Chen Gang >> wrote: >>> >>> It is about bug63510: current input_location isn't

Re: [PATCH] gcc/ira.c: Check !HAVE_FP_INSTEAD_INSNS when frame pointer is needed and as global register

2015-10-12 Thread Chen Gang
On 10/12/15 18:49, Bernd Schmidt wrote: > On 10/11/2015 05:16 PM, Chen Gang wrote: >> For some architectures (e.g. bfin), when this case occurs, they will use >> another instructions instead of frame pointer (e.g. LINK for bfin), so >> they can still generate correct output assembly code. > > Wha

Re: [patch 3/6] scalar-storage-order merge: C++ front-end

2015-10-12 Thread Jeff Law
On 10/06/2015 05:03 AM, Eric Botcazou wrote: This is the C++ front-end part, probably incomplete but passes the testsuite. cp/ * class.c: Add c-family/c-pragma.h. (finish_struct_1): If structure has reverse scalar storage order, rewrite the type of array fields with scala

Re: [patch 6/6] scalar-storage-order merge: testsuite

2015-10-12 Thread Jeff Law
On 10/06/2015 05:07 AM, Eric Botcazou wrote: This is the testsuite part. testsuite/ * c-c++-common/sso-1.c: New test. * c-c++-common/sso-2.c: Likewise. * c-c++-common/sso-3.c: Likewise. * c-c++-common/sso-4.c: Likewise. * c-c++-common/sso-5.c: Likewise.

Re: [patch 1/6] scalar-storage-order merge: Ada front-end

2015-10-12 Thread Jeff Law
On 10/06/2015 05:00 AM, Eric Botcazou wrote: This is the Ada front-end (in fact mostly gigi) part. ada/ * freeze.adb (Check_Component_Storage_Order): Skip a record component if it has Complex_Representation. (Freeze_Record_Type): If the type has Complex_Representation, sk

Re: Possible patch for PR fortran/67806

2015-10-12 Thread Louis Krupp
On Mon, 12 Oct 2015 08:41:43 -0700 Steve Kargl wrote > On Sun, Oct 11, 2015 at 10:18:48PM -0700, Louis Krupp wrote: > > The problem involves a derived type with a character component declared > > CHARACTER(NULL()) or CHARACTER(NULL(n)), where mold argument n is an > > integer poi

[PATCH] Allow FSM threader to thread more complex conditions

2015-10-12 Thread Jeff Law
Right now the FSM threader only handles trivial conditions. Specifically looking up a naked SSA_NAME (used for GIMPLE_SWITCH) and SSA_NAME != 0. The FSM threader need not be so restrictive. We can easily use it to lookup things like SSA_NAME constant for integral and pointer names. Essen

Re: [PATCH 2/9] ENABLE_CHECKING refactoring: libcpp

2015-10-12 Thread Jeff Law
On 10/06/2015 06:40 AM, Bernd Schmidt wrote: I'm not entirely sure what to make of this series. There seem to be good bits in there but also some things I find questionable. I'll add some comments on things that occur to me. Maybe we should start pulling out the bits that we think are ready & go

Re: [PATCH 1/9] ENABLE_CHECKING refactoring

2015-10-12 Thread Jeff Law
On 10/05/2015 05:27 PM, Mikhail Maltsev wrote: 3. Another one: gcc.c-torture/compile/pr52073.c which is, I guess, caused by https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67816 (the backtrace is the same, at least). FYI, this is fixed on the trunk. jeff

Re: [RFC VTV] Fix VTV for targets that have section anchors.

2015-10-12 Thread Jeff Law
On 10/09/2015 03:17 AM, Ramana Radhakrishnan wrote: This started as a Friday afternoon project ... It turned out enabling VTV for AArch64 and ARM was a matter of fixing PR67868 which essentially comes from building libvtv with section anchors turned on. The problem was that the flow of control f

[gomp4] Backport from trunk

2015-10-12 Thread James Norris
Hi, The attached patch was backported from trunk. commit 140722d9d2d574574c982f4616a80bb0ef766276 Author: jnorris Date: Mon Oct 12 20:22:30 2015 + * testsuite/libgomp.oacc-c-c++-common/vector-loop.c: Fix loop initializer. Jim diff --git a/libgomp/testsuite/libgomp.oacc-

Re: [PATCH] Check no unreachable blocks in inverted_post_order_compute

2015-10-12 Thread Jeff Law
On 10/12/2015 07:10 AM, Tom de Vries wrote: Hi, in the header comment of function inverted_post_order_compute in cfganal.c we find: ... This function assumes that all blocks in the CFG are reachable from the ENTRY (but not necessarily from EXIT). ... This patch checks that there are ind

[PATCH] Fix libgomp OpenACC test

2015-10-12 Thread James Norris
Hi, The attached patch fixes a test where the for-loop iterator was not initialized. Committed to trunk as obvious. Jim diff --git a/libgomp/testsuite/libgomp.oacc-c-c++-common/vector-loop.c b/libgomp/testsuite/libgomp.oacc-c-c++-common/vector-loop.c index cc915a9..8a51ee3 100644 --- a/libgomp/

[gomp4, committed] Neuter gang-single code in gang-redundant mode

2015-10-12 Thread Tom de Vries
Hi, ATM stores in the non-loop parts of an oacc kernels region are executed by all gangs. This patch makes sure that those stores are only executed by gang 0. I'm not aware atm of any related failing test cases, but it at least reduces the amount of stores performed. Bootstrapped and reg-t

Re: vector lightweight debug mode

2015-10-12 Thread François Dumont
On 07/10/2015 22:09, Jonathan Wakely wrote: > On 07/10/15 21:38 +0200, François Dumont wrote: >> Hi >> >>I completed vector assertion mode. Here is the result of the new >> test you will find in the attached patch. >> >> With debug mode: >> /home/fdt/dev/gcc/build_git/x86_64-unknown-linux-gnu/l

Re: Make cgraph frequencies more precise

2015-10-12 Thread Jan Hubicka
> On Sun, Oct 11, 2015 at 11:07 PM, Jan Hubicka wrote: > > Hi, > > this patch fixes a case of extreme imprecision I noticed while looking into > > profiles of PHP interpretter. There is a function that is called 22 times > > and contains the main loop. Now since the frequency of entry block is >

[hsa] Make debug stores conditional on a parameter

2015-10-12 Thread Martin Jambor
Hi, because HSA run-time currently offers very few options to debug the HSAIL, especially when it comes to tricky things like executing a kernel from kernel, we have resorted to introducing memory stores solely for the purpose of debugging. While we will gladly throw them away when HSA supports a

[hsa] Silence a warning in emit_directive_variable

2015-10-12 Thread Martin Jambor
Hi, in the previous commit I have introduced a warning, this one silences it. Thanks, Martin 2015-10-12 Martin Jambor * hsa-brig.c (emit_directive_variable): Add typecast. diff --git a/gcc/hsa-brig.c b/gcc/hsa-brig.c index 49d9e1d..e9712b5 100644 --- a/gcc/hsa-brig.c +++ b/gcc/hsa

[hsa] Introduce alignment to hsa_insn_mem

2015-10-12 Thread Martin Jambor
Hi, the newest version of the finalizer actually honors alignment information and therefore the compiler has to provide the correct values, which is what the following patch does. We may need to be more conservative in the cases when we expand memset and memcpy inline, but I have committed this p

[hsa] Export dump_hsa_insn

2015-10-12 Thread Martin Jambor
Hi, this small patch makes dump_hsa_insn available for dumping in other compilation units. Committed to the branch. Thanks, Martin 2015-10-12 Martin Jambor * hsa-dump.c (dump_hsa_insn): Rename to dump_hsa_insn_1. (dump_hsa_insn): New function. (dump_hsa_bb): Use dum

[committed, PATCH] [gcc-5-branch] Wrong stack alignment adjustment

2015-10-12 Thread H.J. Lu
Committed as an obvious fix. H.J. --- Index: gcc/ChangeLog === --- gcc/ChangeLog (revision 228731) +++ gcc/ChangeLog (working copy) @@ -1,3 +1,10 @@ +2015-10-12 H.J. Lu + + PR target/67940 + * config/i386/i3

[committed, gomp4, 3/3] Handle sequential code in kernels region - Testcases

2015-10-12 Thread Tom de Vries
On 12/10/15 19:12, Tom de Vries wrote: Hi, I've committed the following patch series. 1Add get_bbs_in_oacc_kernels_region 2Handle sequential code in kernels region 3Handle sequential code in kernels region - Testcases The patch series adds detection of whether seq

[committed, gomp4, 2/3] Handle sequential code in kernels region

2015-10-12 Thread Tom de Vries
On 12/10/15 19:12, Tom de Vries wrote: Hi, I've committed the following patch series. 1Add get_bbs_in_oacc_kernels_region 2Handle sequential code in kernels region 3Handle sequential code in kernels region - Testcases The patch series adds detection of whether seq

Re: [PATCH, 3/5] Handle original loop tree in expand_omp_for_generic

2015-10-12 Thread Bernd Schmidt
Does this version look better? In terms of clarity, yes. Only one thing: + if (/* If we already have a loop struct for the original loop, don't +allocate a new one. */ + !orig_loop_has_loop_struct Don't really like the formatting with this comment. I'd pull it in fr

[PATCH, mips]: Use ROUND_UP and ROUND_DOWN macros

2015-10-12 Thread Uros Bizjak
Fairly trivial patch that introduces no functional changes. * config/mips/mips.h (MIPS_STACK_ALIGN): Implement using ROUND_UP macro. * config/mips/mips.c (mips_setup_incoming_varargs): Use ROUND_DOWN to calculate off. (mips_gimplify_va_arg_expr): Use ROUND_UP to calculate rsize

Re: [committed, gomp4] Handle sequential code in kernels region patch series

2015-10-12 Thread Tom de Vries
On 12/10/15 19:12, Tom de Vries wrote: Hi, I've committed the following patch series. 1Add get_bbs_in_oacc_kernels_region 2Handle sequential code in kernels region 3Handle sequential code in kernels region - Testcases The patch series adds detection of whether seq

[committed, gomp4] Handle sequential code in kernels region patch series

2015-10-12 Thread Tom de Vries
Hi, I've committed the following patch series. 1 Add get_bbs_in_oacc_kernels_region 2 Handle sequential code in kernels region 3 Handle sequential code in kernels region - Testcases The patch series adds detection of whether sequential code (that is, code in the oacc kernels

Re: Test for __cxa_thread_atexit_impl when cross-compiling libstdc++ for GNU targets

2015-10-12 Thread Bernd Schmidt
On 10/10/2015 01:18 AM, Joseph Myers wrote: I noticed that when testing glibc with a cross compiler I got UNSUPPORTED: nptl/tst-thread_local1 because the libstdc++-v3 configuration for cross compiling defaulted to __cxa_thread_atexit_impl not being available. This patch fixes GLIBCXX_CROSSCONF

[PATCH, sparc]: Use ROUND_UP and ROUND_DOWN macros

2015-10-12 Thread Uros Bizjak
Two functional changes I'd like to point out: /* ALIGN FRAMES on double word boundaries */ -#define SPARC_STACK_ALIGN(LOC) \ - (TARGET_ARCH64 ? (((LOC)+15) & ~15) : (((LOC)+7) & ~7)) +#define SPARC_STACK_ALIGN(LOC) ROUND_UP ((LOC), UNITS_PER_WORD * 2) The one above uses UNITS_PER_WORD in stack

Re: [PATCH, 3/5] Handle original loop tree in expand_omp_for_generic

2015-10-12 Thread Tom de Vries
On 12/10/15 16:11, Bernd Schmidt wrote: On 10/10/2015 01:58 PM, Tom de Vries wrote: Handle original loop tree in expand_omp_for_generic 2015-09-10 Tom de Vries PR tree-optimization/67476 * omp-low.c (expand_omp_for_generic): Handle original loop tree. This one I find slightly confu

Re: [PATCH] v4 of diagnostic_show_locus and rich_location

2015-10-12 Thread Manuel López-Ibáñez
On 12 October 2015 at 16:44, David Malcolm wrote: > v4 of the patch does the conversion of Fortran, and eliminates the > adaptation layer. No partial transitions here! > > Manu: I hope this addresses your concerns. Yes, it looks great. I don't understand how this - and for two locations that

[PATCH] Improve FSM threader to handle compiler temporaries too

2015-10-12 Thread Jeff Law
The FSM jump threader currently will not handle threading for compiler generated temporaries. I discovered this when looking at what tests regress if I remove the ability of the old threader to thread across backedges and why the FSM threader doesn't handle them. bitmap.c has a multitude o

[PATCH] v4 of diagnostic_show_locus and rich_location

2015-10-12 Thread David Malcolm
On Sun, 2015-09-27 at 02:55 +0200, Dodji Seketeli wrote: > [Note to libcpp, C, and Fortran maintainers: we still need your input :-)] Updated version of patch attached (v4); a diff relative to v3 can be seen at: https://dmalcolm.fedorapeople.org/gcc/2015-10-12/0003-Eliminate-special-casing-for-For

Re: Possible patch for PR fortran/67806

2015-10-12 Thread Steve Kargl
On Sun, Oct 11, 2015 at 10:18:48PM -0700, Louis Krupp wrote: > The problem involves a derived type with a character component declared > CHARACTER(NULL()) or CHARACTER(NULL(n)), where mold argument n is an integer > pointer. > I was looking at 67805 this weekend, which is somewhat related to th

[committed. gomp4, 5/6] Add oacc kernels related infra functions

2015-10-12 Thread Tom de Vries
On 12/10/15 16:49, Tom de Vries wrote: Hi, I've committed the following patch series to the gomp-4_0-branch. 1Add pass_dominator::jump_threading_p () 2Add dom_walker::walk_until 3Add pass_dominator::sese_mode_p () 4Add skip_stmt parm to pass_dominator::ge

[committed. gomp4, 6/6] Add pass_dominator_oacc_kernels

2015-10-12 Thread Tom de Vries
On 12/10/15 16:49, Tom de Vries wrote: Hi, I've committed the following patch series to the gomp-4_0-branch. 1Add pass_dominator::jump_threading_p () 2Add dom_walker::walk_until 3Add pass_dominator::sese_mode_p () 4Add skip_stmt parm to pass_dominator::ge

[committed, gomp4, 4/6] Add skip_stmt parm to pass_dominator::get_sese ()

2015-10-12 Thread Tom de Vries
On 12/10/15 16:49, Tom de Vries wrote: Hi, I've committed the following patch series to the gomp-4_0-branch. 1Add pass_dominator::jump_threading_p () 2Add dom_walker::walk_until 3Add pass_dominator::sese_mode_p () 4Add skip_stmt parm to pass_dominator::ge

Re: [PATCH] PR66870 PowerPC64 Enable gold linker with split stack

2015-10-12 Thread Lynn A. Boger
Thanks for doing this Alan. I agree this looks better to me. I assume by "etc" you mean you did biarch builds for your bootstraps on BE? On 10/11/2015 08:07 AM, Alan Modra wrote: On Sat, Oct 10, 2015 at 11:25:38PM +0200, Andreas Schwab wrote: "Lynn A. Boger" writes: Index: gcc/config/rs600

Re: Make cgraph frequencies more precise

2015-10-12 Thread H.J. Lu
On Sun, Oct 11, 2015 at 11:07 PM, Jan Hubicka wrote: > Hi, > this patch fixes a case of extreme imprecision I noticed while looking into > profiles of PHP interpretter. There is a function that is called 22 times > and contains the main loop. Now since the frequency of entry block is dropped > to

[committed, gomp4, 3/6] Add pass_dominator::sese_mode_p ()

2015-10-12 Thread Tom de Vries
On 12/10/15 16:49, Tom de Vries wrote: Hi, I've committed the following patch series to the gomp-4_0-branch. 1Add pass_dominator::jump_threading_p () 2Add dom_walker::walk_until 3Add pass_dominator::sese_mode_p () 4Add skip_stmt parm to pass_dominator::ge

[committed, gomp4, 2/6] Add dom_walker::walk_until

2015-10-12 Thread Tom de Vries
On 12/10/15 16:49, Tom de Vries wrote: Hi, I've committed the following patch series to the gomp-4_0-branch. 1Add pass_dominator::jump_threading_p () 2Add dom_walker::walk_until 3Add pass_dominator::sese_mode_p () 4Add skip_stmt parm to pass_dominator::ge

[committed, gomp4, 1/6] Add pass_dominator::jump_threading_p ()

2015-10-12 Thread Tom de Vries
On 12/10/15 16:49, Tom de Vries wrote: Hi, I've committed the following patch series to the gomp-4_0-branch. 1Add pass_dominator::jump_threading_p () 2Add dom_walker::walk_until 3Add pass_dominator::sese_mode_p () 4Add skip_stmt parm to pass_dominator::ge

[committed. gomp4] pass_dominator_oacc_kernels patch series

2015-10-12 Thread Tom de Vries
Hi, I've committed the following patch series to the gomp-4_0-branch. 1 Add pass_dominator::jump_threading_p () 2 Add dom_walker::walk_until 3 Add pass_dominator::sese_mode_p () 4 Add skip_stmt parm to pass_dominator::get_sese () 5 Add oacc kernels related infra fun

Re: [PATCH, 4/5] Support DEFPARAMENUM in params.def

2015-10-12 Thread Bernd Schmidt
On 10/10/2015 02:05 PM, Tom de Vries wrote: On 10/10/15 13:06, Tom de Vries wrote: OK, I'll repost with the patch split up, as follows: 1Handle simple latch in expand_omp_for_generic 2Add missing phis in expand_omp_for_generic 3Handle original loop tree in expand_o

Re: [PATCH, 2/5] Add missing phis in expand_omp_for_generic

2015-10-12 Thread Tom de Vries
On 12/10/15 16:05, Bernd Schmidt wrote: This seems to be slightly different from the one originally submitted? Hi Bernd, As I mentioned here ( https://gcc.gnu.org/ml/gcc-patches/2015-10/msg01043.html ), I've moved the ssa-support bit into the !broken_loop condition. I think that's the onl

Re: [PATCH, 3/5] Handle original loop tree in expand_omp_for_generic

2015-10-12 Thread Bernd Schmidt
On 10/10/2015 01:58 PM, Tom de Vries wrote: Handle original loop tree in expand_omp_for_generic 2015-09-10 Tom de Vries PR tree-optimization/67476 * omp-low.c (expand_omp_for_generic): Handle original loop tree. This one I find slightly confusing. - add_bb_to_loop (l2

Re: Move sqrt and cbrt simplifications to match.pd

2015-10-12 Thread Christophe Lyon
On 9 October 2015 at 18:17, Richard Sandiford wrote: > Richard Sandiford writes: >> Christophe Lyon writes: >>> On 8 October 2015 at 18:55, Richard Sandiford >>> wrote: Marc Glisse writes: > On Mon, 5 Oct 2015, Richard Sandiford wrote: > >> + /* cbrt(sqrt(x)) -> pow(x,1/6).

Re: [PATCH, 2/5] Add missing phis in expand_omp_for_generic

2015-10-12 Thread Bernd Schmidt
On 10/10/2015 01:49 PM, Tom de Vries wrote: On 10/10/15 13:06, Tom de Vries wrote: OK, I'll repost with the patch split up, as follows: 1Handle simple latch in expand_omp_for_generic 2Add missing phis in expand_omp_for_generic 3Handle original loop tree in expand_o

[PR debug/67192] Fix C loops' back-jump location

2015-10-12 Thread Andreas Arnez
Since r223098 ("Implement -Wmisleading-indentation") the backward-jump generated for a C while- or for-loop can get the wrong line number. This is because the check for misleading indentation peeks ahead one token, advancing input_location to after the loop, and then c_finish_loop() creates the bac

Re: [PATCH, 1/5] Handle simple latch in expand_omp_for_generic

2015-10-12 Thread Bernd Schmidt
On 10/10/2015 01:24 PM, Tom de Vries wrote: On 10/10/15 13:06, Tom de Vries wrote: OK, I'll repost with the patch split up, as follows: 1Handle simple latch in expand_omp_for_generic 2Add missing phis in expand_omp_for_generic 3Handle original loop tree in expand_o

Re: [AArch64_be] Fix vtbl[34] and vtbx4

2015-10-12 Thread James Greenhalgh
On Fri, Oct 09, 2015 at 05:16:05PM +0100, Christophe Lyon wrote: > On 8 October 2015 at 11:12, James Greenhalgh wrote: > > On Wed, Oct 07, 2015 at 09:07:30PM +0100, Christophe Lyon wrote: > >> On 7 October 2015 at 17:09, James Greenhalgh > >> wrote: > >> > On Tue, Sep 15, 2015 at 05:25:25PM +010

Re: Move some bit and binary optimizations in simplify and match

2015-10-12 Thread Richard Biener
On Mon, Oct 12, 2015 at 12:22 PM, Hurugalawadi, Naveen wrote: > Hi Richard, > > Thanks for your review and useful comments. > I will move the future optimization patterns with all the conditions > present in fold-const or builtins file as per your suggestions. > > Please find attached the patch a

[gomp4] OpenACC loop expand reorg

2015-10-12 Thread Nathan Sidwell
I've committed this to gomp4 branch. It reworks the loop expansion code for OpenACC loops in the following ways 1) Removes OpenACC handling from expand_omp_for_static_{,no}chunk}. These are thus now OpenMP only. (Jakub, that should reduce conflicts between the two implemenations) 2) Imple

[PATCH] Check no unreachable blocks in inverted_post_order_compute

2015-10-12 Thread Tom de Vries
Hi, in the header comment of function inverted_post_order_compute in cfganal.c we find: ... This function assumes that all blocks in the CFG are reachable from the ENTRY (but not necessarily from EXIT). ... This patch checks that there are indeed no unreachable blocks when calling inver

Re: [PATCH, aarch64]: Remove AARCH64_ROUND_UP and AARCH64_ROUND_DOWN defines

2015-10-12 Thread Marcus Shawcroft
On 12 October 2015 at 12:28, Uros Bizjak wrote: > Remove private definitions and use equivalent global macros instead. > > 2015-10-12 Uros Bizjak > > * config/aarch/aarch64.h (AARCH64_ROUND_UP): Remove. > (AARCH64_ROUND_DOWN): Ditto. > * config/aarch64/aarch64.c: Use ROUND_UP instea

Re: [PATCH, rs6000]: Use ROUND_UP and ROUND_DOWN macros

2015-10-12 Thread David Edelsohn
> Fairly trivial patch that introduces no functional changes. 2015-10-12 Uros Bizjak * config/rs6000/rs6000.h (RS6000_ALIGN): Implement using ROUND_UP macro. * config/rs6000/rs6000.c (rs6000_darwin64_record_arg_advance_flush): Use ROUND_UP and ROUND_DOWN macros where applicable

Re: [gomp4, committed] Add goacc/kernels-acc-on-device.c

2015-10-12 Thread Tom de Vries
On 12/10/15 12:49, Thomas Schwinge wrote: Hi Tom! On Sat, 10 Oct 2015 12:49:01 +0200, Tom de Vries wrote: >--- /dev/null >+++ b/gcc/testsuite/c-c++-common/goacc/kernels-acc-on-device.c >@@ -0,0 +1,39 @@ >+/* { dg-additional-options "-O2" } */ >+ >+#include Hi Thomas, That doesn't work (at

Re: [PATCH] PR target/67850: Wrong call_used_regs used in aggregate_value_p

2015-10-12 Thread H.J. Lu
On Wed, Oct 7, 2015 at 2:01 AM, Uros Bizjak wrote: > On Wed, Oct 7, 2015 at 10:53 AM, Richard Biener wrote: > >>> >>> > > Since targetm.expand_to_rtl_hook may be called to switch ABI, it >>> >>> > > should >>> >>> > > be called for each function before expanding to RTL. Otherwise, we >>> >>> >

Re: Move some bit and binary optimizations in simplify and match

2015-10-12 Thread Marc Glisse
On Mon, 12 Oct 2015, Hurugalawadi, Naveen wrote: +/* Fold X + (X / CST) * -CST to X % CST. */ +(simplify + (plus (convert1? @0) (convert2? (mult (trunc_div @0 INTEGER_CST@1) INTEGER_CST@2))) + (if ((INTEGRAL_TYPE_P (type) || VECTOR_INTEGER_TYPE_P (type)) + && wi::add (@1, @2) == 0) + (

Re: [1/7] Add new tree code SEXT_EXPR

2015-10-12 Thread Richard Biener
On Sun, Oct 11, 2015 at 12:35 PM, Kugan wrote: > > > On 15/09/15 23:18, Richard Biener wrote: >> On Mon, Sep 7, 2015 at 4:55 AM, Kugan >> wrote: >>> >>> This patch adds support for new tree code SEXT_EXPR. >> >> diff --git a/gcc/cfgexpand.c b/gcc/cfgexpand.c >> index d567a87..bbc3c10 100644 >> -

Re: [3/7] Optimize ZEXT_EXPR with tree-vrp

2015-10-12 Thread Richard Biener
On Sun, Oct 11, 2015 at 4:56 AM, Kugan wrote: > > > On 09/10/15 21:29, Richard Biener wrote: >> + unsigned int prec = tree_to_uhwi (vr1.min); >> >> this should use unsigned HOST_WIDE_INT >> >> + wide_int sign_bit = wi::shwi (1ULL << (prec - 1), >> + TYPE

[PATCH, aarch64]: Remove AARCH64_ROUND_UP and AARCH64_ROUND_DOWN defines

2015-10-12 Thread Uros Bizjak
Remove private definitions and use equivalent global macros instead. 2015-10-12 Uros Bizjak * config/aarch/aarch64.h (AARCH64_ROUND_UP): Remove. (AARCH64_ROUND_DOWN): Ditto. * config/aarch64/aarch64.c: Use ROUND_UP instead of AARCH64_ROUND_UP. Tested by building a crosscompiler to

Re: [PATCH ARM]: PR67745: Fix function alignment after __attribute__ 2/2

2015-10-12 Thread Bernd Schmidt
On 10/12/2015 12:56 PM, Christian Bruel wrote: yes I see, I was hoping to avoid a new hook, but as you said it seems mandatory for the mere declaration case. Here is one proposal, it defaults to nothing and the ARM implementation does not need to handle the vptr bit setting. so that simplifies a

[PATCH][5] Backport ISL 0.15 support

2015-10-12 Thread Richard Biener
This backports the patch to allow bootstrapping with ISL 0.15 to the GCC 5 branch (the GCC 4.9 branch will require backporting of some dependencies). Bootstrapped with ISL 0.15 (in-tree), ISL 0.14 and ISL 0.12 (both installed). Committed to the branch. Richard. 2015-10-12 Richard Biener

Re: [PATCH 1/3] [ARM] PR63870 Add qualifiers for NEON builtins

2015-10-12 Thread Alan Lawrence
On 07/10/15 00:59, charles.bay...@linaro.org wrote: diff --git a/gcc/config/arm/arm-builtins.c b/gcc/config/arm/arm-builtins.c ... case NEON_ARG_MEMORY: /* Check if expand failed. */ if (op[argc] == const0_rtx) { - va_end (a

Re: [RFA 1/2]: Don't ignore target_header_dir when deciding inhibit_libc

2015-10-12 Thread Hans-Peter Nilsson
> From: Ulrich Weigand > Date: Mon, 12 Oct 2015 11:58:40 +0200 (cutting *only* because I had a comment; not an indication of preference.) > --or-- > > 2b) Change target_header_dir from a single directory to a list of > directories, and check all of these for header files. This list > w

Re: [gomp4, committed] Add goacc/kernels-acc-on-device.c

2015-10-12 Thread Thomas Schwinge
Hi Tom! On Sat, 10 Oct 2015 12:49:01 +0200, Tom de Vries wrote: > --- /dev/null > +++ b/gcc/testsuite/c-c++-common/goacc/kernels-acc-on-device.c > @@ -0,0 +1,39 @@ > +/* { dg-additional-options "-O2" } */ > + > +#include That doesn't work (at least in build-tree testing), as gcc/testsuite/ is n

Re: [PATCH 2/3] [ARM] PR63870 Mark lane indices of vldN/vstN with appropriate qualifier

2015-10-12 Thread Alan Lawrence
On 07/10/15 00:59, charles.bay...@linaro.org wrote: diff --git a/gcc/config/arm/neon.md b/gcc/config/arm/neon.md index 2667866..251afdc 100644 --- a/gcc/config/arm/neon.md +++ b/gcc/config/arm/neon.md @@ -4261,8 +4261,9 @@ if (BYTES_BIG_ENDIAN) UNSPEC_VLD1_LANE))] "TARG

Re: [PATCH] gcc/ira.c: Check !HAVE_FP_INSTEAD_INSNS when frame pointer is needed and as global register

2015-10-12 Thread Bernd Schmidt
On 10/11/2015 05:16 PM, Chen Gang wrote: For some architectures (e.g. bfin), when this case occurs, they will use another instructions instead of frame pointer (e.g. LINK for bfin), so they can still generate correct output assembly code. What is "this case"? I don't think you have explained th

Re: [[Boolean Vector, patch 5/5] Support boolean vectors in vector lowering

2015-10-12 Thread Alan Lawrence
On 09/10/15 22:01, Jeff Law wrote: So my question for the series as a whole is whether or not we need to do something for the other languages, particularly Fortran. I was a bit surprised to see this stuff bleed into the C/C++ front-ends and obviously wonder if it's bled into Fortran, Ada, Java,

Re: Move some bit and binary optimizations in simplify and match

2015-10-12 Thread Hurugalawadi, Naveen
Hi Richard, Thanks for your review and useful comments. I will move the future optimization patterns with all the conditions present in fold-const or builtins file as per your suggestions. Please find attached the patch as per your comments. Please review the patch and let me know if any further

Re: [PR67891] drop is_gimple_reg test from set_parm_rtl

2015-10-12 Thread Richard Biener
On Sat, Oct 10, 2015 at 3:16 PM, Alexandre Oliva wrote: > On Oct 9, 2015, Richard Biener wrote: > >> Ok. Note that I think emit_block_move shouldn't mess with the addressable >> flag. > > I have successfully tested a patch that stops it from doing so, > reverting https://gcc.gnu.org/bugzilla/s

Re: [PATCH 2/6] always define SETUP_FRAME_ADDRESSES

2015-10-12 Thread Bernd Schmidt
On 10/11/2015 02:25 AM, tbsaunde+...@tbsaunde.org wrote: From: Trevor Saunders gcc/ChangeLog: 2015-10-10 Trevor Saunders * defaults.h (SETUP_FRAME_ADDRESSES): New default definition. * builtins.c (expand_builtin_return_addr): Adjust. * doc/tm.texi: Likewise.

Re: [PATCH 8/9] Add TARGET_ADDR_SPACE_ZERO_ADDRESS_VALID

2015-10-12 Thread Richard Biener
On Thu, Oct 8, 2015 at 11:10 PM, Richard Henderson wrote: > On 10/08/2015 09:20 PM, Richard Biener wrote: >> >> On Thu, Oct 8, 2015 at 6:59 AM, Richard Henderson wrote: >>> >>> * target.def (TARGET_ADDR_SPACE_ZERO_ADDRESS_VALID): New. >>> * targhooks.h (default_addr_space_zero_a

Re: [PATCH] Fix parloops gimple_uid usage

2015-10-12 Thread Richard Biener
On Fri, Oct 9, 2015 at 11:09 PM, Tom de Vries wrote: > Hi, > > In tree-parloops.c:gather_scalar_reductions, we find the comment: > ... > /* As gimple_uid is used by the vectorizer in between > vect_analyze_loop_form and destroy_loop_vec_info, we can set > gimple_uid of reduc_phi stmts

Re: Move sqrt and cbrt simplifications to match.pd

2015-10-12 Thread Richard Biener
On Fri, Oct 9, 2015 at 6:17 PM, Richard Sandiford wrote: > Richard Sandiford writes: >> Christophe Lyon writes: >>> On 8 October 2015 at 18:55, Richard Sandiford >>> wrote: Marc Glisse writes: > On Mon, 5 Oct 2015, Richard Sandiford wrote: > >> + /* cbrt(sqrt(x)) -> pow(x,1/6

Re: [RFA 1/2]: Don't ignore target_header_dir when deciding inhibit_libc

2015-10-12 Thread Ulrich Weigand
Hans-Peter Nilsson wrote: > > > So, ISTM we should change --with-headers (=yes) to either look > > > in sys-include or in include. Setting it to sys-include > > > wouldn't help you or anyone else as it's already the default... > > > > On the other hand, the current docs appear to imply that the >

Re: [PATCH] Fix PR67783, quadraticness in IPA inline analysis

2015-10-12 Thread Richard Biener
On Sun, 11 Oct 2015, Dominique d'Humières wrote: > > It seems there was regression on fatigue/fatigue2 > > http://gcc.opensuse.org/c++bench/pb11/ Fatigue > > was one of reasons to intorduce the heuristics, so it may be related to the > > patch :( > > The test in pr 64099 comment 14 now require

[PATCH, rs6000]: Use ROUND_UP and ROUND_DOWN macros

2015-10-12 Thread Uros Bizjak
Fairly trivial patch that introduces no functional changes. 2015-10-12 Uros Bizjak * config/rs6000/rs6000.h (RS6000_ALIGN): Implement using ROUND_UP macro. * config/rs6000/rs6000.c (rs6000_darwin64_record_arg_advance_flush): Use ROUND_UP and ROUND_DOWN macros where applicable.

Re: [PATCH] hurd: align -p and -pg behavior on Linux

2015-10-12 Thread Thomas Schwinge
Hi Samuel! On Sat, 19 Sep 2015 14:00:23 +0200, Samuel Thibault wrote: > On Linux, -p and -pg do not make gcc link against libc_p.a, only > -profile does (as documented in r11246), and thus people expect -p > and -pg to work without libc_p.a installed (it is actually even not > available any more

[PATCH] More vectorizer TLC

2015-10-12 Thread Richard Biener
Bootstrapped & tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-10-12 Richard Biener * tree-vect-loop.c (vect_analyze_loop_operations): Move cost related code ... (vect_analyze_loop_2): ... here. Index: gcc/tree-vect-loop.c =

Re: [patch] header file re-ordering.

2015-10-12 Thread Jeff Law
On 10/08/2015 07:37 AM, Andrew MacLeod wrote: On 10/07/2015 06:02 PM, Jeff Law wrote: On 10/01/2015 08:33 PM, Andrew MacLeod wrote: these are all in the main gcc directory. 297 files total. Everything bootstraps on x86_64-pc-linux-gnu and powerpc64le-unknown-linux-gnu. All targets in config-l

Re: [C++ PATCH] PR c++/58566

2015-10-12 Thread Jason Merrill
OK. Jason

Re: Fix more of C/fortran canonical type issues

2015-10-12 Thread Richard Biener
On Mon, 12 Oct 2015, Jan Hubicka wrote: > > Honza, > > > this is a variant of patch I commited (adding the suggested predicate) > > > > This caused https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67923 > > Hmm, strange, I do not seem to be able to reproduce this locally. Is it x86? Can't reproduce

Re: Fix more of C/fortran canonical type issues

2015-10-12 Thread Jan Hubicka
> Honza, > > this is a variant of patch I commited (adding the suggested predicate) > > This caused https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67923 Hmm, strange, I do not seem to be able to reproduce this locally. Is it x86? /opt/gcc/_clean/gcc/testsuite/gfortran.dg/pr56015.f90:12:0: error: t