On Wed, May 15, 2013 at 03:24:37PM +0200, Richard Biener wrote:
We have the same issue in some other places where we insert invariant
code into the loop body - one reason there is another LIM pass
after vectorization.
Well, in this case it causes the shift amount to be loaded into a vector
Followup (and depending on) to the C binding patches for
* COMMON: http://gcc.gnu.org/ml/fortran/2013-05/msg00048.html
* Procedures: http://gcc.gnu.org/ml/fortran/2013-05/msg00051.html
which honour Fortran 2008, where the Fortran name is no longer a global
identifier if a binding name has been
On Thu, May 16, 2013 at 07:59:00PM +0200, Mikael Pettersson wrote:
Jakub Jelinek writes:
On Thu, Apr 25, 2013 at 11:47:02PM +0200, Jakub Jelinek wrote:
This patch adds folding of constant arguments v and v, which helps to
optimize the testcase from the PR back into constant store after
Hi!
http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00282.html
Reject -fsanitize=address -fsanitize=thread linking that won't ever work at
runtime.
Jakub
On 05/16/13 15:32, Rainer Orth wrote:
The new gcc.dg/visibility-21.c testcase fails on i386-pc-solaris2.11 and
x86_64-unknown-linux-gnu:
FAIL: gcc.dg/visibility-21.c (test for excess errors)
Excess errors:
/vol/gcc/src/hg/trunk/local/gcc/testsuite/gcc.dg/visibility-21.c:1:0: warning:
this
Am 17.05.2013 05:22, schrieb Bud Davis:
Not to much to add beyond the title and the patch.
The test file fails before (eventually, when you run out of stack) and passes
after the patch is applied. No new testsuite failures.
--bud
Index: gcc/gcc/fortran/resolve.c
ping?
Am 12.05.2013 11:58, schrieb Matthias Klose:
{bits,ext}/opt_random.h became host specific in 4.8, but are still installed
into the host independent c++ include dir. install them into the host
specific
include dir instead. Ok for 4.8 and trunk?
Matthias
On Fri, 17 May 2013, Jakub Jelinek wrote:
On Wed, May 15, 2013 at 03:24:37PM +0200, Richard Biener wrote:
We have the same issue in some other places where we insert invariant
code into the loop body - one reason there is another LIM pass
after vectorization.
Well, in this case it
On Fri, 17 May 2013, Jakub Jelinek wrote:
On Thu, May 16, 2013 at 07:59:00PM +0200, Mikael Pettersson wrote:
Jakub Jelinek writes:
On Thu, Apr 25, 2013 at 11:47:02PM +0200, Jakub Jelinek wrote:
This patch adds folding of constant arguments v and v, which helps
to
optimize the
On Wed, May 15, 2013 at 6:50 PM, Cary Coutant ccout...@google.com wrote:
gcc/ChangeLog:
* tree-cfg.c (locus_descrim_hasher::hash): Only hash lineno.
(locus_descrim_hasher::equal): Likewise.
(next_discriminator_for_locus): Likewise.
(assign_discriminator): Add return value.
(make_edges):
On Thu, May 16, 2013 at 11:04 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Thu, May 16, 2013 at 2:02 PM, Richard Sandiford
rdsandif...@googlemail.com wrote:
Steven Bosscher stevenb@gmail.com writes:
On Wed, May 15, 2013 at 12:14 AM, Mike Stump wrote:
I don't what to bike shed. So, I'm happy
On Thu, May 16, 2013 at 06:22:10PM +0200, Jakub Jelinek wrote:
As discussed in the PR, there seem to be only 3 define_split
patterns that use dead_or_set_p, one in i386.md and two in s390.md,
but unfortunately insn splitting is done in many passes
(combine, split{1,2,3,4,5}, dbr,
On Fri, May 17, 2013 at 4:40 AM, Bill Schmidt
wschm...@linux.vnet.ibm.com wrote:
This removes two degradations in CPU2006 for 32-bit PowerPC due to lost
vectorization opportunities. Previously, GCC treated malloc'd arrays as
only guaranteeing 4-byte alignment, even though the glibc
Bruce Korb bk...@gnu.org writes:
On 05/16/13 06:41, Rainer Orth wrote:
Work is going on to incorporate all applicable fixincludes fixes into
the Solaris headers proper. One fix is currently problematic since it
uses an G++-internal macro (__GXX_EXPERIMENTAL_CXX0X__) where libstdc++
cmath
On Fri, May 17, 2013 at 10:32:11AM +0200, Richard Biener wrote:
On Fri, May 17, 2013 at 4:40 AM, Bill Schmidt
wschm...@linux.vnet.ibm.com wrote:
This removes two degradations in CPU2006 for 32-bit PowerPC due to lost
vectorization opportunities. Previously, GCC treated malloc'd arrays as
On Fri, May 17, 2013 at 4:47 AM, Jakub Jelinek ja...@redhat.com wrote:
On Fri, May 17, 2013 at 10:32:11AM +0200, Richard Biener wrote:
On Fri, May 17, 2013 at 4:40 AM, Bill Schmidt
wschm...@linux.vnet.ibm.com wrote:
This removes two degradations in CPU2006 for 32-bit PowerPC due to lost
On Fri, May 17, 2013 at 04:55:20AM -0400, David Edelsohn wrote:
As Bill wrote earlier, 2 * POINTER_SIZE causes a different performance
regression for 16 byte alignment on PPC64. 8 bytes is a work-around
for now.
I was talking about the config/linux.h definition, PPC64 can of course
override
On Thu, May 16, 2013 at 10:33:27PM +0200, Jakub Jelinek wrote:
On Thu, May 16, 2013 at 06:44:03PM +0200, Marek Polacek wrote:
--- gcc/tree-ssa-strlen.c.mp2013-05-15 14:11:20.079707492 +0200
+++ gcc/tree-ssa-strlen.c 2013-05-16 17:57:33.963150006 +0200
@@ -1693,8 +1693,10 @@
On Fri, May 17, 2013 at 10:47 AM, Jakub Jelinek ja...@redhat.com wrote:
On Fri, May 17, 2013 at 10:32:11AM +0200, Richard Biener wrote:
On Fri, May 17, 2013 at 4:40 AM, Bill Schmidt
wschm...@linux.vnet.ibm.com wrote:
This removes two degradations in CPU2006 for 32-bit PowerPC due to lost
On Fri, May 17, 2013 at 11:17:26AM +0200, Richard Biener wrote:
Yes - note that it's called MALLOC_*ABI*_ALIGNMENT for a reason - it
is supposed to be the alignment that is required for conforming to the C ABI
on the target. For different allocators I'd rather have a new function
attribute
From: Marc Glisse marc.gli...@inria.fr
Date: Tue, 14 May 2013 13:47:23 +0200
Here is what I tested during the night, I'll just rename the function.
I took the chance to remove an unnecessary alternative in TRUTH_XOR_EXPR.
Passes bootstrap+testsuite on x86_64-linux-gnu.
2013-05-14 Marc
On 13/05/13 17:47, Richard Biener wrote:
On Mon, May 13, 2013 at 5:45 AM, Kugan
kugan.vivekanandara...@linaro.org wrote:
Hi,
This patch removes some of the redundant sign/zero
extensions using value ranges informations generated by VRP.
When GIMPLE_ASSIGN stmts with LHS type smaller than word
OK.
Jason
Jakub Jelinek ja...@redhat.com writes:
2013-05-06 Jakub Jelinek ja...@redhat.com
* gcc.c (SANITIZER_SPEC): Reject -fsanitize=address -fsanitize=thread
linking.
This looks OK to me, thanks.
--
Dodji
On Fri, 2013-05-17 at 01:11 -0400, David Edelsohn wrote:
On Thu, May 16, 2013 at 10:40 PM, Bill Schmidt
wschm...@linux.vnet.ibm.com wrote:
This removes two degradations in CPU2006 for 32-bit PowerPC due to lost
vectorization opportunities. Previously, GCC treated malloc'd arrays as
only
On Fri, 17 May 2013, Hans-Peter Nilsson wrote:
From: Marc Glisse marc.gli...@inria.fr
Date: Tue, 14 May 2013 13:47:23 +0200
Here is what I tested during the night, I'll just rename the function.
I took the chance to remove an unnecessary alternative in TRUTH_XOR_EXPR.
Passes
On Fri, May 17, 2013 at 10:25:01AM +0200, Jakub Jelinek wrote:
Alternative, so far untested, patch is let the register is not dead splitter
do its job always during split2 and just fix it up during peephole2, if the
register was dead.
Now fully bootstrapped/regtested on x86_64-linux and
Hi,
the example we provide for the usage of the random_seed intrinsic
could be better. At least one user has already been tripped over by
the fact that on some targets the first call to system_clock returns
0, resulting in a poor seed. Below is an improved(?!) example program.
Ok for
stmt_kills_ref_p_1 wants to compare MEM_REF offset but compares
the pointers instead (which are previously compared as equal).
This breaks my fix for PR57303.
Fixed thus, applied as obvious.
Richard.
2013-05-17 Richard Biener rguent...@suse.de
* tree-ssa-alias.c
On Fri, May 17, 2013 at 02:46:56PM +0200, Richard Biener wrote:
stmt_kills_ref_p_1 wants to compare MEM_REF offset but compares
the pointers instead (which are previously compared as equal).
This breaks my fix for PR57303.
Fixed thus, applied as obvious.
Richard.
2013-05-17 Richard
2013/5/17 Marc Glisse marc.gli...@inria.fr:
Yes, LOGICAL_OP_NON_SHORT_CIRCUIT seems to be it.
What is the proper thing to do here? If I add the generic xfail back, we'll
get xpass on some platforms, now we have fails on some platforms, and
listing the platforms where we want the
On Fri, May 17, 2013 at 09:54:14PM +0800, Chung-Ju Wu wrote:
2013/5/17 Marc Glisse marc.gli...@inria.fr:
Yes, LOGICAL_OP_NON_SHORT_CIRCUIT seems to be it.
What is the proper thing to do here? If I add the generic xfail back, we'll
get xpass on some platforms, now we have fails on some
On 05/16/13 07:27, Zhenqiang Chen wrote:
On 15 May 2013 06:31, Ramana Radhakrishnan ramana@googlemail.com wrote:
Sorry this had dropped off my list of patches to review somehow but
anyway here's a first cut review.
On Thu, Mar 21, 2013 at 6:58 AM, Zhenqiang Chen
zhenqiang.c...@linaro.org
On Fri, 17 May 2013, Jakub Jelinek wrote:
On Fri, May 17, 2013 at 09:54:14PM +0800, Chung-Ju Wu wrote:
2013/5/17 Marc Glisse marc.gli...@inria.fr:
Yes, LOGICAL_OP_NON_SHORT_CIRCUIT seems to be it.
What is the proper thing to do here? If I add the generic xfail back, we'll
get xpass on some
On 05/16/13 07:46, Zhenqiang Chen wrote:
On 15 May 2013 06:36, Ramana Radhakrishnan ramana@googlemail.com wrote:
What happens to the *arm_simple_return pattern that already exists in
the backend ? Can you rebase to trunk ?
The *arm_simple_return is the same as simple_return used by
On Fri, May 17, 2013 at 04:23:08PM +0200, Marc Glisse wrote:
2013-05-17 Marc Glisse marc.gli...@inria.fr
PR regression/57313
* gcc.dg/binop-xor3.c: Restrict to platforms known to work (x86).
I'd say it should be PR testsuite/57313 (and the PR changed to that).
Ok with that
On Fri, 17 May 2013, Jakub Jelinek wrote:
On Fri, May 17, 2013 at 04:23:08PM +0200, Marc Glisse wrote:
2013-05-17 Marc Glisse marc.gli...@inria.fr
PR regression/57313
* gcc.dg/binop-xor3.c: Restrict to platforms known to work (x86).
I'd say it should be PR testsuite/57313
On Fri, May 17, 2013 at 04:37:45PM +0200, Marc Glisse wrote:
On Fri, 17 May 2013, Jakub Jelinek wrote:
On Fri, May 17, 2013 at 04:23:08PM +0200, Marc Glisse wrote:
2013-05-17 Marc Glisse marc.gli...@inria.fr
PR regression/57313
* gcc.dg/binop-xor3.c: Restrict to platforms known
On 05/17/2013 01:25 AM, Jakub Jelinek wrote:
+(define_peephole2
+ [(set (match_operand:SI 0 memory_operand)
+ (match_operand:SI 1 register_operand))
+ (set (match_operand:SI 2 register_operand) (match_dup 1))
+ (parallel [(set (match_dup 2)
+(ashiftrt:SI (match_dup
On Fri, May 17, 2013 at 7:38 AM, Gopalasubramanian, Ganesh
ganesh.gopalasubraman...@amd.com wrote:
Thank you Uros for the patch. Could you backport this to the 4.8.0?
-Original Message-
From: Uros Bizjak [mailto:ubiz...@gmail.com]
Sent: Wednesday, May 15, 2013 11:16 PM
To:
The rationale is captured in the comment in the first hunk.
Committed.
r~
PR target/49146
* unwind-dw2.c (UNWIND_COLUMN_IN_RANGE): New macro.
(execute_cfa_program): Use it when storing to fs-regs.
diff --git a/libgcc/unwind-dw2.c b/libgcc/unwind-dw2.c
index
On 05/17/2013 12:49 AM, Jakub Jelinek wrote:
Hi!
http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00282.html
Reject -fsanitize=address -fsanitize=thread linking that won't ever work at
runtime.
I thought Dodji already OK's this.If there's any concern about the
scope going outside Dodji's area,
On Fri, May 17, 2013 at 1:22 AM, Richard Biener
richard.guent...@gmail.com wrote:
On Wed, May 15, 2013 at 6:50 PM, Cary Coutant ccout...@google.com wrote:
gcc/ChangeLog:
* tree-cfg.c (locus_descrim_hasher::hash): Only hash lineno.
(locus_descrim_hasher::equal): Likewise.
As I believe I pointed out in a follow-up message, 254.gap is depending
on signed overflow semantics.
This patch avoids eliminating a cast feeding a conditional when the
SSA_NAME's range has overflowed unless -fstrict-overflow is in effect.
Thus 254.gap should be building with
Adding myself to the MAINTAINERS in the Write After Approval category
Index: ChangeLog
===
--- ChangeLog (revision 199021)
+++ ChangeLog (revision 199022)
@@ -1,3 +1,7 @@
+2013-05-17 David Malcolm dmalc...@redhat.com
+
+ *
Errr, how did we miss this? Ok, I'm partly to blame for the lack of
transactional memory in changes.html, but something as big as getting
rid of reload?!
Would it be preferable to mention IRA in the target specific x86 section
instead? I figured it was an important enough change to warrant
The MOVK instruction is currently not used when operand 2 is
more than 16 bits, which leads to sub-optimal code.
This patch improves those situations by removing the check and
instead masking down to 16 bits within the new X format specifier
I added recently.
OK for trunk?
Cheers,
Ian
Ping
On Sun, May 12, 2013 at 5:45 PM, Steven Bosscher wrote:
Hello,
While working on a sched-deps based delay slot scheduler, I've come to
the conclusion that the dependencies themselves must indicate whether
the dependent ref is delayed. So I started hacking sched-deps and ran
into
On Wed, 2013-05-15 at 11:51 -0600, Jeff Law wrote:
On 05/14/2013 01:07 PM, David Malcolm wrote:
On Tue, 2013-05-14 at 11:44 -0600, Jeff Law wrote:
On 05/06/2013 03:05 PM, David Malcolm wrote:
[...snip review and comments about auto-checking of formatting...]
Anyway, the patch is good
On May 16, 2013, at 5:26 PM, David Edelsohn dje@gmail.com wrote:
This patch is creating new segfaults for 32 bit POWER AIX.
Thanks for the heads up. Fixed in r199030.
2013-05-17 Mike Stump mikest...@comcast.net
PR rtl-optimization/57304
* web.c (union_match_dups): Ensure
Hello,
Trying to dump the CFG as a graph fails after the pro_and_epilogue
pass because it leaves unreachable basic blocks. This patch fixes that
issue.
Will commit sometime next week if no-one objects.
Ciao!
Steven
* function.c (rest_of_handle_thread_prologue_and_epilogue):
On 05/17/2013 01:49 PM, Steven Bosscher wrote:
Hello,
Trying to dump the CFG as a graph fails after the pro_and_epilogue
pass because it leaves unreachable basic blocks. This patch fixes that
issue.
Will commit sometime next week if no-one objects.
Ciao!
Steven
* function.c
On 05/15/2013 04:50 AM, Ramana Radhakrishnan wrote:
/* Cannot tail-call to long calls, since these are out of range of
a branch instruction. */
- if (arm_is_long_call_p (decl))
+ if (decl arm_is_long_call_p (decl))
return false;
You can tail call long calls via indirection
Hello!
For some reason we didn't pass mmx, 3dnow and sseX options to cc1 with
-march=native. Passing these options will help older processors that
doesn't get detected through architecture recognition functionality to
get all their ISAs enabled.
2013-05-17 Uros Bizjak ubiz...@gmail.com
*
On 05/14/2013 09:43 PM, Kaushik Phatak wrote:
Hi,
The below patch adds expanders and insns for QI and HI mode for the RL78
target.
The QI mode uses a generic 'mulu' instruction supported by all variants, while
the HI mode creates insn for G13 and G14 target variants using hardware
multiply
On Fri, May 17, 2013 at 9:52 PM, Jeff Law l...@redhat.com wrote:
On 05/17/2013 01:49 PM, Steven Bosscher wrote:
Hello,
Trying to dump the CFG as a graph fails after the pro_and_epilogue
pass because it leaves unreachable basic blocks. This patch fixes that
issue.
Will commit sometime next
On 05/17/2013 02:53 PM, Steven Bosscher wrote:
On Fri, May 17, 2013 at 9:52 PM, Jeff Law l...@redhat.com wrote:
On 05/17/2013 01:49 PM, Steven Bosscher wrote:
Hello,
Trying to dump the CFG as a graph fails after the pro_and_epilogue
pass because it leaves unreachable basic blocks. This patch
On Fri, May 17, 2013 at 11:16 PM, Jeff Law wrote:
What's happened, is that emitting the epilogue at the end of basic
block 4 (with a barrier at the end) has made the use insn 43
unreachable.
But from the description you've given, it appears that the epilogue itself
has unreachable code, and
On Fri, May 17, 2013 at 9:49 PM, Steven Bosscher wrote:
Will commit sometime next week if no-one objects.
Obviously there are objections.
Patch withdrawn.
Ciao!
Steven
On 05/17/2013 03:53 PM, Steven Bosscher wrote:
On Fri, May 17, 2013 at 11:16 PM, Jeff Law wrote:
What's happened, is that emitting the epilogue at the end of basic
block 4 (with a barrier at the end) has made the use insn 43
unreachable.
But from the description you've given, it appears that
Hi,
privatize_symbol_name skips privatizing of symbols where we know for some reason
that it is needed. Loop in rename_statics however expect privatize_symbol_name
to always rename the var and it can get into an infinite loop.
Bootstrapped/regtested x86_64-linux, comitted.
Honza
*
On Fri, May 17, 2013 at 9:12 PM, Richard Henderson r...@redhat.com wrote:
On 05/15/2013 04:50 AM, Ramana Radhakrishnan wrote:
/* Cannot tail-call to long calls, since these are out of range of
a branch instruction. */
- if (arm_is_long_call_p (decl))
+ if (decl arm_is_long_call_p
The warning was attributed to the wrong lineno. Looks like
discriminator does not work well when it coexists with macros.
I think the problem is with same_line_p. It's using expand_location to
test whether two locations refer to the same line, but expand_location
always unwinds the macro stack
On Fri, May 17, 2013 at 4:35 PM, Cary Coutant ccout...@google.com wrote:
The warning was attributed to the wrong lineno. Looks like
discriminator does not work well when it coexists with macros.
I think the problem is with same_line_p. It's using expand_location to
test whether two locations
On Thu, Apr 25, 2013 at 4:42 AM, Richard Biener
richard.guent...@gmail.com wrote:
On Fri, Dec 7, 2012 at 9:01 PM, Easwaran Raman era...@google.com wrote:
It seems I need to reset the debug uses of a statement before moving
the statement itself. The attached patch starts from the leaf to root
Hi,
in this even older ;) and related issue, we reject empty initializer
lists in compound-literals. The fix seems simple: just use
cp_parser_braced_list instead of cp_parser_initializer_list, which
allows for the special case of empty list. Tested x86_64-linux.
There is a nit which I don't
Ping: http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00429.html
The patch is to fix dependency issue of libgcc Makefile.in
by adding 'rm libgcc_tm.stamp' in the clean rule.
That was fixed on main trunk (r187838) but not on gcc-4_7-branch.
Since gcc-4_7-branch is still open, is it OK to backport
Hi all,
Using current trunk repository, it is now able to build
compiler with in-tree isl and cloog.
This patch is to extend ./contrib/download_prerequisites
usage to download isl and cloog conditionally in case
people would like to build gcc with graphite loop optimizations.
OK for the trunk?
On 05/07/2013 09:05 AM, Julian Brown wrote:
On Mon, 6 May 2013 11:53:20 -0600
I don't know. My assumption was that a simple mem would be one that
the PRE pass could handle -- that clause was intended to handle simple
mems in REG_EQUAL notes the same as simple mems as the source of a set.
Maybe
69 matches
Mail list logo
