[PATCH] Add script to contrib to format email body and patch for submission

2013-03-30 Thread Tom de Vries
-patch.body echo cat compose-body-patch.log) \ compose-body-patch.patch) \ message.txt ... OK for trunk? Thanks, - Tom 2013-03-30 Tom de Vries t...@codesourcery.com * compose-body-patch.py: New script. Index: contrib/compose-body-patch.py

Re: [PATCH][IRA] Analysis of register usage of functions for usage by IRA.

2013-03-30 Thread Tom de Vries
On 29/03/13 13:54, Tom de Vries wrote: I split the patch up into 10 patches, to facilitate further review: ... 0001-Add-command-line-option.patch 0002-Add-new-reg-note-REG_CALL_DECL.patch 0003-Add-implicit-parameter-to-find_all_hard_reg_sets.patch 0004-Add-TARGET_FN_OTHER_HARD_REG_USAGE

[PATCH][01/10] -fuse-caller-save - Add command line option

2013-03-30 Thread Tom de Vries
Vladimir, This patch adds the -fuse-caller-save command line option. Thanks, -Tom 2013-03-30 Radovan Obradovic robrado...@mips.com Tom de Vries t...@codesourcery.com * common.opt (fuse-caller-save): New option. diff --git a/gcc/common.opt b/gcc/common.opt index bdbd3b6

[PATCH][03/10] -fuse-caller-save - Add implicit parameter to find_all_hard_reg_sets

2013-03-30 Thread Tom de Vries
Vladimir, This patch adds an implicit parameter to find_all_hard_reg_sets. Thanks, -Tom 2013-03-30 Radovan Obradovic robrado...@mips.com Tom de Vries t...@codesourcery.com * rtlanal.c (find_all_hard_reg_sets): Add bool implicit parameter and handle

[PATCH][04/10] -fuse-caller-save - Add TARGET_FN_OTHER_HARD_REG_USAGE hook

2013-03-30 Thread Tom de Vries
-03-30 Radovan Obradovic robrado...@mips.com Tom de Vries t...@codesourcery.com * hooks.c (hook_void_hard_reg_set_containerp): New function. * hooks.h (hook_void_hard_reg_set_containerp): Declare. * target.def (fn_other_hard_reg_usage): New DEFHOOK

[PATCH][05/10] -fuse-caller-save - Implement TARGET_FN_OTHER_HARD_REG_USAGE hook for ARM

2013-03-30 Thread Tom de Vries
TARGET_FN_OTHER_HARD_REG_USAGE was introduced in the previous patch in this patch series. Build and reg-tested on ARM. OK for trunk? Thanks, -Tom 2013-03-30 Radovan Obradovic robrado...@mips.com Tom de Vries t...@codesourcery.com * config/arm/arm.c (TARGET_FN_OTHER_HARD_REG_USAGE): Redefine

[PATCH][06/10] -fuse-caller-save - Collect register usage information

2013-03-30 Thread Tom de Vries
Vladimir, This patch adds analysis in pass_final to track which hard registers are set or clobbered by the function body, and stores that information in a struct cgraph_node. Thanks, -Tom 2013-03-30 Radovan Obradovic robrado...@mips.com Tom de Vries t...@codesourcery.com

[PATCH][07/10] -fuse-caller-save - Use collected register usage information

2013-03-30 Thread Tom de Vries
-tested on x86_64, Ada inclusive. Build and reg-tested on mips, arm, ppc and sh. Can you approve the df-scan part for trunk? Thanks, -Tom 2013-03-30 Radovan Obradovic robrado...@mips.com Tom de Vries t...@codesourcery.com * df-scan.c (df_get_call_refs): Use

[PATCH][08/10] -fuse-caller-save - Enable by default at O2 and higher

2013-03-30 Thread Tom de Vries
Vladimir, This patch enables the -fuse-caller-save optimization by default. Thanks, -Tom 2013-03-30 Radovan Obradovic robrado...@mips.com Tom de Vries t...@codesourcery.com * opts.c (default_options_table): Add OPT_LEVELS_2_PLUS entry with OPT_fuse_caller_save

[PATCH][09/10] -fuse-caller-save - Add documentation

2013-03-30 Thread Tom de Vries
Vladimir, This patch adds the documentation of -fuse-caller-save. Thanks, -Tom 2013-03-30 Radovan Obradovic robrado...@mips.com Tom de Vries t...@codesourcery.com * doc/invoke.texi (@item Optimization Options): Add -fuse-caller-save to gccoptlist

[PATCH][10/10] -fuse-caller-save - Add test-case

