Re: [rfc] gcc trunk - libgomp thread affinity for freebsd

2015-05-07 Thread Jakub Jelinek
On Wed, May 06, 2015 at 04:29:02PM -0700, Adrian Chadd wrote: This patch implements basic top level thread affinity support for freebsd. It doesn't yet implement thread affinity support for core/socket grouping yet; I'm working on a library to extract that out to userland and plan on teaching

[Cec-weeklies] ST40 tools #1517,#1518,#1519

2015-05-07 Thread Christian Bruel
Highlights --- - Beta Breakpad delived - Upstreaming (almost accepted) Issues --- - Current FSF trunk boostrap broken (delay validations) Completed -- - Fixed FSF attribute optimize inlining - Fixed FSF bad handling of ipa-cp (inlining of functions with constant

Re: [PATCH] Fix(?) PR66002

2015-05-07 Thread Richard Biener
On Mon, 4 May 2015, Richard Biener wrote: This fixes a missed vectorization of a function in paq8p. Without merged PHI nodes phiopt doesn't recognize adjacent MIN/MAX_EXPRs. Certainly no other pass I schedule mergephi over cares for merged PHIs (DCE might even be confused here).

[nvptx] Re: Mostly rewrite genrecog

2015-05-07 Thread Thomas Schwinge
Hi! On Mon, 27 Apr 2015 11:20:30 +0100, Richard Sandiford richard.sandif...@arm.com wrote: I think it's been the case for a while that parallel builds of GCC tend to serialise around the compilation of insn-recog.c, especially with higher --enable-checking settings. This patch tries to speed

RE: [patch 1/28] top-level: Use automake-1.11.6

2015-05-07 Thread Bernd Edlinger
Hi, On Wed, 6 May 2015 15:55:57, Joseph S. Myers wrote: On Wed, 6 May 2015, Michael Haubenwallner wrote: Am 2015-05-05 um 18:03 schrieb Michael Haubenwallner: Now that gcc-5 is out, what about an automake-1.11.6 update for gcc-6? Patch for top-level files. I don't think this top-level

Re: [PATCH, ARM] attribute target (thumb,arm) [4/6] respin (4th)

2015-05-07 Thread Christian Bruel
+ Sandra's doc review fixes. tested with make doc pdf thanks Christian On 05/06/2015 04:24 PM, Christian Bruel wrote: Implements and document the hooks to support target_attributes. The emission of blx is handled directly for armv5 to overcome a bug with the current binutils that fails

Re: [Cec-weeklies] ST40 tools #1517,#1518,#1519

2015-05-07 Thread Christian Bruel
please ignore On 05/07/2015 10:43 AM, Christian Bruel wrote: Highlights --- - Beta Breakpad delived - Upstreaming (almost accepted) Issues --- - Current FSF trunk boostrap broken (delay validations) Completed -- - Fixed FSF attribute optimize inlining -

Re: [nvptx] Re: Mostly rewrite genrecog

2015-05-07 Thread Jakub Jelinek
On Thu, May 07, 2015 at 10:59:01AM +0200, Thomas Schwinge wrote: build/genrecog [...]/source-gcc/gcc/common.md [...]/source-gcc/gcc/config/nvptx/nvptx.md \ insn-conditions.md tmp-recog.c -[...]/source-gcc/gcc/config/nvptx/nvptx.md:1206: warning: operand 0 missing

Re: [AArch64][PR65375] Fix RTX cost for vector SET

2015-05-07 Thread James Greenhalgh
On Wed, May 06, 2015 at 03:12:33AM +0100, Kugan wrote: gcc/ChangeLog: 2015-04-24 Kugan Vivekanandarajah kug...@linaro.org Jim Wilson jim.wil...@linaro.org * config/arm/aarch-common-protos.h (struct mem_cost_table): Added new fields loadv and storev. *

Re: [PATCH, ARM] attribute target (thumb,arm) [1/6] respin (4th)

2015-05-07 Thread Ramana Radhakrishnan
2014-09-23 Christian Bruel christian.br...@st.com * config/arm/arm.h (arm_option_override): Reoganized and split. Reorganized and split into (arm_option_params_internal); New function. s/;/: New function. (arm_option_check_internal): New function.

Re: [PATCH, ARM] attribute target (thumb,arm) [3/6] respin (4th)

2015-05-07 Thread Ramana Radhakrishnan
On 06/05/15 15:22, Christian Bruel wrote: Re-implement ARM_DECLARE_FUNCTION_NAME as a function. That will make changed related to unified/divided and mode directives easier to insert. Patch could be smaller as below. Thanks Christian 2014-09-23 Christian Bruel christian.br...@st.com

[PATCH, alpha]: Fix PR66015, align directives not propagated after __attribute__ ((__optimize__ (O2)))

2015-05-07 Thread Uros Bizjak
Hello! 2015-05-07 Uros Bizjak ubiz...@gmail.com PR target/66015 * config/alpha/alpha.c (alpha_override_options_after_change): New. (TARGET_OVERRIDE_OPTIONS_AFTER_CHANGE): New. (alpha_override_options): Move align_loops, align_jumps and align_functions handling into

Re: [PATCH] Make mergeable read-only sections per-function, if requested.

2015-05-07 Thread Marek Polacek
On Wed, May 06, 2015 at 03:59:33PM -0700, Segher Boessenkool wrote: Currently GCC does not put mergeable read-only data in a per-function section, so the -gc-sections linker option does not do much for such data. Fix that. Dunno if it is a good idea, but... diff --git a/gcc/varasm.c

[Patch, fortran, pr65894, v1] [6 Regression] severe regression in gfortran 6.0.0

2015-05-07 Thread Andre Vehreschild
Hi all, my work on pr60322 caused a regression on trunk. This patch fixes it. The regression had two causes: 1. Not taking the correct attribute for BT_CLASS objects with allocatable components into account (chunk 1), and 2. taking the address of an address (chunk 2). When a class or derived

Re: [patch 0/29] RFC: Use automake-1.11.6 across the tree

2015-05-07 Thread Michael Haubenwallner
On 05/06/2015 03:01 PM, Bernd Edlinger wrote: Hi, On Tue, 5 May 2015 18:03:15, Michael Haubenwallner wrote: Now that gcc-5 is out, what about an automake-1.11.6 update for gcc-6? BTW, the actual commands I use to re-run automake for everything (I found) is: $ export

[gomp4] Merge trunk r222860 (2015-05-06) into gomp-4_0-branch

2015-05-07 Thread Thomas Schwinge
Hi! Committed to gomp-4_0-branch in r222872: commit 8c59bfcd251f08d23767789b9eae4c1ffb6e391d Merge: be14a10 09f66ac Author: tschwinge tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Thu May 7 08:14:12 2015 + svn merge -r 222446:222860 svn+ssh://gcc.gnu.org/svn/gcc/trunk

Re: acc_on_device for device_type_host_nonshm (was: libgomp nvptx plugin: rework initialisation and support the proposed load/unload hooks) (PR65742)

2015-05-07 Thread Julian Brown
On Fri, 17 Apr 2015 15:16:19 +0200 Jakub Jelinek ja...@redhat.com wrote: On Tue, Apr 14, 2015 at 05:43:26PM +0200, Thomas Schwinge wrote: On Tue, 14 Apr 2015 15:15:02 +0100, Julian Brown jul...@codesourcery.com wrote: On Wed, 8 Apr 2015 17:58:56 +0300 Ilya Verbin iver...@gmail.com

Re: [C/C++ PATCH] Implement -Wshift-negative-value (PR c/65179)

2015-05-07 Thread Marek Polacek
On Thu, May 07, 2015 at 12:00:20PM -0600, Jeff Law wrote: OK. Please install if you haven't already. I have not, so will do momentarily. Thanks, Marek

Re: [PATCH] Expand PIC calls without PLT with -fno-plt

2015-05-07 Thread Jeff Law
On 05/06/2015 09:24 AM, Alexander Monakov wrote: On Mon, 4 May 2015, Jeff Law wrote: On 05/04/2015 11:39 AM, Jakub Jelinek wrote: On Mon, May 04, 2015 at 11:34:05AM -0600, Jeff Law wrote: On 05/04/2015 10:37 AM, Alexander Monakov wrote: This patch introduces option -fno-plt that allows to

Re: [rfc] gcc trunk - libgomp thread affinity for freebsd

2015-05-07 Thread Adrian Chadd
On 7 May 2015 at 00:57, Jakub Jelinek ja...@redhat.com wrote: On Wed, May 06, 2015 at 04:29:02PM -0700, Adrian Chadd wrote: This patch implements basic top level thread affinity support for freebsd. It doesn't yet implement thread affinity support for core/socket grouping yet; I'm working on a

RE: Fix compiler warnings

2015-05-07 Thread Aditya K
Thank you very much Jakub. -Aditya Date: Thu, 7 May 2015 19:58:11 +0200 From: ja...@redhat.com To: hiradi...@msn.com CC: gcc-patches@gcc.gnu.org Subject: Re: Fix compiler warnings On Wed, May 06, 2015 at 04:37:49PM +, Aditya K wrote: Thanks!

Re: Fix compiler warnings

2015-05-07 Thread Jakub Jelinek
On Wed, May 06, 2015 at 04:37:49PM +, Aditya K wrote: Thanks! Updated patch. 2015-05-06  Aditya Kumar  hiradi...@msn.com     * gcov-tool.c (do_merge): Refactored to remove int ret.     * ipa-icf.c (sem_item::hash_referenced_symbol_properties): Changed (!type == FUNC) to

Re: [PATCH] PR debug/61352 back port from mainline

2015-05-07 Thread Joel Brobecker
The attached patch is a back port of the change from https://gcc.gnu.org/viewcvs/gcc?view=revisionrevision=211067 for gcc-4_9-branch. Bootstrap and regression tested on x86_64-apple-darwin14 with Xcode 6.3. Okay for gcc-4_9-branch? Jack PR61352_backport.diff Ok.

RFC: Patch to align spills beyond what the stack supports

2015-05-07 Thread Steve Ellcey
I would like to get some feedback on an idea of how to spill registers that require (or perhaps only prefer for performance reasons) an alignment greater than that supported by the stack. If you look at how GCC supports local variables with alignment requirements greater than the stack supports,

Re: [PATCH 0/13] Add musl support to GCC

2015-05-07 Thread Jeff Law
On 05/06/2015 05:36 AM, Szabolcs Nagy wrote: On 30/04/15 00:18, Joseph Myers wrote: On Wed, 29 Apr 2015, Szabolcs Nagy wrote: only affects [u]int_fastN_t types (on 64bit systems for N=16,32 musl uses int but glibc uses long) i can fix glibc-stdint.h, but it's yet another way in which the

Re: [PATCH] Expand PIC calls without PLT with -fno-plt

2015-05-07 Thread H.J. Lu
On Thu, May 7, 2015 at 11:22 AM, Jeff Law l...@redhat.com wrote: On 05/06/2015 09:24 AM, Alexander Monakov wrote: On Mon, 4 May 2015, Jeff Law wrote: On 05/04/2015 11:39 AM, Jakub Jelinek wrote: On Mon, May 04, 2015 at 11:34:05AM -0600, Jeff Law wrote: On 05/04/2015 10:37 AM, Alexander

Re: Patch: Refactor number_of_iterations_exit

2015-05-07 Thread Richard Biener
On May 7, 2015 6:27:28 PM GMT+02:00, Aditya K hiradi...@msn.com wrote: Hi, I refactored number_of_iterations_exit a little bit. I hope it is helpful. The idea is to move the call to function dominated_by_p after some sanity checks so as to avoid call to it. The call is very cheap though. Do you

Re: [PATCH 2/13] musl libc config

2015-05-07 Thread Jeff Law
On 05/06/2015 05:24 AM, Szabolcs Nagy wrote: On 29/04/15 00:30, Joseph Myers wrote: On Mon, 20 Apr 2015, Szabolcs Nagy wrote: * config/linux.opt (mmusl): New option. New -m options need documenting in invoke.texi. Patch v3. Now with documentation in invoke.texi. Based on

Re: [libgomp, testsuite] Support parallel testing in libgomp (PR libgomp/66005)

2015-05-07 Thread Mike Stump
On May 7, 2015, at 4:39 AM, Jakub Jelinek ja...@redhat.com wrote: On Thu, May 07, 2015 at 01:26:57PM +0200, Rainer Orth wrote: As reported in the PR, with the addition of all those OpenACC tests, libgomp make check times have skyrocketed since the testsuite is still run sequentially. Even

Re: C/C++ PATCH to allow deprecating enum values (PR c/47043)

2015-05-07 Thread Marek Polacek
On Thu, May 07, 2015 at 11:23:51AM -0600, Sandra Loosemore wrote: How about making the new Enumerator Attributes node a subsection of the Type Attributes section, instead of a section of its own at the same level? Sorry, I don't particularly like this idea. I think an enumerator is not

Re: [PATCH diagnostics/fortran] Handle two locations for the same diagnostic. Convert all gfc_warning_1 and gfc_notify_std_1 calls

2015-05-07 Thread Dodji Seketeli
Hello Manuel, Sorry for my late reply, and thank you very much for working on this. I have looked at the patch and I like it! I guess I just have some few lateral nits to pick. The Fortran FE allows diagnostics with two different locations. Depending on whether these locations are on the

[RFC 0/6] Flags outputs for asms

2015-05-07 Thread Richard Henderson
Here's a prototype for i386 only, which stands up to light testing. I'd rather post this tonight rather than wait until tomorrow when I can write more proper dejagnu tests. I've tested the intermedate patches via config-list.mk, so despite mucking around with vec.h vs target.h, all targets still

[PATCH 1/6] Only resolve_asm_operand_names once

2015-05-07 Thread Richard Henderson
We do it in the front end already; no need to repeat. --- gcc/cfgexpand.c | 2 -- gcc/stmt.c | 7 --- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/gcc/cfgexpand.c b/gcc/cfgexpand.c index 5905ddb..c77a4ac 100644 --- a/gcc/cfgexpand.c +++ b/gcc/cfgexpand.c @@ -2554,8

[patch 9/10] debug-early merge: testsuite changes

2015-05-07 Thread Aldy Hernandez
gcc/testsuite/ * g++.dg/cpp0x/pr57101.C: Add warning. * g++.dg/debug/dwarf2/dwarf4-typedef.C: Same. * g++.dg/ext/visibility/anon7.C: Same. * g++.dg/ipa/pr63587-2.C: Same. * g++.dg/opt/dump1.C: Same. * g++.dg/opt/pr59622-3.C: Same. * g++.dg/opt/pr59622.C: Same. * g++.dg/torture/pr46383.C:

[patch 10/10] debug-early merge: compiler proper

2015-05-07 Thread Aldy Hernandez
As seen on TV. gcc/ * dwarf2out.c: Remove deferred_locations*. (dwarf2_debug_hooks): Add early_finish hook. Remove global_decl hook. Add early_global_decl and late_global_decl hook. Add dumped_early bit to die_struct. New global early_dwarf_dumping. (struct limbo_die_struct): Document

[PATCH, doc] fix match-and-simplify API doc errors

2015-05-07 Thread Jim Wilson
I noticed this while reading the match-and-simplify docs. The gimple_simplify API list has two built-in function cases with three tree args. The last one is supposed to have four tree args for a ternary function (3 inputs and one output). Similarly, in the gimple_build API list, the built-in

[patch 1/10] debug-early merge: Ada front-end

2015-05-07 Thread Aldy Hernandez
gcc/ada/ * gcc-interface/gigi.h (note_types_used_by_globals): New prototype. * gcc-interface/misc.c (gnat_parse_file): Call note_types_used_by_globals. * gcc-interface/utils.c: Remove global_decls. (gnat_pushdecl): Remove use of global_decls. (gnat_write_global_declarations): Rename to

[patch 0/10] debug-early merge

2015-05-07 Thread Aldy Hernandez
Hi folks. I have divided the patches into 10 pieces. The patches are interdependent and cannot be applied independently. I am merely dividing them up to aid the relevant reviewers. As I've mentioned elsewhere, the patchset as posted has been bootstrapped and GCC tested on:

[patch 3/10] debug-early merge: C++ front-end

2015-05-07 Thread Aldy Hernandez
gcc/cp/ * cp-objcp-common.c: Adjust comment for cxx_warn_unused_global_decl. * cp-objcp-common.h (LANG_HOOKS_WRITE_GLOBALS): Remove (LANG_HOOKS_POST_COMPILATION_PARSING_CLEANUPS): New. * cp-tree.h (note_mangling_alias): Protoize. (cp_write_global_declarations): Remove.

[patch 2/10] debug-early merge: C front-end (include c-family/)

2015-05-07 Thread Aldy Hernandez
gcc/c-family/ * c-common.h (c_parse_final_cleanups): New prototype. * c-opts.c (c_common_parse_file): Call c_parse_final_cleanups. gcc/c/ * c-decl.c (finish_struct): Save C_TYPE_INCOMPLETE_VARS and immediately clobber it. (c_write_global_declarations_1): Remove call to

[patch 8/10] debug-early merge: Objective-C front-end

2015-05-07 Thread Aldy Hernandez
gcc/objc/ * objc-act.c (objc_write_global_declarations): Adjust comment. * objc-gnu-runtime-abi-01.c (build_selector_table_decl): Set TREE_USED bit. diff --git a/gcc/objc/objc-act.c b/gcc/objc/objc-act.c index 718fdf0..3528c89 100644 --- a/gcc/objc/objc-act.c +++ b/gcc/objc/objc-act.c @@

[patch 7/10] debug-early merge: LTO

2015-05-07 Thread Aldy Hernandez
gcc/lto/ * lto-lang.c (lto_write_globals): Remove. (LANG_HOOKS_WRITE_GLOBALS): Same. diff --git a/gcc/lto/lto-lang.c b/gcc/lto/lto-lang.c index 073bf17..094ed1f 100644 --- a/gcc/lto/lto-lang.c +++ b/gcc/lto/lto-lang.c @@ -1116,19 +1116,6 @@ lto_getdecls (void) return NULL_TREE; }

