On Fri, Jul 27, 2012 at 6:47 AM, Teresa Johnson wrote:
> * gcc/gcov-io.h (GCOV_TAG_SUMMARY_LENGTH): Update for new summary
> info.
> (struct gcov_ctr_summary): Add new summary info: num_hot_counters and
> hot_cutoff_value.
You should also update the description for the da
On 21 Jul 2012, at 18:04, Iain Sandoe wrote:
> On 21 Jul 2012, at 17:43, Andrew Pinski wrote:
>> On Sat, Jul 21, 2012 at 9:12 AM, Iain Sandoe wrote:
>>> The following patch was been in use internally, for some time, to handle
>>> two further cases where the processor does not have lwsync. Veri
Hello,
Ryan Mansfield a écrit:
> On 12-07-19 06:06 PM, Gabriel Dos Reis wrote:
[...]
>>> Would moving the GCC_DRIVER_HOST_INITIALIZATION after diagnostic_initialize
>>> be OK?
>>
>> yes, I think so.
>
> OK, then here's a changelog entry for the diff.
>
> 2012-07-20 Ryan Mansfield
>
>
On Thu, Jul 26, 2012 at 11:58 PM, Richard Henderson wrote:
> On 07/26/2012 02:41 PM, Richard Henderson wrote:
>> This is a patch...
>
> ... that I should have attached. Bah.
Do we need to mark the labels so we preserve them? Consider
goto foo;
foo:
bar __attribute__((cold)):
...
so bar wi
On Fri, Jul 27, 2012 at 10:57 AM, Richard Guenther
wrote:
> On Thu, Jul 26, 2012 at 11:58 PM, Richard Henderson wrote:
>> On 07/26/2012 02:41 PM, Richard Henderson wrote:
>>> This is a patch...
>>
>> ... that I should have attached. Bah.
>
> Do we need to mark the labels so we preserve them? Co
On Thu, Jul 26, 2012 at 3:38 AM, Segher Boessenkool
wrote:
> Move those parts of rios.md that apply to 601 to a new file
> 601.md, renaming everything from rios1* to ppc601*.
>
> 2012-07-26 Segher Boessenkool
>
> gcc/
> * config/rs6000/601.md: New file.
> * config/rs6000/aix43.h
On Fri, Jul 27, 2012 at 11:08 AM, Steven Bosscher wrote:
> On Fri, Jul 27, 2012 at 10:57 AM, Richard Guenther
> wrote:
>> On Thu, Jul 26, 2012 at 11:58 PM, Richard Henderson wrote:
>>> On 07/26/2012 02:41 PM, Richard Henderson wrote:
This is a patch...
>>>
>>> ... that I should have attache
On Thu, Jul 26, 2012 at 3:38 AM, Segher Boessenkool
wrote:
> That is, -mpower and friends, TARGET_POWER and friends.
> These are always disabled now.
>
> 2012-07-26 Segher Boessenkool
>
> gcc/
> * common/config/rs6000/rs6000-common.c (rs6000_handle_option):
> Delete code for -mn
his is okay, but why does the ChangeLog line
"> * config/rs6000/rs6000-tables.opt: Regenerate.X
FIXME"
have "X FIXME"?
Just to check if you are paying attention :-) It was a
reminder to myself to force a regenerate of the file, because
the timestamps get messed up duri
Hello!
Attached patch enables ix86_avoid_lea_for_addr to process
zero-extended addresses. This patch should help atom performance,
especially in x32 mode.
Please note the complication with insn re-recognition in
ix86_avoid_lea_for_addr, to solve the problem as described in the
comment:
/* ix86
On Thu, Jul 26, 2012 at 3:38 AM, Segher Boessenkool
wrote:
> gcc/
> * config/rs6000/constraints.md: Delete "q" constraint.
> * config/rs6000/dfp.md (movsd_hardfloat, movsd_softfloat):
> Delete the "q" alternative.
> * config/rs6000/predicates.md (gpc_reg_operand): R
On Fri, Jul 27, 2012 at 11:15 AM, Richard Guenther
wrote:
>> Right. I don't like the use of this attribute on labels at all, for
>> the reasons you list here. I think it would be much cleaner to add a
>> branch hint on the label in the asm goto, to contain this extension
>> and to also to make it
This is okay, but gpc_reg_operand in predicates.md needs something
better than a magic "64" constant in the test. Should this be a
comment or a new symbolic value related the max/last FPR or number of
general registers?
The test assumes a lot about the relative ordering of the
various regnos (l
>>
>> OK if you remove the declarations for abort, exit, rand, and srand;
>> they're no longer needed. Presumably an alternate fix would be to
>> add "extern" before the declarations of rand and srand.
>>
>> Janis
Comitted to trunk and 4.7 branch
http://gcc.gnu.org/ml/gcc-cvs/2012-07/msg00811.ht
On Fri, Jul 27, 2012 at 11:40 AM, Steven Bosscher wrote:
> On Fri, Jul 27, 2012 at 11:15 AM, Richard Guenther
> wrote:
>>> Right. I don't like the use of this attribute on labels at all, for
>>> the reasons you list here. I think it would be much cleaner to add a
>>> branch hint on the label in t
This removes one FOR_EACH_REFERENCED vars loop from into-ssa.
insert_phi_nodes, which is only called when we rewrite the whole
function into SSA, not from SSA updating, has an awkward way
of inserting PHI nodes for all relevant vars which uses
three hashtable lookups for all vars we insert PHIs fo
As suggested by rth here:
http://gcc.gnu.org/ml/gcc-patches/2012-07/msg01281.html
this patch converts all the #define accessors in expmed.h to use inline
functions instead.
By itself, doing that conversion is not very exciting. Followup patches
might:
* Move setters into expmed.c;
* Reduce spa
Kirill has already checked it in for me, thanks!
Btw, I have FSF copyright assignment.
Is it possible to commit this patch to 4.6 branch? Since current ndk
is based on 4.6
2012/7/25 Janis Johnson :
> On 07/25/2012 03:58 AM, Anna Tikhonova wrote:
>> Thanks!
>>
>> I've removed declarations. New pat
On Wed, Jul 25, 2012 at 01:43:59PM +0200, Richard Guenther wrote:
> On Tue, Jul 24, 2012 at 9:27 PM, Marek Polacek wrote:
> > Ping.
>
> Ok.
Sorry, can't commit myself. Thanks,
Marek
Hello!
Without -mno-explicit-relocs, alpha generates:
ldq $27,myBindC($29)!literal!6
jsr $26,($27),myBindC !lituse_jsr!6
which confuses scan-assembler-times "..." 1.
Also, added appropriate cleanup-tree-dump while there.
Tested on alphaev68-pc-linux-gnu, c
When Diego committed rev 119760 he basically disabled the codepath
that switched virtual symbols to full rewrite. Basically because
you could still trigger it by --param virtual-mappings-ratio=0.
As the code has not been excercised in the last 6(!) years I
opted to remove it as the underlying iss
On Fri, Jul 27, 2012 at 1:44 PM, Marek Polacek wrote:
> On Wed, Jul 25, 2012 at 01:43:59PM +0200, Richard Guenther wrote:
>> On Tue, Jul 24, 2012 at 9:27 PM, Marek Polacek wrote:
>> > Ping.
>>
>> Ok.
>
> Sorry, can't commit myself. Thanks,
Committed.
Richard.
> Marek
This tries to more clearly separate per-SSA name held information
from per-DECL held information during update-ssa. We already have
a global array of SSA name informations so it is pointless to
have a hashtable mapping SSA names to yet another piece of information
(a bitmap). This patch simply p
Il 27/07/2012 11:40, Steven Bosscher ha scritto:
>> >
>> > As in the case where you have both an unlikely and likely jump to a
>> > basic-block. But what I understand is that rth adds a way to mark
>> > a basic-block as hot or cold, not a way to mark an edge as hot or cold
>> > (that would be what
This is okay, but gpc_reg_operand in predicates.md needs something
better than a magic "64" constant in the test. Should this be a
comment or a new symbolic value related the max/last FPR or number of
general registers?
The test assumes a lot about the relative ordering of the
various regnos (l
This is a slightly expanded version of the patch in [1]. The main difference is
that I expanded gengtype diagnostics to error on GTY length option applied to
strings too, in addition to other arrays of atomic types. This in turn
uncovered another place where a correct GTY option should be used .
This avoids triggering update-ssa right after into-ssa just because
we didn't rename virtual operands yet. Simply do that on-the-fly,
update_stmt will have added bare symbols as operands already.
Surprisingly simple ... no idea why I chose the "simple" route
when merging alias-improvements (origi
On Fri, Jul 27, 2012 at 2:59 PM, Laurynas Biveinis
wrote:
> This is a slightly expanded version of the patch in [1]. The main difference
> is that I expanded gengtype diagnostics to error on GTY length option applied
> to strings too, in addition to other arrays of atomic types. This in turn
>
This fixes a thinko and typo in
http://gcc.gnu.org/ml/gcc-patches/2012-07/msg01168.html that shows up
as an ICE on e500. Two issues really. One is that the secondary
reload insn emitted by rs6000_secondary_reload_gpr had better be
valid. The other is that we only need these reloads when the
insn
On 12-07-27 03:41 AM, Dodji Seketeli wrote:
2012-07-20 Ryan Mansfield
* gcc.c (main): Move GCC_DRIVER_HOST_INITIALIZATION after
diagnostic_initialize.
Could someone please apply the change?
The change seems small and obvious enough to not require copyright
assignment on f
On Thu, Jul 26, 2012 at 11:57 AM, Steven Bosscher wrote:
> On Thu, Jul 26, 2012 at 11:23 AM, Richard Guenther
> wrote:
>> Ok! Thanks for adding this exhaustive documentation.
>
> There's more to come! I want to add some explanations to ebitmap,
> pointer-set, fibheap, and splay-tree as sets, and
On 07/27/2012 01:57 AM, Richard Guenther wrote:
> On Thu, Jul 26, 2012 at 11:58 PM, Richard Henderson wrote:
>> On 07/26/2012 02:41 PM, Richard Henderson wrote:
>>> This is a patch...
>>
>> ... that I should have attached. Bah.
>
> Do we need to mark the labels so we preserve them? Consider
>
On 07/27/2012 02:08 AM, Steven Bosscher wrote:
> Right. I don't like the use of this attribute on labels at all, for
> the reasons you list here. I think it would be much cleaner to add a
> branch hint on the label in the asm goto, to contain this extension
> and to also to make it clear that it's
On Fri, Jul 27, 2012 at 12:31 AM, Steven Bosscher wrote:
> On Fri, Jul 27, 2012 at 6:47 AM, Teresa Johnson wrote:
>> * gcc/gcov-io.h (GCOV_TAG_SUMMARY_LENGTH): Update for new summary
>> info.
>> (struct gcov_ctr_summary): Add new summary info: num_hot_counters and
>> hot_
On Fri, Jul 27, 2012 at 9:26 AM, Alan Modra wrote:
> This fixes a thinko and typo in
> http://gcc.gnu.org/ml/gcc-patches/2012-07/msg01168.html that shows up
> as an ICE on e500. Two issues really. One is that the secondary
> reload insn emitted by rs6000_secondary_reload_gpr had better be
> vali
Dear Uros,
It looks good an clear to me!
Thanks for the patch and, in particular, for adding the cleanup line.
Paul
On 27 July 2012 13:53, Uros Bizjak wrote:
> Hello!
>
> Without -mno-explicit-relocs, alpha generates:
>
> ldq $27,myBindC($29)!literal!6
> jsr $26,($2
On Fri, Jul 27, 2012 at 8:58 AM, Segher Boessenkool
wrote:
> I changed the test to INT_REGNO_P || FP_REGNO_P; this isn't
> totally obvious since INT_REGNO_P includes ap and fp, but those
> are covered by the other arms of the conditional already: in
> fact, it probably would be better to rewrite
On Fri, Jul 27, 2012 at 3:14 AM, Jan Hubicka wrote:
> Hi,
>> Index: libgcc/libgcov.c
>> ===
>> --- libgcc/libgcov.c(revision 189893)
>> +++ libgcc/libgcov.c(working copy)
>> @@ -276,6 +276,120 @@ gcov_version (struct gcov_info
Richard Guenther wrote:
> On Mon, Jun 11, 2012 at 5:25 PM, Richard Earnshaw wrote:
> > On 11/06/12 15:53, Richard Guenther wrote:
> >> The type argument or the size argument looks redundant.
> >
> > Technically, yes, we could get rid of "tree_low_cst (TYPE_SIZE (type)"
> > and calculate it inside
Please undo that change. You made the test for that heavily used
function even more expensive.
It generates machine code identical to the original.
Segher
On 07/26/2012 11:19 AM, Dodji Seketeli wrote:
+ struct scoped_attributes *ns = set_attributes_namespace (attrs,
+ attrs_len,
+ name_space);
+ if (ns == NULL)
+return NULL;
+
+
The offset field is never set; this patch removes it.
Regression tested on x86_64-unknown-linux-gnu. OK for trunk?
Mikael
2012-07-27 Mikael Morin
* array.c (gfc_copy_array_ref): Don't copy the offset field.
* expr.c (find_array_section): Ignore the offset field.
* trans-expr.c (gfc_find_i
On Fri, 2012-07-27 at 15:40 +0200, Richard Guenther wrote:
> On Thu, Jul 26, 2012 at 11:57 AM, Steven Bosscher
> wrote:
> > On Thu, Jul 26, 2012 at 11:23 AM, Richard Guenther
> > wrote:
> >> Ok! Thanks for adding this exhaustive documentation.
> >
> > There's more to come! I want to add some ex
On Fri, Jul 27, 2012 at 11:29 AM, Uros Bizjak wrote:
> Attached patch enables ix86_avoid_lea_for_addr to process
> zero-extended addresses. This patch should help atom performance,
> especially in x32 mode.
>
> Please note the complication with insn re-recognition in
> ix86_avoid_lea_for_addr, to
On 26/07/2012 17:32, Tobias Burnus wrote:
> On 07/26/2012 05:12 PM, Mikael Morin wrote:
>> On 26/07/2012 16:53, Mikael Morin wrote:
>>> Here is a draft for those. Lightly tested with print *, ...
>
> Looks rather nice. The output for test1 is also good:
>
> integer :: a(1:3,-2:5)
> gives
>
On Fri, Jul 27, 2012 at 7:16 PM, Uros Bizjak wrote:
> On Fri, Jul 27, 2012 at 11:29 AM, Uros Bizjak wrote:
>
>> Attached patch enables ix86_avoid_lea_for_addr to process
>> zero-extended addresses. This patch should help atom performance,
>> especially in x32 mode.
>>
>> Please note the complicat
Tejas Belagod wrote:
> Tejas Belagod wrote:
> > This is because offsettable_address_addr_space_p () gets as far as calling
> > strict_memory_address_addr_space_p () with a QImode and (mode_sz - 1) which
> > returns true. The only way I see offsettable_address_addr_space_p ()
> > returning
> > fals
Gabriel Dos Reis a écrit:
>> > 2012-07-20 Ryan Mansfield >
>> >
>> > * gcc.c (main): Move GCC_DRIVER_HOST_INITIALIZATION after
>> > diagnostic_initialize.
>> >
>> > Could someone please apply the change?
>>
>> The change seems small and obvious enough to not require copyright
>>
Current libiberty md5 code triggers these warnings with gcc-4.7.1 for me:
libiberty/md5.c: In function ‘md5_finish_ctx’:
libiberty/md5.c:117:3: warning: dereferencing type-punned pointer will break
strict-aliasing rules [-Wstrict-aliasing]
libiberty/md5.c:118:3: warning: dereferencing type-punned
On Fri, Jul 27, 2012 at 11:36 AM, Mike Frysinger wrote:
>
> 2012-07-27 Mike Frysinger
>
> * md5.c (md5_finish_ctx): Declare swap_bytes. Assign SWAP() output
> to swap_bytes, and then call memcpy to move it to ctx->buffer.
>
>/* Take yet unprocessed bytes into account. */
>
On 07/27/2012 04:41 AM, Nathan Froyd wrote:
> Tested on x86_64-unknown-linux-gnu. OK to commit?
>
> -Nathan
>
> * expmed.h (alg_hash, alg_hash_used_p, sdiv_pow2_cheap,
> smod_pow2_cheap, zero_cost, add_cost, neg_cost, shift_cost)
> shiftadd_cost, shiftsub0_cost, shiftsub1_cost,
[Please don't top post.]
> Please undo that change. You made the test for that heavily used
> function even more expensive.
It generates machine code identical to the original.
I suspect David is refering to execution time of the compiler itself
And I am talking about the machine code of
Richard,
This ancient patch to tweak mips_legitimize_address
http://gcc.gnu.org/ml/gcc/2008-11/msg00294.html
seems to never have been applied. Do you have any idea whether this is
still a useful change? The test case given in the first message in that
thread no longer reproduces with a rece
This patch attempts to implement the C11 _Generic feature.
Based on the last comment in
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46073
I am not at all sure I've done it correctly.
There are a couple of other things that aren't clear to me.
First, should c_parser_generic_selection call ma
Could you explain the choices you have made for the issues raised on
comp.std.c last month (regarding the handling of qualifiers on controlling
expressions) and the rationale for those choices (and make sure there are
appropriate testcases)?
--
Joseph S. Myers
jos...@codesourcery.com
Hi Janis,
On 07/26/2012 10:16 AM, Thomas Koenig wrote:
No test case because I couldn't figure out how to test for a
warning with no line number.
Try using line number 0.
That didn't work for me. Using
! { dg-do compile }
! { dg-options "-I include_6.f90 -I missing_dir" }
! { dg-warning "n
> "Joseph" == Joseph S Myers writes:
Joseph> Could you explain the choices you have made for the issues
Joseph> raised on comp.std.c last month (regarding the handling of
Joseph> qualifiers on controlling expressions) and the rationale for
Joseph> those choices (and make sure there are approp
On Fri, Jul 27, 2012 at 12:15 PM, Sandra Loosemore
wrote:
> Richard,
>
> This ancient patch to tweak mips_legitimize_address
>
> http://gcc.gnu.org/ml/gcc/2008-11/msg00294.html
>
> seems to never have been applied. Do you have any idea whether this is
> still a useful change? The test case given
On Fri, 27 Jul 2012, Tom Tromey wrote:
> I found this:
>
> https://groups.google.com/forum/?fromgroups#!topic/comp.std.c/InNlRotSWTc
>
> I wasn't really aware of 6.3.2.1, but after reading it and re-reading
> 6.5.1.1, I think I agree with his "model 0" interpretation: no promotion
> or conve
On 07/27/2012 01:06 PM, Thomas Koenig wrote:
> Hi Janis,
>
>> On 07/26/2012 10:16 AM, Thomas Koenig wrote:
>>
>>> No test case because I couldn't figure out how to test for a
>>> warning with no line number.
>>
>> Try using line number 0.
>
> That didn't work for me. Using
>
> ! { dg-do compile
On 07/26/2012 11:19 AM, Dodji Seketeli wrote:
+// Example taken from dcl.attr.grammar:
+
+int p[10];
+void f()
+{
+int x = 42, y[5];
+/* Here, the '[[gnu::' should have introduced an attribute, ont a
+ lambda invocation an array subscripting expression. */
+int(p[[gnu::x] { ret
Subject says it all, really. Two targets with redundant definitions, and
two targets with trivial definitions. Time to remove this.
Tested on x86_64-unknown-linux-gnu. Crosses to {alpha,vax}-linux-gnu built
as well. OK to commit?
-Nathan
* defaults.h (GO_IF_MODE_DEPENDENT_ADDRESS): D
On Fri, 2012-07-27 19:21:40 -0400, Nathan Froyd wrote:
> * config/vax/vax-protos.h (vax_mode_dependent_address_p): Delete.
> * config/vax/vax.h (GO_IF_MODE_DEPENDENT_ADDRESS): Delete.
> * config/vax/vax.c (vax_mode_dependent_address_p): Make static.
> Take a const_rtx.
>
63 matches
Mail list logo
