Re: aarch64 simd index out of range message not correct on 32 bit host

2015-05-29 Thread Andrew Pinski
On Fri, May 29, 2015 at 2:33 PM, Shiva Chen wrote: > Hi, > > I notice that aarch64 simd index range message not correct on 32 bit host. > > The message print by the function aarch64_simd_lane_bounds in aarch64.c. > > The function print HOST_WIDE_INT variable by %ld which is correct on > 64 bit hos

Re: aarch64 simd index out of range message not correct on 32 bit host

2015-05-29 Thread Shiva Chen
Hi, Andrew You are right, it's much simpler. Could you give me a tip how %wd works on GCC ? Could you send a new patch to fix the message since you have better solution ? Thanks, Shiva 2015-05-29 15:13 GMT+08:00 Andrew Pinski : > On Fri, May 29, 2015 at 2:33 PM, Shiva Chen wrote: >> Hi, >> >

Re: aarch64 simd index out of range message not correct on 32 bit host

2015-05-29 Thread Andrew Pinski
On Fri, May 29, 2015 at 3:46 PM, Shiva Chen wrote: > Hi, Andrew > > You are right, it's much simpler. > > Could you give me a tip how %wd works on GCC ? the w modifier is the GCC diagnostic format modifier that specifies HOST_WIDE_INT. > > Could you send a new patch to fix the message since you

Re: [PATCH, testsuite]: Cleanup dumps

2015-05-29 Thread Bernhard Reutner-Fischer
On 31 January 2015 at 22:10, Bernhard Reutner-Fischer wrote: > On January 31, 2015 9:17:57 PM GMT+01:00, Mike Stump > wrote: >>On Jan 31, 2015, at 11:50 AM, Bernhard Reutner-Fischer >> wrote: >>> On January 31, 2015 10:53:39 AM GMT+01:00, Uros Bizjak >> wrote: Hello! >>> >>> Reminds me of j

Re: aarch64 simd index out of range message not correct on 32 bit host

2015-05-29 Thread Shiva Chen
Hi, Andrew I modify the patch as you suggestion and testing on 32/64 bit host. Thanks your tips. I really appreciate for your help. Shiva 2015-05-29 15:57 GMT+08:00 Andrew Pinski : > On Fri, May 29, 2015 at 3:46 PM, Shiva Chen wrote: >> Hi, Andrew >> >> You are right, it's much simpler. >> >>

Re: [PATCH] Don't instrument DECL_INITIAL of statics (PR sanitizer/66190)

2015-05-29 Thread Marek Polacek
Ping. On Thu, May 21, 2015 at 09:36:59PM +0200, Marek Polacek wrote: > In this PR, we find ourselves instrumenting a static initializer and > then crashing when expanding an unlowered UBSAN_NULL. Jakub suggests > to not instrument DECL_INITIAL of a static variable. The following > patch is an at

Re: [PATCH, testsuite]: Cleanup dumps

2015-05-29 Thread Bernhard Reutner-Fischer
On 29 May 2015 at 10:21, Bernhard Reutner-Fischer wrote: > On 31 January 2015 at 22:10, Bernhard Reutner-Fischer > wrote: >> On January 31, 2015 9:17:57 PM GMT+01:00, Mike Stump >> wrote: >>>If you want to wait until stage 1, that’s fine too. >> >> I'd only want that to go in in stage 1. > > I

Re: [Patch]: libbacktrace - add support of PE/COFF

2015-05-29 Thread Tristan Gingold
> On 28 May 2015, at 17:14, Ian Lance Taylor wrote: > > On Thu, May 28, 2015 at 5:01 AM, Tristan Gingold wrote: >> >>> On 28 May 2015, at 02:26, Ian Lance Taylor wrote: >> >>> The #include will break cross-compilers. It's not OK for >>> trunk until that is fixed. >> >> I am confused by th

Re: arm memcpy of aligned data

2015-05-29 Thread Kyrill Tkachov
Hi Mike, On 28/05/15 22:15, Mike Stump wrote: So, the arm memcpy code of aligned data isn’t as good as it can be. void *memcpy(void *dest, const void *src, unsigned int n); void foo(char *dst, int i) { memcpy (dst, &i, sizeof (i)); } generates horrible code, but, it we are willing to notic

Re: [patch] fix bootstrap on FreeBSD i386/arm

2015-05-29 Thread Rainer Orth
Eric Botcazou writes: >> This patch restores bootstrap on i386-*-freebsd*. >> The build was failing after the introduction of -std=c++98 >> configure/build flag. The -std=c++98 enables strict_ansi and on FreeBSD >> the libc function atoll is not defined for this. > > Solaris (x86 and SPARC) is al

Re: [patch] fix bootstrap on FreeBSD i386/arm

2015-05-29 Thread Eric Botcazou
> Only Solaris 10, actually: Solaris 11 isn't affected since it has > clearenv(3C). That's not the only problem AFAICS: /usr/include/sys/resource.h: At global scope: /usr/include/sys/resource.h:79:25: error: declaration does not declare anything [-fpermissive] /homes/botcazou/gcc-head/src/gcc/s

