On Fri, May 17, 2013 at 09:00:21PM -0700, Sriraman Tallam wrote:
> --- ipa-inline.c (revision 198950)
> +++ ipa-inline.c (working copy)
> @@ -374,7 +374,33 @@ can_early_inline_edge_p (struct cgraph_edge *e)
>return false;
> }
>if (!can_inline_edge_p (e, true))
> -retu
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
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?
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 r18
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
On Thu, Apr 25, 2013 at 4:42 AM, Richard Biener
wrote:
> On Fri, Dec 7, 2012 at 9:01 PM, Easwaran Raman 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
>> of the tree to be reassociated and pla
On Fri, May 17, 2013 at 4:35 PM, Cary Coutant 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 refer to the
> 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 9:12 PM, Richard Henderson 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 (dec
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
* lto-pa
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 t
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 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 co
On 05/17/2013 02:53 PM, Steven Bosscher wrote:
On Fri, May 17, 2013 at 9:52 PM, Jeff Law 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
issu
On Fri, May 17, 2013 at 9:52 PM, Jeff Law 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 nex
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
> mul
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
* config/i386/driver
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 indire
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 (rest_of
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):
Cle
On May 16, 2013, at 5:26 PM, David Edelsohn 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
PR rtl-optimization/57304
* web.c (union_match_dups): Ensure that DF_REF_LOC exists before
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
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 tr
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
2013-0
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
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
+
+ * MAINTAINERS (Write After Ap
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 -fno-strict-overf
On Fri, May 17, 2013 at 1:22 AM, Richard Biener
wrote:
> On Wed, May 15, 2013 at 6:50 PM, Cary Coutant wrote:
>>> gcc/ChangeLog:
>>>
>>> * tree-cfg.c (locus_descrim_hasher::hash): Only hash lineno.
>>> (locus_descrim_hasher::equal): Likewise.
>>> (next_discriminator_for_locus): Likewise.
>>> (ass
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,
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 80de5ab..041
On Fri, May 17, 2013 at 7:38 AM, Gopalasubramanian, Ganesh
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: gcc-patches@gcc.gnu.org
> Cc: Gopala
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 (
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
> >>
> >>PR regression/57313
> >>* gcc.dg/binop-xor3.c: Restrict to platforms known to wo
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
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 t
On Fri, May 17, 2013 at 04:23:08PM +0200, Marc Glisse wrote:
> 2013-05-17 Marc Glisse
>
> 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 change, if some othe
On 05/16/13 07:46, Zhenqiang Chen wrote:
On 15 May 2013 06:36, Ramana Radhakrishnan 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
shrink-wrap. *arm_return and *
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 :
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 hav
On 05/16/13 07:27, Zhenqiang Chen wrote:
On 15 May 2013 06:31, Ramana Radhakrishnan 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
wrote:
Hi,
When shrink-wrap is enabled, the "retur
On Fri, May 17, 2013 at 09:54:14PM +0800, Chung-Ju Wu wrote:
> 2013/5/17 Marc Glisse :
> >
> > 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,
2013/5/17 Marc Glisse :
>
> 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 transformation to happen is
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
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
* tree-ssa-alias.c (stmt_kills_ref_p_1): Properly compare
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 trunk/4.8/4.
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 i686
On Fri, 17 May 2013, Hans-Peter Nilsson wrote:
From: Marc Glisse
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-
On Fri, 2013-05-17 at 01:11 -0400, David Edelsohn wrote:
> On Thu, May 16, 2013 at 10:40 PM, Bill Schmidt
> 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
Jakub Jelinek writes:
> 2013-05-06 Jakub Jelinek
>
> * gcc.c (SANITIZER_SPEC): Reject -fsanitize=address -fsanitize=thread
> linking.
This looks OK to me, thanks.
--
Dodji
OK.
Jason
On 13/05/13 17:47, Richard Biener wrote:
On Mon, May 13, 2013 at 5:45 AM, Kugan
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 is expanded to
rtl, if we can pro
> From: Marc Glisse
> 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 Glisse
>
>
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
On Fri, May 17, 2013 at 10:47 AM, Jakub Jelinek 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
>> wrote:
>> > This removes two degradations in CPU2006 for 32-bit PowerPC due to lost
>> > vectorization opportunities. Previ
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 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 Fri, May 17, 2013 at 4:47 AM, Jakub Jelinek 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
>> wrote:
>> > This removes two degradations in CPU2006 for 32-bit PowerPC due to lost
>> > vectorization opportunities. Previo
On Fri, May 17, 2013 at 10:32:11AM +0200, Richard Biener wrote:
> On Fri, May 17, 2013 at 4:40 AM, Bill Schmidt
> 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-
Bruce Korb 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++
>> already swit
On Fri, May 17, 2013 at 4:40 AM, Bill Schmidt
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 implementation
> guarantees 8-byte a
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, pro_and_ep
On Thu, May 16, 2013 at 11:04 PM, H.J. Lu wrote:
> On Thu, May 16, 2013 at 2:02 PM, Richard Sandiford
> wrote:
>> Steven Bosscher writes:
>>> On Wed, May 15, 2013 at 12:14 AM, Mike Stump wrote:
I don't what to bike shed. So, I'm happy if the next poor soul that
touches it just does so
On Wed, May 15, 2013 at 6:50 PM, Cary Coutant 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): Assign mo
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
> >
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
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
>
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
=
65 matches
Mail list logo
