.
The only functional difference between the patches is that we no longer add
printing a debug message in lra_assign. Committed (since the difference between
the approved and new patch is trivial).
Thanks,
- Tom
2014-05-30 Tom de Vries t...@codesourcery.com
* lra-int.h (struct lra_reg
to true for i386? Are there clobbers which
need to be added?
If it's safe to set this hook to true, OK for trunk?
Thanks,
- Tom
2014-05-30 Tom de Vries t...@codesourcery.com
* config/i386/i386.c (TARGET_CALL_FUSAGE_CONTAINS_NON_CALLEE_CLOBBERS):
Redefine as true.
* gcc.target/i386/fuse-caller
@@
+/* Testcase for PR target/46219. */
+/* { dg-do compile { xfail { *-*-* } } */
Hi,
I've committed this follow-up patch to add a missing closing brace.
Thanks,
- Tom
2014-05-31 Tom de Vries t...@codesourcery.com
* gcc.target/i386/sibcall-4.c: Add missing closing brace.
diff --git a/gcc
tmp.patch
$ touch tmp.patch
$ ls -la tmp.patch
-rw-rw-r-- 1 vries vries 0 mei 31 09:41 tmp.patch
$ ./contrib/mklog tmp.patch
$ ls -la tmp.patch
-rw-rw-r-- 1 vries vries 59 mei 31 09:41 tmp.patch
...
OK for trunk?
Thanks,
- Tom
2014-05-31 Tom de Vries t...@codesourcery.com
* mklog: Keep permissions
.
Build and tested on aarch64-linux-gnu.
OK for trunk?
Thanks,
- Tom
2014-06-01 Tom de Vries t...@codesourcery.com
* config/aarch64/aarch64-protos.h (aarch64_emit_call_insn): Declare.
* config/aarch64/aarch64.c
(TARGET_CALL_FUSAGE_CONTAINS_NON_CALLEE_CLOBBERS): Redefine as true
On 30-05-14 10:20, Ramana Radhakrishnan wrote:
On Fri, May 30, 2014 at 9:14 AM, Tom de Vries tom_devr...@mentor.com wrote:
Marcus,
when building for aarch64-linux-gnu with --enable-checking=yes,rtl, I run
into the following error:
...
In file included from src/libgcc/libgcc2.c:56:0:
src/libgcc
.
Build and tested on arm-linux-gnueabi.
OK for trunk?
Thanks,
- Tom
2014-06-01 Radovan Obradovic robrado...@mips.com
Tom de Vries t...@codesourcery.com
* config/arm/arm-protos.h (arm_emit_call_insn): Add bool parameter.
* config/arm/arm.c
Andreas,
This patch ensures that INTVAL is only used on a CONST_INT in define_expands
addptrdi3 and addptrsi3. I ran into this issue when building gcc for target s390
with --enable-checking=yes,rtl.
Finished s390 build with patch, and committed as trivial.
Thanks,
- Tom
2014-06-04 Tom de
Richard,
Ping for these patches:
- [PATCH, ARM] Enable fuse-caller-save for ARM
https://gcc.gnu.org/ml/gcc-patches/2014-06/msg8.html
- [PATCH, AARCH64] Enable fuse-caller-save for AARCH64
https://gcc.gnu.org/ml/gcc-patches/2014-06/msg4.html
The patches enable the -fuse-caller-save
Thomas,
This patch marks fopenacc as LTO option.
This allows fopenacc to be passed to lto, which means the openacc builtins are
recognized when reading in the LTO stream. This fixes a number of testcases in
the libgomp testsuite.
ok for gomp-4_0-branch?
Thanks,
- Tom
2014-11-03 Tom de
Tom de Vries t...@codesourcery.com
* testsuite/libgomp.oacc-c++/c++.exp: Set default LIBGOMP_PLUGIN_PATH.
Print used LIBGOMP_PLUGIN_PATH.
* testsuite/libgomp.oacc-c/c.exp: Same.
* testsuite/libgomp.oacc-fortran/fortran.exp: Same.
diff --git a/libgomp/testsuite/libgomp.oacc-c++/c++.exp b
Thomas,
This patch fixes the libgomp-oacc.c/lib-66.c testcase.
It allows the test to run for non-shared mem accelerators, and skips the test
otherwise.
ok for gomp-4_0-branch?
Thanks,
- Tom
2014-11-03 Tom de Vries t...@codesourcery.com
* testsuite/libgomp.oacc-c/lib-66.c: Skip
Thomas,
this patch skips the nvidia accelerator in testing libgomp, if the nvidia
accelerator is not present.
OK for gomp-4_0-branch?
Thanks,
- Tom
2014-11-03 Tom de Vries t...@codesourcery.com
* testsuite/libgomp.oacc-c++/c++.exp: Only use nvidia accelerator if
present.
* testsuite
On 04-11-14 23:34, Tom de Vries wrote:
Thomas,
this patch sets LIBGOMP_PLUGIN_PATH to the .libs dir in the build area, if
LIBGOMP_PLUGIN_PATH has not been defined.
This allows f.i. a gcc build without an accelerator configured, to automatically
pick up the host_nonshm plugin.
Updated
On 04-11-14 23:46, Tom de Vries wrote:
Thomas,
This patch fixes the libgomp-oacc.c/lib-66.c testcase.
It allows the test to run for non-shared mem accelerators, and skips the test
otherwise.
Fixed path in log message (testsuite/libgomp.oacc-c/lib-66.c -
testsuite/libgomp.oacc-c-c++-common
On 01-09-14 18:41, Ulrich Weigand wrote:
Tom de Vries wrote:
* ira-costs.c (ira_tune_allocno_costs): Use
ALLOCNO_CROSSED_CALLS_CLOBBERED_REGS to adjust costs.
In debugging PR 53864 on s390x-linux, I ran into a weird change in behavior
that occurs when the following part
On 03-09-14 20:12, Ulrich Weigand wrote:
Just for my curiosity, why is the second condition (after )
needed in this clause in the first place?
if (ira_hard_reg_set_intersection_p (regno, mode,
+ *crossed_calls_clobber_regs)
+
On 03-09-14 18:58, Tom de Vries wrote:
I've build the patch and ran the fuse-caller-save tests, and I'm currently
bootstrapping and reg-testing it on x86_64.
Vladimir,
This patch fixes a problem (found on s390) in one of the committed
fuse-caller-save patches. s390 is the only user
On 18-09-14 19:46, Diego Novillo wrote:
On Thu, Sep 18, 2014 at 10:56 AM, Yury Gribov y.gri...@samsung.com wrote:
On 08/04/2014 12:14 PM, Tom de Vries wrote:
On 04-08-14 08:45, Yury Gribov wrote:
Thanks! My 2 (actually 4) cents below.
Hi Yuri,
thanks for the review.
+if ($#ARGV
[with CC to gcc-patches]
Original Message
Subject: [PATCH] Fix PR55876 - Make generation of paradoxical subreg in
widen_operand more robust
Date: Mon, 07 Jan 2013 11:13:49 +0100
From: Tom de Vries tom_devr...@mentor.com
To: Richard Henderson r...@redhat.com
CC: Steve Ellcey sell
the patch and a cc1 recompile, I'm able to run pr55890-{1,2,3}.c
successfully.
OK for trunk after I've tested this on mips64?
Thanks,
- Tom
2013-01-09 Tom de Vries t...@codesourcery.com
PR middle-end/55890
* calls.c (expand_call): Check if arg_nr is valid.
Index: gcc/calls.c
On 11/01/13 12:02, Greta Yorsh wrote:
Tom, are you going to apply this patch?
Greta,
While testing I've ran into several issues with timeouts (not related to this
patch) that I needed to address, so it took me a bit, but I started the latest
build test run yesterday evening so I expect to
On 07/01/13 18:48, Richard Henderson wrote:
On 01/07/2013 02:16 AM, Tom de Vries wrote:
- /* If MODE is no wider than a single word, we return a paradoxical
- SUBREG. */
+ /* If MODE is no wider than a single word, we return a
+ lowpart or paradoxical SUBREG
Steven,
On 19/07/12 16:43, Steven Bosscher wrote:
On Thu, Jul 19, 2012 at 3:43 PM, Tom de Vries tom_devr...@mentor.com wrote:
I think you should compare your method to the one described in the
paper, and at least reference the paper if it's somehow similar --
Interesting, thanks. Will do
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.
* config/arm/arm.c
.
Bootstrapped and reg-tested by Mikael Pettersson on both x86_64-linux and
sparc64-linux.
OK for trunk?
Thanks,
- Tom
2013-02-04 Tom de Vries t...@codesourcery.com
PR rtl-optimization/56131
* cfgrtl.c (delete_insn): Use NOTE_BASIC_BLOCK instead of BLOCK_FOR_INSN
to get
fixes the problems by:
- using NOTE_BASIC_BLOCK to find the bb of NOTE_INSN_BASIC_BLOCK, and
- explicitly handling the cases that the bb of either the label or the note
is NULL.
I don't think that we need to handle the very last case.
OK.
2013-02-04 Tom de Vries t...@codesourcery.com
On 05/02/13 10:50, Jakub Jelinek wrote:
On Tue, Feb 05, 2013 at 10:41:51AM +0100, Tom de Vries wrote:
On 05/02/13 10:02, Eric Botcazou wrote:
The problem is that in delete_insn, while deleting an undeletable label (in
other words, transforming a label into a INSN_NOTE_DELETED_LABEL):
- we try
On 05/02/13 11:12, Jakub Jelinek wrote:
On Tue, Feb 05, 2013 at 11:01:22AM +0100, Tom de Vries wrote:
I'm not sure I understand your comment.
The BLOCK_FOR_INSN of the note was NULL. The NOTE_BASIC_BLOCK of the note was
correct. Are you saying that the BLOCK_FOR_INSN should not have been NULL
Vladimir,
On 25/01/13 16:36, Vladimir Makarov wrote:
On 01/25/2013 08:05 AM, Tom de Vries wrote:
Vladimir,
this patch adds analysis of register usage of functions for usage by IRA.
The patch:
- adds analysis in pass_final to track which hard registers are set or
clobbered
On 03/04/2011 08:37 AM, Paolo Bonzini wrote:
On 03/03/2011 03:28 PM, Tom de Vries wrote:
reg-tested on x86_64. Better?
Yes, very much so
Great. Thanks for the review.
(talking about patch 6.5; the other one is an
optimization but not essential based on the new comments).
Just one
Hi Zdenek,
I rewrote the patch to remove the use of use_uses_inced_iv.
On 03/14/2011 03:03 PM, Zdenek Dvorak wrote:
Hi,
(since the use_uses_inced_iv test is meaningless).
To me it seems use_uses_inced_iv has meaning:
- it models something: it states whether the comparison is using
the
This patch set fixes PR 43920 - 'Choosing conditional execution over
conditional branches for code size in some cases'.
The patch set was tested on ARM and x86_64. The codesize changes were
benchmarked for ARM Thumb-2. For SPEC2000 and EEMBC CoreMark, a
reduction of 1.1% in the geomean of the
A testcase for the code in arm-size-branch_cost.patch.
Thanks,
- Tom
Index: gcc/testsuite/gcc.target/arm/pr43920-1.c
===
--- gcc/testsuite/gcc.target/arm/pr43920-1.c (revision 0)
+++ gcc/testsuite/gcc.target/arm/pr43920-1.c (revision
The impact of patches 4-9 of the patch set on the example from the bug
report for ARM Thumb-2 -Os is a further size reduction of 7%, from 58 to
54 bytes.
This size reduction is illustrated in this diff of the assembly
(left, without patches, size 58. right, with patches, size 54):
...
push
Currently uses and clobbers are counted as insns in crossjumping, which
can cause undesirable crossjumping. The patch fixes this.
Thanks,
- Tom
Index: gcc/cfgcleanup.c
===
--- gcc/cfgcleanup.c (revision 170556)
+++ gcc/cfgcleanup.c
Inserts use of return register in epilogue threading, to keep
representation consistent, and prevent mismatch in crossjump matching.
Thanks,
- Tom
Index: gcc/function.c
===
--- gcc/function.c (revision 170556)
+++ gcc/function.c
Allows crossjump over fallthru paths.
Thanks,
- Tom
diff -u gcc/cfgcleanup.c gcc/cfgcleanup.c
--- gcc/cfgcleanup.c (working copy)
+++ gcc/cfgcleanup.c (working copy)
@@ -1139,6 +1139,43 @@
}
}
+ /* Walks from I1 in BB1 backward till the next non-debug insn, and returns the
+resulting
Test-case for crossjump-fallthru-ml.patch.
Thanks,
- Tom
Index: gcc/testsuite/gcc.target/arm/pr43920-2.c
===
--- gcc/testsuite/gcc.target/arm/pr43920-2.c (revision 0)
+++ gcc/testsuite/gcc.target/arm/pr43920-2.c (revision 0)
@@ -0,0
Allow crossjumping in both directions. Crossjump was assumed to be
symmetric, and therefore only applied on edges e1,e2 and not on e2,e1.
Now given both the fallthru fix and the regnotes fix, crossjumping is
not symmetrical anymore, and we allow both directions (but not by
testing twice).
Thanks,
Hi Jeff,
On 03/31/2011 08:36 PM, Jeff Law wrote:
On 03/31/11 12:35, Tom de Vries wrote:
Currently uses and clobbers are counted as insns in crossjumping, which
can cause undesirable crossjumping. The patch fixes this.
OK. Please install.
jeff
I don't have an account at sourceware.org
Reposting, with ChangeLog.
2011-04-01 Tom de Vries t...@codesourcery.com
PR target/43920
* config/arm/arm.h (BRANCH_COST): Set to 1 for Thumb-2 when optimizing
for size.
Index: gcc/config/arm/arm.h
===
--- gcc/config/arm/arm.h
Reposting, with ChangeLog.
2011-04-01 Tom de Vries t...@codesourcery.com
PR target/43920
* gcc/testsuite/gcc.target/arm/pr43920-1.c: New test.
Index: gcc/testsuite/gcc.target/arm/pr43920-1.c
===
--- gcc/testsuite/gcc.target/arm
Reposting, with ChangeLog.
2011-04-01 Tom de Vries t...@codesourcery.com
PR target/43920
* cfgcleanup.c (flow_find_cross_jump): Don't count USE or CLOBBER as
insn.
Index: gcc/cfgcleanup.c
===
--- gcc/cfgcleanup.c (revision
Reposting, with ChangeLog.
2011-04-01 Tom de Vries t...@codesourcery.com
PR target/43920
* function.c (emit_use_return_register_into_block): New function.
(thread_prologue_and_epilogue_insns): Use
emit_use_return_register_into_block.
Index: gcc/function.c
On 03/31/2011 11:16 PM, Tom de Vries wrote:
On 03/31/2011 08:52 PM, Jeff Law wrote:
On 03/31/11 12:42, Tom de Vries wrote:
Uses regnotes to analyze whether we can replace insn a by insn b, even
if we cannot replace insn b by insn a. Uses this info in crossjumping.
Shouldn't this be using
Reposting, with ChangeLog.
2011-04-01 Tom de Vries t...@codesourcery.com
PR target/43920
* cfgcleanup.c (try_crossjump_to_edge): Add dir parameter. Pass dir to
flow_find_cross_jump. Swap variables to implement backward replacement.
(try_crossjump_bb): Add argument to try_crossjump_to_edge
Hi Eric,
On 03/31/2011 11:02 PM, Eric Botcazou wrote:
Patches will be posted separately.
Always post a ChangeLog entry with a patch.
Sorry about that. I reposted the patches with ChangeLog entries.
Thanks,
- Tom
On 04/01/2011 05:01 PM, Jakub Jelinek wrote:
On Fri, Apr 01, 2011 at 04:56:10PM +0200, Tom de Vries wrote:
Reposting, with ChangeLog.
Similarly one of your patches had
* gcc/cfgcleanup.c (whatever): Whatever.
in the ChangeLog, should be just
* cfgcleanup.c (whatever): Whatever
On 04/01/2011 05:01 PM, Jakub Jelinek wrote:
On Fri, Apr 01, 2011 at 04:56:10PM +0200, Tom de Vries wrote:
Reposting, with ChangeLog.
2011-04-01 Tom de Vries t...@codesourcery.com
PR target/43920
* testsuite/gcc.target/arm/pr43920-2.c: New test.
gcc/testsuite/ has its own
On 04/01/2011 05:18 PM, Richard Earnshaw wrote:
On Fri, 2011-04-01 at 16:45 +0200, Tom de Vries wrote:
Reposting, with ChangeLog.
#define BRANCH_COST(speed_p, predictable_p) \
- (TARGET_32BIT ? 4 : (optimize 0 ? 2 : 0))
+ (TARGET_32BIT ? (TARGET_THUMB2 optimize_size ? 1 : 4
On 04/01/2011 05:34 PM, Richard Earnshaw wrote:
On Fri, 2011-04-01 at 16:47 +0200, Tom de Vries wrote:
Reposting, with ChangeLog.
+/* { dg-options -march=armv7-a -mthumb -Os } */
No, use dg-require-effective-target. The above doesn't work properly
with multilib testing.
Changed
On 04/01/2011 05:34 PM, Richard Earnshaw wrote:
On Fri, 2011-04-01 at 16:47 +0200, Tom de Vries wrote:
Reposting, with ChangeLog.
+/* { dg-options -march=armv7-a -mthumb -Os } */
No, use dg-require-effective-target. The above doesn't work properly
with multilib testing.
Changed
On 04/02/2011 09:47 AM, Richard Guenther wrote:
On Fri, Apr 1, 2011 at 6:06 PM, Tom de Vries vr...@codesourcery.com wrote:
On 04/01/2011 05:18 PM, Richard Earnshaw wrote:
On Fri, 2011-04-01 at 16:45 +0200, Tom de Vries wrote:
Reposting, with ChangeLog.
#define BRANCH_COST(speed_p
Hi Jeff,
On 04/01/2011 04:54 PM, Tom de Vries wrote:
On 03/31/2011 11:16 PM, Tom de Vries wrote:
On 03/31/2011 08:52 PM, Jeff Law wrote:
On 03/31/11 12:42, Tom de Vries wrote:
Uses regnotes to analyze whether we can replace insn a by insn b, even
if we cannot replace insn b by insn a. Uses
it, here's a patch to access nm -S.
Thanks,
- Tom
2011-04-05 Tom de Vries t...@codesourcery.com
* lib/scanasm.exp (object-symbol-size): New proc.
Index: gcc/testsuite/lib/scanasm.exp
===
--- gcc/testsuite/lib/scanasm.exp (revision
Hi,
I introduced the function emit_use_return_register_into_block for
PR43920, but should have guarded it with HAVE_return.
The patch fixes this.
Tested by doing x86_64 non-bootstrap c/fortran build.
OK for trunk?
Thanks,
- Tom
2011-04-05 Tom de Vries t...@codesourcery.com
PR middle-end
On 09/24/2011 05:29 PM, Eric Botcazou wrote:
This is an updated version of the patch. I have 2 new patches and an
updated testcase which I will sent out individually.
Patch set was bootstrapped and reg-tested on x86_64.
Ok for trunk?
Thanks,
- Tom
2011-07-30 Tom de Vries t
on x86_64.
Ok for trunk?
Thanks,
- Tom
2011-07-30 Tom de Vries t...@codesourcery.com
PR middle-end/43513
* Makefile.in (tree-ssa-ccp.o): Add $(PARAMS_H) to rule.
* tree-ssa-ccp.c (params.h): Include.
(fold_builtin_alloca_for_var): New function.
(ccp_fold_stmt
On 09/26/2011 12:29 PM, Richard Guenther wrote:
On Sun, 25 Sep 2011, Tom de Vries wrote:
On 09/25/2011 10:57 AM, Richard Guenther wrote:
On Sat, Sep 24, 2011 at 5:29 PM, Eric Botcazou ebotca...@adacore.com
wrote:
This is an updated version of the patch. I have 2 new patches and an
updated
Committed as obvious.
Thanks,
- Tom
2011-09-28 Tom de Vries t...@codesourcery.com
PR testsuite/50485
* gcc.target/i386/sse4_1-blendps.c: Include stdlib.h.
(TEST): Initialize src3 with random floats.
* gcc.target/i386/sse4_1-blendps-2.c (sse4_1_test): Remove
Richard,
I got a patch for PR50527.
The patch prevents the alignment of vla-related allocas to be set to
BIGGEST_ALIGNMENT in ccp. The alignment may turn out smaller after folding
the alloca.
Bootstrapped and regtested on x86_64.
OK for trunk?
Thanks,
- Tom
2011-09-27 Tom de Vries t
(2))) int_unaligned;
int foo (int_unaligned *p)
{
return *(p+1);
}
...
test.c.023t.ccp1:
...
# PT = anything
# ALIGN = 2, MISALIGN = 0
D.2723_2 = pD.1604_1(D) + 4;
...
Thanks,
- Tom
thanks
Carrot
On Tue, Sep 20, 2011 at 7:13 PM, Tom de Vries vr...@codesourcery.com wrote:
Hi Richard
On 09/24/2011 11:31 AM, Richard Guenther wrote:
On Tue, Sep 20, 2011 at 1:13 PM, Tom de Vries vr...@codesourcery.com wrote:
Hi Richard,
I have a patch for PR43814. It introduces an option that assumes that
function
arguments of pointer type are aligned, and uses that information in
tree
On 09/29/2011 12:21 AM, Tom de Vries wrote:
On 09/24/2011 11:31 AM, Richard Guenther wrote:
On Tue, Sep 20, 2011 at 1:13 PM, Tom de Vries vr...@codesourcery.com wrote:
Hi Richard,
I have a patch for PR43814. It introduces an option that assumes that
function
arguments of pointer type
On 09/28/2011 11:53 AM, Richard Guenther wrote:
On Wed, Sep 28, 2011 at 11:34 AM, Tom de Vries tom_devr...@mentor.com wrote:
Richard,
I got a patch for PR50527.
The patch prevents the alignment of vla-related allocas to be set to
BIGGEST_ALIGNMENT in ccp. The alignment may turn out smaller
Richard,
in gimplify_vla_decl, the alloca argument seems to be the size of the vla in
units:
...
t = build_call_expr (t, 1, DECL_SIZE_UNIT (decl));
...
I wonder why we are going through this 8 vs. BITS_PER_UNIT conversion here:
...
elem_type = build_nonstandard_integer_type (BITS_PER_UNIT,
On 10/01/2011 05:46 PM, Tom de Vries wrote:
On 09/30/2011 03:29 PM, Richard Guenther wrote:
On Thu, Sep 29, 2011 at 3:15 PM, Tom de Vries tom_devr...@mentor.com wrote:
On 09/28/2011 11:53 AM, Richard Guenther wrote:
On Wed, Sep 28, 2011 at 11:34 AM, Tom de Vries tom_devr...@mentor.com
wrote
On 10/04/2011 03:03 PM, Richard Guenther wrote:
On Tue, Oct 4, 2011 at 9:43 AM, Tom de Vries tom_devr...@mentor.com wrote:
On 10/01/2011 05:46 PM, Tom de Vries wrote:
On 09/30/2011 03:29 PM, Richard Guenther wrote:
On Thu, Sep 29, 2011 at 3:15 PM, Tom de Vries tom_devr...@mentor.com
wrote
On 10/04/2011 08:51 PM, Richard Henderson wrote:
On 10/04/2011 09:28 AM, Tom de Vries wrote:
Well, that was the idea. But now I wonder, isn't it better to do this in
expand_builtin_alloca:
...
/* Compute the argument. */
op0 = expand_normal (CALL_EXPR_ARG (exp, 0));
+ align
On 10/04/2011 10:58 PM, Richard Henderson wrote:
On 10/04/2011 01:17 PM, Tom de Vries wrote:
In general, to fold vlas (which are lowered to allocas) to normal
declarations,
if the alloca argument is constant.
Ah. Ok, I suppose. How often are you seeing this happening? I can imagine
On 10/04/2011 11:31 PM, Tom de Vries wrote:
On 10/04/2011 10:58 PM, Richard Henderson wrote:
On 10/04/2011 01:17 PM, Tom de Vries wrote:
In general, to fold vlas (which are lowered to allocas) to normal
declarations,
if the alloca argument is constant.
Ah. Ok, I suppose. How often
On 10/05/2011 10:46 AM, Richard Guenther wrote:
On Tue, Oct 4, 2011 at 6:28 PM, Tom de Vries tom_devr...@mentor.com wrote:
On 10/04/2011 03:03 PM, Richard Guenther wrote:
On Tue, Oct 4, 2011 at 9:43 AM, Tom de Vries tom_devr...@mentor.com wrote:
On 10/01/2011 05:46 PM, Tom de Vries wrote
$2,2($4)
...
The patch makes sure that the src_mem computed here in expand_builtin_memcpy is
marked as MEM_READONLY_P:
...
src_mem = get_memory_rtx (src, len);
...
build and reg-tested on i686, arm, and mips.
OK for trunk?
Thanks,
- Tom
2011-10-10 Tom de Vries t
On 10/10/2011 03:44 PM, Eric Botcazou wrote:
The patch makes sure that the src_mem computed here in
expand_builtin_memcpy is marked as MEM_READONLY_P:
...
src_mem = get_memory_rtx (src, len);
...
build and reg-tested on i686, arm, and mips.
OK for trunk?
Do you get the same
delinked:
...
if (rdef rdef != use)
SET_USE (use_p, rdef);
...
The patch fixes this by forcing SET_USE for delinked uses.
Bootstrapped and reg-tested on x86_64.
OK for trunk?
Thanks,
- Tom
2011-10-12 Tom de Vries t...@codesourcery.com
PR tree-optimization/50672
* tree
On 10/10/2011 05:50 PM, Eric Botcazou wrote:
So, the patch for build_constant_desc does not have the desired effect.
OK, too bad that we need to play this back-and-forth game with MEMs. So the
original patch is OK (with TREE_READONLY (base) on the next line to mimic
what
is done just
(including Ada) on x86_64.
OK for trunk?
Thanks,
- Tom
2011-10-12 Tom de Vries t...@codesourcery.com
* function.c (gimplify_parameters): Set number of arguments of call to
BUILT_IN_ALLOCA_WITH_ALIGN to 2.
Index: gcc/function.c
On 10/12/2011 10:13 AM, Tom de Vries wrote:
On 10/10/2011 05:50 PM, Eric Botcazou wrote:
So, the patch for build_constant_desc does not have the desired effect.
OK, too bad that we need to play this back-and-forth game with MEMs. So the
original patch is OK (with TREE_READONLY (base
On 10/12/2011 02:19 PM, Richard Guenther wrote:
On Wed, Oct 12, 2011 at 8:35 AM, Tom de Vries vr...@codesourcery.com wrote:
Richard,
I have a patch for PR50672.
When compiling the testcase from the PR with -ftree-tail-merge, the scenario
is
as follows:
We start out tail_merge_optimize
On 10/14/2011 12:00 PM, Richard Guenther wrote:
On Fri, Oct 14, 2011 at 1:12 AM, Tom de Vries tom_devr...@mentor.com wrote:
On 10/12/2011 02:19 PM, Richard Guenther wrote:
On Wed, Oct 12, 2011 at 8:35 AM, Tom de Vries vr...@codesourcery.com
wrote:
Richard,
I have a patch for PR50672
On 10/17/2011 01:51 PM, Richard Guenther wrote:
On Sun, Oct 16, 2011 at 12:05 PM, Tom de Vries tom_devr...@mentor.com wrote:
On 10/14/2011 12:00 PM, Richard Guenther wrote:
On Fri, Oct 14, 2011 at 1:12 AM, Tom de Vries tom_devr...@mentor.com
wrote:
On 10/12/2011 02:19 PM, Richard Guenther
are
defined.
The patch fixes the ICE by setting the empty phi argument to .MEM.
bootstrapped and reg-tested on x86_64.
Ok for trunk?
Thanks,
- Tom
2011-10-19 Tom de Vries t...@codesourcery.com
PR 50769/tree-optimization
* tree-ssa-tail-merge.c (replace_block_by): Calculate phi_vuse2
that update_vuses leaves virtual phis with only
one argument (see unlink_virtual_phi).
bootstrapped and reg-tested on x86_64. The tested patch had one addition to the
attached patch: calling verify_dominators at the end of replace_block_by.
OK for trunk?
Thanks,
- Tom
2011-10-20 Tom de Vries t
the original fix to PR50763, this forces us to update the vuses, while
that is not necessary.
Attached patch fixes this problem by determining more precisely when the vuses
need to be updated.
bootstrapped and reg-tested on x86_64 and i686.
OK for trunk?
Thanks,
- Tom
2011-10-25 Tom de Vries
-tested on MIPS and ARM.
Ok for trunk?
Thanks,
- Tom
Richard.
Jakub
2011-10-25 Tom de Vries t...@codesourcery.com
PR target/43814
* tree-ssa-ccp.c (get_align_value): New function, factored out of
get_value_from_alignment.
(get_value_from_alignment): Use
, but for now I
implemented a more coarse-grained fix: if the incoming vuses are not equal, and
at least one of the vuses influenced a non-virtual result, we don't consider the
blocks equal.
Bootstrapped and reg-tested on x86_64.
ok for trunk, 4.7.1?
Thanks,
- Tom
2012-04-13 Tom de Vries t
On 13/04/12 11:13, Richard Guenther wrote:
On Fri, Apr 13, 2012 at 10:32 AM, Tom de Vries tom_devr...@mentor.com wrote:
Richard,
this patch fixes PR52743.
The problem is as follows: blocks 3 and 5, with successor 6 are considered
equal
and merged.
...
# BLOCK 3 freq:6102
# PRED: 2
On 06/03/12 15:21, Richard Guenther wrote:
On Mon, Feb 13, 2012 at 1:36 PM, Tom de Vries tom_devr...@mentor.com wrote:
On 13/02/12 12:54, Richard Guenther wrote:
On Thu, Feb 2, 2012 at 11:44 AM, Tom de Vries tom_devr...@mentor.com
wrote:
Richard,
this patch fixes PR52801.
Consider test
On 27/01/12 21:37, Tom de Vries wrote:
On 24/01/12 11:40, Richard Guenther wrote:
On Mon, Jan 23, 2012 at 10:27 PM, Tom de Vries tom_devr...@mentor.com
wrote:
Richard,
Jakub,
the following patch fixes PR51879.
Consider the following test-case:
...
int bar (int);
void baz (int);
void
On 09/12/11 14:59, Eric Botcazou wrote:
2011-12-07 Tom de Vries t...@codesourcery.com
* cfgcleanup.c (try_optimize_cfg): Replace call to delete_isns_chain by
delete_insn_chain
call to delete_insn. Remove code to reorder BASIC_BLOCK note and
DELETED_LABEL note, and move
Jason,
On 18/02/12 09:33, Jason Merrill wrote:
On 01/22/2012 03:38 AM, Tom de Vries wrote:
Sorry I didn't notice this patch until now; please CC me on C++ patches,
or at least mention C++ in the subject line.
OK, will do.
+ tree expr = NULL;
+ append_to_statement_list
On 16/04/12 16:23, Jason Merrill wrote:
On 04/14/2012 05:43 PM, Tom de Vries wrote:
+ tree expr = NULL;
+ append_to_statement_list (*block,expr);
+ *block = expr;
Rather than doing this dance here, I think it would be better to enhance
append_to_statement_list to handle
On 17/04/12 14:24, Richard Guenther wrote:
On Sat, Apr 14, 2012 at 9:26 AM, Tom de Vries tom_devr...@mentor.com wrote:
On 27/01/12 21:37, Tom de Vries wrote:
On 24/01/12 11:40, Richard Guenther wrote:
On Mon, Jan 23, 2012 at 10:27 PM, Tom de Vries tom_devr...@mentor.com
wrote:
Richard
On 25/04/12 12:09, Jakub Jelinek wrote:
On Wed, Apr 25, 2012 at 11:57:09AM +0200, Richard Guenther wrote:
void *foo ()
{
return __builtin_return_address (0);
}
void *bar (_Bool b)
{
if (b)
return foo ();
else
return foo ();
}
int main()
{
if (bar(true) ==
give this one more iteration, but I guess the basic idea should work.
bootstrapped and reg-tested on x86_64 (ada inclusive).
Is this patch ok, or is the todo required?
Thanks,
- Tom
2012-04-25 Tom de Vries t...@codesourcery.com
PR tree-optimization/51879
* tree-ssa-sccvn.h
On 26/04/12 12:20, Richard Guenther wrote:
On Wed, Apr 25, 2012 at 11:56 PM, Tom de Vries tom_devr...@mentor.com wrote:
On 25/04/12 11:57, Richard Guenther wrote:
SNIP
Hmm. I'm not sure we can conclude that they have the same value!
+int bar (int);
+void baz (int);
+void bla (int
.
bootstrapped and reg-tested on x86_64.
Ok for trunk?
Thanks,
- Tom
2012-05-02 Tom de Vries t...@codesourcery.com
* tree-ssa-sccvn.c (copy_reference_ops_from_call)
(visit_reference_op_call): Handle case that lhs is not an SSA_NAME.
(visit_use): Call visit_reference_op_call
and reg-tested (Ada inclusive) on x86_64.
OK for trunk?
Thanks,
- Tom
2012-07-21 Tom de Vries t...@codesourcery.com
* tree-vrp.c (find_case_label_ranges): New function.
(vrp_visit_switch_stmt, simplify_switch_using_ranges): Use
find_case_label_ranges instead
On 01/08/12 10:47, Richard Guenther wrote:
* tree-ssa-tail-merge.c (release_last_vdef): Remove.
(replace_block_by): Adjust.
(tail_merge_optimize): Use mark_virtual_operands_for_renaming.
Richard,
this caused PR50672 to trigger again.
Thanks,
- Tom
101 - 200 of 2297 matches
Mail list logo
