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

2015-02-22 Thread Tom de Vries
' in menu but not in sectioning ... Attached patch fixes these. Ok for stage4? Thanks, - Tom 2015-02-22 Tom de Vries t...@codesourcery.com PR ada/65102 * doc/include/fdl.texi: Add nodefaultgnufreedocumentationlicensenode ifdef to allow disabling default @node GNU Free Documentation License

[PATCH, PR65126, testsuite] Add additional_sources_used

2015-02-22 Thread Tom de Vries
. With this patch, after running test I no longer find the leftover files in the test directories. Build and reg-tested on x86_64. OK for stage4? Thanks, - Tom 2015-02-22 Tom de Vries t...@codesourcery.com * lib/gcc-defs.exp (additional_sources_used): Add new var. (dg-additional-files-options

Re: [Ada] convert GNAT doc to sphinx

2015-02-23 Thread Tom de Vries
On 22-02-15 21:48, Arnaud Charlet wrote: I didn't see a question here: ... As for the @dircategory I do not know, I couldn't find a proper documentation for this node other than: http://www.gnu.org/software/texinfo/manual/texinfo/html_node/Installing-Dir-Entries.html#index-dircategory which is

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

2015-02-23 Thread Tom de Vries
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 tentatively set PROP_gimple_lva at the start of the gimple pass, and unset it in gimplify_va_arg_expr. That way we

[Committed] Add missing cleanup in gfortran.dg/readwrite_unf_direct_eor_1.f90

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

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

2015-02-24 Thread Tom de Vries
[ forwarding. for some reason, this email didn't make it to gcc-patches ml archive ] Forwarded Message Subject: Re: [PATCH][4/5] Handle internal_fn in operand_equal_p Date: Mon, 23 Feb 2015 10:03:34 +0100 From: Richard Biener rguent...@suse.de To: Tom de Vries tom_devr

[committed] Add missing cleanup in gcc.misc-tests/godump-1.c

2015-02-26 Thread Tom de Vries
Hi, Committed as obvious. Thanks, - Tom 2015-02-26 Tom de Vries t...@codesourcery.com * gcc.misc-tests/godump-1.c: Add missing cleanup of godump-1.out. --- gcc/testsuite/ChangeLog | 4 gcc/testsuite/gcc.misc-tests/godump-1.c | 2 ++ 2 files changed, 6 insertions

[patch, testsuite] Add profopt-target-cleanup

2015-02-26 Thread Tom de Vries
for stage4 trunk? Thanks, - Tom 2015-02-26 Tom de Vries t...@codesourcery.com * lib/profopt.exp (profopt-target-cleanup): New proc. (profopt-execute): Use profopt-target-cleanup to handle cleanup of files from sources from dg-additional-sources. --- gcc/testsuite/lib/profopt.exp | 37

[PATCH, testsuite] Add missing cleanup in run-gcov

2015-02-26 Thread Tom de Vries
Hi, This patch adds cleaning up of uninteresting leftover coverage files in case of xfail, f.i. for gcc.misc-tests/gcov-13.c. OK for stage4 trunk? Thanks, - Tom 2015-02-26 Tom de Vries t...@codesourcery.com * lib/gcov.exp: Cleanup in case of xfail. --- gcc/testsuite/lib/gcov.exp | 3

[committed] Add missing cleanup in gcc.dg/aru-2.c

2015-02-26 Thread Tom de Vries
Hi, committed as obvious. Thanks, - Tom 2015-02-26 Tom de Vries t...@codesourcery.com * gcc.dg/aru-2.c: Add missing cleanup-profile-file. diff --git a/gcc/testsuite/gcc.dg/aru-2.c b/gcc/testsuite/gcc.dg/aru-2.c index d36adc1..054223c 100644 --- a/gcc/testsuite/gcc.dg/aru-2.c +++ b/gcc

[committed] Add missing cleanup in gcc.target/i386/mcount_pic.c

2015-02-26 Thread Tom de Vries
Hi, committed as obvious. Thanks, - Tom 2015-02-26 Tom de Vries t...@codesourcery.com * gcc.target/i386/mcount_pic.c: Add missing cleanup-profile-file. diff --git a/gcc/testsuite/gcc.target/i386/mcount_pic.c b/gcc/testsuite/gcc.target/i386/mcount_pic.c index 806b7d5..1e59995 100644

[committed] Add missing cleanup in pch-init

2015-02-26 Thread Tom de Vries
Hi, committed as obvious. Thanks, - Tom 2015-02-26 Tom de Vries t...@codesourcery.com * lib/dg-pch.exp (pch-init): Add missing cleanup of pchtestn.o. diff --git a/gcc/testsuite/lib/dg-pch.exp b/gcc/testsuite/lib/dg-pch.exp index e250301..9b4f40b 100644 --- a/gcc/testsuite/lib/dg-pch.exp

[committed] Add missing cleanup in g++.dg/gcov/gcov-14.C

2015-02-26 Thread Tom de Vries
Hi, committed as obvious. Thanks, - Tom 2015-02-26 Tom de Vries t...@codesourcery.com * g++.dg/gcov/gcov-14.C: Add cleanup of iostream.gcov, ostream.gcov and locale_facets.h.gcov. diff --git a/gcc/testsuite/g++.dg/gcov/gcov-14.C b/gcc/testsuite/g++.dg/gcov/gcov-14.C index 8f8e388..b069d0e

[committed] Add missing cleanup in go-gc-tests for cmpout

2015-02-26 Thread Tom de Vries
gcc/testsuite/go1: bug206.exe bug328.exe go.log.sep goprint.exe go.sum.sep helloworld.exe issue6899.exe sigchld.exe site.exp ... Committed as obvious. Thanks, - Tom 2015-02-26 Tom de Vries t...@codesourcery.com * go.test/go-test.exp (go-gc-tests): Add missing cleanup for cmpout. diff

Re: [PATCH, stage1][PR65443] Add transform_to_exit_first_loop_alt

2015-04-03 Thread Tom de Vries
On 27-03-15 15:10, Tom de Vries wrote: Hi, this patch fixes PR65443, a todo in the parloops pass for function transform_to_exit_first_loop: ... TODO: the common case is that latch of the loop is empty and immediately follows the loop exit. In this case, it would be better not to copy

[PATCH][PR65802] Mark ifn_va_arg with ECF_NOTHROW

2015-04-21 Thread Tom de Vries
also didn't contain any throwing statements. This patch fixes the ICE by marking ifn_va_arg with ECF_NOTHROW. Bootstrapped and reg-tested on x86_64. OK for trunk? Thanks, - Tom Mark ifn_va_arg with ECF_NOTHROW 2015-04-20 Tom de Vries t...@codesourcery.com PR tree-optimization/65802

[RFC][wwwdocs] bug report reopen policy

2015-04-20 Thread Tom de Vries
Hi, In PR64683 comment 11, Ian mentioned: ... This bug may have the same symptoms but it has a completely different cause. Next time, please do not reopen the bug unless you are certain it has the same cause. Please open a new bug instead. Thanks. ... I couldn't find a similar rule in the

Re: [PATCH] Skip preprocessor directives in mklog

2015-04-21 Thread Tom de Vries
On 21-04-15 13:26, Yury Gribov wrote: Hi all, Contrib/mklog is currently faked by preprocessor directives inside functions to produce invalid ChangeLog. Hi Yury, The effect of the patch on the mklog output using the pastebin input is: ... @@ -2,11 +2,13 @@ 2015-04-21 x y@z - *

Re: [PATCH][PR65802] Mark ifn_va_arg with ECF_NOTHROW

2015-04-26 Thread Tom de Vries
ifn_va_arg as ECF_LEAF 2015-04-26 Tom de Vries t...@codesourcery.com PR tree-optimization/65826 * internal-fn.def: Mark VA_ARG with ECF_LEAF. diff --git a/gcc/internal-fn.def b/gcc/internal-fn.def index 7e19313..ba5c2c1 100644 --- a/gcc/internal-fn.def +++ b/gcc/internal-fn.def @@ -62,4 +62,4

Re: [PATCH][PR65823] Fix va_arg ap_copy nop detection

2015-04-22 Thread Tom de Vries
On 22-04-15 10:06, Richard Biener wrote: On Wed, Apr 22, 2015 at 9:41 AM, Tom de Vries tom_devr...@mentor.com wrote: Hi, this patch fixes PR65823. SNIP The patches fixes the problem by using operand_equal_p to do the equality test. Bootstrapped and reg-tested on x86_64. Did minimal

Re: [PATCH][PR65802] Mark ifn_va_arg with ECF_NOTHROW

2015-04-24 Thread Tom de Vries
On 24-04-15 05:25, Bin.Cheng wrote: On Tue, Apr 21, 2015 at 3:10 PM, Tom de Vries tom_devr...@mentor.com wrote: Hi, this patch fixes PR65802. diff --git a/gcc/testsuite/g++.dg/ pr65802.C b/gcc/testsuite/g++.dg/pr65802.C new file mode 100644 index 000..26e5317 --- /dev/null +++ b/gcc

[PATCH][PR65818][bootstrap,hppa] Return side-effect free result in gimplify_va_arg_internal

2015-04-25 Thread Tom de Vries
side-effect free result in gimplify_va_arg_internal 2015-04-22 Tom de Vries t...@codesourcery.com PR tree-optimization/65818 * gimplify.c (gimplify_va_arg_internal): Ensure that only side-effect free values are returned. --- gcc/gimplify.c | 7 ++- 1 file changed, 6 insertions(+), 1

[PATCH][PR65823] Fix va_arg ap_copy nop detection

2015-04-22 Thread Tom de Vries
-22 Tom de Vries t...@codesourcery.com PR tree-optimization/65823 * gimplify.c (gimplify_modify_expr): Use operand_equal_p to test for equality between ap_copy and ap. --- gcc/gimplify.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/gimplify.c b/gcc/gimplify.c index

[PATCH][PR65893] Move pass_stdarg to after pass_dce in pass_all_optimizations

2015-04-29 Thread Tom de Vries
); NEXT_PASS (pass_rename_ssa_copies); ... Bootstrapped and reg-tested on x86_64. OK for trunk? Thanks, - Tom Move pass_stdarg to after pass_dce in pass_all_optimizations 2015-04-29 Tom de Vries t...@codesourcery.com PR tree-optimization/65893 * passes.def (pass_all_optimizations): Move

Re: [PATCH][PR65818][bootstrap,hppa] Return side-effect free result in gimplify_va_arg_internal

2015-04-28 Thread Tom de Vries
On 28-04-15 12:34, Richard Biener wrote: On Mon, Apr 27, 2015 at 5:04 PM, Tom de Vries tom_devr...@mentor.com wrote: On 27-04-15 15:40, Richard Biener wrote: On Mon, Apr 27, 2015 at 3:06 PM, Tom de Vries tom_devr...@mentor.com wrote: On 27-04-15 10:17, Richard Biener wrote: This patch

Re: [PATCH][PR65823] Fix va_arg ap_copy nop detection

2015-04-27 Thread Tom de Vries
On 22-04-15 15:50, Richard Biener wrote: On Wed, Apr 22, 2015 at 3:38 PM, Tom de Vries tom_devr...@mentor.com wrote: On 22-04-15 10:06, Richard Biener wrote: On Wed, Apr 22, 2015 at 9:41 AM, Tom de Vries tom_devr...@mentor.com wrote: Hi, this patch fixes PR65823. SNIP The patches

Re: [PATCH][PR65818][bootstrap,hppa] Return side-effect free result in gimplify_va_arg_internal

2015-04-27 Thread Tom de Vries
(with in front of expr). OK if bootstrap and reg-test on x86_64 succeeds? Thanks, - Tom Return side-effect free result in gimplify_va_arg_internal 2015-04-27 Tom de Vries t...@codesourcery.com PR tree-optimization/65818 * gimplify.c (gimplify_va_arg_internal): Ensure that only side-effect free values

Re: [PATCH][PR65818][bootstrap,hppa] Return side-effect free result in gimplify_va_arg_internal

2015-04-27 Thread Tom de Vries
On 27-04-15 15:40, Richard Biener wrote: On Mon, Apr 27, 2015 at 3:06 PM, Tom de Vries tom_devr...@mentor.com wrote: On 27-04-15 10:17, Richard Biener wrote: This patch fixes that by gimplifying the address expression of the mem-ref returned by the target hook (borrowing code from

[committed] Fix example in header comment in tree-call-cdce.c

2015-04-28 Thread Tom de Vries
] bb 4: return; } ... The error condition for log is d = 0.0 (zero is pole error, negative is domain error). Updated the example in the header comment in tree-calll-cdce.c accordingly. Thanks, - Tom Fix example in header comment in tree-call-cdce.c 2015-04-28 Tom de Vries t

