On 24-04-15 05:25, Bin.Cheng wrote:
On Tue, Apr 21, 2015 at 3:10 PM, Tom de Vries 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/testsuite/g++.dg/pr65802
1.11.
OK for trunk?
Thanks,
- Tom
Return side-effect free result in gimplify_va_arg_internal
2015-04-22 Tom de Vries
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
Thanks,
- Tom
Mark ifn_va_arg as ECF_LEAF
2015-04-26 Tom de Vries
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,
On 22-04-15 15:50, Richard Biener wrote:
On Wed, Apr 22, 2015 at 3:38 PM, Tom de Vries wrote:
On 22-04-15 10:06, Richard Biener wrote:
On Wed, Apr 22, 2015 at 9:41 AM, Tom de Vries
wrote:
Hi,
this patch fixes PR65823.
The patches fixes the problem by using operand_equal_p to do
lue.
But, it seems to work (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
PR tree-optimization/65818
* gimplify.c (gimplify_va_arg_internal): Ensure that only
On 27-04-15 15:40, Richard Biener wrote:
On Mon, Apr 27, 2015 at 3:06 PM, Tom de Vries 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 gimplify_expr, case
MEM_REF
urn;
}
...
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
* tree-call-cdce.c: Fix exam
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 wrote:
On 22-04-15 10:06, Richard Biener wrote:
On Wed, Apr 22, 2015 at 9:41 AM, Tom de Vries
wrote:
Hi,
this patch fixes PR65823.
The patches fixes the
On 28-04-15 12:34, Richard Biener wrote:
On Mon, Apr 27, 2015 at 5:04 PM, Tom de Vries wrote:
On 27-04-15 15:40, Richard Biener wrote:
On Mon, Apr 27, 2015 at 3:06 PM, Tom de Vries
wrote:
On 27-04-15 10:17, Richard Biener wrote:
This patch fixes that by gimplifying the address
);
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
PR tree-optimization/65893
* passes.def (pass_all_optimizations): Move pass_stdarg to after
of 'va_list escapes 0' to fail.
I'd like 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 gc
6_64 with and without -m32.
OK for trunk?
Thanks,
- Tom
Update address_taken after ifn_va_arg expansion
2015-05-08 Tom de Vries
PR tree-optimization/66013
* tree-stdarg.c: Include tree-ssa.h.
(expand_ifn_va_arg_1): Call execute_update_addresses_taken after
TODO_update_ssa.
* gcc.dg/tree
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 c
_8 = VA_ARG (&apD.1839, 0B, 0);
...
That allows the pass_stdarg optimization to succeed:
...
f2: va_list escapes 0, needs to save 8 GPR units and 0 FPR units.
...
Bootstrapped and reg-tested on x86_64, with and without -m32.
OK for trunk?
[ FWIW, I suspect this patch will make life easier for
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)
+{
+
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
* check_GNU_style.sh: Check if files exists.
---
contrib/check_GNU_style.sh | 7
hanks,
- Tom
[PATCH 1/5] check_GNU_style.sh: Don't use filename prefix for one
patch
2015-05-11 Tom de Vries
* 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
, or from files
2015-05-11 Tom de Vries
* 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/check_GNU_style.sh
index 2c4d9e2
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
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
gling.
The patch fixes this by copying the diff_lines before processing, and using the
copy at the end to reproduce the .diff file.
Committed as obvious.
Thanks,
- Tom
2014-11-14 Tom de Vries
* mklog: Move reading of .diff file up and add comment. Copy diff_lines
to orig_diff_lines.
$enable_linux_futex = yes; then
...
The patch makes sure we can handle the case that enable_linux_futex is empty.
Committed as obvious.
Thanks,
- Tom
2014-11-14 Tom de Vries
* configure.tgt: Handle case that $enable_linux_futex is empty.
---
libgomp/ChangeLog.gomp | 4
libgomp/configure.tgt | 2
sertions(+)
diff --git a/libgomp/ChangeLog.gomp b/libgomp/ChangeLog.gomp
index abceb62..b2c2526 100644
--- a/libgomp/ChangeLog.gomp
+++ b/libgomp/ChangeLog.gomp
@@ -1,5 +1,10 @@
2014-11-14 Tom de Vries
+ * Makefile.am: Add missing dependency "openacc.mod: openacc.lo".
+ * Makefile.
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch series consists of these 8 patches:
...
1 Expand oacc kernels after pass_build_
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch series consists of these 8 pa
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch series consists of these 8 pa
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch series consists of these 8 pa
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch series consists of these 8 pa
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch series consists of these 8 pa
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch series consists of these 8 pa
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch series consists of these 8 pa
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch series consists of these 8 pa
mmitted as obvious.
Thanks,
- Tom
>From 70b69e3572414bd486cd9c25ed77216975136e21 Mon Sep 17 00:00:00 2001
From: Tom de Vries
Date: Mon, 17 Nov 2014 19:00:12 +0100
Subject: [PATCH 1/5] Add -ftree-tail-merge to tail-merge testcases
2014-11-17 Tom de Vries
* gcc.dg/pr43864-2.c: Add -ftree-tai
Hi,
this patch fixes the scan patterns for test-cases pr43864-{2,3,4].c.
The patterns matched over several lines, this is fixed in the patch by using
(?n).
Committed as obvious.
Thanks,
- Tom
2014-11-17 Tom de Vries
* gcc.dg/pr43864-2.c: Fix scan-tree-dump-times scan pattern.
* gcc.dg
and trunk as well.
The patch introduces an xfail for pr51879-12.c. I can follow up with a patch to
improve upon that, but I think that's better limited to trunk only.
Bootstrapped and reg-tested on x86_64/trunk.
OK for trunk/stage3, 4.8, 4.9?
Thanks,
- Tom
2014-11-17 Tom de Vries
PR
On 15-11-14 13:14, Tom de Vries wrote:
Don't allow flto-partition=balance for fopenacc
Unsubmitted. This works around a compilation problem for
libgomp/testsuite/libgomp.oacc-c-c++-common/asyncwait-2.c that I ran into on
our internal dev branch. I'll investigate whe
clobbers to epilogue_insns is not trivial, and probably not a
good idea for stage3. The patch works around the problem by disabling
fuse-caller-save in Thumb1 mode.
Build and reg-tested on arm-none-eabi.
OK for stage3?
Thanks,
- Tom
2014-11-20 Tom de Vries
PR rtl-optimization/63718
* config
would
work for both svn and git checkouts.
Diego,
this patch implements both:
- it uses the ~/.mklog file proposed by Peter
- in absence of a ~/.mklog file, it uses git config, also when not in a git
repository
OK?
Thanks,
- Tom
2014-11-20 Tom de Vries
Peter Bergner
* mklog: Handle .
On 20-11-14 17:43, Segher Boessenkool wrote:
On Thu, Nov 20, 2014 at 05:22:20PM +0100, Tom de Vries wrote:
+my $conf = "$ENV{HOME}/.mklog";
+if (-f "$conf") {
+open (CONF, "$conf")
+ or die "Could not open file '$conf' for reading: $!\n&q
actually does something.
Committed as trivial.
Thanks,
- Tom
2014-11-21 Tom de Vries
* gcc.dg/store-motion-fgcse-sm.c: New test.
---
gcc/testsuite/gcc.dg/store-motion-fgcse-sm.c | 32
1 file changed, 32 insertions(+)
create mode 100644 gcc/testsuite/gcc.dg/store
On 15-11-14 18:19, Tom de Vries wrote:
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch s
On 17-11-14 11:13, Richard Biener wrote:
On Sat, 15 Nov 2014, Tom de Vries wrote:
>On 15-11-14 13:14, Tom de Vries wrote:
> >Hi,
> >
> >I'm submitting a patch series with initial support for the oacc kernels
> >directive.
> >
> >The patch ser
On 24-11-14 12:28, Tom de Vries wrote:
On 17-11-14 11:13, Richard Biener wrote:
On Sat, 15 Nov 2014, Tom de Vries wrote:
>On 15-11-14 13:14, Tom de Vries wrote:
> >Hi,
> >
> >I'm submitting a patch series with initial support for the oacc kernels
> >directi
On 14-11-14 13:18, Jonathan Wakely wrote:
This adds system_error support to iostreams, including the required
base class changes to std::ios_base::failure. The abi_tag is used to
make it a distinct type. This changes the type of I/O exceptions
thrown by the library but exceptions are very rarely
On 24-11-14 18:12, Jonathan Wakely wrote:
On 24/11/14 17:48 +0100, Tom de Vries wrote:
On 14-11-14 13:18, Jonathan Wakely wrote:
This adds system_error support to iostreams, including the required
base class changes to std::ios_base::failure. The abi_tag is used to
make it a distinct type
On 24-11-14 13:12, Richard Biener wrote:
On Mon, 24 Nov 2014, Tom de Vries wrote:
On 24-11-14 12:28, Tom de Vries wrote:
On 17-11-14 11:13, Richard Biener wrote:
On Sat, 15 Nov 2014, Tom de Vries wrote:
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with in
ge3?
Thanks,
- Tom
2014-11-23 Tom de Vries
* tree-cfg.c (verify_sese): New function.
(move_sese_region_to_fn): Call verify_sese.
* tree-cfg.h (verify_sese): Declare.
---
gcc/tree-cfg.c | 55 +++
gcc/tree-cfg.h | 1 +
2 files changed,
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:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loo
On 15-11-14 18:20, Tom de Vries wrote:
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch s
On 15-11-14 18:21, Tom de Vries wrote:
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch s
On 15-11-14 18:21, Tom de Vries wrote:
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch s
On 15-11-14 18:22, Tom de Vries wrote:
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch s
On 15-11-14 18:22, Tom de Vries wrote:
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch s
On 15-11-14 18:23, Tom de Vries wrote:
On 15-11-14 13:14, Tom de Vries wrote:
Hi,
I'm submitting a patch series with initial support for the oacc kernels
directive.
The patch series uses pass_parallelize_loops to implement parallelization of
loops in the oacc kernels region.
The patch s
ish in case the prettyprinters.exp file is
unsupported, which AFAIU is also required in that case.
Bootstrapped and reg-tested on x86_64.
OK for trunk/stage3?
Thanks,
- Tom
2014-11-25 Tom de Vries
* testsuite/libstdc++-prettyprinters/prettyprinters.exp: Add missing
dg-finish. Only print unsupport
On 25-11-14 10:28, Richard Biener wrote:
On Tue, Nov 25, 2014 at 1:01 AM, Tom de Vries wrote:
Richard,
I ran into a problem with my oacc kernels directive patch series where
tail-merge added another entry into a region that was previously
single-entry-single-exit.
That resulted in hitting
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:
...
vec *head = redirect_edge_var_map_vector (re);
ene = single_succ_edge (entry_bb);
psi = gsi_start_phis
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
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
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
PR rtl-optimization/63957
* doc
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 w
Committed as obvious.
Thanks,
- Tom
2015-02-24 Tom de Vries
* 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/readwrite_unf_direct_eor_1
[ 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
To: Tom de Vries
CC: Jakub Jelinek , GCC Pa
On 25-02-15 12:40, Thomas Schwinge wrote:
Hi!
On Mon, 23 Feb 2015 18:14:35 +0100, Tom de Vries 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 expansion using
the
Hi,
Committed as obvious.
Thanks,
- Tom
2015-02-26 Tom de Vries
* 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(+)
diff --git a/gcc
Hi,
committed as obvious.
Thanks,
- Tom
2015-02-26 Tom de Vries
* lib/dg-pch.exp (pch-init): Add missing cleanup of pchtest.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
+++ b/gcc/testsuite/lib
Hi,
committed as obvious.
Thanks,
- Tom
2015-02-26 Tom de Vries
* 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/testsuite/gcc.dg/aru-2.c
Hi,
committed as obvious.
Thanks,
- Tom
2015-02-26 Tom de Vries
* 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
--- a/gcc/testsuite
Hi,
committed as obvious.
Thanks,
- Tom
2015-02-26 Tom de Vries
* 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 100644
--- a/gcc
for stage4 trunk?
Thanks,
- Tom
2015-02-26 Tom de Vries
* 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
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
* lib/gcov.exp: Cleanup in case of xfail.
---
gcc/testsuite/lib/gcov.exp | 3 +++
1 file changed, 3
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
* go.test/go-test.exp (go-gc-tests): Add missing cleanup for cmpout.
diff --git a/gcc
[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 w
[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
-
On 12-03-15 10:57, Uros Bizjak wrote:
On Thu, Mar 12, 2015 at 9:11 AM, Tom de Vries 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
* gcc.target/i386/fuse-caller-save
/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
PR rtl-optimization/
?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
PR rtl-optimization/64895
* lra-lives.c (check_pseudos_live_through_calls
86_64.
OK for stage1 trunk?
Thanks,
- Tom
2015-03-10 Tom de Vries
* 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/tree-parloops.c
On 13-03-15 11:36, Richard Biener wrote:
On Fri, Mar 13, 2015 at 11:32 AM, Tom de Vries 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 in the parloops
On 13-03-15 13:36, Richard Biener wrote:
On Fri, Mar 13, 2015 at 1:07 PM, Jakub Jelinek 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 runs very early and
On 12-03-15 11:51, Uros Bizjak wrote:
On Thu, Mar 12, 2015 at 11:41 AM, Tom de Vries 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
* gcc.target/i386/fuse-caller-save
On 18-03-15 11:16, Richard Biener wrote:
On Fri, Mar 13, 2015 at 4:28 PM, Tom de Vries wrote:
On 13-03-15 13:36, Richard Biener wrote:
On Fri, Mar 13, 2015 at 1:07 PM, Jakub Jelinek wrote:
On Fri, Mar 13, 2015 at 01:04:57PM +0100, Richard Biener wrote:
Not really (I don't like -
On 18-03-15 12:18, Richard Biener wrote:
On Wed, Mar 18, 2015 at 12:03 PM, Tom de Vries wrote:
On 18-03-15 11:16, Richard Biener wrote:
On Fri, Mar 13, 2015 at 4:28 PM, Tom de Vries
wrote:
On 13-03-15 13:36, Richard Biener wrote:
On Fri, Mar 13, 2015 at 1:07 PM, Jakub Jelinek wrote
de Vries
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-parloops.h
de Vries
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/gcc/omp-low.c
Tom de Vries
* 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/gcc/tree-parloops.c
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
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 is
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 conv
ction.
>>
>
>Like this?
Looks good to me.
Added 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
* function.c (push_dummy_function): New function.
(init_dummy_function_start): Use pu
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 `G
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_fun
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
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 (chi
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 to
Scale down libgomp.graphite/force-parallel-6.c
2015-03-27 Tom de Vries
* 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, 3 insertions(+), 2
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
* testsuite
goto ;
:
ivtmp = ivtmp + inc;
goto
...
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
PR tree-optimization/65443
* tree-parloops.c (replace_imm_uses, replace_uses_in_bb_by)
(replace_use
lse
goto ;
;; succ: 13 [99.0%] (TRUE_VALUE,EXECUTABLE)
;; 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
PR tree-opt
Tom de Vries
* 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-cfg.c
index 6db6dff
1101 - 1200 of 2350 matches
Mail list logo