Hans, here is what I'm playing with right now against current
trunk.
I looked at the use cases for making use of the scale factor in the
VIS %gsr register and it's used similar to how rounding modes are
modified in the FPU control register.
You have a function, or family of functions, that want
OK.
Jason
I missed the fact that some of the updated Go tests were getting Tcl
errors leading to an unresolved state, rather than actually failing.
The errors were because the regular expressions which appear in the file
need to be translated into expressions which can be used with dg-error,
which is mainly
Hi,
as agreed on the audit trail. Tested x86_64-linux.
Ok for mainline?
Thanks,
Paolo.
/cp
2011-09-23 Paolo Carlini
PR c++/44267
* class.c (build_base_path): Add a tsubst_flags_t parameter.
(convert_to_base): Adjust call.
* typeck.c
This patch adds a new flag to mark sections as exclude sections. Such
sections will be discarded by the linker. Also, mark .gnu.callgraph
sections which store the callgraph edge profile info as exclude
sections.
I also want the SECTION_EXCLUDE part alone to be considered for trunk.
Index: final
Cary Coutant writes:
>>> This patch is OK, with or without the testsuite additions Jakub
>>> mentions.
>>
>> Thanks, I added the new tests, and tweaked the clone suffix parsing to
>> allow the leading "_".
>>
>> Committed as r179132.
>
> Should I backport this patch to gcc-4_6-branch?
Since you
This patch to the Go frontend improves the parsing of a non-identifier
on the left hand side of :=, and of a non-type in a type switch.
Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committed to mainline.
Ian
diff -r ffdb8cbbc880 go/parse.cc
--- a/go/parse.cc Fri Sep 23 08:28:44
Issue 253 was raised in 2000, but was considered low priority at the
time; it pointed out that classes with no actual data don't really need
an initializer, so we shouldn't require one. This becomes more
important now that we have non-static data member initializers that can
initialize all the
This patch adds a new linker plugin to re-order functions. The plugin
constructs an annotated callgraph with edge profile information and then
repeatedly groups sections that are connected by hot edges and passes the new
function layout to the linker. The grouping is done similar to the Petti
>> This patch is OK, with or without the testsuite additions Jakub
>> mentions.
>
> Thanks, I added the new tests, and tweaked the clone suffix parsing to
> allow the leading "_".
>
> Committed as r179132.
Should I backport this patch to gcc-4_6-branch?
-cary
The following pretty big patch is to remove complicated support of bound
pseudos (pseudos which should be always assigned to the same location).
It should make code more clear and speed LRA up.
The patch also
o improves inheritance by fixes some typos and using attribute value
(instead of ju
> This patch is OK, with or without the testsuite additions Jakub
> mentions.
Thanks, I added the new tests, and tweaked the clone suffix parsing to
allow the leading "_".
Committed as r179132.
-cary
Hi there,
2011/8/17 Rainer Orth :
> Hi Robert,
>
>> My patch still applies cleanly to current HEAD, has this migration
>> happened already? If not, what's the current ETA? I'll have almost
>> no spare time after this week, I'd like to sort this out before/during
>> the weekend if possible.
>
> a
On Sun, Sep 18, 2011 at 11:00 AM, Ira Rosen wrote:
> Hi,
>
> Strided accesses of single element or with gaps may require creation
> of epilogue loop. At the moment we don't support peeling for outer
> loops, therefore, we should not allow such strided accesses in outer
> loops.
>
> Bootstrapped an
On Wed, Sep 14, 2011 at 2:01 PM, Ira Rosen wrote:
> Hi,
>
> This patch makes data-refs analysis to not fail if simple_iv returns
> false in basic block SLP.
>
> Bootstrapped and tested on powerpc64-suse-linux.
> OK for mainline?
Ok if you instead of wrapping dr_analyze_innermost change the remain
ping?
On Thu, Aug 25, 2011 at 5:44 PM, Doug Kwan (關振德) wrote:
> Hi,
>
> We received this from Intel and would like to check in the trunk.
> Could the maintainers of gcc/config take a look?
>
> Thanks.
>
> -Doug
>
>
> 2011-08-25 Mark D Horn
>
> config/linux-android.h (ANDROID_STARTFILE
Pierre Vittet writes:
> Thanks for your interest,
>
> I just checked revision 179127 of GCC. Last revision is 177700, it has
> not been change for 6 weeks.
>
> My file is the same as this one:
> http://gcc.gnu.org/viewcvs/trunk/libiberty/md5.c?revision=177700&view=markup
>
> in libiberty/md5.c, f
Ian Lance Taylor writes:
> Rainer Orth writes:
>
>> Unfortunately, the
>>
>> /vol/gcc/src/hg/trunk/local/libgo/syscalls/exec.go:11:14: error: imported
>> and not used: unsafe
>>
>> error remains. I've no idea what triggers it.
>
> Bother. Neither do I. I just looked through the code and it a
Rainer Orth writes:
> Unfortunately, the
>
> /vol/gcc/src/hg/trunk/local/libgo/syscalls/exec.go:11:14: error: imported and
> not used: unsafe
>
> error remains. I've no idea what triggers it.
Bother. Neither do I. I just looked through the code and it all looks
fine. The reference to unsafe
Hi,
this patch extends inliner's predicate infrastructure to track optimization
oppurtunities that depends on properties of call site parameters that are
not readilly available (and do not belong to) jump functions.
For this new inline_param_summary is introduced that is filled in
estimate_functio
Hi,
> Martin,
>
> thanks for the explanation.
>
> I knew inliner did a good job of maintaining callgraph and profile
> information, but did not know about removing dead edges etc.
This was fixed with introduction of virtual clones, where the edges are still
needed post inlining. We however do
Hi,
I found typo in the patch instead of checking *set_after != 0 it was
set_after != 0, here is corrected version of patch. I retested the
patch without typo on mipsel-unknown-linux-gnu with no new
regressions.
thanks, Dinar.
On Fri, Sep 23, 2011 at 3:11 PM
Hi!
This simple patch improves the f3 function in the testcase below,
a parameter with TYPE_RESTRICT REFERENCE_TYPE IMHO can be safely treated
like the DECL_BY_REFERENCE case where the source actually didn't contain
a reference, but compiler created it anyway. If source contains &__restrict
param
2011/9/23 Georg-Johann Lay :
> This is the compare part to fix PR50447.
>
> Just like the preceding changes to fix PR50447 it's a micro-optimization to
> smarter print-out of instructions.
>
> The patch covers comparisons against HI and SI integers.
>
> The byte-wide comparison allows reusing the v
On Wed, Sep 21, 2011 at 11:51 AM, Iyer, Balaji V
wrote:
> Hello Everyone,
> This patch is for the Cilkplus branch.This patch will change the
> cilk-plus testsuite file extensions from ".cpp" to ".cc". In addition, some
> of the long filenames are shortened. The script file is also modified t
Martin,
thanks for the explanation.
I knew inliner did a good job of maintaining callgraph and profile
information, but did not know about removing dead edges etc.
However the situation you described (unwanted edges, post-ipa DCE
eliminating calls without updating cg etc) won't be a big issue fo
Michael Hope wrote:
> On Sat, Sep 10, 2011 at 5:04 AM, Ulrich Weigand wrote:
> > the problem in PR 50305 turned out to be caused by the ARM back-end
> > LEGITIMIZE_RELOAD_ADDRESS implementation.
>
> Interesting the fault goes away with -mfpu=neon, perhaps due to the DI
> mode operations getting p
This is the compare part to fix PR50447.
Just like the preceding changes to fix PR50447 it's a micro-optimization to
smarter print-out of instructions.
The patch covers comparisons against HI and SI integers.
The byte-wide comparison allows reusing the value in the scratch register and
individua
On Fri, Sep 23, 2011 at 4:47 PM, Jakub Jelinek wrote:
>> >> I can surely do that (or e.g.
>> >> (define_mode_attr i128 [(V4DI "%~128") (V4DF "f128") ...])
>> >> and
>> >> "vextract\t{$0x1, %1, %0|%0, %1, 0x1}"
>> >> )
>> >> if you prefer it that way, but that is a functional alternative
>> >> to t
OK.
Jason
Hi!
I've noticed with the
2009-05-29 Eric Botcazou
* tree-ssa-loop-ivopts.c (strip_offset_1) : New case.
(force_expr_to_var_cost) : Likewise.
(ptr_difference_cost): Use affine combinations to compute it.
(difference_cost): Likewise.
(get_computat
Rainer Orth writes:
> that does the trick to correctly define TIOCNOTTY and TIOCSCTTY on
> Solaris. I'm currently running a i386-pc-solaris2.10 bootstrap with a
> single-line change to mksysinfo.sh:
>
> diff --git a/libgo/mksysinfo.sh b/libgo/mksysinfo.sh
> --- a/libgo/mksysinfo.sh
> +++ b/libgo
Rainer Orth writes:
> Ian,
>
Is tIOC defined in gen-sysinfo.go? In ? Does some other
>>>
>>> Only as
>>>
>>> // unknowndefine tIOC ('t'<<8)
>>
>> Ah.
>>
>> I committed this patch which may fix this problem. Bootstrapped and ran
>> Go testsuite on x86_64-unknown-linux-gnu. You'll have to
Jakub Jelinek writes:
> On Thu, Sep 22, 2011 at 05:40:15PM -0700, Cary Coutant wrote:
>> > Ping?
>> >
>> > http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01626.html
>
> It looks good to me, but I'd prefer if Ian with his demangler author
> and libiberty maintainer hats looked at it too.
This patch
Hi,
On Thu, Sep 22, 2011 at 04:24:47PM -0700, Xinliang David Li wrote:
> ok for google branches.
>
> (Did a little digging -- the remove pass is added because ipa-inline
> did not do a good job updating the call graph so there might be some
> inconsistency. However the affinity information needs
On Tue, Sep 20, 2011 at 04:10:56PM +0200, Uros Bizjak wrote:
> >> I can surely do that (or e.g.
> >> (define_mode_attr i128 [(V4DI "%~128") (V4DF "f128") ...])
> >> and
> >> "vextract\t{$0x1, %1, %0|%0, %1, 0x1}"
> >> )
> >> if you prefer it that way, but that is a functional alternative
> >> to th
Ian,
>>> Is tIOC defined in gen-sysinfo.go? In ? Does some other
>>
>> Only as
>>
>> // unknowndefine tIOC ('t'<<8)
>
> Ah.
>
> I committed this patch which may fix this problem. Bootstrapped and ran
> Go testsuite on x86_64-unknown-linux-gnu. You'll have to remove
> TARGET/libgo/s-sysinfo to
Rainer Orth writes:
>> Is tIOC defined in gen-sysinfo.go? In ? Does some other
>
> Only as
>
> // unknowndefine tIOC ('t'<<8)
Ah.
I committed this patch which may fix this problem. Bootstrapped and ran
Go testsuite on x86_64-unknown-linux-gnu. You'll have to remove
TARGET/libgo/s-sysinfo to
Hi,
On Thu, Sep 22, 2011 at 06:36:43PM +1200, Maxim Kuvyrkov wrote:
> On 9/02/2011, at 6:53 AM, Martin Jambor wrote:
>
> >
> > This patch basically disables all intraprocedural devirtualization
> > simply because that transformation relies on assumptions that no
> > longer hold true. That leave
On 09/23/2011 03:59 PM, Paolo Carlini wrote:
On 09/23/2011 03:46 PM, Paolo Carlini wrote:
On 09/23/2011 11:39 AM, JonY wrote:
Ping, any updates?
I'm wondering if it wouldn't be cleaner to handle this in the
appropriate config/os/ headers, something like (with a good comment
before!):
#ifnde
On 09/23/2011 03:46 PM, Paolo Carlini wrote:
On 09/23/2011 11:39 AM, JonY wrote:
Ping, any updates?
I'm wondering if it wouldn't be cleaner to handle this in the
appropriate config/os/ headers, something like (with a good comment
before!):
#ifndef _GLIBCXX_FULLY_DYNAMIC_STRING
#define _GLIBC
On 9/23/2011 21:46, Paolo Carlini wrote:
> On 09/23/2011 11:39 AM, JonY wrote:
>> Ping, any updates?
> I'm wondering if it wouldn't be cleaner to handle this in the
> appropriate config/os/ headers, something like (with a good comment
> before!):
>
> #ifndef _GLIBCXX_FULLY_DYNAMIC_STRING
> #defin
On 09/23/2011 11:39 AM, JonY wrote:
Ping, any updates?
I'm wondering if it wouldn't be cleaner to handle this in the
appropriate config/os/ headers, something like (with a good comment
before!):
#ifndef _GLIBCXX_FULLY_DYNAMIC_STRING
#define _GLIBCXX_FULLY_DYNAMIC_STRING
#endif
Paolo.
Hi,
so this is what I'm finishing testing on x86_64-linux. Ok?
Thanks,
Paolo.
/cp
2011-09-23 Paolo Carlini
PR c++/50258
* decl.c (check_static_variable_definition): Allow in-class
initialization of static data member of non-integral type in
Hi,
On Fri, Sep 16, 2011 at 04:29:44PM +0200, Jan Hubicka wrote:
> >
> > * ipa-prop.h (jump_func_type): Updated comments.
> > (ipa_known_type_data): New type.
> > (ipa_jump_func): Use it to describe known type jump functions.
> > * ipa-prop.c (ipa_print_node_jump_functions_for_edg
Mike Stump writes:
> On Sep 1, 2011, at 6:20 AM, Bernd Schmidt wrote:
>> The following change
>>
>> 2011-05-26 Rainer Orth
>> PR gcov-profile/48845
>>
>> causes testsuite failures on C6X.
>
> I was hoping that Rainer would comment... or a profiling person, or an avr
> person
Bernd Schmidt writes:
> The following change
>
> 2011-05-26 Rainer Orth
> PR gcov-profile/48845
>
> causes testsuite failures on C6X. Specifically,
>
> @@ -495,11 +495,16 @@ proc check_profiling_available { test_wh
> # These conditions depend on the argument so examine them be
Ian Lance Taylor writes:
>> * The Solaris 11/x86 libgo bootstrap dies like this:
>>
>> /vol/gcc/src/hg/trunk/local/libgo/syscalls/exec.go:11:14: error: imported
>> and not used: unsafe
>
> That is odd since the file clearly does use unsafe. Perhaps it is
> somehow a consequence of the other err
hi, Richard,
This version of patch showed no regressions on mipsel-unknown-linux-gnu.
Thanks, Dinar.
On Wed, Sep 21, 2011 at 4:59 PM, Richard Sandiford
wrote:
> Dinar Temirbulatov writes:
>> @@ -14696,7 +14696,11 @@ mips_avoid_hazard (rtx after, rtx insn,
>> clobb
On 09/23/2011 11:36 AM, Georg-Johann Lay wrote:
Paolo Bonzini schrieb:
On 09/23/2011 10:56 AM, Paolo Bonzini wrote:
Also, I am curious about one thing: while this is of course a very
pragmatic solution, you could also convert AVR to get rid of CC0, do
this at expansion time, and get split-wide-
> Mainline Ada bootstrap is currently broken on Solaris 8 and 9/x86 with
> Sun as:
>
> Assembler: a-strunb.adb
> "/var/tmp//cc6ZxWWa.s", line 2395 : Syntax error
> "/var/tmp//cc6ZxWWa.s", line 2591 : Syntax error
> make[7]: *** [a-strunb.o] Error 1
>
> The problem is that s-atocou-x86
Mainline Ada bootstrap is currently broken on Solaris 8 and 9/x86 with
Sun as:
Assembler: a-strunb.adb
"/var/tmp//cc6ZxWWa.s", line 2395 : Syntax error
"/var/tmp//cc6ZxWWa.s", line 2591 : Syntax error
make[7]: *** [a-strunb.o] Error 1
The problem is that s-atocou-x86.adb uses lock
On Fri, Sep 23, 2011 at 10:12 AM, Jakub Jelinek wrote:
>> > --- gcc/config/i386/i386.c.jj 2011-09-02 16:29:38.0 +0200
>> > +++ gcc/config/i386/i386.c 2011-09-07 21:57:52.0 +0200
>> > @@ -18304,6 +18304,11 @@ ix86_prepare_sse_fp_compare_args (rtx de
>> > {
>> > rtx tmp;
>>
On 09/23/2011 11:23 AM, Rainer Orth wrote:
Rainer Orth writes:
John David Anglin writes:
I could find no indication that HP-UX 10 supported anything but DCE
threads (especially no POSIX threads), so _REENTRANT being defined must
meant DCE threads, unless I'm mistaken.
_REENTRANT doesn't i
On 9/21/2011 17:08, xunxun wrote:
> 于 2011/9/21 10:56, Cesar Strauss 写道:
>> Please let me present an opposing view.
>>
>> On the one hand, according to comment 4 of [1], by using
>> --enable-fully-dynamic-string, all other users will miss a very good
>> optimization. On the other hand, these users
Paolo Bonzini schrieb:
> On 09/23/2011 10:56 AM, Paolo Bonzini wrote:
>> Also, I am curious about one thing: while this is of course a very
>> pragmatic solution, you could also convert AVR to get rid of CC0, do
>> this at expansion time, and get split-wide-types to work as intended.
My changes ar
Rainer Orth writes:
> John David Anglin writes:
>
>>> I could find no indication that HP-UX 10 supported anything but DCE
>>> threads (especially no POSIX threads), so _REENTRANT being defined must
>>> meant DCE threads, unless I'm mistaken.
>>
>> _REENTRANT doesn't imply DCE threads. The reent
On 14 September 2011 13:30, Richard Sandiford
wrote:
> neon_vget_high and neon_vget_low extract one half of a vector.
> The patterns look like:
>
> (define_insn "neon_vget_highv16qi"
> [(set (match_operand:V8QI 0 "s_register_operand" "=w")
> (vec_select:V8QI (match_operand:V16QI 1 "s_regis
On 09/23/2011 10:56 AM, Paolo Bonzini wrote:
Also, I am curious about one thing: while this is of course a very
pragmatic solution, you could also convert AVR to get rid of CC0, do
this at expansion time, and get split-wide-types to work as intended.
compare-elim.c makes it relatively easy to re
On 20/09/11 10:35, Terry Guo wrote:
> Hello,
>
>>>
>>> I suppose you want a torture that excercises different -march/-mtune
>>> combinations then.
>>>
>>> But can't you do the pruning somewhere in an .exp file then instead
>>> of sprinkling it all over the tests itself?
>>>
>>
>> It seems not. At
On 09/23/2011 10:41 AM, Georg-Johann Lay wrote:
Yes, can be done for comparisons, too. I'd prefer a separate patch for it. The
subject is bit misleading; should be (2/n).
Also, I am curious about one thing: while this is of course a very
pragmatic solution, you could also convert AVR to get r
Paolo Bonzini schrieb:
> On 09/22/2011 07:43 PM, Georg-Johann Lay wrote:
>> This patch adds the PLUS part to fix the PR.
>>
>> addsi3 has a 8-bit scratch register now so that constants that are not
>> covered
>> by the constraints won't force a reload of the constant.
>>
>> The output routine tries
Denis Chertykov schrieb:
> 2011/9/22 Georg-Johann Lay :
>> This patch adds the PLUS part to fix the PR.
>>
>> addsi3 has a 8-bit scratch register now so that constants that are not
>> covered
>> by the constraints won't force a reload of the constant.
>>
>> The output routine tries adding the cons
On 20/09/11 02:52, Terry Guo wrote:
> Hello,
>
> This is an old case written in 1997. Perhaps it intends to cover an old gcc
> bug. In fact the case itself passed on QEMU for cortex-m3. So I am proposing
> a patch here to remove the test script to make this pass as a normal pass.
> Otherwise the p
On Fri, Sep 23, 2011 at 12:09:18AM +0200, Uros Bizjak wrote:
> > --- gcc/config/i386/i386.c.jj 2011-09-02 16:29:38.0 +0200
> > +++ gcc/config/i386/i386.c 2011-09-07 21:57:52.0 +0200
> > @@ -18304,6 +18304,11 @@ ix86_prepare_sse_fp_compare_args (rtx de
> > {
> > rtx tmp;
>
2011/9/23 Jason Merrill :
> On 09/22/2011 05:11 PM, Fabien Chêne wrote:
>>
>> 2011/9/22 Jason Merrill:
>
>>> I don't, it just seemed strange to handle functions differently from
>>> other
>>> decls here. But when I look more closely I see that we're in
>>> lookup_field_1, which isn't interested in
On Thu, Sep 22, 2011 at 05:40:15PM -0700, Cary Coutant wrote:
> > Ping?
> >
> > http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01626.html
It looks good to me, but I'd prefer if Ian with his demangler author
and libiberty maintainer hats looked at it too.
> > include/ChangeLog:
> >
> > PR 408
On 09/22/2011 07:43 PM, Georg-Johann Lay wrote:
This patch adds the PLUS part to fix the PR.
addsi3 has a 8-bit scratch register now so that constants that are not covered
by the constraints won't force a reload of the constant.
The output routine tries adding the constant and subtracting the n
68 matches
Mail list logo
