> Hello,
>
> There is code in stmt.c since the initial checkin, that tries to
> balance a switch tree according to some ascii heuristics. I see a
> couple of problems with this code:
>
> 1. It doesn't seem to help much. With the attached patch to remove the
> code, I see no compile time changes t
2012/4/16 Georg-Johann Lay :
> This patch adds a new file ./gcc/doc/avr-mmcu.texi that lists all valid
> -mmcu= settings and replaces the respective text in invoke.texi by
> @include avr-mmcu.texi
>
> Up to now, there is no complete list of -mmcu= options, and a list is
> hard to maintain by hand b
Hi,
When dealing with Multilib, I find it isn't so easy to come up with a
complete MULTILIB_EXCEPTIONS rules to cover all unnecessary cases,
especially on ARM where the majority of combinations need to be filtered
out. This patch is proposing to consider this thing from another angle:
specifying t
When substituting into a template template parameter pack pattern with a
set of template arguments, we want to use tsubst_expr so that we get the
substituted result; tsubst tries and fails to build up a new template
template parameter.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit d04f
Here, we were getting confused and thinking that an argument pack of
template template arguments wasn't suitable for a template template
parameter pack. But it is.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 6143aa83bb48ef7c9d2bbcb0dff13a964036b3d1
Author: Jason Merrill
Date: Mon
In this testcase, the partial specialization has fewer template
arguments than the primary template has parameters, which was confusing
GCC. Let's give an error for this case because the partial
specialization isn't more specialized than the primary template.
Tested x86_64-pc-linux-gnu, apply
Hi,
I had a look to this Segmentation fault in cp_parser_member_declaration
and what happens is that initializer_token_start is still null (as
initialized) when we get to:
if (initializer)
error_at (initializer_token_start->location,
"pure-specifier on
This patch from Rémy Oudompheng avoids an infinite recursion in the Go
frontend when evaluating a string constant. It avoids a compiler crash
on invalid code like
const f, g = g, f
func S() []byte { return []byte(f) }
Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committe
Hello,
There is code in stmt.c since the initial checkin, that tries to
balance a switch tree according to some ascii heuristics. I see a
couple of problems with this code:
1. It doesn't seem to help much. With the attached patch to remove the
code, I see no compile time changes to e.g. compile G
Thomas Koenig wrote:
- [patch, fortran] Trim spaces on list-directed reads
http://gcc.gnu.org/ml/fortran/2012-04/msg00040.html
That one was committed:
http://gcc.gnu.org/ml/gcc-cvs/2012-04/msg00417.html
Jerry indicated that this would also be OK for a backport; I'll
do that within a few day
The old code that compares the number of parameter types doesn't work in
the age of variadic templates. Instead, we should take the deduced
template arguments, substitute them in, and see if the result is what
we're looking for.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 84857b9698
Ping.
On Mon, Apr 9, 2012 at 2:33 PM, Easwaran Raman wrote:
> On Sun, Mar 25, 2012 at 9:40 PM, Easwaran Raman wrote:
>> On Sun, Mar 25, 2012 at 12:22 PM, Jan Hubicka wrote:
This patch propagates execution count of thee case labels of a
switch-case statement after its expansion. Bootst
On Fri, Apr 13, 2012 at 4:20 PM, Andrey Belevantsev wrote:
> On 13.04.2012 14:18, Igor Zamyatin wrote:
>>
>> On Thu, Apr 12, 2012 at 5:01 PM, Andrey Belevantsev
>> wrote:
>>>
>>> On 12.04.2012 16:38, Richard Guenther wrote:
On Thu, Apr 12, 2012 at 2:36 PM, Igor Zamyatin
wrote
On Apr 16, 2012, at 6:43 AM, Richard Guenther wrote:
> Well, the number one advantage of replacing PCH is that it would kill
> the current PCH implementation which is tied to our garbage collector...
If compile speed isn't important to people, there is little reason to keep pch
around. Once that
This patch teaches the arm target about the rev16 and revsh
instructions which can
efficiently execute the recently added __builtin_bswap16() function
(BSWAP:HI in RTL)
These instructions are available in thumb and arm modes on armv6
architecture and up.
Before this patch,
short swaps16(short x)
This patch adds a new file ./gcc/doc/avr-mmcu.texi that lists all valid
-mmcu= settings and replaces the respective text in invoke.texi by
@include avr-mmcu.texi
Up to now, there is no complete list of -mmcu= options, and a list is
hard to maintain by hand because it contains more than 180 devices
On 04/10/2012 09:35 AM, Richard Sandiford wrote:
Hi Vlad,
Back in Decemember, when we were still very much in stage 3, I sent
an RFC about an alternative implementation of -fsched-pressure.
Just wanted to send a reminder now that we're in the proper stage:
http://gcc.gnu.org/ml/gcc-patches/2011
On Apr 16, 2012, at 8:03 AM, Rainer Orth wrote:
> I've long noticed that libstdc++.log (unlike libstdc++.sum) doesn't
> contain log entries for tests run from abi.exp, but hadn't looked
> closer, getting used to check libstdc++.log.sep instead which contained
> everything I expected.
> ok for main
On Tue, Apr 3, 2012 at 7:49 AM, H.J. Lu wrote:
> On Thu, Mar 29, 2012 at 7:34 AM, H.J. Lu wrote:
>> On Sat, Mar 3, 2012 at 9:54 AM, H.J. Lu wrote:
>>> Hi,
>>>
>>> This patch backports x32 support to libtool:
>>>
>>> http://git.savannah.gnu.org/cgit/libtool.git/commit/?id=88992fe6771ec3258bde1b03
2012/4/16 Richard Guenther :
> On Mon, Apr 16, 2012 at 3:58 PM, Kai Tietz wrote:
>> 2012/4/12 Richard Guenther :
>>> On Thu, Apr 5, 2012 at 6:15 PM, Kai Tietz wrote:
Hello,
this patch adds some basic folding capabilities to fold-const for
equal and none-equal comparisons
cc Richard.
thanks,
David
On Mon, Apr 16, 2012 at 3:24 AM, Rainer Orth
wrote:
> As reported in PR testsuite/52948, several plugin testcases were failing
> since the removal of TODO_dump_func:
>
> UNRESOLVED: selfassign.c compilation, -I.
> -I/vol/gcc/src/hg/trunk/local/gcc/testsuite
> -I/vol/
On Mon, Apr 16, 2012 at 3:58 PM, Kai Tietz wrote:
> 2012/4/12 Richard Guenther :
>> On Thu, Apr 5, 2012 at 6:15 PM, Kai Tietz wrote:
>>> Hello,
>>>
>>> this patch adds some basic folding capabilities to fold-const for
>>> equal and none-equal comparisons
>>> on integer typed argument.
>>>
>>> Cha
On Mon, Apr 16, 2012 at 3:38 PM, Diego Novillo wrote:
>> I'm not sure PPH will replace PCH - will it?
>
> In theory, it may. But it's not in the immediate plans.
IMHO, the case for including PPH would be much stronger if replacing
PCH would be included in the immediate to mid-term plans. The cur
Hi Tobias,
- [patch, fortran] Trim spaces on list-directed reads
http://gcc.gnu.org/ml/fortran/2012-04/msg00040.html
That one was committed:
http://gcc.gnu.org/ml/gcc-cvs/2012-04/msg00417.html
Jerry indicated that this would also be OK for a backport; I'll
do that within a few days unless
On 04/15/2012 02:29 AM, Richard Sandiford wrote:
alpha-linux-gnu arm-linux-gnueabi avr-rtems bfin-elf c6x-elf
cr16-elf cris-elf epiphany-elf fr30-elf frv-linux-gnu h8300-elf
ia64-linux-gnu iq2000-elf lm32-elf m32c-elf m32r-elf
m68k-linux-gnu mcore-elf mep-elf micr
On Mon, Apr 16, 2012 at 9:08 AM, Ilya Enkovich wrote:
>> On Mon, Apr 16, 2012 at 9:01 AM, Ilya Enkovich
>> wrote:
Maybe it is better to break it into 2 patches:
1. Introduce config/i386/linux-common.h with any new functionalities
2. Add Android support.
Thanks.
Hi,
this patch moves cgraph/varpool hashes into symbol table hashes, so the
symbol table is actually almost a symbol table ;)
Work done.
Bootstrapped/regtested x86_64-linux. Will commit it after bit of more
testing - the assembler name handling is slipperly wrt aliases.
Honza
* cgraph.c
> On Mon, Apr 16, 2012 at 9:01 AM, Ilya Enkovich wrote:
>>>
>>> Maybe it is better to break it into 2 patches:
>>>
>>> 1. Introduce config/i386/linux-common.h with any new functionalities
>>> 2. Add Android support.
>>>
>>> Thanks.
>>>
>>>
>>> H.J.
>>> ---
>> Hi,
>>
>> The first part will just add
On Mon, Apr 16, 2012 at 9:01 AM, Ilya Enkovich wrote:
>>
>> Maybe it is better to break it into 2 patches:
>>
>> 1. Introduce config/i386/linux-common.h with any new functionalities
>> 2. Add Android support.
>>
>> Thanks.
>>
>>
>> H.J.
>> ---
> Hi,
>
> The first part will just add empty file. Wha
>
> Maybe it is better to break it into 2 patches:
>
> 1. Introduce config/i386/linux-common.h with any new functionalities
> 2. Add Android support.
>
> Thanks.
>
>
> H.J.
> ---
Hi,
The first part will just add empty file. What is the reason to do it?
Testing will not even reveal if I added it in
Ramana Radhakrishnan wrote:
> This looks OK to me. Looking at the ISA documents and the variants of
> the vldn instructions your summary is correct. The alignment specifier
> should not be greater than the memory size being transferred and
> checking this in this form is OK .
Thanks, Ramana! I'v
On Mon, Apr 16, 2012 at 8:33 AM, Ilya Enkovich wrote:
> Hello,
>
> Here is patch for i386 Android support. It is the second part of
> previously splitted patch
> (http://gcc.gnu.org/ml/gcc-patches/2012-04/msg00127.html). The first
> part has been committed
> (http://gcc.gnu.org/ml/gcc-cvs/2012-04/
Hello,
Here is patch for i386 Android support. It is the second part of
previously splitted patch
(http://gcc.gnu.org/ml/gcc-patches/2012-04/msg00127.html). The first
part has been committed
(http://gcc.gnu.org/ml/gcc-cvs/2012-04/msg00378.html). Bootstrapped
and checked on linux-x86_64. OK for tru
I've long noticed that libstdc++.log (unlike libstdc++.sum) doesn't
contain log entries for tests run from abi.exp, but hadn't looked
closer, getting used to check libstdc++.log.sep instead which contained
everything I expected.
This weekend, I've observed the same for the prettyprinters.exp failu
On 04/14/2012 05:43 PM, Tom de Vries wrote:
>> + tree expr = NULL;
>> + append_to_statement_list (*block,&expr);
>> + *block = expr;
>
> Rather than doing this dance here, I think it would be better to enhance
> append_to_statement_list to handle the case of the list argument
Just another place we need to call check_for_bare_parameter_packs.
Tested x86_64-pc-linux-gnu, applying to trunk.
commit 616afbea9bf859633ade63b6536e5b63be5a19e6
Author: Jason Merrill
Date: Mon Apr 16 09:23:41 2012 -0400
PR c++/51148
* friend.c (make_friend_class): Call check_for_ba
On 4/16/12 7:32 AM, Rainer Orth wrote:
Btw., it occured to me that it might be useful to add an option to
locate out-of-tree manifests. I often have several source trees
(unmodified sources, ones with local patches) and would like to share
manifests between them. While this can be achieved wit
On 4/16/12 10:43 AM, Richard Guenther wrote:
Well, the number one advantage of replacing PCH is that it would kill
the current PCH implementation which is tied to our garbage collector...
Sure. That too.
Diego.
2012/4/12 Richard Guenther :
> On Thu, Apr 5, 2012 at 6:15 PM, Kai Tietz wrote:
>> Hello,
>>
>> this patch adds some basic folding capabilities to fold-const for
>> equal and none-equal comparisons
>> on integer typed argument.
>>
>> ChangeLog
>>
>> 2012-04-05 Kai Tietz
>>
>> * fold-cons
Hi Uli,
Apologies for the delayed response.
>
> Shouldn't the check be implemented along the following lines?
>
> if (memsize == 32 && (align % 32) == 0)
> align_bits = 256;
> else if ((memsize == 16 || memsize == 32) && (align % 16) == 0)
> align_bits = 128;
>
On Mon, Apr 16, 2012 at 8:31 AM, Paolo Carlini wrote:
> Hi,
>
>> On Mon, Apr 16, 2012 at 12:42 AM, Marc Glisse
>> wrote:
>>>
>>> On Sun, 15 Apr 2012, Gabriel Dos Reis wrote:
>>>
a hybrid approach; I would suggest something like this: (a) if caret
is in effect, then print
the caret
Hi,
this patch starts with process of commonizing code that should be in symtab.c.
It moves linked lists out of cgraph and varpool nodes into the symbol base
type and compensates all remaining places we walk the lists directly to use
the accessor functions.
Finally bit of benefits and code unifica
On Mon, 16 Apr 2012, Diego Novillo wrote:
> On Mon, Apr 16, 2012 at 10:23, Richard Guenther wrote:
> > On Mon, 16 Apr 2012, Steven Bosscher wrote:
> >
> >> On Mon, Apr 16, 2012 at 3:19 PM, Richard Guenther
> >> wrote:
> >> >
> >> > This fixes PR52977 - for PCH to work with its pointer relocatio
On Mon, Apr 16, 2012 at 10:23, Richard Guenther wrote:
> On Mon, 16 Apr 2012, Steven Bosscher wrote:
>
>> On Mon, Apr 16, 2012 at 3:19 PM, Richard Guenther wrote:
>> >
>> > This fixes PR52977 - for PCH to work with its pointer relocation code
>> > we have to avoid dereferencing pointers to comput
On 15/04/12 10:29, Richard Sandiford wrote:
> As mentioned in the discussion of Mike's >2HWI patch, plus_constant is
> AFAIK the only rtl routine that handles arithmetic on CONST_INT without
> specifying the mode of the CONST_INT. That means it isn't always able
> to generate the right result. E.
Hi,
On Mon, Apr 16, 2012 at 12:42 AM, Marc Glisse wrote:
On Sun, 15 Apr 2012, Gabriel Dos Reis wrote:
a hybrid approach; I would suggest something like this: (a) if caret
is in effect, then print
the caret pointing to the symbol in question; otherwise (b) print the
symbol and the type (as sug
On Mon, 16 Apr 2012, Steven Bosscher wrote:
> On Mon, Apr 16, 2012 at 3:19 PM, Richard Guenther wrote:
> >
> > This fixes PR52977 - for PCH to work with its pointer relocation code
> > we have to avoid dereferencing pointers to compute array lengths
> > in structures. So we have to unfortunately
On Mon, Apr 16, 2012 at 3:19 PM, Richard Guenther wrote:
>
> This fixes PR52977 - for PCH to work with its pointer relocation code
> we have to avoid dereferencing pointers to compute array lengths
> in structures. So we have to unfortunately keep duplicated info about
> VECTOR_CST vector lengths
This fixes PR52977 - for PCH to work with its pointer relocation code
we have to avoid dereferencing pointers to compute array lengths
in structures. So we have to unfortunately keep duplicated info about
VECTOR_CST vector lengths.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to
Ping?
On 29.03.2012 13:48, Christophe Lyon wrote:
Hello,
According to a comment in configure/configure.ac:
# We want to ensure that TARGET libraries (which we know are built with
# gcc) are built with "-O2 -g", so include those options when setting
# CFLAGS_FOR_TARGET and CXXFLAGS_FOR_TARGET.
On 14/04/12 14:11, Andrew Stubbs wrote:
> And now with the patch. :(
>
> On 14/04/12 13:48, Andrew Stubbs wrote:
>> On 12/04/12 16:48, Richard Earnshaw wrote:
>>> If negation in Neon needs a scratch register, it seems to me to be
>>> somewhat odd that we're disparaging the ARM version.
>>>
>>> Als
On Mon, 2012-04-16 at 11:01 +0200, Richard Guenther wrote:
> On Sat, Apr 14, 2012 at 7:05 PM, William J. Schmidt
> wrote:
> > This patch corrects two errors in reassociating expressions with
> > repeated factors. First, undistribution needs to recognize repeated
> > factors. For now, repeated fa
This fixes the missed cleanup opportunities from the if-conversion mess.
We should try to produce consistent predicates - thus not do overly
optimistic simplifications on them.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2012-04-16 Richard Guenther
When I tried contrib/testsuite-management/validate_failures.py for the
first time this weekend, I ran into the following problem: while it
works fine for a i386-pc-solaris2.11 bootstrap, it fails instead for a
amd64-pc-solaris2.11 bootstrap:
error: 11-gcc-64/build is not a valid GCC top leve
As reported in PR testsuite/52948, several plugin testcases were failing
since the removal of TODO_dump_func:
UNRESOLVED: selfassign.c compilation, -I.
-I/vol/gcc/src/hg/trunk/local/gcc/testsuite
-I/vol/gcc/src/hg/trunk/local/gcc/testsuite/../../gcc
-I/var/gcc/regression/trunk/11-gcc/build/gcc/
When looking at the code we produce from if-conversion and vectorization
I noticed several things. For one, we are not handling TARGET_MEM_REF
in operand_equal_p which means that DOM does not clean up loads/stores
of that form (to the extent DOM can do that anyway). Also we never
simplify the co
On Mon, Apr 16, 2012 at 2:10 AM, Gerald Pfeifer wrote:
> On Wed, 6 Jul 2011, Paulo J. Matos wrote:
>> On 06/07/11 16:08, Richard Guenther wrote:
>>> See cgraph.c:clone_function_name, it uses the NO_DOT_IN_LABEL and
>>> NO_DOLLAR_IN_LABEL target macros and ASM_FORMAT_PRIVATE_NAME.
>> Why are NO_DOL
On Sun, Apr 15, 2012 at 5:09 PM, Richard Sandiford
wrote:
> ADDRESS has long had no meaning in .md files. This patch removes
> the associated gen* support.
>
> Borders on the obvious, but just to be sure: bootstrapped & regression
> tested on x86_64-linux-gnu. OK to install?
Ok.
Thanks,
Richar
On Sat, Apr 14, 2012 at 7:05 PM, William J. Schmidt
wrote:
> This patch corrects two errors in reassociating expressions with
> repeated factors. First, undistribution needs to recognize repeated
> factors. For now, repeated factors will be ineligible for this
> optimization. In the future, thi
Hi Tobias,
>> here is a small patch for an OOP problem. For discussion see the PR.
>> Regtested on x86_64-unknown-linux-gnu. Ok for trunk?
>
> OK and thanks for the patch.
thanks for the review. Committed as r186486.
> Note, however, that since Bernhard's patch*,
> using cleanup-modules is usua
Dear all,
first, I would like to ping my patch:
- [Patch, Fortran] PR52864 - fix actual/formal checks
http://gcc.gnu.org/ml/fortran/2012-04/msg00059.html
Other patches with pending review:
- [Patch, Fortran, F03] PR52909: Procedure pointers not private to modules
http://gcc.gnu.org/ml/fortran
On Mon, Apr 16, 2012 at 12:42 AM, Marc Glisse wrote:
> On Sun, 15 Apr 2012, Gabriel Dos Reis wrote:
>
>> a hybrid approach; I would suggest something like this: (a) if caret
>> is in effect, then print
>> the caret pointing to the symbol in question; otherwise (b) print the
>> symbol and the type
62 matches
Mail list logo
