Hello Ramana,
Can you please review my patch at
http://gcc.gnu.org/ml/gcc-patches/2013-03/msg01252.html.
Thanks.
Terry
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Terry Guo
> Sent: Friday, March 29, 2013 6:00 PM
> To:
)¡¦:
/home/jasonwucj/tmp/gcc-svn-trunk/gcc/tree-ssa-loop-ivcanon.c:866:46: warning:
¡¥n_unroll¡¦ may be used uninitialized in this function [-Wuninitialized]
/home/jasonwucj/tmp/gcc-svn-trunk/gcc/tree-ssa-loop-ivcanon.c:655:17: note:
¡¥n_unroll¡¦ was declared here
[jasonwucj@sw-compiler]$ tool
On Sat, Apr 13, 2013 at 03:48:13PM -0400, David Edelsohn wrote:
> V2DI mode is allowed in GPRs and the pattern predicate allows easy
> vector constants but the pattern in vsx.md does not provide an
> alternative for that case, which can lead to an ICE where the insn
> does not satisfy its constrain
On Tue, Apr 16, 2013 at 01:03:35AM +0200, Steven Bosscher wrote:
> On Tue, Apr 16, 2013 at 12:48 AM, Michael Meissner wrote:
> > 0x1055e1bf check_rtl
> > /home/meissner/fsf-src/meissner-lra/gcc/lra.c:1999
>
> These are all cases of insns not satisfying their constraints. There
> are n
On Tue, Apr 16, 2013 at 12:48 AM, Michael Meissner wrote:
> 0x1055e1bf check_rtl
> /home/meissner/fsf-src/meissner-lra/gcc/lra.c:1999
These are all cases of insns not satisfying their constraints. There
are no PRs for this, and there are no test suite failures of this kind
in the logs
I built the spec 2006 suite with/without Vlad's patches for enabling using the
LRA register allocator for the powerpc. Because of the bug with the count
register that was in the version I checked out, I have built things with the
-fno-branch-count-reg option.
I created a branch off of subversion
Tobias Burnus wrote:
Am 15.04.2013 23:03, schrieb Thomas Koenig:
I see that it can be useful, but I really dislike disabling the TKR
checks.
Side remark: TS29113 already allows to disable TK checks via TYPE(*).
And I find disabling TRK checks very useful for certain applications -
and MPI bu
Le 09/04/2013 20:33, Janne Blomqvist a écrit :
> Regtested on x86_64-unknown-linux-gnu, Ok for trunk?
>
Looks good. OK. Thanks.
> (IMHO the increase in compile time is modest enough that it's not
> worth doing the caching of uncompressed module files that I was
> previously thinking about, especi
Am 15.04.2013 23:03, schrieb Thomas Koenig:
Hi Tobias,
Minor patch update due to Janus' gfc_explicit_interface_required patch.
Build and regtested on x86-64-gnu-linux.
OK for the trunk?
I see that it can be useful, but I really dislike disabling the TKR
checks. Can you put this behind an opt
Hi, I'm to bring up this patch about '-fstack-protector-strong' for trunk.
Background - some times stack-protector is too-simple while
stack-protector-all over-kills, for example, to build one of our core
systems, we forcibly add "-fstack-protector-all" to all compile
commands, which brings big pe
Hi Tobias,
Minor patch update due to Janus' gfc_explicit_interface_required patch.
Build and regtested on x86-64-gnu-linux.
OK for the trunk?
I see that it can be useful, but I really dislike disabling the TKR
checks.
Can you put this behind an option so the user has to specify that
he reall
*ping*
Also pending is the NO_ARGS_CHECK patch at:
http://gcc.gnu.org/ml/fortran/2013-04/msg00120.html
On April 10, Tobias Burnus wrote:
Fortran 2008 supports C_LOC(array); if the argument is not simply
contiguous, the current code adds a call to __gfortran_intrinsic_pack.
The pack call sh
Hello,
The attached patch enables register pressure aware scheduling for the
ix86 and x86_64 targets. It uses the optimistic algorithm to avoid
being overly conservative.
This is the same as what other CISCy targets, like s390, also do.
The motivation for this patch is the excessive spilling I'v
Le 14/04/2013 16:21, Thomas Koenig a écrit :
> Hi Mikael,
>
>>>
>>> - (void) gfc_expr_walker (&fcn, callback_reduction, NULL);
>>
>> why remove this?
>
> Because it is not needed, as the test case _46 shows. No need
> to run this twice, it doesn't get better :-)
>
Indeed, that's right.
> gf
>> Indeed, that's invalid:
>>
> And then, the call to gfc_match_varspec shouldn't be there in the first
> place. I'll test the following later.
It seems like the parts you're removing have essentially been there
since day zero. Would be interesting to know if (and where) your patch
fails.
Cheers
Sebastian,
Alan's patch is related to some original work by Richard Henderson and
Richard has told both Alan and I that he eventually will review the
patch. If someone else wants to review the patch before he gets to
it, that would be great as well.
Thanks, David
> would one of the global revie
Le 13/04/2013 17:44, Mikael Morin a écrit :
> Indeed, that's invalid:
>
And then, the call to gfc_match_varspec shouldn't be there in the first
place. I'll test the following later.
Index: primary.c
===
--- primary.c (révision
Generate prologue/epilogue using STRD/LDRD in ARM mode, when tuning
prefer_ldrd_strd flag is set, such as in Cortex-A15.
The previous version of this patch was posted for review here:
http://gcc.gnu.org/ml/gcc-patches/2012-10/msg00995.html
The new version includes the following improvements:
(1)
From: Eric Botcazou
Date: Mon, 15 Apr 2013 18:07:05 +0200
>> We can actually support this by adding patterns for the partial store
>> instructions, which can store 8-bit and 16-bit quantities from FP
>> registers.
>
> Ah, indeed, with -mvis. Not clear whether that would really be worthwhile.
W
> No objections.
Thanks.
> We can actually support this by adding patterns for the partial store
> instructions, which can store 8-bit and 16-bit quantities from FP
> registers.
Ah, indeed, with -mvis. Not clear whether that would really be worthwhile.
--
Eric Botcazou
Richard Biener wrote:
On Wed, Apr 10, 2013 at 12:50 PM, James Greenhalgh
wrote:
-Original Message-
From: dosr...@gmail.com [mailto:dosr...@gmail.com] On Behalf Of Gabriel
Dos Reis
Sent: 20 March 2013 19:09
To: James Greenhalgh
Cc: Jakub Jelinek; Richard Biener; gcc-patches@gcc.gnu.org;
While working on the latest 52748 patch, it occurred to me that using
tsubst_expr for DECLTYPE_TYPE is wrong, since that function deals with
statements; since we only have to deal with expressions there,
tsubst_copy_and_build is a better choice.
Tested x86_64-pc-linux-gnu, applying to trunk.
c
I noticed that gcc.dg/torture/pr53922.c FAILs on Tru64 UNIX on the 4.7
branch for the same reason as on HP-UX: no undefined weak symbols. The
following patch accounts for that, together with a small cleanup.
Tested with the appropriate runtest invocation on alpha-dec-osf5.1b,
installed on 4.7 bra
On 04/11/2013 08:50 PM, Jason Merrill wrote:
My earlier N3276 work only affected the function call syntax, but it
needs to affect implicit function calls from overloaded operators as well.
...and in templates.
Tested x86_64-pc-linux-gnu, applying to trunk and 4.8.
commit 9448b41e5f5a64a2a9251
My code for inserting capture proxy DECL_EXPRs was assuming that there
is one stmt_list_stack entry per cp_binding_level entry, but that isn't
always the case. Conveniently, the stmt_list_stack entry we want will
always be at index 1, since each function has its own stack.
Tested x86_64-pc-li
On 04/14/2013 08:20 AM, Steven Bosscher wrote:
Hello,
This patch splits mips_reorg.c in a pre-dbr_schedule part and a new,
machine specific post-dbr_schedule pass. With this patch,
cleanup_barriers and dbr_schedule can be static functions again.
Cross-built&tested mips-sim. OK for trunk?
Ciao!
In reality this was discontinued some time ago. But I was just doing
some local spring-cleaning and realized that wwwdocs had never been updated.
Committed as obvious.
R.
Index: svn.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/svn.h
On the 4.7 branch, 30_threads/condition_variable/members/53841.cc fails
to compile on IRIX and Tru64 UNIX since the required -std=gnu++0x wasn't
passed. Fixed like this, tested with the appropriate runtest
invocation, installed on the 4.7 branch.
Rainer
2013-04-15 Rainer Orth
On 04/15/2013 01:36 PM, Gerald Pfeifer wrote:
On Tue, 2 Apr 2013, Tom G. Christensen wrote:
First round of results for gcc 4.8.x.
There's something in that patch that you probably did not mean
to include:
Missing: 4.8.0#x86_64-winnix-linux-gnu (arch/tune
amdfam10/amdfam10)#http://gcc.gnu.
On Mon, 15 Apr 2013, Jakub Jelinek wrote:
> Hi!
>
> record_increment failed to verify that initializer is usable, which it is
> only if cand_stmt is an addition (CAND_ADD can be e.g. even on a cast of
> addition to some type of the same precision etc.) and one of the operands is
> c->base_expr (b
FWIW, I agree with Jakub's fix. Thanks!
Bill
On Mon, 2013-04-15 at 16:28 +0200, Jakub Jelinek wrote:
> Hi!
>
> record_increment failed to verify that initializer is usable, which it is
> only if cand_stmt is an addition (CAND_ADD can be e.g. even on a cast of
> addition to some type of the same
On Mon, Apr 15, 2013 at 04:14:58AM -0500, Gabriel Dos Reis wrote:
> > Defining a variable in my environment counts as a clear intention.
>
> If you invoke GCC on command line with explicit option requesting
> colors, I don't think there is any doubt that. However, I dispute the
> intent to be so
Hi!
record_increment failed to verify that initializer is usable, which it is
only if cand_stmt is an addition (CAND_ADD can be e.g. even on a cast of
addition to some type of the same precision etc.) and one of the operands is
c->base_expr (because then the other operand necessarily has to be the
On 04/13/2013 07:17 PM, Chung-Ju Wu wrote:
Hi,
I noticed there is an uninitialized variable warning
when compiling tree-ssa-loop-ivcanon.c file.
Attached patch is a slight modification to avoid the warning
and a plaintext ChangeLog is as below.
Is it OK for trunk?
2013-04-14 Chung-Ju Wu
This fixes the wrong-code caused by the use-before-compute
of GROUP_READ_WRITE_DEPENDENCE after I moved data dependence
checking after group analysis in the vectorizer. The fix
is to move the dependence checking completely to the
dependence checking - now possible as we have computed groups
alrea
On 04/14/2013 03:43 AM, Eric Botcazou wrote:
I don't recall ever working on this aspect of reorg. The obvious worry
is that with reorg moving stuff around those notes may not be valid
anymore in the general case.
Yes, in the general case I agree that's too dangerous. In this particular
case,
Hello,
would one of the global reviewers mind having a look at this?
The comment #2 of PR55033 clearly shows that this is a compiler bug:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55033#c2
Alan Modra proposed a patch to fix this problem only one day after the bug
report. This was in Octobe
On Mon, Apr 15, 2013 at 3:51 PM, Tobias Burnus wrote:
> Janne Blomqvist wrote:
>>
>> I committed the attached patch with most of your suggestions as r197968.
>
>
> Thanks! Although, I assume you meant "kind=8" in the last sentence:
>
>> +the underlying platform clock. @var{COUNT_MAX} usually equal
Janne Blomqvist wrote:
I committed the attached patch with most of your suggestions as r197968.
Thanks! Although, I assume you meant "kind=8" in the last sentence:
+the underlying platform clock. @var{COUNT_MAX} usually equals
+@code{HUGE(COUNT_MAX)}. Note that the millisecond resolution of t
Found with the Coverity scanner. It also complains about the
fall-throughs. Either the comment will silence the warning* or at least
it helps human reviewers.
Committed as Rev. 197969 after build+regtesting on x86-64-gnu-linux.
Tobias
(* Well, one can also silence the Coverity diagnostic for
On Mon, Apr 15, 2013 at 11:37 AM, Tobias Burnus wrote:
> Janne Blomqvist wrote:
>>
>> Attached is an updated patch which uses GetTickCount for system_clock_4;
>> this should be fine as system_clock_4 wraps around in ~25 days anyways. For
>> system_clock_8 it uses QueryPerformance{Counter,Frequency
On Mon, Apr 15, 2013 at 11:47 AM, Richard Biener
wrote:
> On Sun, Apr 14, 2013 at 9:46 AM, Eric Botcazou wrote:
>>> This is a quadratic algorithm and as such not ok. We already have
>>> aliasing_component_refs_p in tree-ssa-alias.c which is supposed to be
>>> the non-quadratic replacement. It's
On Mon, Apr 15, 2013 at 12:04 PM, Eric Botcazou wrote:
>> I think I already rejected this and asked you to fix the users (like
>> layout_type is a user).
>
> Yes, but that would be a pain, there are too many users in the Ada front-end.
Users that care about the special casing of 0 - 1 and overfl
On Tue, Apr 2, 2013 at 1:59 PM, Pavel Chupin wrote:
> On Mon, Apr 1, 2013 at 7:07 PM, Pavel Chupin wrote:
>> On Android pthread is integrated into libc.
>> Attached patch fixes configures for this case by trying to build test
>> without -pthread -lpthread.
>>
>> 2013-04-01 Pavel Chupin
>>
>>
On Tue, 2 Apr 2013, Tom G. Christensen wrote:
> First round of results for gcc 4.8.x.
There's something in that patch that you probably did not mean
to include:
Missing: 4.8.0#x86_64-winnix-linux-gnu (arch/tune
amdfam10/amdfam10)#http://gcc.gnu.org/ml/gcc-testresults/2013-03/msg03331.html
Mi
Tobias Burnus writes:
> Rainer Orth wrote:
>> While the Solaris linker doesn't support the --as-needed/--no-as-needed
>> options (yet), it long has provided the equivalent -z ignore/-z record
>> options. This patch makes use of them, avoiding unnecessary
>> dependencies on libgcc_s.so.1.
>
> Cou
On 08/04/13 13:47, Kyrylo Tkachov wrote:
Hi all,
When compiling:
unsigned long long
muld (unsigned long long X, unsigned long long Y)
{
unsigned long long mask = 0xull;
return (X & mask) * (Y & mask);
}
we get a suboptimal sequence:
stmfd sp!, {r4, r5}
mvn
Ping?
Thanks,
Kyrill
> -Original Message-
> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov
> Sent: 08 April 2013 13:47
> To: gcc-patches@gcc.gnu.org
> Cc: Ramana Radhakrishnan; Richard Earnshaw
> Subject: [PATCH][ARM] Improve code
Rainer Orth wrote:
While the Solaris linker doesn't support the --as-needed/--no-as-needed
options (yet), it long has provided the equivalent -z ignore/-z record
options. This patch makes use of them, avoiding unnecessary
dependencies on libgcc_s.so.1.
Could you also do a similar update in lib
On Mon, Apr 15, 2013 at 8:30 AM, Andrew Hsieh wrote:
> Sure. Put back Init(SSP_TLS) and rebase to r197955.
> If no further feedback, please merge.
Committed with slightly changed ChangeLog entry:
2013-04-15 Andrew Hsieh
* config/i386/i386.opt: New option mstack-protector-guard=.
* c
> I think I already rejected this and asked you to fix the users (like
> layout_type is a user).
Yes, but that would be a pain, there are too many users in the Ada front-end.
> Clearly 0 - 1 in unsigned arithmetic overflows. Not indicating this may
> cause bugs elsewhere as easily as it fixes co
Trunk as been merged into the c++-concepts branch.
-- Gaby
Il 08/04/2013 14:20, Rainer Orth ha scritto:
> While the Solaris linker doesn't support the --as-needed/--no-as-needed
> options (yet), it long has provided the equivalent -z ignore/-z record
> options. This patch makes use of them, avoiding unnecessary
> dependencies on libgcc_s.so.1.
>
> Bootst
On Sun, Apr 14, 2013 at 10:05 AM, Eric Botcazou wrote:
> Hi,
>
> this is a regression present on the mainline and 4.8 branch and introduced by
> the latest series of sizetype changes. Associated adjustments were made in
> the various front-ends for it, most notably Ada which was the most affected
On Sun, Apr 14, 2013 at 9:46 AM, Eric Botcazou wrote:
>> This is a quadratic algorithm and as such not ok. We already have
>> aliasing_component_refs_p in tree-ssa-alias.c which is supposed to be
>> the non-quadratic replacement. It's not used via decl_refs_may_alias_p,
>> so that may be the thi
Found with the Coverity scanner.*
Committed as Rev. 197961 after build+regtesting on x86-64-gnu-linux.
Tobias
* There are more issues, some real, some false positive; thus, if
someone wants to reduce the number of true issues ...
2013-04-15 Tobias Burnus
* class.c (gfc_find_intrinsic_vtab
> This didn't really help make things look prettier. The notes are
> enumerated in many other places, for various purposes. I didn't like
> special-casing this one INSN_NOTE oddity when there are so many others
> already. So I created a note_outside_basic_block_p instead, I hope you
> agree with th
The following patch has remained unreviewed for a week:
[build] Use -z ignore instead of --as-needed on Solaris
http://gcc.gnu.org/ml/gcc-patches/2013-04/msg00425.html
It needs build and driver maintainers to review.
Thanks.
Rainer
--
---
On Wed, Apr 10, 2013 at 7:13 PM, Uros Bizjak wrote:
> Attached testsuite patch fixes:
>
> pr33992.c: In function ‘do_test’:
> pr33992.c:11:1: error: inlining failed in call to always_inline ‘foo’:
> function body can be overwritten at link time
> pr33992.c:28:9: error: called from here
> foo
On Sun, Apr 14, 2013 at 4:53 PM, Marc Glisse wrote:
> On Wed, 10 Apr 2013, Gabriel Dos Reis wrote:
>
>> On Wed, Apr 10, 2013 at 1:42 PM, Manuel López-Ibáñez
>> wrote:
>>>
>>> On 9 April 2013 15:21, Jakub Jelinek wrote:
white). The default is still -fdiagnostics-color=never, can be cha
Janne Blomqvist wrote:
Attached is an updated patch which uses GetTickCount for
system_clock_4; this should be fine as system_clock_4 wraps around in
~25 days anyways. For system_clock_8 it uses
QueryPerformance{Counter,Frequency}.
The patch also adds an additional check for _POSIX_MONOTONI
> -Original Message-
> From: Ramana Radhakrishnan
> Sent: Thursday, April 11, 2013 4:40 PM
> To: Joey Ye
> Cc: gcc-patches@gcc.gnu.org
> Subject: Re: [PATCH][ARM][thumb1] Reduce lr save for leaf function with
> non-far jump
>
> On 12/20/12 09:53, Joey Ye wrote:
> > Current GCC thumb1 has
Yvan's patch was checked in a few days ago...
/Marcus
r197770 | clyon | 2013-04-11 13:06:04 +0100 (Thu, 11 Apr 2013) | 12 lines
2013-03-16 Yvan Roux
* include/private/gcconfig.h (AARCH64): New macro (defined only if
__aarch64__).
* include/private/gcconfig.h (mach_type
Andrew Haley writes:
> Looks basically OK. What were the failures, though?
FAIL: TestClosureGC run
FAIL: Array_3 execution - source compiled test
FAIL: Array_3 -findirect-dispatch execution - source compiled test
FAIL: Array_3 -O3 execution - source compiled test
FAIL: Array_3 -O3 -findirect-di
64 matches
Mail list logo
