Here we can get to define_label from within poplevel, which results in
recursive starting of TV_NAME_LOOKUP. The obvious solution is to just
use cond_start/stop in define_label.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 4e0a90cdec5d3e1b29335eb8c63c6a6885e0f674
Author: Jason
Andrey Belevantsev a...@ispras.ru writes:
On 30.09.2011 19:21, Roman Zhuykov wrote:
2011/7/22 Richard Sandifordrichard.sandif...@linaro.org:
That's pre-approved (independently of the other patches) if it works.
...
Changed like the following. Will commit if no objections after a couple of
On 15 February 2012 20:09, DJ Delorie wrote:
The typo is obvious, but whether or not the wrapped code still works
isn't (to me).
* src/c++98/locale.cc (locale::facet::_S_get_c_locale): Fix typo.
I must have looked at that code dozens of times and never seen it! :-\
I wonder if
Hi Diego,
This patch fixes a problem of the validate_failures.py script that
it rejects cross compilers. I have tested it by running the script
in the build directory of gcc configure for arm-unknown-linux-gnueabi.
I also tested it on a native compiler for x86. This should go to both
I committed (Rev. 184331) the attached patch as obvious. Thanks to all
translators at translationproject.org for their work - and in particular
to Roland, who reported all the issues below.
See also PR 52274 for remaining translation-related issues.
* * *
General GCC status: 0 P1, 64 P2, 12
Committed as Rev. 184332.
Tobias
Index: gfortran.texi
===
--- gfortran.texi (revision 184330)
+++ gfortran.texi (working copy)
@@ -795,7 +795,7 @@ override type-bound procedures or to have deferred
@item Polymorphic entities
On 02/17/2012 12:15 PM, Tobias Burnus wrote:
Committed as Rev. 184332.
I just saw that one line didn't only have a typo but also a grammar bug.
Fix that one now as attached. (Rev. 184333.)
Tobias
Index: gfortran.texi
===
---
On Mon, Feb 13, 2012 at 23:04, Steven Bosscher stevenb@gmail.com wrote:
On Mon, Feb 13, 2012 at 7:20 PM, Janne Blomqvist
blomqvist.ja...@gmail.com wrote:
Hi,
the attached patch changes the low-level libgfortran IO dispatching
mechanism to use shared vtables for each stream type, instead
Hi,
this patch fixes old regression about ms-bitfields structure layout.
This patch was
verified by VLC community and libAV people (as to see in bug-report).
The result of
the structure-layout was also compared to VC's 32-bit and 64-bit
result by VLC people.
ChangeLog
2012-02-17 Kai Tietz
On 17/02/12 05:49 , Doug Kwan wrote:
2012-02-17 Doug Kwandougk...@google.com
* contrib/testsuite-management/validate_failures.py
(def GetMakefileValue): Check for cross compilers.
OK. Thanks.
Diego.
On 16/02/12 20:00 , Doug Kwan (關振德) wrote:
Hi Diego Ollie,
Could you guys take a look?
-Doug
On Mon, Feb 13, 2012 at 6:41 PM, Doug Kwandougk...@google.com wrote:
Hi,
This patch adds support for powerpc*-grtev2-linux-gnu. The changes
include:
1. Relocating the dynamic linker using a
I missed one issue because it was in the bug summary while the bug
description (also in the same line) mentioned a different bug. The
attached patch now fixes also the bug of the summary.
(Rev. 184334.)
Note to self: Read bug descriptions more carefully and also read the
summary line.
Jonathan Wakely jwakely@gmail.com writes:
For the HTML docs, just check in without that bit in the ChangeLog.
Regenerating the HTML changes almost every page because links between
sections get randomly-generated anchor names, so I'll just do an
update for doc/html/* once your change is
Hi!
The following testcase is miscompiled (by CCP, or, if -fno-tree-ccp,
by VRP), because the (X C1) | C2 folding changes the C1 constant
into an integer constant with most significant bit set, but not
sign-extended into the double_int. Fixed by using double_int_to_tree
which extends it
Rainer Orth r...@cebitec.uni-bielefeld.de writes:
Richard Guenther richard.guent...@gmail.com writes:
I'm not sure about the varasm.c change - it's definitely not a no-op
(callback will be not set, and the flags will be different). Certainly
As I've demonstrated in my response to H.J., the
An operation declared in the private part of a generic package is overriding
in an instane only if it was overriding in the generic. If an instantiation
fails because a non-abstract type has a remaining abstract inherited operation
it may be because of a failure to override. This patch produces a
Committed the following to backends.html:
-m : The port *does* use define_constants
+b : Port does not use * ... notation for output template code
+g : Port does not define TARGET_ASM_FUNCTION_(PRO|EPI)LOGUE
Johann
Index: backends.html
On Tue, Feb 14, 2012 at 11:12:31AM -0800, Jason Merrill wrote:
On 02/10/2012 06:25 AM, Jakub Jelinek wrote:
PR middle-end/51929
* cgraphunit.c (verify_edge_corresponds_to_fndecl): If node is
a same_body_alias, also test whether e-callee isn't a former
or current clone of the
On 14/02/12 18:00, Andrew Stubbs wrote:
On Tue 14 Feb 2012 17:35:36 GMT, Richard Earnshaw wrote:
Bernds checked in a patch last year (or maybe even before that) to make
the selection of flags clobbered insns run very late (certainly after
condexec had run), so I'm not sure why you're not seeing
On 16/02/12 15:33, Andrew Stubbs wrote:
OK for 4.8?
I forgot to address Ramana's comment about optimize_size.
This update fixes that and leaves everything else untouched.
OK?
Andrew
2012-02-17 Andrew Stubbs a...@codesourcery.com
gcc/
* config/arm/arm-protos.h
On 02/16/2012 03:32 PM, Richard Sandiford wrote:
David Daneydavid.da...@cavium.com writes:
On 02/16/2012 02:12 PM, Richard Henderson wrote:
[...]
Thanks for the patch.
index 1c19f8b..59d4560 100644
--- a/gcc/config/mips/mips.h
+++ b/gcc/config/mips/mips.h
@@ -2920,3 +2920,15 @@ extern
On 02/17/12 09:31, David Daney wrote:
Just off the top of my head (with out actually looking at the code). Is
there anything that could be done with the register zero save slot (if it
even exists)?
No. We're talking about saving information in the exception structure between
unwindings.
Spill failure PR52148 occurs for movmem insn that allocates 2 of AVR's 3
pointer registers. Register allocator is at it's limits and the patch tries to
cure the situation by replacing
(match_operand:HI 0 register_operand x)
with explicit
(reg:HI REG_X)
and similar for Z Register classes x and
On 02/14/2012 11:58 PM, Eric Botcazou wrote:
I see several new fails on s390x with that patch (r184174):
People should realize that -fgnu-tm doesn't really work if libitm hasn't been
ported to the architecture, for example s390/s390x. My understanding is that
Hans-Peter's patch should
On Tue, 14 Feb 2012, Walter Lee wrote:
This patch adds information on the Tile-Gx/TILEPro ports to wwwdocs.
Ok to commit? (assuming I have commit rights which I have not tried.)
Yes (and yes). :-)
If you'd like to mention your name in the news item, like we have
in other cases, that would be
On 2/17/2012 1:30 PM, Gerald Pfeifer wrote:
On Tue, 14 Feb 2012, Walter Lee wrote:
This patch adds information on the Tile-Gx/TILEPro ports to wwwdocs.
Ok to commit? (assuming I have commit rights which I have not tried.)
Yes (and yes). :-)
If you'd like to mention your name in the news
In Go it is possible to have an initialization loop in a constant
initializer, if the initializer is a function pointer. This patch adds
that check to the gccgo frontend. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r e19e56d17d8c
In Go an interface method always takes a receiver parameter. The gccgo
frontend was not correctly representing that in the backend type
information generated for an interface value. This didn't make any
difference for code generation, but it would mess up debug info and
perhaps other things as
Adds support for disabling -Werror with --disable-werror configure flag.
tested x86/linux --enable-maintainer-mode
tested x86/linux --enable-maintainer-mode --disable-werror
-benjamin2012-02-17 Benjamin Kosnik b...@redhat.com
PR libstdc++/47058
* acinclude.m4
Hi!
How do we move this issue forward?
On Mon, 23 Jan 2012 15:46:34 +0100, Bernd Schmidt ber...@codesourcery.com
wrote:
On 11/29/2011 05:35 PM, Mitchell, Mark wrote:
So, I still think this patch is the best way to go forward, and it
does
fix incorrect code generation. Would appreciate an
Noticed that the config for the new atomic builtins was not correct for
targets that did not do link tests. (ie x86-x-cris-elf). Fixed as
follows.
tested x86/linux
tested x86/linux x cris-elf
-benjamin2012-02-17 Benjamin Kosnik b...@redhat.com
PR libstdc++/51798 continued.
*
Am 13.02.2012 21:39, schrieb Thomas Koenig:
Ping?
Hello world,
the attached patch xfails the offencing test case on architectures
which do not allow unaligned access for vecorization. OK for trunk?
Any other architectures which should be XFAILed?
Regression-tested on
The Go compiler needs to see all input files at once, rather than be
invoked separately on each one. The gcc driver will only do that if the
static combine_inputs variable is set. And combine_inputs is only set
if the -o option is used (or LTO-specific -fwpa option is used). The
gccgo driver
The Go frontend was accidentally lowering binary expressions with
mismatched types. It would reject two integer constant expressions with
mismatched types, but would then go on to try to see if the expressions
had floating point constant values. Since all integer constants can be
viewed as
I got a request to add function parameter names to the Go export
information, so that tools that do context-sensitive help, such as some
IDEs, have more information to present to the user. This patch
implements that request. This is an incompatible change to the export
format--all existing
Should I submit toe trunk also, now that it is closed?
On Fri, Feb 17, 2012 at 4:52 AM, Diego Novillo dnovi...@google.com wrote:
On 17/02/12 05:49 , Doug Kwan wrote:
2012-02-17 Doug Kwandougk...@google.com
* contrib/testsuite-management/validate_failures.py
(def
This patch to the Go frontend changes it to list all imported packages
in the export information. This is useful for tools which want to trace
package dependencies. This is another incompatible change to the export
information--might as well do them together. Bootstrapped and ran Go
testsuite
On Fri, Feb 17, 2012 at 18:17, Doug Kwan (關振德) dougk...@google.com wrote:
Should I submit toe trunk also, now that it is closed?
Yes, that's fine. contrib/ is not release-critical.
Diego.
On Fri, Feb 17, 2012 at 12:44 PM, Janne Blomqvist
blomqvist.ja...@gmail.com wrote:
On Mon, Feb 13, 2012 at 23:04, Steven Bosscher stevenb@gmail.com wrote:
On Mon, Feb 13, 2012 at 7:20 PM, Janne Blomqvist
blomqvist.ja...@gmail.com wrote:
Hi,
the attached patch changes the low-level
In an oversight, the gccgo frontend rejected valid Go code
(_) = 1
This patch fixes the problem. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r bcfd45b0bd55 go/parse.cc
--- a/go/parse.cc Fri Feb 17 15:19:05 2012 -0800
+++ b/go/parse.cc
Interface embedding just means copying the interface methods, so there
is nothing wrong with embedding the empty interface more than once.
This patch fixes the gccgo frontend to permit that. Bootstrapped and
ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r
I've checked in this patch to consistently hyphenate big-endian and
little-endian when used as adjectives. I observe that Jonathan Swift
also hyphenated Big-endians when used as a noun in Gulliver's Travels,
but I did not see any uses of either term as a noun in invoke.texi, so
thankfully I
Hi,
This patch backports the fix from trunk:
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46770
for google/gcc-4_6_2-mobile branch. This is needed to support C++
global constructors/destructiors on Android/x86. OK for
google/gcc-4_6_2-mobile branch?
Thanks.
H.J.
---
2011-08-20 H.J. Lu
This patch to libgo adds the mount flags, the GNU/Linux specific
fallocate function, and the GNU/Linux version of the statfs function.
Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committed to mainline.
Ian
diff -r 2b6e531380f6 libgo/configure.ac
--- a/libgo/configure.ac Fri
On 2/17/2012 8:00 PM, Sandra Loosemore wrote:
I've checked in this patch to consistently hyphenate big-endian and
little-endian when used as adjectives. I observe that Jonathan Swift
also hyphenated Big-endians when used as a noun in Gulliver's Travels,
but I did not see any uses of either term
I've checked in this patch to address several nits mentioned on the GCC
coding conventions web page
http://gcc.gnu.org/codingconventions.html
Specific tweaks made in this patch include:
* American spelling
* back end/front end
* bit-field
* enumerated type
* epilogue/prologue
*
OK. Thanks for porting the patch.
I will commit the patch into google/gcc-4_6_2-mobile for you.
I would also like to commit it into google/gcc-4_6 branch if all tests
pass. This patch is almost the same as Google Ref 47894.
Thanks,
Jing
On Fri, Feb 17, 2012 at 5:20 PM, H.J. Lu
47 matches
Mail list logo