Re: [patch] fix bootstrap on FreeBSD i386/arm

2015-05-29 Thread Rainer Orth
Eric Botcazou writes: >> Only Solaris 10, actually: Solaris 11 isn't affected since it has >> clearenv(3C). > > That's not the only problem AFAICS: > > /usr/include/sys/resource.h: At global scope: > /usr/include/sys/resource.h:79:25: error: declaration does not declare > anything [-fpermissive]

Re: arm memcpy of aligned data

2015-05-29 Thread Kyrill Tkachov
On 29/05/15 10:08, Kyrill Tkachov wrote: Hi Mike, On 28/05/15 22:15, Mike Stump wrote: So, the arm memcpy code of aligned data isn’t as good as it can be. void *memcpy(void *dest, const void *src, unsigned int n); void foo(char *dst, int i) { memcpy (dst, &i, sizeof (i)); } generates ho

Commit: MSP430: Add multilibs for different types of hardware multiply

2015-05-29 Thread Nick Clifton
Hi Guys, I am applying the patch below to the MSP430 backend to add new multilibs based upon the type of hardware multiply support used. Cheers Nick gcc/ChangeLog 2015-05-29 Nick Clifton * config/msp430/t-msp430 (MULTILIB_OPTIONS): Add multilibs for the different types

Re: [PATCH] Don't instrument DECL_INITIAL of statics (PR sanitizer/66190)

2015-05-29 Thread Jakub Jelinek
On Fri, May 29, 2015 at 10:41:34AM +0200, Marek Polacek wrote: > Ping. > > On Thu, May 21, 2015 at 09:36:59PM +0200, Marek Polacek wrote: > > In this PR, we find ourselves instrumenting a static initializer and > > then crashing when expanding an unlowered UBSAN_NULL. Jakub suggests > > to not in

[PATCH] Fix PR66314

2015-05-29 Thread Richard Biener
This fixes jump threading which fails to update the loop ownership of duplicated blocks (and was probably just lucky that loops were fixed up usually). The blocks of the jump thread path always belong to the loop of the final threading destination. Bootstrapped on x86_64-unknown-linux-gnu, testi

Re: debug mode maintenance patch

