Hi Janus,
OK for trunk?
Looks good to m
Committed as rev. 201658; also committed a
snippet to the documentation.
Thanks for the review!
Thomas
In function get_default_value of tree-ssa-ccp.c,
261 else if (is_gimple_assign (stmt)
262/* Value-returning GIMPLE_CALL statements assign to
263 a variable, and are treated similarly to GIMPLE_ASSIGN. */
264|| (is_gimple_call (stmt)
265
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-linux-gnu (pc).
2013-08-10 Perez Read netfirew...@gmail.com
*
Tom == Tom Tromey tro...@redhat.com 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 pola...@redhat.com 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
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 ubiz...@gmail.com 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
On Mon, Aug 12, 2013 at 11:24 AM, Perez Read netfirew...@gmail.com 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
I think this one is obvious/trivial, but I'll ask anyway.
OK?
Andrew.
2013-08-12 Andrew Haley a...@redhat.com
Backport from mainline:
* 2013-07-11 Andreas Schwab sch...@suse.de
* config/aarch64/aarch64-linux.h (CPP_SPEC): Define.
Index:
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,
2013/8/12 Nick Clifton ni...@redhat.com:
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
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 length
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 length
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
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.
Thank you for this patch, Adam.
Adam Butcher a...@jessamine.co.uk 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
Marcus,
On 9 August 2013 18:17, Marcus Shawcroft marcus.shawcr...@arm.com wrote:
On 03/08/13 19:01, Venkataramanan Kumar wrote:
2013-08-02 Venkataramanan Kumar venkataramanan.ku...@linaro.org
* config/aarch64/aarch64.h (MCOUNT_NAME): Define.
(NO_PROFILE_COUNTERS):
On Mon, Aug 12, 2013 at 5:26 PM, Uros Bizjak ubiz...@gmail.com wrote:
On Mon, Aug 12, 2013 at 11:24 AM, Perez Read netfirew...@gmail.com 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
On Tue, Aug 6, 2013 at 10:23 PM, Teresa Johnson tejohn...@google.com wrote:
On Tue, Aug 6, 2013 at 9:29 AM, Teresa Johnson tejohn...@google.com wrote:
On Tue, Aug 6, 2013 at 9:01 AM, Martin Jambor mjam...@suse.cz wrote:
Hi,
On Tue, Aug 06, 2013 at 07:14:42AM -0700, Teresa Johnson wrote:
On
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 ubiz...@gmail.com wrote:
On Mon, Aug 12, 2013 at 3:39 PM, Perez Read netfirew...@gmail.com 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
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
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 change
Xinliang David Li davi...@google.com 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
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 bug,
* 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 kirill.yuk...@gmail.com 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
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
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
To: 'Richard
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,
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
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
seems
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 davi...@google.com wrote:
Hi, the attached is a follow up patch. The patch includes the following
changes:
1. new option
On Mon, Aug 12, 2013 at 4:15 AM, Florian Weimer fwei...@redhat.com wrote:
On 08/12/2013 12:39 AM, Caroline Tice wrote:
On Sun, Aug 11, 2013 at 1:04 PM, Florian Weimer fwei...@redhat.com
wrote:
On 08/11/2013 01:08 AM, Caroline Tice wrote:
OK, I have removed the attempt to use $HOME for the
-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
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
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
On Mon, Aug 12, 2013 at 11:04 AM, Anna Tikhonova
anna.m.tikhon...@gmail.com 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
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.
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 cmt...@google.com wrote:
I would like to make the following changes to the MAINTAINERS file.
May I commit this?
-- Caroline Tice
On Thu, Aug 8, 2013 at 9:34 AM, Caroline Tice cmt...@google.com wrote:
On Thu, Aug 8, 2013 at 3:27 AM, Rainer Orth r...@cebitec.uni-bielefeld.de
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
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 richard.guent...@gmail.com 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
On Aug 10, 2013, at 4:08 PM, Caroline Tice cmt...@google.com 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
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 paolo.carl...@oracle.com
PR c++/57416
* g++.dg/cpp0x/pr57416.C: New.
Index: g++.dg/cpp0x/pr57416.C
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 abutcher@138bc75d-0d04-0410-961f-82ee72b054a4
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
From: abutcher abutcher@138bc75d-0d04-0410-961f-82ee72b054a4
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
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
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
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
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
65 matches
Mail list logo