Re: [RFC 0/6] Flags outputs for asms

2015-05-07 Thread H. Peter Anvin
On 05/07/2015 02:38 PM, Richard Henderson wrote: Here's a prototype for i386 only, which stands up to light testing. I'd rather post this tonight rather than wait until tomorrow when I can write more proper dejagnu tests. I've tested the intermedate patches via config-list.mk, so despite

Re: [RFC 0/6] Flags outputs for asms

2015-05-07 Thread H. Peter Anvin
This is a separate issue which really shouldn't have anything to do with this, but is there a specific reason why: void good1(int x, int y) { _Bool pf; asm(cmpl %2,%1 : =@ccp (pf) : r (x), g (y)); if (pf) beta(); } ... ends up generating a jump to a jump?

[patch 6/10] debug-early merge: Java front-end

2015-05-07 Thread Aldy Hernandez
gcc/java/ * class.c: Remove pending_static_fields. (add_field): Remove use of pending_static_fields. (java_write_globals): Remove. * decl.c (java_mark_class_local): Remove use of pending_static_fields. * java-tree.h: Remove pending_static_fields and java_write_globals. * jcf-parse.c

[patch 4/10] debug-early merge: Fortran front-end

2015-05-07 Thread Aldy Hernandez
gcc/fortran/ * f95-lang.c (gfc_write_global_declarations): Remove. (LANG_HOOKS_WRITE_GLOBALS): Remove. (gfc_write_global_declarations): Move code from here to... (gfc_be_parse_file): ...here. Call global_decl_processing. * trans-decl.c (gfc_emit_parameter_debug_info): Rename global_decl