Re: [PATCH][PR65823] Fix va_arg ap_copy nop detection

2015-04-28 Thread Tom de Vries
On 27-04-15 09:45, Tom de Vries wrote: On 22-04-15 15:50, Richard Biener wrote: On Wed, Apr 22, 2015 at 3:38 PM, Tom de Vries tom_devr...@mentor.com wrote: On 22-04-15 10:06, Richard Biener wrote: On Wed, Apr 22, 2015 at 9:41 AM, Tom de Vries tom_devr...@mentor.com wrote: Hi, this patch

[PATCH][PR66013] Update address_taken after ifn_va_arg expansion

2015-05-08 Thread Tom de Vries
*)ap.4_9]; ... and the pass_stdarg optimization succeeds now: ... f2: va_list escapes 0, needs to save 4 GPR units and all FPR units. ... Bootstrapped and reg-tested on x86_64 with and without -m32. OK for trunk? Thanks, - Tom Update address_taken after ifn_va_arg expansion 2015-05-08 Tom de

Re: [PATCH][PR66010] Don't take address of ap unless necessary

2015-05-11 Thread Tom de Vries
On 11-05-15 09:47, Richard Biener wrote: Bootstrapped and reg-tested on x86_64, with and without -m32. OK for trunk? [ FWIW, I suspect this patch will make life easier for the reimplementation of the pass_stdarg optimization using ifn_va_arg. ] + if (canon_va_type != NULL) +{ + if

[PATCH,testsuite] Xfail gcc.dg/tree-ssa/stdarg-2.c f15 scans

2015-05-05 Thread Tom de Vries
to commit two patches. - The first patch undoes the modification of stdarg-2.c as committed in the original patch series (omitted). - The second patch adds appropriate xfails (attached). OK for trunk? Thanks, - Tom [PATCH 2/2] Xfail gcc.dg/tree-ssa/stdarg-2.c f15 scans 2015-05-05 Tom de Vries t

Re: [PATCH][PR66013] Update address_taken after ifn_va_arg expansion

2015-05-08 Thread Tom de Vries
On 08-05-15 17:31, Richard Biener wrote: OK for trunk? As noted in one of the PRs I think that it is the proper time to re-implement the stdarg optimization on the un-lowered form which should also fix this. AFAIU, the implementation of the stdarg optimization on the un-lowered form should

[PATCH][PR66010] Don't take address of ap unless necessary

2015-05-08 Thread Tom de Vries
on x86_64, with and without -m32. OK for trunk? [ FWIW, I suspect this patch will make life easier for the reimplementation of the pass_stdarg optimization using ifn_va_arg. ] Thanks, - Tom Don't take address of ap unless necessary 2015-05-08 Tom de Vries t...@codesourcery.com PR tree

[PATCH, 5/5] check_GNU_style.sh: Fix tab size in 80 characters check

2015-05-12 Thread Tom de Vries
Hi, this patch fixes a problem in the 80 characters length check. Currently tab width is not properly calculated. The patch uses expand to interpret tabs properly. OK for trunk? Thanks, - Tom [PATCH 5/5] check_GNU_style.sh: Fix tab size in 80 characters check 2015-05-11 Tom de Vries t

[PATCH, 3/5] check_GNU_style.sh: Read either from stdin, or from files

2015-05-12 Thread Tom de Vries
, or from files 2015-05-11 Tom de Vries t...@codesourcery.com * check_GNU_style.sh: Read either from stdin, or from files. --- contrib/check_GNU_style.sh | 22 -- 1 file changed, 16 insertions(+), 6 deletions(-) diff --git a/contrib/check_GNU_style.sh b/contrib

[PATCH, 4/5] check_GNU_style.sh: Put stdin into temp file

2015-05-12 Thread Tom de Vries
Hi, this patch makes sure we dump stdin into a file before we handle it in the rest of the script. This makes the stdin handling more like the handling of normal files. OK for trunk? Thanks, - Tom [PATCH 4/5] check_GNU_style.sh: Put stdin into temp file 2015-05-11 Tom de Vries t

[PATCH, 2/5] check_GNU_style.sh: Check file presence

2015-05-12 Thread Tom de Vries
Hi, this patch checks that the files specified on the check_GNU_style.sh command line are present. OK for trunk? Thanks, - Tom [PATCH 2/5] check_GNU_style.sh: Check file presence 2015-05-11 Tom de Vries t...@codesourcery.com * check_GNU_style.sh: Check if files exists. --- contrib

[PATCH, 1/5] check_GNU_style.sh: Don't use filename prefix for one patch

2015-05-12 Thread Tom de Vries
1/5] check_GNU_style.sh: Don't use filename prefix for one patch 2015-05-11 Tom de Vries t...@codesourcery.com * check_GNU_style.sh: Don't use a filename prefix if we're only processing one patch file. --- contrib/check_GNU_style.sh | 14 +++--- 1 file changed, 11 insertions(+), 3