2013-03-30 Thread Tom de Vries
, new effective targets are introduced. Build and reg-tested on mips. OK for trunk? Thanks, -Tom 2013-03-30 Radovan Obradovic robrado...@mips.com Tom de Vries t...@codesourcery.com * lib/target-supports.exp (check_effective_target_mips16

Re: [PATCH] Fix PR56426

2014-07-31 Thread Tom de Vries
-ssa.exp. Committed as trivial. Thanks, - Tom 2014-07-30 Tom de Vries t...@codesourcery.com * tree-ssa-loop.c (pass_tree_loop_init::execute): Remove dead code. diff --git a/gcc/tree-ssa-loop.c b/gcc/tree-ssa-loop.c index 7c52748..d0c9980 100644 --- a/gcc/tree-ssa-loop.c +++ b/gcc/tree-ssa

[PATCH][committed] Remove superfluous declaration in testcase

2014-07-31 Thread Tom de Vries
Hi, This patch removes a superfluous declaration in a test-case. Committed to trunk. Thanks, - Tom 2014-07-31 Tom de Vries t...@codesourcery.com * gcc.dg/pr51879-7.c: Remove superfluous declaration of bar. diff --git a/gcc/testsuite/gcc.dg/pr51879-7.c b/gcc/testsuite/gcc.dg/pr51879-7.c

[PATCH][testsuite] Don't run cproj-fails-with-broken-glibc.c for broken glibc

2014-07-31 Thread Tom de Vries
implementation which will conflict with GCC's builtin cproj(). GLIBC 2.12+ should be okay. ... This patch skips the test for known broken glibcs. OK for trunk? Thanks, - Tom 2014-07-31 Tom de Vries t...@codesourcery.com * lib/target-supports.exp (check_effective_target_glibc

Re: [PATCH] Keep patch file permissions in mklog

2014-08-01 Thread Tom de Vries
On 01-08-14 06:21, Yury Gribov wrote: Cc Diego. --- From: Tom de Vries tom_devr...@mentor.com Sent: Saturday, May 31, 2014 11:47AM To: Geoff Keating geo...@geoffk.org Cc: GCC Patches Subject: [PATCH] Keep patch file permissions in mklog On 05/31

Re: [PATCH][testsuite] Don't run cproj-fails-with-broken-glibc.c for broken glibc

2014-08-02 Thread Tom de Vries
2014-08-01 Tom de Vries t...@codesourcery.com * gcc.dg/cproj-fails-with-broken-glibc.c: Use xfail for broken glibc version instead of required-target. * lib/target-supports.exp (check_effective_target_not_glibc_2_11_or_earlier): Replace by ... (check_effective_target_glibc_2_11_or_earlier

Re: [PATCH] Keep patch file permissions in mklog

2014-08-02 Thread Tom de Vries
On 01-08-14 09:18, Yury Gribov wrote: On 08/01/2014 10:52 AM, Tom de Vries wrote: This patch adds a script contrib/mklog-in-patch, which uses mklog to generate the skeleton log, but generates the log at the start of the patch as mklog did before (which is how I like to use it). Yeah, we had

Re: [PATCH] Keep patch file permissions in mklog

2014-08-04 Thread Tom de Vries
don't check for result of mktemp... I've added a check for the result of mktemp, and removed the unlink result check. I've left in the Could not append patch to temp file check because the patch file might be read-only. OK for trunk? Thanks, - Tom 2014-08-04 Tom de Vries t

Re: [PATCH][testsuite] Don't run cproj-fails-with-broken-glibc.c for broken glibc

2014-08-07 Thread Tom de Vries
://gcc.gnu.org/ml/gcc-patches/2010-04/msg01285.html I'd strongly suggest just removing the test and the supporting effective-target keywords. Done. Committed as attached. Thanks, - Tom 2014-08-07 Tom de Vries t...@codesourcery.com * gcc.dg/cproj-fails-with-broken-glibc.c: Remove test. * lib

Fix if-conversion pass for dead type-unsafe code

2014-08-08 Thread Tom de Vries
on x86_64, trunk. No issue found other than https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62060 , which looks like a test-case issue. OK for trunk, 4.9, 4.8? Thanks, - Tom 2014-08-07 Tom de Vries t...@codesourcery.com * ifcvt.c (mem_alias_equal_p, rtx_interchangeable_p): New function

Re: Fix if-conversion pass for dead type-unsafe code

2014-08-08 Thread Tom de Vries
On 08-08-14 13:36, Richard Biener wrote: On Fri, 8 Aug 2014, Tom de Vries wrote: Steven, this patch fixes: - PR62004 (the if-conversion pass part, the tail-merge part is still todo), and - PR62030. In both cases, a valid program with a dead type-unsafe access is transformed

Re: Fix if-conversion pass for dead type-unsafe code

2014-08-08 Thread Tom de Vries
On 08-08-14 17:17, Tom de Vries wrote: Maybe instead of a new mem_alias_equal_p simply compare MEM_ATTRs with mem_attrs_eq_p? I propose to fix it this way (as attached) on 4.8/4.9/trunk, and maybe do a more efficient handling on trunk as a follow-up patch. I'll put this through bootstrap/test

Re: [PATCH] Keep patch file permissions in mklog

2014-08-11 Thread Tom de Vries
mktemp from File::Temp. With these modification, I've eliminated system() calls and backticks from the patch. BTW you may want to wait for Diego's and Trevor's comments (Diego is the maintainer and approver for this code). ok. Thanks, - Tom 2014-08-11 Tom de Vries t...@codesourcery.com

Re: [PATCH] Keep patch file permissions in mklog

2014-08-11 Thread Tom de Vries
On 11-08-14 10:18, Yury Gribov wrote: On 08/11/2014 11:22 AM, Tom de Vries wrote: I'm now using mktemp from File::Temp. The script now relies on external Perl modules. This is probably fine because AFAIK File::Temp and File::Copy are available included in all distributions. +use File::Copy

Re: Fix if-conversion pass for dead type-unsafe code

2014-08-18 Thread Tom de Vries
On 14-08-14 16:34, Richard Biener wrote: On Sat, Aug 9, 2014 at 7:14 AM, Tom de Vries tom_devr...@mentor.com wrote: On 08-08-14 17:17, Tom de Vries wrote: Maybe instead of a new mem_alias_equal_p simply compare MEM_ATTRs with mem_attrs_eq_p? I propose to fix it this way (as attached

[PING][PATCH, ARM] Fix PR63718, Thumb1 bootstrap -- disable fuse-caller-save for Thumb1

2014-12-02 Thread Tom de Vries
On 20-11-14 12:54, Tom de Vries wrote: Richard, This patch fixes PR63718, which currently breaks Thumb1 bootstrap. The problem is that in Thumb1 mode, we emit the epilogue in RTL, but the last insn - epilogue_insns - does not accurately model the corresponding insns emitted in the asm file

Re: [4.8, Fortran, Patch] PR 48820 - Support TYPE(*) of TS29113

2014-12-02 Thread Tom de Vries
On 03/03/12 08:41, Paul Richard Thomas wrote: Dear Tobias, This is certainly OK for 4.8. I have a couple of remarks: (i) The DTYPE_TYPE_MASK is 0x38 so that we saturated it a long time since! At the moment it does not cause any problems because of the extremely limited use of the dtype

[PATCH, PR63957] Rename fuse-caller-save to -fipa-ra

2014-12-03 Thread Tom de Vries
Jakub, As discussed*, this patch renames fuse-caller-save to fipa-ra. Bootstrapped and reg-tested on x86_64. Did minimal arm build. OK for stage3? Thanks, - Tom (*) https://gcc.gnu.org/ml/gcc-patches/2014-10/msg01943.html 2014-12-03 Tom de Vries t...@codesourcery.com PR rtl-optimization

Fix scan-tree-dump for scop-19.c

2015-02-04 Thread Tom de Vries
to match the pic/nonpic behaviour. OK for trunk? Thanks, - Tom 2015-02-04 Tom de Vries t...@codesourcery.com * gcc.dg/graphite/scop-19.c: Fix scan-tree-dump for fpic. diff --git a/gcc/testsuite/gcc.dg/graphite/scop-19.c b/gcc/testsuite/gcc.dg/graphite/scop-19.c index 9dfc865..ab23bb4 100644

Re: [PATCH] Fix PR64764

2015-02-05 Thread Tom de Vries
On 26-01-15 15:47, Richard Biener wrote: Index: gcc/testsuite/gcc.dg/uninit-19.c === --- gcc/testsuite/gcc.dg/uninit-19.c(revision 0) +++ gcc/testsuite/gcc.dg/uninit-19.c(working copy) @@ -0,0 +1,23 @@ +/* { dg-do compile }

Re: [PATCH] Fix PR64764 Fix scan-tree-dump for scop-19.c

2015-02-11 Thread Tom de Vries
On 11-02-15 10:22, Dominique d'Humières wrote: Le 10 févr. 2015 à 12:42, Tom de Vries tom_devr...@mentor.com a écrit : I think we need to understand first what's going on. Sure, my patch was mainly to silence the failures on my working tree. In both test-cases, on Linux with -fpic

[committed] Add missing cleanup in gfortran.dg/pr37287-1.f90

2015-02-18 Thread Tom de Vries
Tom de Vries t...@codesourcery.com * gfortran.dg/pr37287-1.f90: Add missing cleanup-modules. --- gcc/testsuite/gfortran.dg/pr37287-1.f90 | 1 + 1 file changed, 1 insertion(+) diff --git a/gcc/testsuite/gfortran.dg/pr37287-1.f90 b/gcc/testsuite/gfortran.dg/pr37287-1.f90 index c2d42e6..ca8b879

[committed][PR65107] Add missing cleanup in gfortran.dg/read_eof_8.f90

2015-02-18 Thread Tom de Vries
in the test directories. Committed as trivial. Thanks, - Tom 2015-02-18 Tom de Vries t...@codesourcery.com PR testsuite/65107 * gfortran.dg/read_eof_8.f90: Add missing close. gcc/testsuite/gfortran.dg/read_eof_8.f90 | 1 + 1 file changed, 1 insertion(+) diff --git a/gcc/testsuite

[committed] Add missing cleanup in gfortran.dg/fmt_cache_1.f

2015-02-18 Thread Tom de Vries
. Committed as obvious. Thanks, - Tom 2015-02-18 Tom de Vries t...@codesourcery.com * gfortran.dg/fmt_cache_1.f: Add missing close. --- gcc/testsuite/gfortran.dg/fmt_cache_1.f | 1 + 1 file changed, 1 insertion(+) diff --git a/gcc/testsuite/gfortran.dg/fmt_cache_1.f b/gcc/testsuite/gfortran.dg

[committed] Add missing cleanup in gfortran.dg/finalize_28.f90

2015-02-18 Thread Tom de Vries
Hi, I found finalize_28.f90.003t.original in a gfortran test directory. This patch adds the missing cleanup-tree-dump. Tested by running the test-case and checking that the file does not occur anymore in the test directory. Committed as obvious. Thanks, - Tom 2015-02-18 Tom de Vries t

[committed] Add missing cleanup in gfortran.dg/coarray_35a.f90

2015-02-18 Thread Tom de Vries
2015-02-18 Tom de Vries t...@codesourcery.com * gfortran.dg/coarray_35a.f90: Add missing cleanup-modules. --- gcc/testsuite/gfortran.dg/coarray_35a.f90 | 1 + 1 file changed, 1 insertion(+) diff --git a/gcc/testsuite/gfortran.dg/coarray_35a.f90 b/gcc/testsuite/gfortran.dg/coarray_35a.f90

Re: [Patch] Add option ftree-stdarg-opt

2015-02-18 Thread Tom de Vries
On 17-02-15 13:26, Richard Biener wrote: On Tue, Feb 17, 2015 at 1:12 PM, Tom de Vries tom_devr...@mentor.com wrote: Hi, this patch adds option ftree-stdarg-opt, which switches pass_stdarg on or off. Pass_stdarg does an optimization on cfun-va_list_gpr/fpr_size, and since it's an optimization

[committed] Add gcc.target/x86_64/abi/callabi/vaarg-6.c

2015-02-17 Thread Tom de Vries
Committed new test-case. Thanks, - Tom 2015-02-17 Tom de Vries t...@codesourcery.com * gcc.target/x86_64/abi/callabi/vaarg-6.c: New test. --- .../gcc.target/x86_64/abi/callabi/vaarg-6.c| 40 ++ 1 file changed, 40 insertions(+) create mode 100644 gcc/testsuite

[stage1] Postpone expanding va_arg until pass_stdarg

2015-02-19 Thread Tom de Vries
Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002-Add-gimple_find_sub_bbs.patch - 0003-Factor-optimize_va_list_gpr_fpr_size-out-of-pass_std.patch - 0004-Handle-internal_fn-in-operand_equal_p.patch -

[PATCH][5/5] Postpone expanding va_arg until pass_stdarg

2015-02-19 Thread Tom de Vries
On 19-02-15 11:29, Tom de Vries wrote: Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002-Add-gimple_find_sub_bbs.patch - 0003-Factor-optimize_va_list_gpr_fpr_size-out-of-pass_std.patch - 0004-Handle-internal_fn

[PATCH][1/5] Disable lang_hooks.gimplify_expr in free_lang_data

2015-02-19 Thread Tom de Vries
On 19-02-15 11:29, Tom de Vries wrote: Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002-Add-gimple_find_sub_bbs.patch - 0003-Factor-optimize_va_list_gpr_fpr_size-out-of-pass_std.patch - 0004-Handle-internal_fn

[PATCH][3/5] Factor optimize_va_list_gpr_fpr_size out of pass_stdarg::execute

2015-02-19 Thread Tom de Vries
On 19-02-15 11:29, Tom de Vries wrote: Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002-Add-gimple_find_sub_bbs.patch - 0003-Factor-optimize_va_list_gpr_fpr_size-out-of-pass_std.patch - 0004-Handle-internal_fn

[PATCH][4/5] Handle internal_fn in operand_equal_p

2015-02-19 Thread Tom de Vries
On 19-02-15 11:29, Tom de Vries wrote: Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002-Add-gimple_find_sub_bbs.patch - 0003-Factor-optimize_va_list_gpr_fpr_size-out-of-pass_std.patch - 0004-Handle-internal_fn

[PATCH][2/5] Add gimple_find_sub_bbs

2015-02-19 Thread Tom de Vries
On 19-02-15 11:29, Tom de Vries wrote: Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002-Add-gimple_find_sub_bbs.patch - 0003-Factor-optimize_va_list_gpr_fpr_size-out-of-pass_std.patch - 0004-Handle-internal_fn

[PATCH][testsuite] Abort on failure in gcc.dg/pr30957-1.c

2015-02-20 Thread Tom de Vries
Hi, this patch reverses the abort logic in pr30957-1.c, such that it aborts on failure rather than on success. OK for stage4? Thanks, - Tom 2015-02-20 Tom de Vries t...@codesourcery.com * gcc.dg/pr30957-1.c (main): Abort on failure. --- gcc/testsuite/gcc.dg/pr30957-1.c | 6

Re: [PATCH][testsuite] Abort on failure in gcc.dg/pr30957-1.c

2015-02-20 Thread Tom de Vries
On 20-02-15 10:42, Jakub Jelinek wrote: On Fri, Feb 20, 2015 at 10:25:54AM +0100, Tom de Vries wrote: this patch reverses the abort logic in pr30957-1.c, such that it aborts on failure rather than on success. That sounds really weird. From the description it looks like it is a known bug

Re: [PATCH][4/5] Handle internal_fn in operand_equal_p

2015-02-19 Thread Tom de Vries
On 19-02-15 14:07, Richard Biener wrote: On Thu, 19 Feb 2015, Jakub Jelinek wrote: On Thu, Feb 19, 2015 at 01:44:46PM +0100, Richard Biener wrote: I'd call it a bug though, and we do have internal fns in generic already thus the issue is latent (with ubsan at least). Which means ok for trunk

[Patch] Add option ftree-stdarg-opt

2015-02-17 Thread Tom de Vries
or documentation fix, so it doesn't classify as a stage 4 patch. I could imagine it still being included in stage4 because it adds the possibility for a workaround in case of problems. Bootstrapped and reg-tested on x86_64. OK for stage1 (or even stage 4)? Thanks, - Tom 2015-02-17 Tom de Vries t

Re: Add to maintainers list.

2015-01-25 Thread Tom de Vries
On 24-01-15 20:41, Gerald Pfeifer wrote: Hi Alex, On Friday 2014-11-21 10:07, Alex Velenko wrote: Can someone, please, approve? we tried to document this in https://gcc.gnu.org/svnwrite.html . Can you perhaps suggest a way for us to improve this to make it more clear or easier to find?

Re: [PATCH x86] Add march/mtune=knl

2015-01-24 Thread Tom de Vries
patch ok for stage4 trunk? Thanks, - Tom 2015-01-24 Tom de Vries t...@codesourcery.com * gcc.target/i386/funcspec-5.c: Replace avx512 with avx512f. --- gcc/testsuite/gcc.target/i386/funcspec-5.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/gcc/testsuite/gcc.target

[PATCH, fortran] Add gfc_define_builtin_with_spec

2015-01-09 Thread Tom de Vries
no user yet in trunk, so I've declared it unused. Bootstrapped and reg-tested on x86_64. OK for stage 3 trunk? Thanks, - Tom 2015-01-09 Tom de Vries t...@codesourcery.com * f95-lang.c (gfc_define_builtin_with_spec): New function. --- gcc/fortran/f95-lang.c | 21 + 1 file

[PING^2] Fix gcc_assert in expand_omp_for_static_chunk

2015-01-09 Thread Tom de Vries
On 26-11-14 14:25, Tom de Vries wrote: On 12-11-14 11:00, Tom de Vries wrote: Jakub, this patch fixes a gcc_assert in expand_omp_for_static_chunk. The assert follows a loop with composite loop condition: ... vecedge_var_map *head = redirect_edge_var_map_vector (re); ene

Re: [PATCH] Handle CALL_INSN_FUNCTION_USAGE clobbers in regcprop.c

2015-01-09 Thread Tom de Vries
On 09-01-15 11:48, Jakub Jelinek wrote: On Fri, Jan 09, 2015 at 11:35:41AM +0100, Tom de Vries wrote: 2015-01-09 Tom de Vries t...@codesourcery.com PR rtl-optimization/64539 * regcprop.c (copyprop_hardreg_forward_1): Handle clobbers in CALL_INSN_FUNCTION_USAGE

Re: [PATCH, fortran] Add gfc_define_builtin_with_spec

2015-01-09 Thread Tom de Vries
janv. 2015 à 16:37, Tom de Vries tom_devr...@mentor.com a écrit : Jakub, For the oacc kernels patch series I need a fortran builtin with fn spec attribute (as mentioned here: https://gcc.gnu.org/ml/gcc/2014-12/msg1.html ). Attached patch adds a function gfc_define_builtin_with_spec that allows

[PATCH] Handle CALL_INSN_FUNCTION_USAGE clobbers in regcprop.c

2015-01-09 Thread Tom de Vries
after the earlier 'note_stores (PATTERN (insn), kill_clobbered_value, vd)' call. Terry reported that the patch fixes the problem for him. Bootstrapped and reg-tested on x86_64, no issues found. OK for stage3 trunk? Thanks, - Tom 2015-01-09 Tom de Vries t...@codesourcery.com PR rtl

Re: [PATCH] Fix PR64764 Fix scan-tree-dump for scop-19.c

2015-02-09 Thread Tom de Vries
On 09-02-15 18:23, Dominique Dhumieres wrote: Tom, After these changes I have the following regressions on x86_64-apple-darwin1[04]*: FAIL: gcc.dg/uninit-19.c (test for warnings, line 22) FAIL: gcc.dg/uninit-19.c (test for excess errors) FAIL: gcc.dg/graphite/scop-19.c scan-tree-dump-times

Re: [PATCH] Fix PR64764 Fix scan-tree-dump for scop-19.c

2015-02-10 Thread Tom de Vries
On 09-02-15 20:01, Dominique d'Humières wrote: Le 9 févr. 2015 à 19:10, Tom de Vries tom_devr...@mentor.com mailto:tom_devr...@mentor.com a écrit : On 09-02-15 18:23, Dominique Dhumieres wrote: Tom, After these changes I have the following regressions on x86_64-apple-darwin1[04]*: FAIL

Re: [PATCH] Fix PR64764

2015-02-09 Thread Tom de Vries
On 09-02-15 09:59, Richard Biener wrote: On Thu, 5 Feb 2015, Tom de Vries wrote: On 26-01-15 15:47, Richard Biener wrote: Index: gcc/testsuite/gcc.dg/uninit-19.c === --- gcc/testsuite/gcc.dg/uninit-19.c(revision 0) +++ gcc

Re: [PATCH, stage1] Make parloops gate more strict

2015-03-18 Thread Tom de Vries
On 18-03-15 11:16, Richard Biener wrote: On Fri, Mar 13, 2015 at 4:28 PM, Tom de Vries tom_devr...@mentor.com wrote: On 13-03-15 13:36, Richard Biener wrote: On Fri, Mar 13, 2015 at 1:07 PM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Mar 13, 2015 at 01:04:57PM +0100, Richard Biener wrote

[PATCH][2/3][PR65458] Mark omp thread functions as parallelized

2015-03-18 Thread Tom de Vries
de Vries t...@codesourcery.com PR tree-optimization/65458 * omp-low.c: Add include of tree-parloops.h. (expand_omp_taskreg): Call mark_parallelized_function for child_fn. * tree-parloops.c (mark_parallelized_function): New function. Factor out of .. (create_loop_fn): ... here. * tree

[PATCH][3/3][PR65460] Mark offloaded functions as parallelized

2015-03-18 Thread Tom de Vries
de Vries t...@codesourcery.com PR tree-optimization/65460 * omp-low.c (expand_omp_target): Call mark_parallelized_function for child_fn. --- gcc/omp-low.c | 1 + 1 file changed, 1 insertion(+) diff --git a/gcc/omp-low.c b/gcc/omp-low.c index c5c0ccf..e7ceee2 100644 --- a/gcc/omp-low.c +++ b

[PATCH][1/3] Make parallelize_loops static

2015-03-18 Thread Tom de Vries
Tom de Vries t...@codesourcery.com * tree-parloops.c (parallelize_loops): Make static. * tree-parloops.h (parallelize_loops): Remove extern declaration. --- gcc/tree-parloops.c | 2 +- gcc/tree-parloops.h | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/gcc/tree-parloops.c b

Re: [PATCH, stage1] Make parloops gate more strict

2015-03-18 Thread Tom de Vries
On 18-03-15 12:18, Richard Biener wrote: On Wed, Mar 18, 2015 at 12:03 PM, Tom de Vries tom_devr...@mentor.com wrote: On 18-03-15 11:16, Richard Biener wrote: On Fri, Mar 13, 2015 at 4:28 PM, Tom de Vries tom_devr...@mentor.com wrote: On 13-03-15 13:36, Richard Biener wrote: On Fri, Mar

Re: [PATCH, i386 testsuite]: Require nonpic target for some tests

2015-03-16 Thread Tom de Vries
On 12-03-15 11:51, Uros Bizjak wrote: On Thu, Mar 12, 2015 at 11:41 AM, Tom de Vries tom_devr...@mentor.com wrote: Attached patch adds nonpic target requirement for some (obvious) cases, where data access or PIC register setup confuses scan-asms. 2015-01-30 Uros Bizjak ubiz...@gmail.com

[PATCH] Fix fdump-passes

2015-03-19 Thread Tom de Vries
ChangeLog entry, bootstrapped and reg-tested on x86_64. OK for stage1 (or stage4) trunk? Thanks, - Tom Fix fdump-passes 2015-03-19 Tom de Vries t...@codesourcery.com * function.c (push_dummy_function): New function. (init_dummy_function_start): Use push_dummy_function. (pop_dummy_function): New

Re: [PATCH][2/3][PR65458] Mark omp thread functions as parallelized

2015-03-19 Thread Tom de Vries
On 19-03-15 12:11, Jakub Jelinek wrote: On Thu, Mar 19, 2015 at 12:02:01PM +0100, Tom de Vries wrote: +void +mark_parallelized_function (tree fndecl) +{ + cgraph_node *node = cgraph_node::get (fndecl); + gcc_assert (node != NULL); + node-parallelized_function = 1; } I'm not convinced we

Re: [PATCH][2/3][PR65458] Mark omp thread functions as parallelized

2015-03-19 Thread Tom de Vries
On 18-03-15 18:25, Jakub Jelinek wrote: On Wed, Mar 18, 2015 at 06:21:51PM +0100, Tom de Vries wrote: this patch fixes PR65458. The patch marks omp thread functions as parallelized, which means the parloops pass no longer attempts to modify that function. Bootstrapped and reg-tested on x86_64

Re: [PATCH][3/3][PR65460] Mark offloaded functions as parallelized

2015-03-19 Thread Tom de Vries
On 18-03-15 18:22, Tom de Vries wrote: Hi, this patch fixes PR65460. The patch marks offloaded functions as parallelized, which means the parloops pass no longer attempts to modify that function. Updated patch to postpone mark_parallelized_function until the corresponding cgraph_node

Re: [PATCH] Fix warnings from including fdl.texi into gnat-style.texi

2015-03-20 Thread Tom de Vries
On 22-02-15 18:37, Arnaud Charlet wrote: While generating gnat-style.info, we see these warnings: ... src/gcc/doc/include/fdl.texi:33: warning: node `Index' is next for `GNU Free Documentation License' in menu but not in sectioning src/gcc/doc/include/fdl.texi:33: warning: node `Top' is up for

Re: [PATCH][2/3][PR65458] Mark omp thread functions as parallelized

2015-03-20 Thread Tom de Vries
On 19-03-15 12:29, Jakub Jelinek wrote: On Thu, Mar 19, 2015 at 12:27:04PM +0100, Tom de Vries wrote: Sure, I can update that, I'll retest and repost. Yes, please. Probably the tree-parloops.h include will not be needed either then. Updated to eliminate mark_parallelized_function

Re: [PATCH][3/3][PR65460] Mark offloaded functions as parallelized

2015-03-20 Thread Tom de Vries
On 19-03-15 12:05, Tom de Vries wrote: On 18-03-15 18:22, Tom de Vries wrote: Hi, this patch fixes PR65460. The patch marks offloaded functions as parallelized, which means the parloops pass no longer attempts to modify that function. Updated patch to postpone mark_parallelized_function

Re: [PATCH][2/3][PR65458] Mark omp thread functions as parallelized

2015-03-20 Thread Tom de Vries
On 20-03-15 12:57, Jakub Jelinek wrote: @@ -5569,6 +5569,7 @@ expand_omp_taskreg (struct omp_region *region) /* Inform the callgraph about the new function. */ DECL_STRUCT_FUNCTION (child_fn)-curr_properties = cfun-curr_properties; cgraph_node::add_new_function

Re: [PATCH][3/3][PR65460] Mark offloaded functions as parallelized

2015-03-21 Thread Tom de Vries
On 20-03-15 12:38, Tom de Vries wrote: On 19-03-15 12:05, Tom de Vries wrote: On 18-03-15 18:22, Tom de Vries wrote: Hi, this patch fixes PR65460. The patch marks offloaded functions as parallelized, which means the parloops pass no longer attempts to modify that function. Updated patch

[PING] [PATCH][5a/5] Postpone expanding va_arg until pass_stdarg

2015-03-10 Thread Tom de Vries
[stage1 ping] On 22-02-15 14:13, Tom de Vries wrote: On 19-02-15 14:03, Richard Biener wrote: On Thu, 19 Feb 2015, Tom de Vries wrote: On 19-02-15 11:29, Tom de Vries wrote: Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002

[PING] [PATCH][5b/5] Avoid running expand_ifn_va_arg_1 in functions without va_arg

2015-03-10 Thread Tom de Vries
[stage1 ping] [was: Postpone expanding va_arg until pass_stdarg] On 24-02-15 07:48, Tom de Vries wrote: On 23-02-15 11:09, Tom de Vries wrote: On 23-02-15 09:26, Michael Matz wrote: Hi, On Sun, 22 Feb 2015, Tom de Vries wrote: Btw, I'm wondering if as run-time optimization we can

[PATCH, stage1] Make parloops gate more strict

2015-03-13 Thread Tom de Vries
. OK for stage1 trunk? Thanks, - Tom 2015-03-10 Tom de Vries t...@codesourcery.com * tree-parloops.c (parallelize_loops): Move early-out tests from here ... (pass_parallelize_loops::execute): ... and here ... (pass_parallelize_loops::gate): ... to here. diff --git a/gcc/tree-parloops.c b/gcc

Re: [PATCH, stage1] Make parloops gate more strict

2015-03-13 Thread Tom de Vries
On 13-03-15 11:36, Richard Biener wrote: On Fri, Mar 13, 2015 at 11:32 AM, Tom de Vries tom_devr...@mentor.com wrote: Hi, this patch moves a bunch of early-out tests from the parloops pass to the gate function. The only effect is for functions that we don't consider at all for parallelization

Re: [PATCH, stage1] Make parloops gate more strict

2015-03-13 Thread Tom de Vries
On 13-03-15 13:36, Richard Biener wrote: On Fri, Mar 13, 2015 at 1:07 PM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Mar 13, 2015 at 01:04:57PM +0100, Richard Biener wrote: Not really (I don't like -fdump-passes ...), but we need to make sure that -fdump-passes doesn't crash (because it

Re: [PATCH, i386 testsuite]: Require nonpic target for some tests

2015-03-12 Thread Tom de Vries
On 12-03-15 10:57, Uros Bizjak wrote: On Thu, Mar 12, 2015 at 9:11 AM, Tom de Vries tom_devr...@mentor.com wrote: Attached patch adds nonpic target requirement for some (obvious) cases, where data access or PIC register setup confuses scan-asms. 2015-01-30 Uros Bizjak ubiz...@gmail.com

Re: [PATCH, i386 testsuite]: Require nonpic target for some tests

2015-03-12 Thread Tom de Vries
. * gcc.target/i386/fuse-caller-save-xmm.c: Ditto. * gcc.target/i386/fuse-caller-save.c: Ditto. Hi, I've reverted this part of the patch. The scans were failing because the -fipa-ra optimization was broken for -m32 -fpic (PR64895). Thanks, - Tom 2015-03-12 Tom de Vries t

[Committed][PR64895] Use actual_call_used_reg_set to find conflicting regs

2015-03-12 Thread Tom de Vries
?id=64895#c7 ): ... The patch looks ok to me. Tom, could you prepare the patch (check it mostly for x86-64 bootstrap and testsuite) and commit it to the trunk. I approve it. ... Thanks, - Tom 2015-03-12 Tom de Vries t...@codesourcery.com PR rtl-optimization/64895 * lra-lives.c

[PATCH][PR65511] Fix edge probabilities in gimple_duplicate_sese_tail

2015-03-30 Thread Tom de Vries
) ;; 14 [1.0%] (FALSE_VALUE,EXECUTABLE) ... Bootstrapped and reg-tested on x86_64. OK for, I'd guess, stage1? Thanks, - Tom Fix edge probabilities in gimple_duplicate_sese_tail 2015-03-27 Tom de Vries t...@codesourcery.com PR tree-optimization/65511 * tree-cfg.c

Re: [PATCH][PR65511] Fix edge probabilities in gimple_duplicate_sese_tail

2015-03-30 Thread Tom de Vries
Tom de Vries t...@codesourcery.com * tree-cfg.c (move_sese_region_to_fn): Add entry_count and exit_count arrays, and used the to propagate edge counts alongside edge probabilities. --- gcc/tree-cfg.c | 10 ++ 1 file changed, 10 insertions(+) diff --git a/gcc/tree-cfg.c b/gcc/tree

Re: [PATCH][PR65511] Fix edge probabilities in gimple_duplicate_sese_tail

2015-03-30 Thread Tom de Vries
on x86_64 are ok? Thanks, - Tom Fix edge probabilities in gimple_duplicate_sese_tail 2015-03-27 Tom de Vries t...@codesourcery.com PR tree-optimization/65511 * tree-cfg.c (gimple_duplicate_sese_tail): Fix edge probabilities and counts. * gcc.dg/parloops-prob.c: New test. --- gcc/testsuite

[PATCH][ada][PR65490] Fix bzero warning in child_setup_tty

2015-03-30 Thread Tom de Vries
, - Tom Fix bzero warning in child_setup_tty 2015-03-30 Tom de Vries t...@codesourcery.com PR ada/65490 * terminals.c (child_setup_tty): Fix warning 'argument to sizeof in bzero call is the same expression as the destination'. --- gcc/ada/terminals.c | 4 ++-- 1 file changed, 2 insertions(+), 2

[Committed][testsuite][PR65594] Add verification to libgomp.graphite/force-parallel-6.c

2015-03-27 Thread Tom de Vries
Hi, this patch adds verification to libgomp.graphite/force-parallel-6.c. Committed as approved by Jakub here: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65594#c8 . Thanks, - Tom Add verification to libgomp.graphite/force-parallel-6.c 2015-03-27 Tom de Vries t...@codesourcery.com

[Committed][testsuite][PR65594] Scale down libgomp.graphite/force-parallel-6.c

2015-03-27 Thread Tom de Vries
Scale down libgomp.graphite/force-parallel-6.c 2015-03-27 Tom de Vries t...@codesourcery.com * testsuite/libgomp.graphite/force-parallel-6.c (M): Define. (foo): Use M for non-inner loops to scale down test-case. --- libgomp/testsuite/libgomp.graphite/force-parallel-6.c | 5 +++-- 1 file changed

[PATCH, stage1][PR65443] Add transform_to_exit_first_loop_alt

2015-03-27 Thread Tom de Vries
; bb newheader: if (ivtmp n + 1) goto bb header; else goto bb exit; bb latch: ivtmp = ivtmp + inc; goto bb newheader ... Bootstrapped and reg-tested on x86_64. OK for stage1 trunk? Thanks, - Tom Add transform_to_exit_first_loop_alt 2015-03-27 Tom de Vries

Re: [PATCH][5/5] Postpone expanding va_arg until pass_stdarg

2015-02-23 Thread Tom de Vries
On 23-02-15 09:26, Michael Matz wrote: Hi, On Sun, 22 Feb 2015, Tom de Vries wrote: Btw, I'm wondering if as run-time optimization we can tentatively set PROP_gimple_lva at the start of the gimple pass, and unset it in gimplify_va_arg_expr. That way we would avoid the loop

Re: [PATCH][2/5] Add gimple_find_sub_bbs

2015-02-22 Thread Tom de Vries
On 19-02-15 13:41, Richard Biener wrote: On Thu, 19 Feb 2015, Tom de Vries wrote: On 19-02-15 11:29, Tom de Vries wrote: Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002-Add-gimple_find_sub_bbs.patch - 0003-Factor

Re: [PATCH][testsuite] Abort on failure in gcc.dg/pr30957-1.c

2015-02-22 Thread Tom de Vries
On 20-02-15 20:18, Mike Stump wrote: On Feb 20, 2015, at 4:36 AM, Tom de Vries tom_devr...@mentor.com wrote: On 20-02-15 10:42, Jakub Jelinek wrote: On Fri, Feb 20, 2015 at 10:25:54AM +0100, Tom de Vries wrote: this patch reverses the abort logic in pr30957-1.c, such that it aborts on failure

Re: [PATCH][4/5] Handle internal_fn in operand_equal_p

2015-02-22 Thread Tom de Vries
On 20-02-15 12:54, Richard Biener wrote: On Thu, 19 Feb 2015, Tom de Vries wrote: On 19-02-15 14:07, Richard Biener wrote: On Thu, 19 Feb 2015, Jakub Jelinek wrote: On Thu, Feb 19, 2015 at 01:44:46PM +0100, Richard Biener wrote: I'd call it a bug though, and we do have internal fns

Re: [PATCH][5/5] Postpone expanding va_arg until pass_stdarg

2015-02-22 Thread Tom de Vries
On 19-02-15 14:03, Richard Biener wrote: On Thu, 19 Feb 2015, Tom de Vries wrote: On 19-02-15 11:29, Tom de Vries wrote: Hi, I'm posting this patch series for stage1: - 0001-Disable-lang_hooks.gimplify_expr-in-free_lang_data.patch - 0002-Add-gimple_find_sub_bbs.patch - 0003-Factor

[PATCH] Use DO_PRAGMA in libgomp.oacc-c-c++-common/reduction-1.c

2015-02-23 Thread Tom de Vries
; } 2015-02-23 Tom de Vries t...@codesourcery.com * testsuite/libgomp.oacc-c-c++-common/reduction-1.c (DO_PRAGMA, check_reduction, apply_bin_int_op, check_reduction_int) (apply_max_op, apply_min_op, check_reduction_minmax, apply_bin_bool_op) (check_reduction_bool): Declare. (test_reductions_int

Re: [PATCH] Use DO_PRAGMA in libgomp.oacc-c-c++-common/reduction-1.c

2015-02-23 Thread Tom de Vries
On 23-02-15 17:08, Jakub Jelinek wrote: On Mon, Feb 23, 2015 at 04:52:56PM +0100, Tom de Vries wrote: The only thing I'm not sure about is the two-level pragma expansion using the apply pragmas. It maximizes factoring out common parts, but it makes things less readable. Tested on x86_64. OK

Re: [PATCH] Use DO_PRAGMA in libgomp.oacc-c-c++-common/reduction-1.c

2015-02-25 Thread Tom de Vries
On 25-02-15 12:40, Thomas Schwinge wrote: Hi! On Mon, 23 Feb 2015 18:14:35 +0100, Tom de Vries tom_devr...@mentor.com wrote: On 23-02-15 17:08, Jakub Jelinek wrote: On Mon, Feb 23, 2015 at 04:52:56PM +0100, Tom de Vries wrote: The only thing I'm not sure about is the two-level pragma

Re: [Ada] convert GNAT doc to sphinx

2015-02-22 Thread Tom de Vries
On 22-02-15 20:15, Arnaud Charlet wrote: Well OK but these are automatically generated now, and this doesn't really answer my question about the documentation of @dircategory. I didn't see a question here: ... As for the @dircategory I do not know, I couldn't find a proper documentation for

[committed][ada] Fix warning in 'Loop Statements' in gnat-style.texi

2015-02-22 Thread Tom de Vries
Hi, Committed this warning fix as obvious. Thanks, - Tom 2015-02-22 Tom de Vries t...@codesourcery.com * gnat-style.texi (@subsection Loop Statements): Replace @noindent by @item, and fix warning '@itemize has text but no @item'. --- gcc/ada/gnat-style.texi | 2 +- 1 file changed, 1

Re: [Ada] convert GNAT doc to sphinx

2015-02-22 Thread Tom de Vries
On 20-02-15 10:17, Arnaud Charlet wrote: Tested on x86_64-pc-linux-gnu, committed on trunk. 2015-02-20 Arnaud Charlet char...@adacore.com * gnat_rm.texi, gnat_ugn.texi: Now automatically generated from sphinx in the doc directory. * doc: New directory containing

Re: [Ada] convert GNAT doc to sphinx

2015-02-22 Thread Tom de Vries
On 22-02-15 19:04, Arnaud Charlet wrote: 2015-02-20 Arnaud Charlet char...@adacore.com * gnat_rm.texi, gnat_ugn.texi: Now automatically generated from sphinx in the doc directory. * doc: New directory containing sphinx versions of gnat_rm and

<    1   2   3   4   5   6   7   8   9   10   >