[patch 5/10] debug-early merge: Go front-end

2015-05-07 Thread Aldy Hernandez
gcc/go/ * go-gcc.cc (write_global_definitions): Remove call to finalize_compilation_unit. Remove Go specific debug generation. * go-lang.c (go_langhook_parse_file): Call go_write_globals. (go_langhook_write_globals): Remove. Remove LANG_HOOKS_WRITE_GLOBALS everywhere. diff --git

[PATCH 6/6] i386: Implement asm flag outputs

2015-05-07 Thread Richard Henderson
All jcc mnemonics implemented as =@ to make it easy for someone reading the manual to figure out what condition is desired. --- gcc/config/i386/constraints.md | 5 ++ gcc/config/i386/i386.c | 132 ++--- 2 files changed, 130 insertions(+), 7

[PATCH 5/6] i386: Add CCPmode

2015-05-07 Thread Richard Henderson
For testing parity coming out of asm flags. --- gcc/config/i386/i386-modes.def | 2 ++ gcc/config/i386/i386.c | 19 +++ 2 files changed, 13 insertions(+), 8 deletions(-) diff --git a/gcc/config/i386/i386-modes.def b/gcc/config/i386/i386-modes.def index 0b6a1f1..714c138

[PATCH 2/6] Merge expand_asm_operands into expand_asm_stmt