Re: [PATCH][PR66010] Don't take address of ap unless necessary

2015-05-12 Thread Tom de Vries
On 12-05-15 09:45, Richard Biener wrote: On Mon, 11 May 2015, Tom de Vries wrote: On 11-05-15 09:47, Richard Biener wrote: Bootstrapped and reg-tested on x86_64, with and without -m32. OK for trunk? [ FWIW, I suspect this patch will make life easier for the reimplementation

Re: [PATCH][PR66010] Don't take address of ap unless necessary

2015-05-12 Thread Tom de Vries
On 12-05-15 12:12, Michael Matz wrote: Hi, On Fri, 8 May 2015, Tom de Vries wrote: III. Using the patch, before inlining we can see the address operator has been removed in va_arg: ... f2_1 (struct * apD.1832) { intD.6 _4; # .MEM_3 = VDEF .MEM_1(D) # USE = anything # CLB

Re: [PATCH][PR66010] Don't take address of ap unless necessary

2015-05-14 Thread Tom de Vries
and compiled a few va_arg test-cases on the other targets. OK for trunk? Thanks, - Tom Move array-type va_list handling to build_va_arg 2015-05-13 Tom de Vries t...@codesourcery.com * gimplify.c (gimplify_modify_expr): Remove do_deref handling. (gimplify_va_arg_expr): Remove do_deref handling

Re: [PING][PATCH][PR65443] Add transform_to_exit_first_loop_alt

2015-05-14 Thread Tom de Vries
On 20-04-15 14:25, Richard Biener wrote: On Wed, 15 Apr 2015, Tom de Vries wrote: On 03-04-15 14:39, Tom de Vries wrote: On 27-03-15 15:10, Tom de Vries wrote: Hi, this patch fixes PR65443, a todo in the parloops pass for function transform_to_exit_first_loop: ... TODO: the common case

Re: [PING][PR65637] Fix ssa-handling code in expand_omp_for_static_chunk

2015-05-18 Thread Tom de Vries
On 18-05-15 14:53, Tom de Vries wrote: On 15-04-15 15:10, Tom de Vries wrote: Hi, This patch series fixes PR65637. Currently, ssa-handling code in expand_omp_for_static_chunk is dead and not exercised by testing. Ssa-handling code in omp-low.c is only triggered by pass_parallelize_loops

[PING][PR65637] Fix ssa-handling code in expand_omp_for_static_chunk

2015-05-18 Thread Tom de Vries
On 15-04-15 15:10, Tom de Vries wrote: Hi, This patch series fixes PR65637. Currently, ssa-handling code in expand_omp_for_static_chunk is dead and not exercised by testing. Ssa-handling code in omp-low.c is only triggered by pass_parallelize_loops, and that pass doesn't specify a chunk size

Re: [PATCH, 5/5] check_GNU_style.sh: Fix tab size in 80 characters check

2015-05-18 Thread Tom de Vries
is now printed in red, similar to how the other checks show the output. OK for trunk? Thanks, - Tom check_GNU_style.sh: Don't do 80 char check line by line 2015-05-18 Tom de Vries t...@codesourcery.com * check_GNU_style.sh: Add temp files tmp2 and tmp3. (cat_with_prefix): New function, using

Re: [PATCH][PR66010] Don't take address of ap unless necessary

2015-05-12 Thread Tom de Vries
On 12-05-15 12:04, Tom de Vries wrote: Committed with comments below added. Hmm, this causes an ice for s390 in gcc.dg/tree-ssa/stdarg-2.c: ... gimplification failed: ap addr_expr 0x7f132f54fb20 type pointer_type 0x7f132f55f1f8 type array_type 0x7f132f53fe70 va_list type

Re: [gomp4] bootstrap broken, function enclosing_target_ctx defined but not used

2015-05-19 Thread Tom de Vries
[ moved to gcc-patches ml ] On 18-05-15 17:31, Tom de Vries wrote: Thomas, In ran into this bootstrap failure with branch gomp-4_0-branch: ... src/gcc-gomp-4_0-branch/gcc/omp-low.c:2897:1: error: 'omp_context* enclosing_target_ctx(omp_context*)' defined but not used [-Werror=unused-function

[PATCH] Fix va_arg gimplification error for s390

2015-05-13 Thread Tom de Vries
va_arg ap based on do_deref 2015-05-13 Tom de Vries t...@codesourcery.com PR tree-optimization/66010 * gimplify.h (gimplify_va_arg_internal): Remove declaration. * gimplify.c (gimplify_va_arg_internal): Remove and inline into ... * tree-stdarg.c (expand_ifn_va_arg_1): ... here. Choose

Re: [PATCH][PING^2] Skip preprocessor directives in mklog

2015-05-12 Thread Tom de Vries
On 12-05-15 17:33, Yury Gribov wrote: On 05/12/2015 06:23 PM, Diego Novillo wrote: The patch looks fine to me. I'm not really involved in GCC development anymore. I would suggest that this script should be maintained by whoever's been hacking on it the most. It's a simple script, so it

[Committed] check_GNU_style.sh: Improve readability function calls

2015-05-18 Thread Tom de Vries
Hi, committed as obvious. Thanks, - Tom check_GNU_style.sh: Improve readability function calls 2015-05-18 Tom de Vries t...@codesourcery.com * check_GNU_style.sh: Improve readability function calls. --- contrib/check_GNU_style.sh | 7 --- 1 file changed, 4 insertions(+), 3 deletions

check_GNU_style.sh: Don't cat empty file

2015-05-18 Thread Tom de Vries
Hi, committed as obvious. Thanks, - Tom check_GNU_style.sh: Don't cat empty file 2015-05-18 Tom de Vries t...@codesourcery.com * check_GNU_style.sh (g, ag, vg): Don't cat empty file. --- contrib/check_GNU_style.sh | 30 -- 1 file changed, 24 insertions(+), 6

[Committed] check_GNU_style.sh: Declare local vars with local

2015-05-18 Thread Tom de Vries
Hi, committed as obvious. Thanks, - Tom check_GNU_style.sh: Declare local vars with local 2015-05-18 Tom de Vries t...@codesourcery.com * check_GNU_style.sh (g, ag, vg, col): Declare local vars with local. --- contrib/check_GNU_style.sh | 18 +- 1 file changed, 9

Re: check_GNU_style.sh: Don't cat empty file

2015-05-18 Thread Tom de Vries
On 18-05-15 10:31, Andreas Schwab wrote: Tom de Vries tom_devr...@mentor.com writes: diff --git a/contrib/check_GNU_style.sh b/contrib/check_GNU_style.sh index 728c11a..ab59b1e 100755 --- a/contrib/check_GNU_style.sh +++ b/contrib/check_GNU_style.sh @@ -84,10 +84,16 @@ grep $format '^+' $files

[PR65637][PATCH][3/3] Handle 2 preds for fin_bb in expand_omp_for_static_chunk

2015-04-15 Thread Tom de Vries
On 15-04-15 15:10, Tom de Vries wrote: Hi, This patch series fixes PR65637. Currently, ssa-handling code in expand_omp_for_static_chunk is dead and not exercised by testing. Ssa-handling code in omp-low.c is only triggered by pass_parallelize_loops, and that pass doesn't specify a chunk size

[PR65637][PATCH][1/3] Fix gcc_assert in expand_omp_for_static_chunk

2015-04-15 Thread Tom de Vries
On 15-04-15 15:10, Tom de Vries wrote: Hi, This patch series fixes PR65637. Currently, ssa-handling code in expand_omp_for_static_chunk is dead and not exercised by testing. Ssa-handling code in omp-low.c is only triggered by pass_parallelize_loops, and that pass doesn't specify a chunk size

[PR65637] Fix ssa-handling code in expand_omp_for_static_chunk

2015-04-15 Thread Tom de Vries
Hi, This patch series fixes PR65637. Currently, ssa-handling code in expand_omp_for_static_chunk is dead and not exercised by testing. Ssa-handling code in omp-low.c is only triggered by pass_parallelize_loops, and that pass doesn't specify a chunk size on the GIMPLE_OMP_FOR it constructs,

[PING][PATCH][PR65443] Add transform_to_exit_first_loop_alt

2015-04-15 Thread Tom de Vries
On 03-04-15 14:39, Tom de Vries wrote: On 27-03-15 15:10, Tom de Vries wrote: Hi, this patch fixes PR65443, a todo in the parloops pass for function transform_to_exit_first_loop: ... TODO: the common case is that latch of the loop is empty and immediately follows the loop exit

[PR65637][PATCH][2/3] Fix inner loop phi in expand_omp_for_static_chunk

2015-04-15 Thread Tom de Vries
On 15-04-15 15:10, Tom de Vries wrote: Hi, This patch series fixes PR65637. Currently, ssa-handling code in expand_omp_for_static_chunk is dead and not exercised by testing. Ssa-handling code in omp-low.c is only triggered by pass_parallelize_loops, and that pass doesn't specify a chunk size

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

2015-04-16 Thread Tom de Vries
[stage1 ping^2] On 10-03-15 16:30, Tom de Vries wrote: [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

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

2015-04-16 Thread Tom de Vries
[stage1 ping^2] On 10-03-15 16:31, Tom de Vries wrote: [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

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

2015-04-17 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

[gomp4, committed] Add checks for num_gangs(32) in goacc kernels tests

2015-06-02 Thread Tom de Vries
Hi, this patch adds an extra check in the goacc kernels testcases. Committed to gomp-4_0-branch. Thanks, - Tom Add checks for num_gangs(32) in goacc kernels tests 2015-05-28 Tom de Vries t...@codesourcery.com * c-c++-common/goacc/kernels-counter-vars-function-scope.c: Add check

Re: Heads-up: testsuite: removed cleanup-ipa-dump, cleanup-rtl-dump,cleanup-tree-dump, cleanup-dump, cleanup-saved-temps

2015-06-05 Thread Tom de Vries
On 02/06/15 20:40, Bernhard Reutner-Fischer wrote: On June 2, 2015 5:38:06 PM GMT+02:00, Bernhard Reutner-Fischerrep.dot@gmail.com wrote: On June 2, 2015 2:08:47 PM GMT+02:00, Richard Biener rguent...@suse.de wrote: Committed. Seems to cause half of the vectorizer tests to be dropped

Re: [PATCH, PR66444] Handle -fipa-ra in reload_combine

2015-06-08 Thread Tom de Vries
On 08/06/15 17:31, Jakub Jelinek wrote: On Mon, Jun 08, 2015 at 02:04:12PM +0200, Tom de Vries wrote: this patch fixes PR66444, a problem with -fipa-ra in reload_combine. The problem is that for the test-case, reload_combine combines these two insns: Please work out with Vlad whether

Re: [PATCH, PR66444] Handle -fipa-ra in reload_combine

2015-06-08 Thread Tom de Vries
On 08/06/15 17:31, Jakub Jelinek wrote: On Mon, Jun 08, 2015 at 02:04:12PM +0200, Tom de Vries wrote: this patch fixes PR66444, a problem with -fipa-ra in reload_combine. The problem is that for the test-case, reload_combine combines these two insns: Please work out with Vlad whether

Re: [PING][PATCH][PR65443] Add transform_to_exit_first_loop_alt

2015-06-08 Thread Tom de Vries
On 08/06/15 17:55, Thomas Schwinge wrote: Hi Tom! On Mon, 8 Jun 2015 12:43:01 +0200, Tom de Vries tom_devr...@mentor.com wrote: There are two problems in try_transform_to_exit_first_loop_alt: 1. In case the latch is not a singleton bb, the function should return false rather than true. 2

Re: [PING][PATCH][PR65443] Add transform_to_exit_first_loop_alt

2015-06-09 Thread Tom de Vries
On 09/06/15 00:05, Tom de Vries wrote: On 08/06/15 17:55, Thomas Schwinge wrote: Hi Tom! On Mon, 8 Jun 2015 12:43:01 +0200, Tom de Vries tom_devr...@mentor.com wrote: There are two problems in try_transform_to_exit_first_loop_alt: 1. In case the latch is not a singleton bb, the function

[patch, testsuite] Remove superfluous -std={c99,gnu99} in libgomp.c

2015-06-05 Thread Tom de Vries
in expand_GOMP_SIMD_LAST_LANE'. OK for trunk? Thanks, - Tom Remove superfluous -std={c99,gnu99} in libgomp.c 2015-06-05 Tom de Vries t...@codesourcery.com * testsuite/libgomp.c/collapse-3.c: Remove superfluous dg-options line with -std={gnu99,c99}. * testsuite/libgomp.c/for-1.c: Same. * testsuite

[patch, testsuite] Remove -fopenmp in dg-options in libgomp.c

2015-06-05 Thread Tom de Vries
for trunk? Thanks, - Tom Remove -fopenmp in dg-options in libgomp.c 2015-06-05 Tom de Vries t...@codesourcery.com * testsuite/libgomp.c/atomic-18.c: Remove superfluous -fopenmp setting in dg-options. * testsuite/libgomp.c/atomic-3.c: Same. * testsuite/libgomp.c/debug-1.c: Same. * testsuite

Re: [PING][PATCH][PR65443] Add transform_to_exit_first_loop_alt

2015-06-08 Thread Tom de Vries
On 04/06/15 10:28, Tom de Vries wrote: I'm ok with the patch and count on you to fix eventual fallout ;) Great, will do. And here is the fallout: * PR66442 - [6 regression] FAIL: gcc.dg/autopar/pr46885.c (test for excess errors) There are two problems

[PATCH, PR66444] Handle -fipa-ra in reload_combine

2015-06-08 Thread Tom de Vries
get_call_reg_set_usage to find out that r8 is actually not killed by the call_insn. Bootstrapped and reg-tested on x86_64 on top of trunk. OK for trunk and gcc-5-branch? Thanks, - Tom Handle -fipa-ra in reload_combine 2015-06-08 Tom de Vries t...@codesourcery.com PR rtl-optimization/66444

[PING^2][PATCH][3/3][PR65460] Mark offloaded functions as parallelized

2015-06-08 Thread Tom de Vries
On 17/04/15 12:08, Tom de Vries wrote: 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

[PING^2][PR65637] Fix ssa-handling code in expand_omp_for_static_chunk

2015-06-08 Thread Tom de Vries
On 18/05/15 14:53, Tom de Vries wrote: On 15-04-15 15:10, Tom de Vries wrote: Hi, This patch series fixes PR65637. Currently, ssa-handling code in expand_omp_for_static_chunk is dead and not exercised by testing. Ssa-handling code in omp-low.c is only triggered by pass_parallelize_loops

Re: [PATCH, 3/8] Add pass_ch_oacc_kernels to pass_oacc_kernels

2015-06-03 Thread Tom de Vries
On 22/04/15 09:39, Richard Biener wrote: Committed to gomp-4_0-branch in r81: commit 58c33a7965c379b55b549d50e3b79b2252bcc876 Author: tschwinge tschwinge@138bc75d-0d04-0410-961f-82ee72b054a4 Date: Tue Apr 21 19:48:16 2015 + Add pass_ch_oacc_kernels to pass_oacc_kernels

Re: [PATCH, 3/8] Add pass_ch_oacc_kernels to pass_oacc_kernels

2015-06-03 Thread Tom de Vries
-branch? Thanks, - Tom Add in_oacc_kernels_region field to struct loop 2015-06-03 Tom de Vries t...@codesourcery.com * cfgloop.h (struct loop): Add in_oacc_kernels_region field. * omp-low.c (mark_loops_in_oacc_kernels_region): New function. (loop_get_oacc_kernels_region_entry): New function

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

2015-06-09 Thread Tom de Vries
On 09/06/15 13:07, Richard Biener wrote: On Mon, 8 Jun 2015, Tom de Vries wrote: On 17/04/15 12:08, Tom de Vries wrote: 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

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

2015-06-10 Thread Tom de Vries
On 10/06/15 00:15, Thomas Schwinge wrote: Hi Tom! On Tue, 9 Jun 2015 16:12:12 +0200, Tom de Vries tom_devr...@mentor.com wrote: On 09/06/15 13:07, Richard Biener wrote: On Mon, 8 Jun 2015, Tom de Vries wrote: On 17/04/15 12:08, Tom de Vries wrote: On 20-03-15 12:38, Tom de Vries wrote

Re: [PATCH, 3/8] Add pass_ch_oacc_kernels to pass_oacc_kernels

2015-06-04 Thread Tom de Vries
On 03/06/15 13:20, Richard Biener wrote: On Wed, 3 Jun 2015, Tom de Vries wrote: On 22/04/15 09:39, Richard Biener wrote: Ehm. So why not simply add a flag to struct loop instead and set it during OMP region parsing/lowering? Attached patch adds an in_oacc_kernels_region flag to struct

Re: Expand oacc kernels after pass_fre

2015-06-04 Thread Tom de Vries
On 22/04/15 09:36, Richard Biener wrote: On Tue, 21 Apr 2015, Thomas Schwinge wrote: Hi! On Tue, 25 Nov 2014 12:22:02 +0100, Tom de Vries tom_devr...@mentor.com wrote: On 24-11-14 11:56, Tom de Vries wrote: On 15-11-14 18:19, Tom de Vries wrote: On 15-11-14 13:14, Tom de Vries wrote: I'm

Re: Don't dump low gimple functions in gimple dump

2015-06-04 Thread Tom de Vries
approach OK? On Tue, 20 May 2014 10:16:45 +0200, Tom de Vries tom_devr...@mentor.com wrote: Honza, Consider this program: ... int main(void) { #pragma omp parallel { extern void foo(void); foo (); } return 0; } ... When compiling this program with -fopenmp, the ompexp pass

Re: [PATCH] Add debug msg to dump_file in add_new_function

2015-06-04 Thread Tom de Vries
On 04/06/15 15:20, Tom de Vries wrote: Hi, [ posted earlier as part of Don't dump low gimple functions in gimple dump, https://gcc.gnu.org/ml/gcc-patches/2014-05/msg01586.html, currently discussed at: https://gcc.gnu.org/ml/gcc-patches/2015-05/msg02076.html ] This patch adds a debug msg

[gomp4] Re: acc_on_device for device_type_host_nonshm

2015-06-03 Thread Tom de Vries
parameters with ATTRIBUTE_UNUSED 2015-06-04 Tom de Vries t...@codesourcery.com backport from trunk: 2015-05-28 H.J. Lu hongjiu...@intel.com * builtins.c (expand_builtin_acc_on_device): Mark parameters with ATTRIBUTE_UNUSED. --- gcc/builtins.c | 3 ++- 1 file changed, 2 insertions(+), 1

Re: [PING][PATCH][PR65443] Add transform_to_exit_first_loop_alt

2015-06-04 Thread Tom de Vries
On 26/05/15 12:39, Richard Biener wrote: On Thu, 14 May 2015, Tom de Vries wrote: On 20-04-15 14:25, Richard Biener wrote: On Wed, 15 Apr 2015, Tom de Vries wrote: On 03-04-15 14:39, Tom de Vries wrote: On 27-03-15 15:10, Tom de Vries wrote: Hi, this patch fixes PR65443, a todo

Re: [PING^2] [PATCH][5a/5] Postpone expanding va_arg until pass_stdarg

2015-06-09 Thread Tom de Vries
with that hunk. I'll put it through bootstrap and reg-test on x86_64. OK for trunk if that goes well? Thanks, - Tom Handle location in expand_ifn_va_arg_1 2015-06-09 Tom de Vries t...@codesourcery.com * tree-stdarg.c (expand_ifn_va_arg_1): Handle location. * gcc.target/i386/vararg-loc.c

Re: [gomp4, committed] Fix parallelization for fortran oacc kernels tests

2015-06-18 Thread Tom de Vries
On 18/06/15 12:48, Richard Biener wrote: On Thu, 18 Jun 2015, Tom de Vries wrote: Hi, I ran into a problem with fortran loops in oacc kernels regions not being parallelized, after introducting transform_to_exit_first_loop_alt. For gfortran.dg/goacc/kernels-loop.f95, we get: ... #pragma omp

[committed] Add missing update_stmt in transform_to_exit_first_loop_alt

2015-06-22 Thread Tom de Vries
update_stmt in transform_to_exit_first_loop_alt 2015-06-22 Tom de Vries t...@codesourcery.com * tree-parloops.c (transform_to_exit_first_loop_alt): Add update_stmt for cond_stmt. --- gcc/tree-parloops.c | 1 + 1 file changed, 1 insertion(+) diff --git a/gcc/tree-parloops.c b/gcc/tree-parloops.c index

Re: [PATCH] Check dominator info in compute_dominance_frontiers

2015-06-22 Thread Tom de Vries
On 22/06/15 13:47, Richard Biener wrote: On Mon, Jun 22, 2015 at 1:33 PM, Tom de Vries tom_devr...@mentor.com wrote: On 22/06/15 12:14, Richard Biener wrote: On Mon, Jun 22, 2015 at 10:04 AM, Tom de Vries tom_devr...@mentor.com wrote: Hi, during development of a patch I ran into a case

[committed] Test for flag_parallelize_loops 1

2015-06-22 Thread Tom de Vries
On 19/06/15 11:26, Tom de Vries wrote: Hi, DEF_GOMP_BUILTIN tests for 'flag_parallelize_loops'. But if flag_parallelize_loops is one (which is also the default), then pass_parloops doesn't do anything, and won't generate any OMP constructs. This patch makes DEF_GOMP_BUILTIN tests

Re: [PATCH] Check dominator info in compute_dominance_frontiers

2015-06-22 Thread Tom de Vries
On 22/06/15 13:47, Richard Biener wrote: (eventually also for the case where we end up only computing the fast-query stuff). Like this? ... diff --git a/gcc/dominance.c b/gcc/dominance.c index 9c66ca2..58fc6fd 100644 --- a/gcc/dominance.c +++ b/gcc/dominance.c @@ -679,6 +679,12 @@

[committed] Use abort in parloops-exit-first-loop-alt-{3,4}.c

2015-06-23 Thread Tom de Vries
Hi, committed attached patch as trivial. Thanks, - Tom Use abort in parloops-exit-first-loop-alt-{3,4}.c 2015-06-23 Tom de Vries t...@codesourcery.com * testsuite/libgomp.c/parloops-exit-first-loop-alt-3.c (main): Use abort. * testsuite/libgomp.c/parloops-exit-first-loop-alt-4.c (main

Re: [PATCH] Check dominator info in compute_dominance_frontiers

2015-06-22 Thread Tom de Vries
On 22/06/15 12:14, Richard Biener wrote: On Mon, Jun 22, 2015 at 10:04 AM, Tom de Vries tom_devr...@mentor.com wrote: Hi, during development of a patch I ran into a case where compute_dominance_frontiers was called with incorrect dominance info. The result was a segmentation violation

[PATCH] Check dominator info in compute_dominance_frontiers

2015-06-22 Thread Tom de Vries
. - there is a level of ENABLE_CHECKING that is more appropriate for the verify_dominators call. Is this ok for trunk if bootstrap and reg-test on x86_64 succeeds? Thanks, - Tom Check dominator info in compute_dominance_frontiers 2015-06-22 Tom de Vries t...@codesourcery.com * cfganal.c

[gomp4] dominance info after predicate_omp_regions

2015-06-22 Thread Tom de Vries
On 21/05/15 13:42, ber...@gcc.gnu.org wrote: Author: bernds Date: Thu May 21 11:42:14 2015 New Revision: 223478 URL: https://gcc.gnu.org/viewcvs?rev=223478root=gccview=rev Log: * omp-low.c (struct omp_region): Add a gwv_this field. (bb_region_map): New variable.

[gomp4, committed] Handle reduction in oacc kernels region

2015-06-22 Thread Tom de Vries
on top of gomp-4_0-branch. Committed to gomp-4_0-branch. Thanks, - Tom Handle reduction in oacc kernels region 2015-06-18 Tom de Vries t...@codesourcery.com * tree-parloops.c (struct reduction_info): Add reduc_addr field. (create_call_for_reduction_1): Handle case that reduc_addr is non

[PATCH, 1/2] Merge rewrite_virtuals_into_loop_closed_ssa from gomp4 branch

2015-06-25 Thread Tom de Vries
2015-06-24 Tom de Vries t...@codesourcery.com merge from gomp4 branch: 2015-06-24 Tom de Vries t...@codesourcery.com * tree-ssa-loop-manip.c (get_virtual_phi): Factor out of ... (rewrite_virtuals_into_loop_closed_ssa): ... here. * tree-ssa-loop-manip.c (replace_uses_in_dominated_bbs

[PATCH, 2/2][PR66642] Add empty loop exit block in transform_to_exit_first_loop_alt

2015-06-25 Thread Tom de Vries
inbetween the main loop header and the old exit block. Bootstrapped and reg-tested on x86_64. OK for trunk? Thanks, - Tom Add empty loop exit block in transform_to_exit_first_loop_alt 2015-06-24 Tom de Vries t...@codesourcery.com PR tree-optimization/66642 * tree-parloops.c

[Committed] Remove 'unsigned' in header comment canonicalize_loop_ivs

2015-06-25 Thread Tom de Vries
Hi, this patch fixes the header comment of canonicalize_loop_ivs. The generated IV is not guaranteed to be unsigned since the fix for PR49471. Thanks, - Tom Remove 'unsigned' in header comment canonicalize_loop_ivs 2015-06-25 Tom de Vries t...@codesourcery.com * tree-ssa-loop-manip.c

[gomp4, committed] Add replace_uses_in_dominated_bbs

2015-06-24 Thread Tom de Vries
Hi, this patch factors out a new function replace_uses_in_dominated_bbs out of rewrite_virtuals_into_loop_closed_ssa. Committed to gomp-4_0-branch. Thanks, - Tom Add replace_uses_in_dominated_bbs 2015-06-18 Tom de Vries t...@codesourcery.com * tree-ssa-loop-manip.c

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