> "Tom" == Tom Tromey writes:
Tom> This is a refresh of my series to resurrect automatic dependency
Tom> tracking.
Ping.
Tom
On Fri, Aug 09, 2013 at 08:40:00PM +0200, Richard Biener wrote:
> Marek Polacek wrote:
> >In this PR the problem was that when dealing with the gimple assign in
> >the tailcall optimization, we, when the rhs operand is of a vector
> >type, need to create -1 also of a vector type, but build_int_cst
On Mon, Aug 12, 2013 at 01:20:03AM +0100, Gerald Pfeifer wrote:
> David suggested adding this link, and I think it fits nicely.
Does this also deserve a news post? I certainly found it to be
interesting news!
James
On Mon, Aug 12, 2013 at 2:52 PM, Uros Bizjak wrote:
> Hello!
>
>> movabs is incorrectly translated into "mov [rax], -1", and causes
>> compile error "Error: ambiguous operand size for `mov' ".
>> It should be "mov QWORD PTR [rax], -1"
>>
>> Bootstrap passed. Regression tested on x86_64-unknown-lin
On Mon, Aug 12, 2013 at 11:24 AM, Perez Read wrote:
>>> movabs is incorrectly translated into "mov [rax], -1", and causes
>>> compile error "Error: ambiguous operand size for `mov' ".
>>> It should be "mov QWORD PTR [rax], -1"
>>>
>>> Bootstrap passed. Regression tested on x86_64-unknown-linux-gn
I think this one is obvious/trivial, but I'll ask anyway.
OK?
Andrew.
2013-08-12 Andrew Haley
Backport from mainline:
* 2013-07-11 Andreas Schwab
* config/aarch64/aarch64-linux.h (CPP_SPEC): Define.
Index: gcc/config/aarch64/aarch64-linux.h
=
Hi Dennis, Hi Anatoly, Hi Eric,
I have run into a small problem building GCC for an AVR Linux target -
glibc-c.o is not being built. It turns out that the section handling
"avr-*-*" in the config.gcc file is redefining tmake_file without
allowing for the fact that t-glibc has already been
Hi,
this patch represents bare bones of what I hope to give me possible targets
of a virtual call.
I basically added One Definition Rule based hash that unify all types that
are same in C++ sense (with LTO many of those are still not merged - I hope
that with few dumps I can improve the merging, t
2013/8/12 Nick Clifton :
> Hi Dennis, Hi Anatoly, Hi Eric,
>
> I have run into a small problem building GCC for an AVR Linux target -
> glibc-c.o is not being built. It turns out that the section handling
> "avr-*-*" in the config.gcc file is redefining tmake_file without
> allowing for th
Hi,
Ping. Any news of the following patch being included into the trunk?
Thanks,
david
On Aug 2, 2013, at 1:45 PM, Martin Jambor wrote:
> Hi,
>
> while compute_record_mode in stor-layout.c makes sure it assigns BLK
> mode to structs with flexible arrays, it has no such provisions for
> zero le
Hi,
Ping. Any news of the following patch being included into the trunk?
Thanks,
david
On Aug 2, 2013, at 1:45 PM, Martin Jambor wrote:
> Hi,
>
> while compute_record_mode in stor-layout.c makes sure it assigns BLK
> mode to structs with flexible arrays, it has no such provisions for
> zero le
On 12 Aug 16:12, Yuri Rumyantsev wrote:
Hello,
part of the thread was accidentally removed from gcc-patches.
I've comitted Yuri's patch into ML:
http://gcc.gnu.org/ml/gcc-cvs/2013-08/msg00272.html
As far as discussion was out of ML - feel free to object.
Thanks, K
> -- Forwarded messa
Ugh. Typos in arm_neon.h macro names mean that scalar intrinsics end
up calling macros which don't exist.
So wherever I have written vget_laneq I should have written
vgetq_lane.
This gets fixed by:
http://gcc.gnu.org/ml/gcc-patches/2013-08/msg00535.html
which I was testing at the same time.
But
Thank you for this patch, Adam.
Adam Butcher a écrit:
> * pt.c: Grammar fix in comments ("it's" to "its").
FWIW, this change seems to fall under the obvious rule and thus, ought
to be committed.
>
> ---
> gcc/cp/pt.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff
Marcus,
On 9 August 2013 18:17, Marcus Shawcroft wrote:
> On 03/08/13 19:01, Venkataramanan Kumar wrote:
>
>
>> 2013-08-02 Venkataramanan Kumar
>>
>> * config/aarch64/aarch64.h (MCOUNT_NAME): Define.
>> (NO_PROFILE_COUNTERS): Likewise.
>> (PROFILE_HOOK): Likew
On Mon, Aug 12, 2013 at 5:26 PM, Uros Bizjak wrote:
> On Mon, Aug 12, 2013 at 11:24 AM, Perez Read wrote:
>
movabs is incorrectly translated into "mov [rax], -1", and causes
compile error "Error: ambiguous operand size for `mov' ".
It should be "mov QWORD PTR [rax], -1"
B
On Tue, Aug 6, 2013 at 10:23 PM, Teresa Johnson wrote:
> On Tue, Aug 6, 2013 at 9:29 AM, Teresa Johnson wrote:
>> On Tue, Aug 6, 2013 at 9:01 AM, Martin Jambor wrote:
>>> Hi,
>>>
>>> On Tue, Aug 06, 2013 at 07:14:42AM -0700, Teresa Johnson wrote:
On Tue, Aug 6, 2013 at 5:37 AM, Martin Jambo
Hi,
At present, mainline fails to build a PowerPC E500v2 cross-compiler for
me because of the bug described in PR57717. The attached patch is a
possible fix for that, although I have been struggling to obtain good
evidence that it is correct due to lack of a working current baseline.
Without the
On Mon, Aug 12, 2013 at 9:51 PM, Uros Bizjak wrote:
> On Mon, Aug 12, 2013 at 3:39 PM, Perez Read wrote:
>
>> movabs is incorrectly translated into "mov [rax], -1", and causes
>> compile error "Error: ambiguous operand size for `mov' ".
>> It should be "mov QWORD PTR [rax], -1"
>>
Hi Guys,
I am applying the patch below to fix a small problem building
m32r-linux toolchains - the glibc-c.o object file was not being built
because the definition of tmake_file in M32R section of config.gcc was
not allowing for the inclusion of t-glibc.
Cheers
Nick
gcc/ChangeLog
2013-
On 08/12/2013 08:16 AM, Jan Hubicka wrote:
With multiple inheritance I need to adjust offsets.
It's not clear to me that you need to worry about that in your search.
A call through a particular vptr can only call overrides that go into a
vtable that vptr can point to, and you can look up any
On 08/12/2013 09:06 AM, Dodji Seketeli wrote:
FWIW, this change seems to fall under the obvious rule and thus, ought
to be committed.
Agreed. But I don't see Adam's name in MAINTAINERS; do you still need
to get set up for write after approval?
Jason
On 08/11/2013 03:49 PM, Adam Butcher wrote:
+ if (cxx_dialect < cxx1y || flag_iso)
+ cp_parser_error (parser,
+"lambda templates are only available with "
+"-std=gnu++1y");
Normally we only use flag_iso to disable extensions that can ch
Xinliang David Li writes:
> I like the approach in general -- in the past, indirect call promotion
> and function inlining heuristics are disconnected -- which can lead to
> either missing promotions or useless ones. This approach solves the
> problem.
>
> On Sun, Aug 11, 2013 at 4:11 PM, Jan Hu
On Mon, Aug 12, 2013 at 01:58:34PM +0100, James Greenhalgh wrote:
>
> Ugh. Typos in arm_neon.h macro names mean that scalar intrinsics end
> up calling macros which don't exist.
>
> So wherever I have written vget_laneq I should have written
> vgetq_lane.
As this patch is obvious, fixes a silly
* config/rs6000/rs6000.c (rs6000_legitimize_reload_address): Don't
perform invalid legitimization on greater-than-word-size modes for
TARGET_E500_DOUBLE.
Disabling greater-than-word-size for E500v2 instead of continually
updating the enumeration seems reasonable.
Thanks, David
On Mon, Aug 12, 2013 at 2:31 PM, Kirill Yukhin wrote:
>
> part of the thread was accidentally removed from gcc-patches.
>
> I've comitted Yuri's patch into ML:
> http://gcc.gnu.org/ml/gcc-cvs/2013-08/msg00272.html
>
> As far as discussion was out of ML - feel free to object.
I have changed X87MO
> On 08/12/2013 08:16 AM, Jan Hubicka wrote:
> >With multiple inheritance I need to adjust offsets.
>
> It's not clear to me that you need to worry about that in your
> search. A call through a particular vptr can only call overrides
> that go into a vtable that vptr can point to, and you can look
> > Ping?
> >
>
> Ping^2
>
Ping^3
http://gcc.gnu.org/ml/gcc-patches/2013-07/msg00929.html
> Thanks,
> Kyrill
> >
> > > -Original Message-
> > > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-
> > > ow...@gcc.gnu.org] On Behalf Of Kyrylo Tkachov
> > > Sent: 23 July 2013 10:09
Hi, GCC Global Maintainers,
The Andes nds32 port has been accepted by GCC SC [1]
and the initial patch is posted on [2] but still needs
global maintainer's approval.
Here are some up-to-date status of nds32 port with current trunk:
a) Using a native compiler from current GCC trunk (20130811, Rev
> > On 08/12/2013 08:16 AM, Jan Hubicka wrote:
> > >With multiple inheritance I need to adjust offsets.
> >
> > It's not clear to me that you need to worry about that in your
> > search. A call through a particular vptr can only call overrides
> > that go into a vtable that vptr can point to, and
On Mon, 2013-08-12 at 01:11 +0200, Jan Hubicka wrote:
> Hi,
> this patch adds simple cost model into indirect call speculation.
[...]
> Index: cgraph.c
[...]
> *** cgraph_resolve_speculation (struct cgrap
> ! fprintf (dump_file, "Speculative indirect call %s/%i => %s/%i has
On 08/11/2013 03:49 PM, Adam Butcher wrote:
+ if (!(cxx_dialect >= cxx1y && (!flag_iso || lambda_p)))
Again, better to use a pedwarn.
+#define fully_implicit_template scope_chain->x_fully_implicit_template
Why did you choose to add this to saved_scope rather than cp_parser? It
see
Fixed some formatting issues and typos. There are no regressions with
the attached patch. Ok for trunk?
thanks,
David
On Wed, Aug 7, 2013 at 10:12 PM, Xinliang David Li wrote:
> Hi, the attached is a follow up patch. The patch includes the following
> changes:
>
> 1. new option -mno-default --
On Mon, Aug 12, 2013 at 4:15 AM, Florian Weimer wrote:
> On 08/12/2013 12:39 AM, Caroline Tice wrote:
>>
>> On Sun, Aug 11, 2013 at 1:04 PM, Florian Weimer
>> wrote:
>>>
>>> On 08/11/2013 01:08 AM, Caroline Tice wrote:
OK, I have removed the attempt to use $HOME for the logs; they
> -Original Message-
> From: Rainer Orth [mailto:r...@cebitec.uni-bielefeld.de]
> Sent: Friday, August 09, 2013 7:54 AM
> To: Iyer, Balaji V
> Cc: Jakub Jelinek; gcc-patches@gcc.gnu.org; Marek Polacek
> (pola...@redhat.com)
> Subject: Re: [PATCH] Fix for PR c/57490
>
> Rainer Orth write
Hi,
today I wanted to add a testcase and noticed again that in quite a few
cases we still use a second error call for the "previous
declaration"-type message. Thus the below, consisting mostly of
testsuite adjustments, the code changes themselves should be largely
obvious.
Tested x86_64-lin
Hi,
I guess I understand you now about the offsets. I do not need to update token,
right? Here is updated and somewhat more complette patch (now I can get list
of possible targets and use it to prune the callgraph)
I also added sanity check that ipa-cp devirtualization actually picks one
of targe
On Mon, Aug 12, 2013 at 11:04 AM, Anna Tikhonova
wrote:
> Also, there should be an issue with -masm=intel for sse3_monitor. Please see
> attached patch.
We can just use "monitor" with implicit operands here, like in
attached patch. It should handle 32bit, 64bit and x32 targets with ATT
and Intel
It might be more flexible to represent the analysis results as a
type/inheritance graph -- i.e. explicitly introduce inheritance edge
class to capture the interitence relationship (offset, etc) between
two class nodes. The 'method' should probably be augmented to include
vtable slot index info. Inh
Looking to simplify our navigation bar, I realised that our
mirrors page fails to link to our releases page.
Applied.
Gerald
Index: mirrors.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/mirrors.html,v
retrieving revision 1.220
diff -u
Ping? Is this one of those I can commit without waiting for approval?
-- Caroline Tice
cmt...@google.com
On Sat, Aug 10, 2013 at 1:16 PM, Caroline Tice wrote:
> I would like to make the following changes to the MAINTAINERS file.
> May I commit this?
>
> -- Caroline Tice
> cmt...@google.com
>
>
On Thu, Aug 8, 2013 at 9:34 AM, Caroline Tice wrote:
> On Thu, Aug 8, 2013 at 3:27 AM, Rainer Orth
> wrote:
>> Caroline,
>>
>
>> As an aside, I had a very quick look at libvtv to determine what's
>> required for a port to a non-Linux platform. It would be good if the
>> requirements (currently,
On Mon, Aug 12, 2013 at 12:53:56PM -0700, Caroline Tice wrote:
> Ping? Is this one of those I can commit without waiting for approval?
Yes.
Marek
One of the Fortran failures in the powerpc64le xfail file I committed
to the Google 4.8 branch last week is no longer failing. This patch
removes it from the xfails file.
Committed as obvious.
- Brooks
2013-08-12_xfail-fixup.diff
Description: Binary data
On Aug 11, 2013, at 1:00 AM, Richard Biener wrote:
> Fact is that we need to separate internal details of tree.h into sth
> shareable from exactly two places. Tree-core.h and tree.h are both 'tree.h'
> in some way. Call it tree-internal.h or tree1.h.
1 is also a bad category; internal or core
On Aug 10, 2013, at 4:08 PM, Caroline Tice wrote:
> OK, I have removed the attempt to use $HOME for the logs;
I've reviewed the code and don't see anything I don't like anymore, thanks for
the work.
On 12.08.2013 16:33, Jason Merrill wrote:
On 08/12/2013 09:06 AM, Dodji Seketeli wrote:
FWIW, this change seems to fall under the obvious rule and thus,
ought
to be committed.
Agreed. But I don't see Adam's name in MAINTAINERS; do you still
need to get set up for write after approval?
Yes,
OK.
Jason
On 08/12/2013 04:52 PM, Adam Butcher wrote:
Yes, I must confess up until now I hadn't considered it. I've just been
submitting patches from a clone of http://gcc.gnu.org/git/gcc.git. Are
you happy to approve this?
Yes.
If so I'll set up a sourceware account at
http://sourceware.org/cgi-bin/p
On 12.08.2013 22:03, Jason Merrill wrote:
On 08/12/2013 04:52 PM, Adam Butcher wrote:
Yes, I must confess up until now I hadn't considered it. I've just
been
submitting patches from a clone of http://gcc.gnu.org/git/gcc.git.
Are
you happy to approve this?
Yes.
Cheers. Submitted.
If so
Hi,
tested x86_64-linux, committed to mainline.
Thanks,
Paolo.
///
2013-08-12 Paolo Carlini
PR c++/57416
* g++.dg/cpp0x/pr57416.C: New.
Index: g++.dg/cpp0x/pr57416.C
===
--- g++.dg/cpp0x/
The attached patch defines a new parameter to decide if icall
promotion (LIPO) is using a new hotness based heuristic or not. (See
Honza's speculative call support patch for reference).
David
Index: params.def
===
--- params.def (
From: abutcher
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@201673
138bc75d-0d04-0410-961f-82ee72b054a4
---
ChangeLog | 4
MAINTAINERS | 1 +
2 files changed, 5 insertions(+)
diff --git a/ChangeLog b/ChangeLog
index a35d04a..8f29bc9 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7
From: abutcher
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@201674
138bc75d-0d04-0410-961f-82ee72b054a4
---
gcc/cp/ChangeLog | 4
gcc/cp/pt.c | 6 +++---
2 files changed, 7 insertions(+), 3 deletions(-)
diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog
index a7462af..6dd0cd8 100644
Someone using buildroot reported a problem to me that I tracked down to
compiling a routine containing __builtin_prefetch with '-march=loongson
-mabi=32'. When doing this the prefetch instruction on loongson generates
'ld' instead of 'lw' and then tries to put it into a delay slot. This
causes th
On 12.08.2013 16:47, Jason Merrill wrote:
On 08/11/2013 03:49 PM, Adam Butcher wrote:
+ if (cxx_dialect < cxx1y || flag_iso)
+ cp_parser_error (parser,
+"lambda templates are only available with "
+"-std=gnu++1y");
Normally we only use
In r201505 I added a reference to $(PIPELINE_H) to
gcc/config/i386/t-i386.
This was a holdover from before this was renamed to "pass_manager", and
should have read $(PASS_MANAGER_H) - I missed this when renaming.
Successfully bootstrapped; I've committed the fix to trunk as being
obvious (as r2016
On 12.08.2013 17:00, Jason Merrill wrote:
On 08/11/2013 03:49 PM, Adam Butcher wrote:
+#define fully_implicit_template
scope_chain->x_fully_implicit_template
Why did you choose to add this to saved_scope rather than cp_parser?
It seems like state that could be local to the parser rather than
s
On Thu, 2013-08-01 at 10:48 -1000, Richard Henderson wrote:
[...]
> Ok.
Thanks; committed to trunk as r201680 (having rebootstrapped and tested)
On Thu, 2013-08-01 at 10:58 -1000, Richard Henderson wrote:
[...]
> Ok.
Thanks; committed to svn trunk as r201681 (having rebootstrap and
tested).
> Does this also deserve a news post? I certainly found it to be
> interesting news!
Sure. Want to add one, David?
Based on another suggestion by David I also added a link to the
gnutools account on Google+. Patch below, gcc.gnu.org updated.
Gerald
Index: style.mhtml
=
62 matches
Mail list logo