2015-05-07 Thread Richard Henderson
Prepatory to converting from tree chains to vectors. --- gcc/cfgexpand.c | 167 1 file changed, 72 insertions(+), 95 deletions(-) diff --git a/gcc/cfgexpand.c b/gcc/cfgexpand.c index c77a4ac..a5de512 100644 --- a/gcc/cfgexpand.c +++

[PATCH 3/6] Canonicalize asm volatility earlier

2015-05-07 Thread Richard Henderson
If gimple_asm_volatile_p is correct, no point re-checking. This is also done by the C and C++ front ends, but not Ada. So we can't yet trust ASM_VOLATILE_P from the front end. --- gcc/cfgexpand.c | 11 +++ gcc/gimplify.c | 2 +- 2 files changed, 4 insertions(+), 9 deletions(-) diff

Re: [PATCH, ARM] attribute target (thumb,arm) [2/6] respin (4th)

2015-05-07 Thread Ramana Radhakrishnan
On 06/05/15 15:20, Christian Bruel wrote: In preparation of the pragma target reorganize ÂTARGET_CPU_CPP_BUILTINSÂ to redefine mode dependent macros based on current thumb_p. I'm not entirely happy with this patch as it appears to be too tied to just the thumbness of the attributes.

Re: [PATCH] Fix OpenACC shutdown and PTX image unloading (PR65904)

