This patch to the Go frontend gives a better error message for a missing
comma in a composite literal. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline and 4.7 branch.
Ian
diff -r f47f5449a663 go/parse.cc
--- a/go/parse.cc Mon Sep 24 15:20:45 2012 -0700
+++ b
On Wed, Sep 26, 2012 at 1:52 PM, Dominique Dhumieres wrote:
>
> It also breaks the building of the Ada tools:
>
> ...
> ../../xgcc -B../../ -I- -I../rts -I. -I/opt/gcc/work/gcc/ada -DIN_GCC -g
> -O2 -W -Wall -o ../../gnatmake b_gnatm.o a-except.o ali.o ali-util.o
> aspects.o s-casuti.o alloc.
Hi,
tested x86_64-linux, committed.
Thanks,
Paolo.
/
2012-09-26 Paolo Carlini
* include/ext/random: Add include checks for c++11 and .
Index: include/ext/random
===
--- include/ext/random (revision
adds in exports for more vtable bits, as per bugzilla
-benjamin
tested x86/linux
tested x86/linux --enable-symvers=gnu-versioned-namespace012-09-26 Benjamin Kosnik
PR libstdc++/54314
* config/abi/pre/gnu.ver: Add vtable exports.
* config/abi/pre/gnu-versioned-namespace.
Hi,
tested x86_64-linux, committed to mainline.
Paolo.
///
2012-09-26 Akira Takahashi
PR libstdc++/53515
* include/bits/stl_iterator_base_funcs.h (__advance(_InputIterator&,
_Distance, input_iterator_tag)): Add _GLIBCXX_DEBUG_ASSERT(__n >= 0).
Ind
Test gcc.target/arm/div64-unwinding.c is known to fail for GNU/Linux
targets, as described in PR54732. This patch adds an XFAIL.
Tested on arm-none-eabi and arm-none-linux-gnueabi, checked in on trunk.
Janis
2012-09-26 Janis Johnson
* gcc.target/arm/div64-unwinding.c: XFAIL for GNU/
Test gcc.target/arm/mmx-2.c tests ARM iWMMXt functionality but fails to
compile if the compiler under test does not support that by default.
This patch adds -mcpu=iwmmxt. Now instead of failing to compile due to
unrecognized builtins, it fails with the ICE described in PR54338.
Tested on arm-none
Test gcc.target/arm/combine-movs.c uses the wrong effective-target
keyword on dg-final scans; arm_thumb2_ok says that the target will
support THUMB2 if -mthumb is specified, and arm_thumb2 says that
with the current options it is supported.
Tested on arm-none-eabi with multiple flags, checked in o
Test gcc.target/arm/pr42879.c fails for big-endian multilibs. Julian
Brown suggested this fix.
Tested on arm-none-eabi with several multilibs, checked in on trunk.
Janis
2012-09-26 Janis Johnson
* gcc.target/arm/pr42879.c: Handle big-endian.
Index: gcc.target/arm/pr42879.c
=
Test gcc.target/arm/unsigned-extend-1.c specifies option -march=armv6,
which causes compile failures when that option conflicts with other
test flags, such as for multilibs. It can also be overridden by other
test flags. The test is scanning that instruction "uxtb" is not
issued. Leaving off the
On Sep 26, 2012, at 1:28 PM, Steve Ellcey wrote:
> While doing some testing and trying to run multiple versions of the GCC
> testsuite (big-endian, little-endian, etc) I looked at the example at
> http://gcc.gnu.org/install/test.html and thought I could run multiple
> versions by using multiple --
On 09/26/2012 01:58 AM, Chung-Lin Tang wrote:
+/* { dg-do compile } */
+/* { dg-options "-mthumb -O1 -march=armv5te -fno-omit-frame-pointer
-fno-forward-propagate" } */
+/* { dg-require-effective-target arm_thumb1_ok } */
This test will fail to compile for test flags that conflict with
the -mar
ARCompact is one of the architectures that have zero-overhead loops that
are initiated with an instruction at the loop top. There is a way to
set up loops before jumping into their middle, by poking values into
control registers, but that method is more costly and thus only pays off
with a higher
On Wed, Sep 26, 2012 at 12:46 PM, Dehao Chen wrote:
> This patch fixes some small nits.
>
> Bootstrapped and passed gcc regression tests.
>
> Is it ok for trunk?
>
> Thanks,
> Dehao
>
> gcc/ChangeLog:
>
> 2012-09-26 Dehao Chen
>
> * tree-ssa-live.c (clear_unused_block_pointer): Fix two small ni
Hi HJ,
The last-year patch is currently almost useless, as efforts needed for
its rebase seem to be almost the same as efforts needed for writing it
from scratch. I hoped to make a patch covering at least subset of
cases, but unfortunately haven't had time even for it yet.
What time do we have for
http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01975.html
Thanks,
Dehao
> I agree (subreg:M (op:N A C) 0) to (op:M (subreg:N (A 0)) C) is
> a good transformation, but why do we need to handle as special
> the case where the subreg is itself the operand of a plus or minus?
> I think it should happen regardless of where the subreg occurs.
Don't we need to restrict this
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'gcc' has been submitted
by the Swedish team of translators. The file is available at:
http://translationproject.org/latest/gcc/sv.po
(This file, 'gcc-4.7.2.sv.po', has just
cpplib-4.7.2.sv.po.gz
Description: Binary data
The Translation Project robot, in the
name of your translation coordinator.
Hello, gentle maintainer.
This is a message from the Translation Project robot.
A revised PO file for textual domain 'cpplib' has been submitted
by the Swedish team of translators. The file is available at:
http://translationproject.org/latest/cpplib/sv.po
(This file, 'cpplib-4.7.2.sv.po',
On 9/26/2012 4:19 PM, Tom Tromey wrote:
"Florian" == Florian Weimer writes:
Florian> This patch adds support for #pragma GCC warning and #pragma GCC
Florian> error. These pragmas can be used from preprocessor macros,
Florian> unlike the existing #warning and #error directives. Library
Florian
On Mon, 2012-09-24 at 09:47 +0200, Oleg Endo wrote:
> On Mon, 2012-09-24 at 09:15 +0200, Eric Botcazou wrote:
> > > A while ago I've tried doing that for SH (implementing bswap16 with RTL
> > > combine). It was like an explosion of patterns, because combine would
> > > try out a lot of things depe
On Thu, Sep 20, 2012 at 10:33 PM, Ian Lance Taylor wrote:
> Ping.
>
> The patch is the same, so I didn't attach it again. It may be found
> at http://gcc.gnu.org/ml/gcc-patches/2012-09/msg01174.html . I think
> I've replied to all comments.
>
It also breaks the building of the Ada tools:
...
..
On Wed, Sep 26, 2012 at 10:28:14PM +0200, Alexandre Lissy wrote:
> Hello,
>
> As I reported on the gcc-m...@googlegroups.com list, I experienced bad
> behavior of the GtkMM probe for MELT. I identified the issue as linked
> to the amount of inline buttons added in the source code, but was not
> ab
While doing some testing and trying to run multiple versions of the GCC
testsuite (big-endian, little-endian, etc) I looked at the example at
http://gcc.gnu.org/install/test.html and thought I could run multiple
versions by using multiple --target_board flags in RUNTESTFLAGS. It turns
out this doe
Hello,
As I reported on the gcc-m...@googlegroups.com list, I experienced bad
behavior of the GtkMM probe for MELT. I identified the issue as linked
to the amount of inline buttons added in the source code, but was not
able to fix or identify the root of the issue. Since I'm not very
skilled with
> "Florian" == Florian Weimer writes:
Florian> This patch adds support for #pragma GCC warning and #pragma GCC
Florian> error. These pragmas can be used from preprocessor macros,
Florian> unlike the existing #warning and #error directives. Library
Florian> authors can use these pragmas to ad
This patch fixes some small nits.
Bootstrapped and passed gcc regression tests.
Is it ok for trunk?
Thanks,
Dehao
gcc/ChangeLog:
2012-09-26 Dehao Chen
* tree-ssa-live.c (clear_unused_block_pointer): Fix two small nits.
Index: gcc/tree-ssa-live.c
===
On Wed, Sep 26, 2012 at 1:09 PM, Lawrence Crowl wrote:
>
> The problem is that while T is customary as a template parameter, I
> have never seen it used as a typedef name. And that's the situation
> that we are in now.
this should be a no-brainer: T should be reserved for the name of the
templat
On 9/26/12, Michael Matz wrote:
> On Tue, 25 Sep 2012, Lawrence Crowl wrote:
> > On 8/15/12, Richard Henderson wrote:
> > > On 2012-08-15 07:29, Richard Guenther wrote:
> > > > typedef typename Element::Element_t Element_t;
> > >
> > > Can we use something less ugly than Element_t?
> > > Such as
"Steve Ellcey " writes:
> 2012-09-26 Steve Ellcey
>
> PR c/37303
> * gcc.dg/pr37303.c: Check for rdata or rodata.
OK, thanks.
Richard
Uros Bizjak writes:
> The solution is, to transform operands of PLUS and MINUS RTX in the
> form of (subreg:M (op:N A C) 0) to (op:M (subreg:N (A 0)) C), as is
> done for standalone subreg operation in 32bit case.
I agree (subreg:M (op:N A C) 0) to (op:M (subreg:N (A 0)) C) is
a good transformati
On Wed, Sep 26, 2012 at 8:32 AM, H.J. Lu wrote:
> On Thu, Sep 20, 2012 at 10:33 PM, Ian Lance Taylor wrote:
>> Ping.
>>
>> The patch is the same, so I didn't attach it again. It may be found
>> at http://gcc.gnu.org/ml/gcc-patches/2012-09/msg01174.html . I think
>> I've replied to all comments.
"Ulrich Weigand" writes:
> ChangeLog:
>
> * lower-subreg.c (enum classify_move_insn): Rename
> SIMPLE_PSEUDO_REG_MOVE to DECOMPOSABLE_SIMPLE_MOVE.
> (find_decomposable_subregs): Update.
> (decompose_multiword_subregs): Add DECOMPOSE_COPIES parameter.
> Only mark pseud
On Wed, Sep 26, 2012 at 12:14 PM, Marc Glisse wrote:
>> Currently the random number engine interface is
>> inefficient since it returns a single number. What we need is an
>> additional interface to return vectors.
>
>
> Isn't the __generate interface good enough?
__generate is for the distribut
"H.J. Lu" writes:
> On Thu, Sep 20, 2012 at 10:33 PM, Ian Lance Taylor wrote:
>> Ping.
>>
>> The patch is the same, so I didn't attach it again. It may be found
>> at http://gcc.gnu.org/ml/gcc-patches/2012-09/msg01174.html . I think
>> I've replied to all comments.
>>
>
> It breaks Linux/i686 b
On Wed, 26 Sep 2012, Ulrich Drepper wrote:
On Wed, Sep 26, 2012 at 7:32 AM, Jakub Jelinek wrote:
Have you considered also an __AVX__ version handling 4 elements at a time?
Without __AVX2__ one would need to cast __m256i to __m256d for and/or, as
AVX1 doesn't have _mm256_and_si256 or _mm256_or_
The test gcc.dg/pr37303.c fails on mips because mips uses rdata instead
of rodata and the test scans for rodata in the assembly file. Rather
then make an special check on mips I modified the test to scan for rodata
or rdata on any plaforms.
Tested on MIPS, OK to checkin?
Steve Ellcey
sell...@mip
On Tue, Sep 25, 2012 at 04:25:08PM +0200, Florian Weimer wrote:
> This patch adds support for #pragma GCC warning and #pragma GCC
> error. These pragmas can be used from preprocessor macros, unlike
> the existing #warning and #error directives. Library authors can
> use these pragmas to add deprec
On Thu, Sep 20, 2012 at 10:33 PM, Ian Lance Taylor wrote:
> Ping.
>
> The patch is the same, so I didn't attach it again. It may be found
> at http://gcc.gnu.org/ml/gcc-patches/2012-09/msg01174.html . I think
> I've replied to all comments.
>
It breaks Linux/i686 build:
../../src-trunk/gcc/diag
The following patch fixes a mis-fired error on extern alias
(same-body). The assembler hash related code needs rework to avoid
patches like this. (The bug was found during a large grouping stress
test).
David
extern_alias.p
Description: Binary data
Adding an x86 maintainer in Cc:
On Tue, 11 Sep 2012, Marc Glisse wrote:
Hello,
any advice?
http://gcc.gnu.org/ml/gcc-patches/2012-09/msg00044.html
On Sun, 2 Sep 2012, Marc Glisse wrote:
Hello,
this patch passes bootstrap+testsuite. It is probably wrong in many ways,
but I don't know enou
On Fri, Aug 31, 2012 at 1:54 AM, Jan Hubicka wrote:
>> On Mon, Dec 12, 2011 at 6:02 AM, Jan Hubicka wrote:
>> >> Any update?
>> >
>> > I will look into it today, but anyway I think it is stage1 material, so we
>> > have some time to progress on it.
>> >
>> > Honza
>>
>> Hi Honza,
>>
>> The old p
On Wed, 26 Sep 2012, Richard Guenther wrote:
> On Wed, 26 Sep 2012, Richard Guenther wrote:
>
> >
> > This fixes PR54709, the location change broke DECL_IS_BUILTIN which
> > is bogously used by streamer_handle_as_builtin_p. I am reverting
> > the previous lto-symtab.c change as it is not necess
Hello,
RTL optimizers looks like a promising category, and Eric like a nice
maintainer for it, adding in Cc: ;-)
(I bothered Richard Sandiford last time)
On Tue, 11 Sep 2012, Marc Glisse wrote:
On Sun, 9 Sep 2012, Marc Glisse wrote:
Hello,
this patch lets the compiler try to rewrite:
(v
Sorry, I'll change that with my next patch.
Dehao
On Wed, Sep 26, 2012 at 12:20 AM, Eric Botcazou wrote:
>> gcc/ChangeLog:
>> 2012-09-21 Dehao Chen
>>
>> PR middle-end/54645
>> * c-family/c-pch.c (c_common_read_pch): Rebuild the location_adhoc_data
>> map when read in the pc
On Wed, Sep 26, 2012 at 4:39 PM, Marc Glisse wrote:
> http://gcc.gnu.org/ml/gcc-patches/2012-09/msg01041.html
>
> Hmm, maybe I shouldn't put "trivial" in a post, that makes it sound like I
> am not waiting for comments.
>
> Comments?
Ok.
Thanks,
Richard.
>
> On Fri, 14 Sep 2012, Marc Glisse wro
http://gcc.gnu.org/ml/gcc-patches/2012-09/msg01041.html
Hmm, maybe I shouldn't put "trivial" in a post, that makes it sound like I
am not waiting for comments.
Comments?
On Fri, 14 Sep 2012, Marc Glisse wrote:
A fairly trivial follow-up to the patch with the code. I added a line for PR
5302
On Tue, Sep 25, 2012 at 4:30 PM, Lawrence Crowl wrote:
> On 8/15/12, Richard Henderson wrote:
>> On 2012-08-15 07:29, Richard Guenther wrote:
>> > + typedef typename Element::Element_t Element_t;
>>
>> Can we use something less ugly than Element_t?
>> Such as
>>
>> typedef typename Element::T
Ping.
2012/9/18 Yuri Rumyantsev :
> Hi All,
>
> Forgot to attch the patch.
>
> 2012/9/18 Yuri Rumyantsev :
>> Hi All,
>>
>> This patch aims to fix all stability issues related to using the first
>> scheduler in gcc
>> for x86 target (there several reported issues related to this problem).
>>
>> Ma
>-Original Message-
>From: Richard Guenther [mailto:richard.guent...@gmail.com]
>Sent: Wednesday, September 26, 2012 8:16 AM
>To: Iyer, Balaji V
>Cc: gcc-patches@gcc.gnu.org; al...@redhat.com; r...@redhat.com;
>l...@redhat.com
>Subject: Re: [PATCH] Cilk Plus merging to trunk (2 of n)
>
>O
On Wed, 26 Sep 2012, Richard Guenther wrote:
>
> This fixes PR54709, the location change broke DECL_IS_BUILTIN which
> is bogously used by streamer_handle_as_builtin_p. I am reverting
> the previous lto-symtab.c change as it is not necessary for fixing
> the testcases. LTO bootstrap fails with
On Mon, Sep 24, 2012 at 10:10 PM, Sharad Singhai wrote:
> Ping.
Some minor issues:
* c/c-decl.c (c_write_global_declarations): Use different method to
determine if the dump has ben initialized.
* cp/decl2.c (cp_write_global_declarations): Ditto.
* testsuite/gcc.ta
Hi,
On Tue, 25 Sep 2012, Lawrence Crowl wrote:
> On 8/15/12, Richard Henderson wrote:
> > On 2012-08-15 07:29, Richard Guenther wrote:
> > > + typedef typename Element::Element_t Element_t;
> >
> > Can we use something less ugly than Element_t?
> > Such as
> >
> > typedef typename Element::T
On Mon, Sep 24, 2012 at 5:05 PM, Iyer, Balaji V wrote:
> Hi Richard,
> Please see my comments embedded below.
>
>>-Original Message-
>>From: Richard Guenther [mailto:richard.guent...@gmail.com]
>>Sent: Monday, September 24, 2012 5:53 AM
>>To: Iyer, Balaji V
>>Cc: gcc-patches@gcc.gn
Hello,
a while ago Andrew Stubbs noticed that the lower-subreg pass seems a little
too aggressive in splitting up DImode moves into SImode moves. See the
discussion starting at:
http://gcc.gnu.org/ml/gcc/2012-04/msg00676.html
The conclusion of this thread was that the mere presence of a DImode m
On Wed, Sep 26, 2012 at 7:32 AM, Jakub Jelinek wrote:
> Have you considered also an __AVX__ version handling 4 elements at a time?
> Without __AVX2__ one would need to cast __m256i to __m256d for and/or, as
> AVX1 doesn't have _mm256_and_si256 or _mm256_or_si256, but _mm256_and_pd
> or _mm256_or_p
This fixes PR54709, the location change broke DECL_IS_BUILTIN which
is bogously used by streamer_handle_as_builtin_p. I am reverting
the previous lto-symtab.c change as it is not necessary for fixing
the testcases. LTO bootstrap fails with this again with
/tmp/cca0yEEE.ltrans17.ltrans.o: In fun
On Wed, Sep 26, 2012 at 07:16:09AM -0400, Ulrich Drepper wrote:
> Here is a patch to accelerate the __generate function for the
> normal_distribution class. The speed-up is quite significant,
> the amount depending on which random number engine is used.
>
> mt19937+20%
>
> mt19937_64
Hi,
On 09/26/2012 01:16 PM, Ulrich Drepper wrote:
Here is a patch to accelerate the __generate function for the
normal_distribution class. The speed-up is quite significant,
the amount depending on which random number engine is used.
mt19937+20%
mt19937_64 +30%
sfmt19937 +30
Here is a patch to accelerate the __generate function for the
normal_distribution class. The speed-up is quite significant,
the amount depending on which random number engine is used.
mt19937+20%
mt19937_64 +30%
sfmt19937 +30%
sfmt19937_64 +30%
This patch introduces a head
On Wed, Sep 26, 2012 at 11:29 AM, Jakub Jelinek wrote:
> Hi!
>
> At least i?86 and rs6000 backends use V*[SD]F AND/IOR/XOR/NOT
> operations, seems most of the simplify-rtx.c code is properly guarded for
> that with INTEGRAL_MODE_P (mode), but the
> (x - (x & y)) -> (x & ~y)
> optimization was not,
Hi!
At least i?86 and rs6000 backends use V*[SD]F AND/IOR/XOR/NOT
operations, seems most of the simplify-rtx.c code is properly guarded for
that with INTEGRAL_MODE_P (mode), but the
(x - (x & y)) -> (x & ~y)
optimization was not, eventhough it is only valid for integral modes.
Fixed thusly, bootst
On Wed, Sep 26, 2012 at 10:37 AM, Georg-Johann Lay wrote:
> http://gcc.gnu.org/ml/gcc-patches/2012-09/msg01448.html
>
> Adds an addr_space_t argument to targetm.mode_dependent_address_p.
>
> Up to now the change is a no-op, but the avr backend needs this.
Ok.
Thanks,
Richard.
> Johann
On Wed, Sep 26, 2012 at 6:29 AM, Dehao Chen wrote:
> IS_UNKNOWN_LOCATION is very misleading. This patch removes this macro
> from input.h. For sites when checking LOCUS is intended, we explicit
> use LOCATION_LOCUS and compare it to UNKNOWN_LOCATION.
>
> Bootstrapped and passed all gcc regression
On Tue, Sep 25, 2012 at 7:10 PM, Joern Rennecke
wrote:
> A few arm targets were removed from config.gcc, and picochip is only
> available with --enable-obsolete. This patch brings config-list.mk
> up-to-date in these respects.
>
> 2012-09-24 Joern Rennecke
>
> * contrib-list.mk (LIST):
On Fri, Sep 21, 2012 at 10:04 AM, Christophe Lyon
wrote:
> On 20 September 2012 09:55, Christophe Lyon
> wrote:
>> On 20 September 2012 09:12, Eric Botcazou wrote:
The attached patch catches C constructs:
(A << 8) | (A >> 8)
where A is unsigned 16 bits
and maps them to built
On 2012/9/16 05:15 PM, Richard Sandiford wrote:
> Second ping for the ARM part of Chung-Lin's __builtin_thread_pointer patch:
>
> http://gcc.gnu.org/ml/gcc-patches/2012-08/msg01914.html
>
> I think this is the only part that hasn't been approved.
>
> Thanks,
> Richard
>
Ping again.
(and t
There was a patch of mine here from over an year ago:
http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01846.html
That old patch contained a constraint modification that was a point of
discussion at the time, but upon re-testing I found it was unneeded for
fixing the PR (don't remember why I included
On 10/09/12 16:11, Tejas Belagod wrote:
Hi,
This patch tightens the predicate for the CMP pattern. It makes it restrictive
to accept reg or zero as prescribed by the architecture.
Regression-tested on aarch64-none-elf. OK for aarch64-branch?
Thanks,
Tejas Belagod
ARM.
PS: This patch applies o
http://gcc.gnu.org/ml/gcc-patches/2012-09/msg01448.html
Adds an addr_space_t argument to targetm.mode_dependent_address_p.
Up to now the change is a no-op, but the avr backend needs this.
Johann
> gcc/ChangeLog:
> 2012-09-21 Dehao Chen
>
> PR middle-end/54645
> * c-family/c-pch.c (c_common_read_pch): Rebuild the location_adhoc_data
> map when read in the pch.
Wrong ChangeLog file, you want gcc/c-family/ChangeLog (and remove c-family/).
--
Eric Botcazou
72 matches
Mail list logo