2015-05-29 Thread Jonathan Wakely
On 28/05/15 22:32 +0200, François Dumont wrote: Sorry, I saw it used so many times for macros that I though it was the right way to report macro modifications. The changelog style is fairly complicated :-) I also replicate Copyrights from debug.h to assertions.h. * include/debug/debug.h (

Re: [PATCH][ARM/AArch64 Testsuite] Cleanup advsimd-intrinsics.exp, removing unnecessary loop

2015-05-29 Thread Alan Lawrence
Christophe Lyon wrote: This looks OK, but why can't you also drop the other torture-related lines as you did in your previous patch? I mean: load_lib c-torture.exp load_lib torture-options.exp etc... We need c-torture.exp in order to set-torture-options; we need to set-torture-options to get

[PATCH] PR 66215: S390: Fix placement of post-label NOPs with -mhotpatch

2015-05-29 Thread Dominik Vogt
The attached patch fixes the problem reported in PR 66215. Patches for gcc-4.8, 4.9 and 5.1 will follow. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog 2015-05-29 Dominik Vogt PR target/66215 * config/s390/s390.c (s390_reorg): Fix placement of post-label N

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

2015-05-29 Thread Richard Biener
On Thu, May 28, 2015 at 10:35 PM, Jason Merrill wrote: > On 05/28/2015 02:53 PM, Aldy Hernandez wrote: >> >> On 05/27/2015 08:39 AM, Jason Merrill wrote: >>> >>> On 05/20/2015 11:50 AM, Aldy Hernandez wrote: >> >> + /* Fill in the size of variable-length fields in late dwarf. */ + if

Re: [PATCH] PR 66215: S390: Fix placement of post-label NOPs with -mhotpatch

2015-05-29 Thread Jakub Jelinek
On Fri, May 29, 2015 at 11:57:00AM +0100, Dominik Vogt wrote: > 2015-05-29 Dominik Vogt > > PR target0/66215 Please remove the 0 above. > * gcc.target/s390/hotpatch-1.c: Improve to detect multi-line patterns. > * gcc.target/s390/hotpatch-2.c: Likewise. > * gcc.target/s

Re: [Patch, fortran, PR44672, v6] [F08] ALLOCATE with SOURCE and no array-spec

2015-05-29 Thread Andre Vehreschild
Hi Mikael, comments inline below: On Thu, 28 May 2015 20:06:57 +0200 Mikael Morin wrote: > Le 28/05/2015 17:29, Andre Vehreschild a écrit : > > *** resolve_allocate_expr (gfc_expr *e, gfc_ > > *** 7103,7112 > > --- 7103,7123 > > if (!ref2 || ref2->type != REF_ARRAY ||

[PATCH] Tiny if-conv improvement (PR tree-optimization/66142)

2015-05-29 Thread Jakub Jelinek
Hi! On the following testcase from the PR, dce2 turns # .MEM_21 = PHI <.MEM_20(5), .MEM_4(D)(2)> into # .MEM_21 = PHI <.MEM_21(5), .MEM_4(D)(2)> because all writes in the loop have been dead code eliminated. But only the phicprop2 pass much later is able to optimize this out (by changing all .

Re: [PATCH] Tiny if-conv improvement (PR tree-optimization/66142)

2015-05-29 Thread Richard Biener
On Fri, 29 May 2015, Jakub Jelinek wrote: > Hi! > > On the following testcase from the PR, dce2 turns > # .MEM_21 = PHI <.MEM_20(5), .MEM_4(D)(2)> > into > # .MEM_21 = PHI <.MEM_21(5), .MEM_4(D)(2)> > because all writes in the loop have been dead code eliminated. > But only the phicprop2 pass

Re: [PATCH, CFT] Fix bootstrap with in-tree ISL

2015-05-29 Thread Bernhard Reutner-Fischer
On 1 April 2015 at 17:55, Bernhard Reutner-Fischer wrote: > On 1 April 2015 at 17:53, Tobias Grosser wrote: >> On 04/01/2015 05:09 PM, Bernhard Reutner-Fischer wrote: >>> >>> Hi, >>> >>> Trying to build config-list.mk for current trunk. >>> The box is a stable debian (7.8 it seems) so the OS' ISL

[PATCH] TLC to reassoc get_rank

2015-05-29 Thread Richard Biener
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2015-05-29 Richard Biener * tree-ssa-reassoc.c (get_rank): Simplify. Index: gcc/tree-ssa-reassoc.c === --- gcc/tree-ssa-reassoc.c (revision 22

[PATCH] Fix PR66280

2015-05-29 Thread Richard Biener
In addition to the required backport of PR66251 (testing right now) we need to fix hybrid stmt detection for trunk and for a related testcase also on the branch. Bootstrap and regtest running on x86_64-unknown-linux-gnu. Richard. 2015-05-29 Richard Biener PR tree-optimization/66280

Re: [PATCH] TLC to reassoc get_rank

2015-05-29 Thread Jakub Jelinek
On Fri, May 29, 2015 at 02:41:28PM +0200, Richard Biener wrote: > @@ -525,7 +498,11 @@ get_rank (tree e) >return (rank + 1); > } > > - /* Globals, etc, are rank 0 */ > + /* Constants have rank 0. */ > + if (is_gimple_min_invariant (e)) > +return 0; > + > + /* Constants, glo

Re: [PATCH] TLC to reassoc get_rank

2015-05-29 Thread Richard Biener
On Fri, 29 May 2015, Jakub Jelinek wrote: > On Fri, May 29, 2015 at 02:41:28PM +0200, Richard Biener wrote: > > @@ -525,7 +498,11 @@ get_rank (tree e) > >return (rank + 1); > > } > > > > - /* Globals, etc, are rank 0 */ > > + /* Constants have rank 0. */ > > + if (is_gimple_min

Re: conditional lim

2015-05-29 Thread Evgeniya Maenkova
Hi Richard, Here is some explanation. I hope you let me know if I need to clarify something. Also, you asked me about concrete example, to make sure you don’t miss my answer here is the link: https://gcc.gnu.org/ml/gcc-patches/2015-05/msg02417.html. Also, I doubt whether it’s convenient for you

Re: [RFC / CFT] PR c++/66192 - Remove TARGET_RELAXED_ORDERING and use load acquires.

2015-05-29 Thread Ramana Radhakrishnan
On 22/05/15 18:36, Jason Merrill wrote: On 05/22/2015 11:23 AM, Ramana Radhakrishnan wrote: On 22/05/15 15:28, Jason Merrill wrote: I do notice that get_guard_bits after build_atomic_load just won't work on non-ARM targets, as it ends up trying to take the address of a value. So on powerpc

Re: [PATCH, testsuite]: Cleanup dumps

2015-05-29 Thread Kyrill Tkachov
On 29/05/15 09:21, Bernhard Reutner-Fischer wrote: On 31 January 2015 at 22:10, Bernhard Reutner-Fischer wrote: On January 31, 2015 9:17:57 PM GMT+01:00, Mike Stump wrote: On Jan 31, 2015, at 11:50 AM, Bernhard Reutner-Fischer wrote: On January 31, 2015 10:53:39 AM GMT+01:00, Uros Bizjak

Re: [PATCH 01/35] Introduce new type-based pool allocator.

2015-05-29 Thread Martin Liška
On 05/28/2015 07:15 PM, Jeff Law wrote: On 05/28/2015 06:49 AM, Martin Liška wrote: . This mechanism has been just adapted. I find it quite useful as we have examples in source code where we allocate same struct/class types from a various pool. For debugging purpose, it helps to identify if rel

Re: [PATCH 02/35] Change use to type-based pool allocator in et-forest.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 07:45 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * et-forest.c (et_new_occ): Use new type-based pool allocator. (et_new_tree): Likewise. (et_free_tree): Likewise. (et_free_tree_force): Likewise. (et_free

Re: [PATCH 03/35] Change use to type-based pool allocator in lra-lives.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 07:47 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * lra-lives.c (free_live_range): Use new type-based pool allocator. (free_live_range_list) Likewise. (create_live_range) Likewise. (copy_live_range) Likewise.

Re: [PATCH 04/35] Change use to type-based pool allocator in lra.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 07:50 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * lra.c (init_insn_regs): Use new type-based pool allocator. (new_insn_reg) Likewise. (free_insn_reg) Likewise. (free_insn_regs) Likewise. (finish_insn_re

Re: [PATCH 05/35] Change use to type-based pool allocator in ira-color.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 07:51 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * ira-color.c (init_update_cost_records):Use new type-based pool allocator. (get_update_cost_record) Likewise. (free_update_cost_record_list) Likewise. (finis

Re: [PATCH 07/35] Change use to type-based pool allocator in var-tracking.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 03:56 PM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * var-tracking.c (variable_htab_free):Use new type-based pool allocator. (attrs_list_clear) Likewise. (attrs_list_insert) Likewise. (attrs_list_copy) Likewise. (shared_hash_un

Re: [PATCH 10/35] Change use to type-based pool allocator in cfg.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 07:57 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * cfg.c (initialize_original_copy_tables):Use new type-based pool allocator. (free_original_copy_tables) Likewise. (copy_original_table_clear) Likewise. (copy

Re: [PATCH 09/35] Change use to type-based pool allocator in c-format.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 07:55 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/c-family/ChangeLog: 2015-04-30 Martin Liska * c-format.c (check_format_arg):Use new type-based pool allocator. (check_format_info_main) Likewise. OK. jeff v2 >From c9d58f9d3e3b16b9ed588a1009efbaf1c

Re: [PATCH 11/35] Change use to type-based pool allocator in sh.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 07:59 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * config/sh/sh.c (add_constant):Use new type-based pool allocator. (sh_reorg) Likewise. OK. jeff v2 >From 68f0b72993d882d5dfe4096806f2bb78c87a37c4 Mon Sep 17 00:0

Re: [PATCH 13/35] Change use to type-based pool allocator in df-problems.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:01 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * df-problems.c (df_chain_create):Use new type-based pool allocator. (df_chain_unlink_1) Likewise. (df_chain_unlink) Likewise. (df_chain_remove_problem) Likew

Re: [PATCH 12/35] Change use to type-based pool allocator in cselib.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 03:56 PM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * cselib.c (new_elt_list):Use new type-based pool allocator. (new_elt_loc_list) Likewise. (unchain_one_elt_list) Likewise. (unchain_one_elt_loc_list) Likewise. (unchain_one_va

Re: [PATCH 14/35] Change use to type-based pool allocator in df-scan.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 03:56 PM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * df-scan.c (struct df_scan_problem_data):Use new type-based pool allocator. (df_scan_free_internal) Likewise. (df_scan_alloc) Likewise. (df_grow_reg_info) Likewise. (df_free

Re: [PATCH 15/35] Change use to type-based pool allocator in dse.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 03:56 PM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * dse.c (get_group_info):Use new type-based pool allocator. (dse_step0) Likewise. (free_store_info) Likewise. (delete_dead_store_insn) Likewise. (free_read_records) Likewise.

Re: [PATCH][ARM/AArch64 Testsuite] Cleanup advsimd-intrinsics.exp, removing unnecessary loop

2015-05-29 Thread Christophe Lyon
On 29 May 2015 at 12:48, Alan Lawrence wrote: > Christophe Lyon wrote: >> >> >> This looks OK, but why can't you also drop the other torture-related >> lines as you did in your previous patch? >> I mean: >> load_lib c-torture.exp >> load_lib torture-options.exp >> etc... > > > We need c-torture.ex

Re: [PATCH 16/35] Change use to type-based pool allocator in tree-sra.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:02 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * tree-sra.c (sra_initialize): Use new type-based pool allocator. (sra_deinitialize) Likewise. (create_access_1) Likewise. (build_accesses_from_assign) Likewi

Re: [PATCH 17/35] Change use to type-based pool allocator in tree-ssa-math-opts.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:03 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * tree-ssa-math-opts.c (occ_new): Use new type-based pool allocator. (free_bb): Likewise. (pass_cse_reciprocals::execute): Likewise. OK. jeff v2 >From 3fffec0

Re: [PATCH 18/35] Change use to type-based pool allocator in stmt.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:09 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * stmt.c (add_case_node): Use new type-based pool allocator. (expand_case): Likewise. (expand_sjlj_dispatch_table): Likewise. OK. jeff v2 >From 7d6b26873043b3

Re: [PATCH 20/35] Change use to type-based pool allocator in ira-build.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:12 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * ira-build.c (initiate_cost_vectors): Use new type-based pool allocator. (ira_allocate_cost_vector): Likewise. (ira_free_cost_vector): Likewise. (finish_cost

Re: [PATCH 19/35] Change use to type-based pool allocator in sel-sched-ir.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:04 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * sel-sched-ir.c (alloc_sched_pools): Use new type-based pool allocator. (free_sched_pools): Likewise. * sel-sched-ir.h (_list_alloc): Likewise. (_list_remove

Re: [PATCH 22/35] Change use to type-based pool allocator in sched-deps.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:14 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * sched-deps.c (create_dep_node): Use new type-based pool allocator. (delete_dep_node): Likewise. (create_deps_list): Likewise. (free_deps_list): Likewise.

Re: [PATCH 21/35] Change use to type-based pool allocator in regcprop.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:11 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * regcprop.c (free_debug_insn_changes): Use new type-based pool allocator. (replace_oldest_value_reg): Likewise. (pass_cprop_hardreg::execute): Likewise. OK. jef

Re: [PATCH 23/35] Change use to type-based pool allocator in tree-ssa-pre.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:18 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * tree-ssa-pre.c (get_or_alloc_expr_for_name): Use new type-based pool allocator. (bitmap_set_new): Likewise. (get_or_alloc_expr_for_constant): Likewise. (ge

Re: [PATCH 25/35] Change use to type-based pool allocator in tree-ssa-sccvn.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:13 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * tree-ssa-sccvn.c (vn_reference_insert): Use new type-based pool allocator. (vn_reference_insert_pieces): Likewise. (vn_phi_insert): Likewise. (visit_referen

Re: [PATCH 26/35] Change use to type-based pool allocator in tree-ssa-strlen.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 03:56 PM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * tree-ssa-strlen.c (new_strinfo): Use new type-based pool allocator. (free_strinfo): Likewise. (pass_strlen::execute): Likewise. --- gcc/tree-ssa-strlen.c | 10 -- 1 file changed,

Re: [PATCH 24/35] Change use to type-based pool allocator in tree-ssa-reassoc.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:12 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * tree-ssa-reassoc.c (add_to_ops_vec): Use new type-based pool allocator. (add_repeat_to_ops_vec): Likewise. (get_ops): Likewise. (maybe_optimize_range_tests)

Re: [PATCH 29/35] Change use to type-based pool allocator in ipa-prop.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:16 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * ipa-prop.c (ipa_set_jf_constant): Use new type-based pool allocator. (ipa_edge_duplication_hook): Likewise. (ipa_free_all_structures_after_ipa_cp): Likewise.

Re: [PATCH 30/35] Change use to type-based pool allocator in ipa-inline-analysis.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 03:56 PM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * ipa-inline-analysis.c (edge_set_predicate): Use new type-based pool allocator. (set_hint_predicate): Likewise. (inline_summary_alloc): Likewise. (reset_inline_edge_summary): Likewi

Re: [PATCH 28/35] Change use to type-based pool allocator in ipa-profile.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:15 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * ipa-profile.c (account_time_size): Use new type-based pool allocator. (ipa_profile_generate_summary): Likewise. (ipa_profile_read_summary): Likewise. (ipa_p

Re: [PATCH 32/35] Change use to type-based pool allocator in ira-build.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:21 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * ira-build.c (finish_allocnos): Use new type-based pool allocator. (finish_prefs): Likewise. (finish_copies): Likewise. Is this a partial duplicate of patch #34

Re: [PATCH 31/35] Change use to type-based pool allocator in ipa-prop.c and ipa-cp.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 03:56 PM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * ipa-cp.c (ipcp_value::add_source): Use new type-based pool allocator. (allocate_and_init_ipcp_value): Likewise. (ipcp_lattice::add_value): Likewise. (merge_agg_lats_step): Likewise.

Re: [PATCH 35/35] Remove old pool allocator.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:24 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * alloc-pool.c (create_alloc_pool): Remove. (empty_alloc_pool): Likewise. (free_alloc_pool): Likewise. (free_alloc_pool_if_empty): Likewise. (pool_alloc):

Re: [PATCH, testsuite]: Cleanup dumps

2015-05-29 Thread Bernhard Reutner-Fischer
On 29 May 2015 at 15:21, Kyrill Tkachov wrote: > Hi Bernhard, > > You asked to be CC'ed ;) > Applied this as obvious with r223864 to. oops, thanks for fixing! This obviously escaped my grep-foo. As said, i'd vote for -Wcomment -fno-ident to be added per default where testcases should opt-out if

[patch] Use std::decay in return type of std::async (LWGW 2021)

2015-05-29 Thread Jonathan Wakely
http://cplusplus.github.io/LWG/lwg-defects.html#2021 corrects the return type of std::async(), which is necessary to ensure that the result_of expression matches the actual result of the callable object passed to async. We already do the right thing for std::bind(), so only std::async() needs cha

Re: [PATCH 27/35] Change use to type-based pool allocator in tree-ssa-structalias.c.

2015-05-29 Thread Martin Liška
On 05/27/2015 08:15 PM, Jeff Law wrote: On 05/27/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-04-30 Martin Liska * tree-ssa-structalias.c (new_var_info): Use new type-based pool allocator. (new_constraint): Likewise. (init_alias_vars): Likewise. (delete_points_to_sets):

Re: [wwwdocs] Buildstat update for 5.1

2015-05-29 Thread Gerald Pfeifer
On Sun, 3 May 2015, Tom G. Christensen wrote: > The first update for 5.x. Thank Tom, and sorry for missing this. Applied, and I'll momentarily check out the other one as well. Did we get more updates for GCC 5.1 in the meantime? Gerald

Re: [PATCH] Simple optimization for MASK_STORE.

2015-05-29 Thread Yuri Rumyantsev
Hi Richard, Did you have a chance to look at my updated patch? Any comments will be appreciated. Yuri. 2015-05-20 17:00 GMT+03:00 Yuri Rumyantsev : > Hi All, > > Here is updated patch to optimize mask stores. The main goal of it is > to avoid execution of mask store if its mask is zero vector si

Re: [wwwdocs] Buildstat update for 4.9

2015-05-29 Thread Gerald Pfeifer
On Sun, 3 May 2015, Tom G. Christensen wrote: > Latest results for 4.9.x This is online now, thank you Tom! Gerald

Re: [Patch]: libbacktrace - add support of PE/COFF

2015-05-29 Thread Ian Lance Taylor
On Fri, May 29, 2015 at 1:43 AM, Tristan Gingold wrote: > >> On 28 May 2015, at 17:14, Ian Lance Taylor wrote: >> >> The way you have written the code, I'm fairly sure that it will be >> compiled for an i386-coff target. > > And the only coff target supported is djgpp, right ? Really? I didn't

Re: [PATCH] PR 66215: S390: Fix placement of post-label NOPs with -mhotpatch

2015-05-29 Thread Dominik Vogt
No bother, new versions attached. Ciao Dominik ^_^ ^_^ -- Dominik Vogt IBM Germany gcc/ChangeLog: -- 2015-05-29 Dominik Vogt PR target/66215 * config/s390/s390.c (s390_reorg): Fix placement of post-label NOPs with -mhotpatch=. gcc/testsuite/ChangeLog:

Re: [PATCH] PR 66215: S390: Fix placement of post-label NOPs with -mhotpatch

2015-05-29 Thread Andreas Krebbel
On 05/29/2015 04:14 PM, Dominik Vogt wrote: > No bother, new versions attached. Applied to mainline. Thanks! -Andreas-

Re: [PATCH] nios2-linux: add missing cpp specs

2015-05-29 Thread Sandra Loosemore
On 05/27/2015 10:00 AM, Mike Frysinger wrote: Define CPP_SPEC for nios2 linux targets so that -posix & -pthread work like on all other linux targets. 2015-05-27 Mike Frysinger * config/nios2/linux.h (CPP_SPEC): Define. I see that -posix is not documented at all in invoke.texi and -

Re: [PATCH 3/4] split-stack for powerpc64

2015-05-29 Thread Alan Modra
Two fixes for -fsplit-stack on powerpc64. I goofed on the block scanned for uses of r12. ENTRY_BLOCK_PTR_FOR_FN is the fake one. The next block is the first one having insns. The second change emits an error if people use a global register variable r29 with -fsplit-stack. For example: register

[patch] Fix duplicate xml:id attribute in libstdc++ manual

2015-05-29 Thread Jonathan Wakely
Self-explanatory. Committed to trunk. commit 1a92bedf73d04fe31eea20d5a7a4fe9407a9e414 Author: Jonathan Wakely Date: Fri May 29 16:16:17 2015 +0100 * doc/xml/manual/status_cxx2017.xml: Fix duplicate ID attribute. * doc/html/manual/*: Regenerate. diff --git a/libstdc++-v3/doc/xml/manu

Re: [RFC / CFT] PR c++/66192 - Remove TARGET_RELAXED_ORDERING and use load acquires.

2015-05-29 Thread Richard Henderson
On 05/29/2015 06:18 AM, Ramana Radhakrishnan wrote: > > PR c++/66192 > * config/alpha/alpha.c (TARGET_RELAXED_ORDERING): Likewise. > * config/ia64/ia64.c (TARGET_RELAXED_ORDERING): Likewise. > * config/rs6000/rs6000.c (TARGET_RELAXED_ORDERING): Likewise. > * config/sparc/linux.

Re: [RFC / CFT] PR c++/66192 - Remove TARGET_RELAXED_ORDERING and use load acquires.

2015-05-29 Thread Richard Henderson
On 05/29/2015 06:18 AM, Ramana Radhakrishnan wrote: > > One of the bits of fallout that I've observed in my testing and that I'm not > sure about what to do is that on *bare-metal* arm-none-eabi targets we still > put out calls to __sync_synchronize on architecture versions that do not have > a >

Re: [PATCH, RFC] New memory usage statistics infrastructure

2015-05-29 Thread Vidya Praveen
Martin, The following change: @@ -2655,10 +2655,10 @@ s-iov: build/gcov-iov$(build_exeext) $(BASEVER) $(DEVPHASE) GCOV_OBJS = gcov.o gcov$(exeext): $(GCOV_OBJS) $(LIBDEPS) - +$(LINKER) $(ALL_LINKERFLAGS) $(LDFLAGS) $(GCOV_OBJS) $(LIBS) -o $@ + +$(LINKER) $(ALL_LINKERFLAGS) $(LDF

[gomp4] Avoiding predication for certain blocks

2015-05-29 Thread Bernd Schmidt
When predicating the code for OpenACC, we should avoid the entry block in an offloaded region, which contains setup code that should be run in every thread. The following patch adds a new marker statement that is used to identify this block. Currently, predication doesn't happen anyway due to a

[gomp4] A thread barrier builtin

2015-05-29 Thread Bernd Schmidt
This adds an OpenACC specific thread synchronization builtin and the nvptx pattern it expands to. Committed to gomp-4_0-branch. Bernd Index: gcc/ChangeLog.gomp === --- gcc/ChangeLog.gomp (revision 223869) +++ gcc/ChangeLog.gomp (wo

[gomp4] Initialize some extra variables at the entry to an OpenACC offloaded region

2015-05-29 Thread Bernd Schmidt
This initializes worker count and worker id variables; these will be used in a followup patch to allocate arrays in gang-local memory to be used for synchronizing threads. Committed to gomp-4_0-branch. Bernd Index: gcc/ChangeLog.gomp

[PATCH, v2] Fix bootstrap with in-tree ISL

2015-05-29 Thread Bernhard Reutner-Fischer
Hi, config/ChangeLog: 2015-05-29 Bernhard Reutner-Fischer * isl.m4 (ISL_CHECK_VERSION): AC_SUBST ENABLE_ISL_CHECK and set gcc_cv_isl. Tidy whitespace. Lowercase first line of help-text like for all the other help texts. ChangeLog: 2015-05-29 Bernhard Reutner-Fischer

Re: [PATCH 3/4] split-stack for powerpc64

2015-05-29 Thread David Edelsohn
On Fri, May 29, 2015 at 10:47 AM, Alan Modra wrote: > Two fixes for -fsplit-stack on powerpc64. I goofed on the block > scanned for uses of r12. ENTRY_BLOCK_PTR_FOR_FN is the fake one. The > next block is the first one having insns. > > The second change emits an error if people use a global re

Re: [patch] fix bootstrap on FreeBSD i386/arm

2015-05-29 Thread Steve Ellcey
On Thu, 2015-05-28 at 23:54 +0200, Eric Botcazou wrote: > > This patch restores bootstrap on i386-*-freebsd*. > > The build was failing after the introduction of -std=c++98 > > configure/build flag. The -std=c++98 enables strict_ansi and on FreeBSD > > the libc function atoll is not defined for thi

Re: [RFC][PATCH][X86_64] Eliminate PLT stubs for specified external functions via -fno-plt=

2015-05-29 Thread Sriraman Tallam
Hi HJ, Is this ok to commit? Thanks Sri On Thu, May 28, 2015 at 11:03 PM, Sriraman Tallam wrote: > On Thu, May 28, 2015 at 5:05 PM, H.J. Lu wrote: >> On Thu, May 28, 2015 at 4:54 PM, Sriraman Tallam wrote: >>> On Thu, May 28, 2015 at 2:52 PM, H.J. Lu wrote: On Thu, May 28, 2015 at 2:27

RFA: Fix mode checks for possibly-constant predicates

2015-05-29 Thread Richard Sandiford
genrecog relies on a predicate foo_operand (op, mode) checking that OP really does have mode MODE, with VOIDmode acting as a wildcard. This was true even with the old genrecog, but as Andreas found on s390x, new genrecog is being a bit more aggressive about it. The problem is that at the moment,

Re: [RFC][PATCH][X86_64] Eliminate PLT stubs for specified external functions via -fno-plt=

2015-05-29 Thread H.J. Lu
On Fri, May 29, 2015 at 10:20 AM, Sriraman Tallam wrote: > Hi HJ, > > Is this ok to commit? > Looks good to me. But I can't approve it. -- H.J.

Re: [RFC][PATCH][X86_64] Eliminate PLT stubs for specified external functions via -fno-plt=

2015-05-29 Thread Sriraman Tallam
+Uros On Fri, May 29, 2015 at 10:25 AM, H.J. Lu wrote: > On Fri, May 29, 2015 at 10:20 AM, Sriraman Tallam wrote: >> Hi HJ, >> >> Is this ok to commit? >> > > Looks good to me. But I can't approve it. > > -- > H.J.

Re: [PATCH] nios2-linux: add missing cpp specs

2015-05-29 Thread Mike Frysinger
On 29 May 2015 08:44, Sandra Loosemore wrote: > On 05/27/2015 10:00 AM, Mike Frysinger wrote: > > Define CPP_SPEC for nios2 linux targets so that -posix & -pthread work > > like on all other linux targets. > > > > 2015-05-27 Mike Frysinger > > > > * config/nios2/linux.h (CPP_SPEC): Define. >

RFC: Add C++ warnings for unnecessarily general return types

2015-05-29 Thread Richard Sandiford
One of the main early aims of the rtl refactoring is to finally separate instructions (rtx_insns) from other rtxes. I thought it would help if the compiler could warn about cases where a function returns rtx when it could return rtx_insn*, or where a variable has type rtx but could have type rtx_i

Re: RFA: Fix mode checks for possibly-constant predicates

2015-05-29 Thread Richard Henderson
On 05/29/2015 10:23 AM, Richard Sandiford wrote: > + /* Check whether the predicate accepts const scalar ints (which always > + have a stored mode of VOIDmode, but logically have a real mode) > + and whether it matches anything besides const scalar ints. */ > + bool matches_const_scalar_

Re: [PATCH] nios2-linux: add missing cpp specs

2015-05-29 Thread Sandra Loosemore
On 05/29/2015 11:36 AM, Mike Frysinger wrote: On 29 May 2015 08:44, Sandra Loosemore wrote: On 05/27/2015 10:00 AM, Mike Frysinger wrote: Define CPP_SPEC for nios2 linux targets so that -posix & -pthread work like on all other linux targets. 2015-05-27 Mike Frysinger * config/nios2

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

2015-05-29 Thread Aldy Hernandez
On 05/28/2015 04:35 PM, Jason Merrill wrote: On 05/28/2015 02:53 PM, Aldy Hernandez wrote: On 05/27/2015 08:39 AM, Jason Merrill wrote: On 05/20/2015 11:50 AM, Aldy Hernandez wrote: + /* Fill in the size of variable-length fields in late dwarf. */ + if (TREE_ASM_WRITTEN (type) + && !

Re: [PATCH, v2] Fix bootstrap with in-tree ISL

2015-05-29 Thread Tobias Grosser
On 05/29/2015 06:54 PM, Bernhard Reutner-Fischer wrote: Hi, config/ChangeLog: 2015-05-29 Bernhard Reutner-Fischer * isl.m4 (ISL_CHECK_VERSION): AC_SUBST ENABLE_ISL_CHECK and set gcc_cv_isl. Tidy whitespace. Lowercase first line of help-text like for all the other he

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

2015-05-29 Thread Richard Biener
On May 29, 2015 8:42:50 PM GMT+02:00, Aldy Hernandez wrote: >On 05/28/2015 04:35 PM, Jason Merrill wrote: >> On 05/28/2015 02:53 PM, Aldy Hernandez wrote: >>> On 05/27/2015 08:39 AM, Jason Merrill wrote: On 05/20/2015 11:50 AM, Aldy Hernandez wrote: >>> > + /* Fill in the size of variabl

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

2015-05-29 Thread Jason Merrill
On 05/29/2015 02:42 PM, Aldy Hernandez wrote: unsigned int i=555; int main() { unsigned int array[i]; ... } For the VLA, I'd like to check if we have an array type with a missing DW_AT_{upper,lower}_bound late in the game, and fill it in. During early dwarf we only have an uninitialized

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

2015-05-29 Thread Jason Merrill
On 05/29/2015 03:26 PM, Richard Biener wrote: ISTR I had to mark the gimple reg used for the bound as non-DECL_IGNORED for the LTO stuff. Let's go with that, then. Jason

Re: [RFC][PATCH][X86_64] Eliminate PLT stubs for specified external functions via -fno-plt=

2015-05-29 Thread Jan Hubicka
> * config/i386/i386.c (avoid_plt_to_call): New function. > (ix86_output_call_insn): Generate indirect call for functions > marked with "noplt" attribute. > (attribute_spec ix86_attribute_): Define new attribute "noplt". > * doc/extend.texi: Document new attribute "nop

Re: [RFC / CFT] PR c++/66192 - Remove TARGET_RELAXED_ORDERING and use load acquires.

2015-05-29 Thread Jason Merrill
On 05/29/2015 09:18 AM, Ramana Radhakrishnan wrote: +static tree +build_atomic_load_byte (tree src, HOST_WIDE_INT model) This function needs a comment. The C++ changes are OK with that. Jason

Re: [PATCH, v2] Fix bootstrap with in-tree ISL

2015-05-29 Thread DJ Delorie
A few minor nits... Your patch includes many whitespace changes, which makes reviewing your patch more difficult. Please limit whitespace changes when they're unrelated to the patch. Assuming you've looked at the actual diffs for them, and see nothing beyond changes related to your patch, it's

  1   2   >