2015-05-07 Thread Julian Brown
On Wed, 6 May 2015 10:32:56 +0200 Thomas Schwinge tho...@codesourcery.com wrote: Hi! On Fri, 1 May 2015 10:47:19 +0100, Julian Brown jul...@codesourcery.com wrote: The patch also fixes a thinko that was revealed in image unloading in the NVPTX backend. Tested for libgomp with PTX

RE: [RFC]: Remove Mem/address type assumption in combiner

2015-05-07 Thread Kumar, Venkataramanan
Hi Segher, Thank you I committed as r222874. Ref: https://gcc.gnu.org/viewcvs/gcc?view=revisionrevision=222874 Regards, Venkat. -Original Message- From: Segher Boessenkool [mailto:seg...@kernel.crashing.org] Sent: Tuesday, May 05, 2015 10:46 PM To: Kumar, Venkataramanan Cc: Jeff Law

Re: [PATCH] Fix eipa_sra AAPCS issue (PR target/65956)

2015-05-07 Thread Jakub Jelinek
On Thu, May 07, 2015 at 12:16:32PM +0100, Alan Lawrence wrote: So for my two cents, or perhaps three: (1) It'd be great to have something in the documentation for TARGET_FUNCTION_ARG that explains what the contract for the type information provided is. Even/especially if some of this is

Re: [PATCH] Make mergeable read-only sections per-function, if requested.

2015-05-07 Thread Segher Boessenkool
On Thu, May 07, 2015 at 10:15:08AM +0200, Marek Polacek wrote: +const char * +function_mergeable_rodata_prefix (void) This function needs a comment. And much more importantly, I forgot to make it static. Fixed. +{ + section *s = targetm.asm_out.function_rodata_section

Re: [PATCH] Fix eipa_sra AAPCS issue (PR target/65956)

2015-05-07 Thread Alan Lawrence
Richard Biener wrote: On May 5, 2015 4:33:58 PM GMT+02:00, Richard Earnshaw richard.earns...@foss.arm.com wrote: On 05/05/15 15:33, Richard Earnshaw wrote: On 05/05/15 15:29, Jakub Jelinek wrote: On Tue, May 05, 2015 at 02:20:43PM +0100, Richard Earnshaw wrote: On 05/05/15 14:06, Jakub

Remove mode argument from gen_rtx_SET

2015-05-07 Thread Richard Sandiford
One problem with the automatically-generated gen_rtx_FOO () macros is that they always have a mode parameter, even for codes like SET where the mode should always be VOIDmode. This inevitably leads to cases where a caller accidentally passes something other than VOIDmode. E.g. when expanding an

Re: [PATCH, ARM] attribute target (thumb,arm) [2/6] respin (4th)

2015-05-07 Thread Christian Bruel
On 05/07/2015 10:49 AM, Ramana Radhakrishnan wrote: On 06/05/15 15:20, Christian Bruel wrote: In preparation of the pragma target reorganize ÂTARGET_CPU_CPP_BUILTINSÂ to redefine mode dependent macros based on current thumb_p. I'm not entirely happy with this patch as it appears to

Re: [libgomp, testsuite] Support parallel testing in libgomp (PR libgomp/66005)

2015-05-07 Thread Jakub Jelinek
On Thu, May 07, 2015 at 01:26:57PM +0200, Rainer Orth wrote: As reported in the PR, with the addition of all those OpenACC tests, libgomp make check times have skyrocketed since the testsuite is still run sequentially. Even on a reasonably fast x86 machine (4 x 2.0 Ghz Xeon E7450) the run

Fix ubsan tests on ARM (PR testsuite/66046)

2015-05-07 Thread Marek Polacek
Trailing \[^\n\r]*(\n|\r\n|\r) in the last dg-output aroused dismay on ARM. Here's a patch to clean this up; can you please test this on ARM? make check-g{cc,++} RUNTESTFLAGS='--target_board=unix\{-m32,-m64\} ubsan.exp' passes on x86_64-linux. 2015-05-07 Marek Polacek pola...@redhat.com

Re: C/C++ PATCH to allow deprecating enum values (PR c/47043)

2015-05-07 Thread Marek Polacek
[ CC'ing Sandra: since you were recently cleaning up the attributes docs (thanks), the doc/extend.texi bits in this patch might be of interest to you. ] On Wed, May 06, 2015 at 08:44:10PM +0200, Marek Polacek wrote: 2015-05-06 Marek Polacek pola...@redhat.com PR c/47043 *

[libgomp, testsuite] Support parallel testing in libgomp (PR libgomp/66005)

2015-05-07 Thread Rainer Orth
As reported in the PR, with the addition of all those OpenACC tests, libgomp make check times have skyrocketed since the testsuite is still run sequentially. Even on a reasonably fast x86 machine (4 x 2.0 Ghz Xeon E7450) the run takes 4286 seconds. On slower sparc boxes (1.2 GHz UltraSPARC-T2)

C++ PATCH for c++/59012 (alignas pack expansion)

2015-05-07 Thread Jason Merrill
When we added support for C++11 attributes, we mentioned pack expansions in the comments but didn't do anything to support them in the code. I'm not sure how that happened. Anyway, this patch adds that support. Tested x86_64-pc-linux-gnu, applying to trunk. commit

RE: [patch, avr] extend part-clobbered check to AVR_TINY architecture

2015-05-07 Thread Sivanupandi, Pitchumani
Ping! -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Sivanupandi, Pitchumani Sent: Tuesday, April 21, 2015 8:21 PM To: Georg-Johann Lay; Denis Chertykov Cc: GCC Patches Subject: [patch, avr] extend part-clobbered check to

Re: Remove mode argument from gen_rtx_SET

2015-05-07 Thread Jeff Law
On 05/07/2015 05:37 AM, Richard Sandiford wrote: One problem with the automatically-generated gen_rtx_FOO () macros is that they always have a mode parameter, even for codes like SET where the mode should always be VOIDmode. This inevitably leads to cases where a caller accidentally passes

Re: Vectorize stores with unknown stride

2015-05-07 Thread Richard Biener
On Wed, May 6, 2015 at 5:37 PM, Michael Matz m...@suse.de wrote: Hi, I'm sitting on this since quite some time already and always missed stage 1. This implements support for vectorizing strided stores with unknown but loop invariant stride, like: sumit (float * __restrict dest,

Re: C/C++ PATCH to allow deprecating enum values (PR c/47043)

2015-05-07 Thread Ed Smith-Rowland
On 05/07/2015 09:59 AM, Marek Polacek wrote: On Wed, May 06, 2015 at 11:17:20PM -0400, Ed Smith-Rowland wrote: In addition to a PR this is 1/2 of a C=+17 feature. (The other half - really a separate thing - is attributes on namespaces). Ah, nice, I wasn't aware. For the record, this is

Re: Remove mode argument from gen_rtx_SET

2015-05-07 Thread Richard Sandiford
Jeff Law l...@redhat.com writes: On 05/07/2015 05:37 AM, Richard Sandiford wrote: One problem with the automatically-generated gen_rtx_FOO () macros is that they always have a mode parameter, even for codes like SET where the mode should always be VOIDmode. This inevitably leads to cases

Re: C/C++ PATCH to allow deprecating enum values (PR c/47043)

2015-05-07 Thread Marek Polacek
On Wed, May 06, 2015 at 11:17:20PM -0400, Ed Smith-Rowland wrote: In addition to a PR this is 1/2 of a C=+17 feature. (The other half - really a separate thing - is attributes on namespaces). Ah, nice, I wasn't aware. For the record, this is

Re: [PATCH] Make mergeable read-only sections per-function, if requested.

2015-05-07 Thread Jeff Law
On 05/07/2015 02:15 AM, Marek Polacek wrote: On Wed, May 06, 2015 at 03:59:33PM -0700, Segher Boessenkool wrote: Currently GCC does not put mergeable read-only data in a per-function section, so the -gc-sections linker option does not do much for such data. Fix that. Dunno if it is a good

Minor operand shortening patch reapplied

2015-05-07 Thread Jeff Law
I found the i686 bootstrap comparison failure reported by HJ after the minor improvement to match.pd was installed. Basically we were violating the ODR via inline functions in *-match-head.c. In an unoptimized compile, those routines would not be inlined and had implicit external linkage

Re: [PATCH] Make mergeable read-only sections per-function, if requested.

2015-05-07 Thread Jeff Law
On 05/07/2015 06:30 AM, Segher Boessenkool wrote: On Thu, May 07, 2015 at 10:15:08AM +0200, Marek Polacek wrote: +const char * +function_mergeable_rodata_prefix (void) This function needs a comment. And much more importantly, I forgot to make it static. Fixed. +{ + section *s =

Re: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option

2015-05-07 Thread Joseph Myers
On Fri, 6 Mar 2015, H.J. Lu wrote: +# We don't want to compile the compiler with -fPIE, it make PCH fail. +COMPILER += @NO_PIE_CFLAGS@ + +# Link with -no-pie since we compile the compiler with -fno-PIE. +LINKER += @NO_PIE_FLAG@ As I understand it, what we don't want is the compiler to be a

[PATCH 4/6] Convert to md_asm_adjust

2015-05-07 Thread Richard Henderson
Using proper vectors instead of lists of trees. --- gcc/cfgexpand.c | 614 --- gcc/config/cris/cris.c | 88 +++ gcc/config/i386/i386.c | 24 +- gcc/config/mn10300/mn10300.c | 20 +- gcc/config/rs6000/rs6000.c | 17 +-

Re: [patch 3/28] fixincludes: Use automake-1.11.6 (across the tree)

2015-05-07 Thread Bruce Korb
On 05/06/15 01:58, Michael Haubenwallner wrote: Trivial patch for fixincludes. A) sufficiently trivial that explicit permission ought not be required B) it is now officially blessed that we can coalesce year lists. Let's do so, okay? Am 2015-05-05 um 18:03 schrieb Michael Haubenwallner:

Re: [PATCH][DRIVER] Wrong C++ include paths when configuring with --with-sysroot=/

2015-05-07 Thread Joseph Myers
On Mon, 20 Apr 2015, Pavel Kopyl wrote: Hi all, To build a GCC-4.9.2 ARM cross-compiler for my setting I need to configure it with --with-sysroot=/ --with-gxx-include-dir=/usr/include/c++/4.9.2. But I found that gcc driver removes the leading slash from resulting paths: `gcc

Re: C PATCH to add -Woverride-init-side-effects (PR c/64918)

2015-05-07 Thread Joseph Myers
On Fri, 24 Apr 2015, Marek Polacek wrote: As discussed in the PR, the initialized field with side-effects overwritten warning is sometimes not so useful, so it probably makes sense to provide an option so that users are able to specifically enable/disable it. Since the warning is enabled by

RE: Patch: Refactor number_of_iterations_exit

2015-05-07 Thread Aditya K
Subject: Re: Patch: Refactor number_of_iterations_exit From: richard.guent...@gmail.com Date: Thu, 7 May 2015 19:47:37 +0200 To: hiradi...@msn.com; gcc-patches@gcc.gnu.org On May 7, 2015 6:27:28 PM GMT+02:00, Aditya K hiradi...@msn.com wrote: Hi, I

[PATCH, ARM, doc] add missing -mtune options

2015-05-07 Thread Jim Wilson
I noticed that the list of -mtune options in the arm-cores.def file didn't match the list in the doc/invoke.texi file. There are 3 cores missing: generic-armv7-a, cortex-a17, and cortex-a17.cortex-a7. This patch adds the missing cores to the docs. Jim 2015-05-07 Jim Wilson

Re: update docs for --enable-languages

2015-05-07 Thread Jim Wilson
ping https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01690.html Jim On Mon, Apr 27, 2015 at 5:09 PM, Jim Wilson jim.wil...@linaro.org wrote: I noticed this while working on my mostlyclean patch. The list of languages in the docs for --enable-languages is incomplete. It is missing jit and

Re: Fix PR48052: loop not vectorized if index is unsigned int

2015-05-07 Thread Abderrazek Zaafrani
Richard, Agree that the code is handling a very special case but this special case is common enough and is limiting the vectorizer in a significant way. The special case is: loops with unsigned index, non-constant start value, and step 1. We have a code for a matrix multiply – loops blocked by

Re: [PATCH, stage1] Better error recovery for merge-conflict markers

2015-05-07 Thread Joseph Myers
On Fri, 17 Apr 2015, David Malcolm wrote: gcc/c-family/ChangeLog: * c-common.h (conflict_marker_get_final_tok_kind): New prototype. * c-lex.c (conflict_marker_get_final_tok_kind): New function. gcc/c/ChangeLog: * c-parser.c (struct c_parser): Expand array tokens_buf from

Re: Fix ubsan tests on ARM (PR testsuite/66046)

2015-05-07 Thread Jakub Jelinek
On Thu, May 07, 2015 at 05:35:30PM +0300, Maxim Ostapenko wrote: This works fine for me on target ARM board, thanks for fixing this! -Maxim On 07/05/15 13:41, Marek Polacek wrote: Trailing \[^\n\r]*(\n|\r\n|\r) in the last dg-output aroused dismay on ARM. Here's a patch to clean this up;

Re: [debug-early] call debug_hooks-late_global_decl for optimized away symbols

2015-05-07 Thread Aldy Hernandez
And now for my next trick, I will revert all changes I had made to the non-dwarf backends. No special kludges will be needed for stabs et al, with my previous patch in place. Tested on: x86_64-unknown-linux-gnu powerpc-ibm-aix7.1.2.0 powerpc64-unknown-linux-gnu

RE: [patch 1/28] top-level: Use automake-1.11.6

2015-05-07 Thread Joseph Myers
On Thu, 7 May 2015, Bernd Edlinger wrote: But that is not the case for other tool scripts.  I think these should be in-sync with the automake version that creates the configure scripts that make use of them. At least some of these scripts are also usable other than from automake-generated

[gomp4.1] Taskloop construct lowering and expansion

2015-05-07 Thread Jakub Jelinek
Hi! This patch gimplifies OMP_TASKLOOP as 3 different gimple statements - two GIMPLE_OMP_FOR with GIMPLE_OMP_TASK sandwiched in between them, so that we can represent properly the effect of what happens in the GOMP_taskloop caller (needs to compute number of iterations, prepare counts of

Re: OpenACC: initialization with unsupported acc_device_t (was: [PR testsuite/65205, libgomp/65993] Fix dg-shouldfail usage in OpenACC libgomp tests)

2015-05-07 Thread Julian Brown
On Tue, 5 May 2015 16:09:18 +0200 Thomas Schwinge tho...@codesourcery.com wrote: Hi! On Tue, 5 May 2015 08:43:48 -0400, John David Anglin dave.ang...@bell.net wrote: On 2015-05-05 5:43 AM, Thomas Schwinge wrote: FAIL: libgomp.oacc-c/../libgomp.oacc-c-c++-common/lib-62.c

Re: [PATCH] Make mergeable read-only sections per-function, if requested.

2015-05-07 Thread Segher Boessenkool
On Thu, May 07, 2015 at 07:31:33AM -0600, Jeff Law wrote: If you could cobble together a little test (ppc specific is fine with me) it'd be appreciated. With that, approved for the trunk. I did one for *-*-linux*. Tested on powerpc64-linux and x86_64-linux. Committed. Segher 2015-05-07

Re: [patch 0/29] RFC: Use automake-1.11.6 across the tree

2015-05-07 Thread Michael Haubenwallner
On 05/06/2015 04:09 PM, Jeff Law wrote: On 05/05/2015 10:03 AM, Michael Haubenwallner wrote: Hello build machinery maintainers, following up http://thread.gmane.org/gmane.comp.gcc.patches/331902/focus=334462 http://thread.gmane.org/gmane.comp.gcc.patches/332160 [ ... snip ... ] Even if I

Re: Vectorize stores with unknown stride

2015-05-07 Thread Michael Matz
On Thu, 7 May 2015, Alan Lawrence wrote: Also update comment? (5 identical cases) Also update comment? Obviously a good idea, thanks :) (s/loads/accesses/ for the commit) @@ -5013,7 +5025,7 @@ vectorizable_store (gimple stmt, gimple_stmt_iterator *gsi, gimple *vec_stmt, tree

Re: [AArch64/GCC] PR64304, miscompilation with -mgeneral-regs-only

2015-05-07 Thread Richard Earnshaw
On 14/01/15 15:31, Jiong Wang wrote: as discussed here https://gcc.gnu.org/ml/gcc-patches/2015-01/msg00563.html the problem is aarch64 hardware only support left shift truncation for SI/DI, while SHIFT_COUNT_TRUNCATED is enabled for all mode including QI/HI, which is inconsistent with

Re: Remove mode argument from gen_rtx_SET

2015-05-07 Thread Jeff Law
On 05/07/2015 07:59 AM, Richard Sandiford wrote: Jeff Law l...@redhat.com writes: On 05/07/2015 05:37 AM, Richard Sandiford wrote: One problem with the automatically-generated gen_rtx_FOO () macros is that they always have a mode parameter, even for codes like SET where the mode should always

Re: [patch 4/28] gotools: Use automake-1.11.6 (across the tree)

2015-05-07 Thread Ian Lance Taylor
The patches to gotools and libbacktrace are fine if the rest of the patches are approved. I will handle the patch to libgo, which is copied from a different repository. Please do not commit it yourself. Thanks. Ian Michael Haubenwallner michael.haubenwall...@ssi-schaefer.com writes: Patch

Re: Fix ubsan tests on ARM (PR testsuite/66046)

2015-05-07 Thread Maxim Ostapenko
This works fine for me on target ARM board, thanks for fixing this! -Maxim On 07/05/15 13:41, Marek Polacek wrote: Trailing \[^\n\r]*(\n|\r\n|\r) in the last dg-output aroused dismay on ARM. Here's a patch to clean this up; can you please test this on ARM? make check-g{cc,++}

Re: Vectorize stores with unknown stride

2015-05-07 Thread Alan Lawrence
(Below are all minor/style points only, no reason for patch not to go in.) Michael Matz wrote: diff --git a/gcc/tree-vect-data-refs.c b/gcc/tree-vect-data-refs.c index 96afc7a..6d8f17e 100644 --- a/gcc/tree-vect-data-refs.c +++ b/gcc/tree-vect-data-refs.c @@ -665,7 +665,7 @@

[PATCH] Fix 65697. Add memory model support for stronger __sync operations.

2015-05-07 Thread Andrew MacLeod
There has been some debate over the strength requirement of barriers for __sync operations... This is documented within the PR. Originally __sync was suppose to be synonymous with SEQ_CST, but there has been a slight slackening of the barrier-ness of SEQ_CST from the language lawyers. Under

Re: C/C++ PATCH to allow deprecating enum values (PR c/47043)

2015-05-07 Thread Marek Polacek
On Thu, May 07, 2015 at 10:21:28AM -0400, Ed Smith-Rowland wrote: Instead of NULL_TREE in pt.c I grabbed the attrs. /* Actually build the enumerator itself. */ build_enumerator (DECL_NAME (decl), value, newtag, DECL_ATTRIBUTES (decl), DECL_SOURCE_LOCATION (decl));

Patch: Refactor number_of_iterations_exit

2015-05-07 Thread Aditya K
Hi, I refactored number_of_iterations_exit a little bit. I hope it is helpful. The idea is to move the call to function dominated_by_p after some sanity checks so as to avoid call to it. Thanks, -Aditya 2015-05-07  Aditya Kumar  hiradi...@msn.com     * tree-ssa-loop-niter.c

Re: [Patch, fortran, pr65894, v1] [6 Regression] severe regression in gfortran 6.0.0

2015-05-07 Thread Mikael Morin
Le 07/05/2015 11:52, Andre Vehreschild a écrit : Hi all, my work on pr60322 caused a regression on trunk. This patch fixes it. The regression had two causes: 1. Not taking the correct attribute for BT_CLASS objects with allocatable components into account (chunk 1), and 2. taking the

  1   2   >