Re: RFA: Add makefile for cross-configuration torture test

2011-04-13 Thread Ralf Wildenhues
Hi Joern, * Joern Rennecke wrote on Wed, Apr 13, 2011 at 11:02:27PM CEST: > 2010-04-13 Joern Rennecke > > * config-list.mk: New file. > --- contrib/config-list.mk(revision 0) > +++ contrib/config-list.mk(revision 0) > +all: $(LIST) > + > +.PHONEY: make-log-dir make-script-dir

[PATCH 7/n, i386]: Merge SSE and AVX patterns using "enable" attribute.

2011-04-13 Thread Uros Bizjak
Hello! Attached patch converts " Miscellaneous" and "SSSE3 instructions" sections of sse.md. 2011-04-12 Uros Bizjak * config/i386/sse.md (*sse2_uavgv16qi3): Merge with *avx_uavgv16qi3. (*sse2_uavgv8hi3): Merge with *avx_uavgv8hi3. (sse2_psadbw): Merge with *avx_psadbw.

Go patch committed: Use backend interface for go and defer statements

2011-04-13 Thread Ian Lance Taylor
This patch to the Go frontend uses the backend interface for go and defer statements. To make that work, I changed the defer stack variable from a tree to an Expression. This in turn required a bit of patching to ensure that the variable is created before it is used. Bootstrapped and ran Go test

Re: ObjC: remove temporary tree list used when calling objc_declare_protocols()

2011-04-13 Thread Mike Stump
On Apr 13, 2011, at 3:57 PM, Nicola Pero wrote: > This patch cleans up objc_declare_protocols() to avoid having to create > temporary tree chains just to perform the function call (in the same way > as my other patch cleaned up objc_declare_class()). > > Ok to commit ? Ok.

Re: ObjC: remove temporary tree list used when calling objc_declare_class()

2011-04-13 Thread Mike Stump
On Apr 13, 2011, at 1:52 PM, Nicola Pero wrote: > This patch removes another case of temporary tree chains from the ObjC > compiler. > Ok to commit ? Ok.

ObjC: remove temporary tree list used when calling objc_declare_protocols()

2011-04-13 Thread Nicola Pero
This patch cleans up objc_declare_protocols() to avoid having to create temporary tree chains just to perform the function call (in the same way as my other patch cleaned up objc_declare_class()). Ok to commit ? Thanks PS: This patch has no measurable effect on compilation speed in my tests beca

Re: FDO usability patch -- cfg and lineno checksum

2011-04-13 Thread Xinliang David Li
This was contributed by: 2011-04-13 Neil Vachharajani On Wed, Apr 13, 2011 at 3:25 PM, Xinliang David Li wrote: > Hi,  in current FDO implementation, the source file version used in > profile-generate needs to strictly match the version used in > profile-use -- simple formating changes will i

FDO usability patch -- cfg and lineno checksum

2011-04-13 Thread Xinliang David Li
Hi, in current FDO implementation, the source file version used in profile-generate needs to strictly match the version used in profile-use -- simple formating changes will invalidate the profile data (use of it will lead to compiler error or ICE). There are two main problems that lead to the weak

More of ipa-inline housekeeping

2011-04-13 Thread Jan Hubicka
Hi, this patch moves inline_summary from field in cgraph_node into its own on side datastructure. This moves it from arcane decision of mine to split all IPA data into global/local datas stored in common datastructure into the scheme we developed for new IPA passes some time ago. The advantage is

Re: RFA: Add makefile for cross-configuration torture test

2011-04-13 Thread Douglas B Rupp
On 4/13/2011 3:10 PM, Joseph S. Myers wrote: The person who volunteered for Interix is Douglas B Rupp. I'm still good for it. --Doug

Re: RFA: Add makefile for cross-configuration torture test

2011-04-13 Thread Joseph S. Myers
On Wed, 13 Apr 2011, Joern Rennecke wrote: > > It's up to the people who volunteered to maintain Interix and SCORE to get > > them into a state that builds cleanly - and you can make sure they are > > CC:ed on the relevant bugs and point out that without fixes the targets > > will be removed later

Re: RFA: Add makefile for cross-configuration torture test

2011-04-13 Thread Joern Rennecke
Quoting "Joseph S. Myers" : I think Interix and SCORE should be included, but probably only one of Solaris 8 and Solaris 9 needs to be included for each of x86 and SPARC since the targets don't appear significantly different - so no need to include Solaris 8 at all if you include Solaris 9 for b

Re: [patch] Split Parse Timevar (issue4378056)

2011-04-13 Thread Lawrence Crowl
On 4/13/11, Richard Guenther wrote: > On Apr 12, 2011 Diego Novillo wrote: > > On Apr 12, 2011 Lawrence Crowl wrote: > > > This patch provides more finer, more precise compile time > > > information. I sent an advisory mail some time ago, and it > > > was good then. Please confirm for trunk. >

ObjC: remove temporary tree list used when calling objc_declare_class()

2011-04-13 Thread Nicola Pero
This patch removes another case of temporary tree chains from the ObjC compiler. When the parser finds a @class NSArray; declaration, it would call objc_declare_class() passing as argument a temporary tree list containing the identifier. This tree list is created, used for the function call,

C++ PATCH for c++/48594 (failure with overloaded ->* in template)

2011-04-13 Thread Jason Merrill
The code in build_offset_ref_from_tree was assuming that any ->* involved calling a bound pointer to member function. But that's not necessarily the case: it could be calling a pointer to function or functor by way of a pointer to data member or overloaded ->* operator. Tested x86_64-pc-linux

New Swedish PO file for 'gcc' (version 4.6.0)

2011-04-13 Thread Translation Project Robot
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.6.0.sv.po', has just

Re: [Committed][AVR]: PR target/45263: Don't use r20 around calls of __tablejump_elpm__

2011-04-13 Thread Eric Botcazou
> PR target/45263 > * config/avr/libgcc.S (__do_global_ctors, __do_global_dtors): Don't use > r20 around calls of __tablejump_elpm__ > > http://gcc.gnu.org/viewcvs?view=revision&revision=172384 > http://gcc.gnu.org/viewcvs?view=revision&revision=172385 Please do not post useless

Re: [PATCH 5/6] Add line map statistics to -fmem-report output

2011-04-13 Thread Dodji Seketeli
Hello, Gabriel Dos Reis writes: > On Fri, Dec 10, 2010 at 5:11 AM, Dodji Seketeli wrote: > > This patch adds statistics about the memory consumption of line maps > > to the output of -fmem-report. It has been useful in trying to reduce > > the memory consumption of the macro maps support. > > >

Re: [PATCH] add statistics counting to postreload, copy-rename, and math-opts

2011-04-13 Thread Nathan Froyd
On Wed, Apr 13, 2011 at 11:07:15AM +0200, Richard Guenther wrote: > On Tue, Apr 12, 2011 at 5:09 PM, Nathan Froyd > wrote: > > Granted, but that fact should still be recorded.  The situation we have > > today, for something like: > > > > func1: statistic for "statx" was 0 > >  - nothing is record

Re: [PATCH, ARM] Switch to EABI version 5 for RTEMS

2011-04-13 Thread Joel Sherrill
On 04/13/2011 03:38 AM, Sebastian Huber wrote: On 04/06/2011 07:20 PM, Sebastian Huber wrote: On 06/04/11 18:24, Ralf Corsepius wrote: On 04/06/2011 05:20 PM, Sebastian Huber wrote: there were several requests for ARM Cortex-M support on RTEMS recently. The first step towards this is a suita

Re: [pph] Test PPH include at global scope. (issue4399041)

2011-04-13 Thread dnovillo
On 2011/04/12 21:40:10, Lawrence Crowl wrote: Add a test to ensure that PPH files are #included at global scope. Initially, this test is XFAIL, as it's a low priority error. Index: gcc/testsuite/ChangeLog.pph 2011-04-12 Lawrence Crowl * g++.dg/pph/y2sm

Re: [build] Avoid ld -v error message with Sun ld on SPARC

2011-04-13 Thread David Miller
From: Rainer Orth Date: Wed, 13 Apr 2011 19:16:32 +0200 > David Miller writes: > >> From: Rainer Orth >> Date: Tue, 12 Apr 2011 15:00:53 +0200 >> >>> On Solaris 10, as assembles the test just fine, but ld cannot deal with >>> gas 2.21 output: >>> >>> ld: fatal: relocation error: R_SPARC_GOTDA

Re: [pph/libcpp] Allow include callback to not read the file (issue4388057)

2011-04-13 Thread Diego Novillo
On Wed, Apr 13, 2011 at 12:21, Tom Tromey wrote: >> "Diego" == Diego Novillo writes: > > Lawrence> This change is not source compatible with existing code using > Lawrence> the callbacks (which may not be in the gcc svn).  Perhaps a new > Lawrence> callback is needed? > > Diego> Well, it only

Re: [build] Avoid ld -v error message with Sun ld on SPARC

2011-04-13 Thread Rainer Orth
David Miller writes: > From: Rainer Orth > Date: Tue, 12 Apr 2011 15:00:53 +0200 > >> On Solaris 10, as assembles the test just fine, but ld cannot deal with >> gas 2.21 output: >> >> ld: fatal: relocation error: R_SPARC_GOTDATA_HIX22: file gotdata.o: symbol >> : offset 0xff370163 is non-align

Re: RFA: patch for PR48455

2011-04-13 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 04/13/11 11:09, Vladimir Makarov wrote: > The following patch should improve code size which degradation for arm > is reported on > > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48455 > > Is it ok to commit? The patch was successfully bootstrappe

RFA: patch for PR48455

2011-04-13 Thread Vladimir Makarov
The following patch should improve code size which degradation for arm is reported on http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48455 Is it ok to commit? The patch was successfully bootstrapped on x86-64 and i686 with H.J.'s autotester options. 2011-04-13 Vladimir Makarov PR rt

[Committed][AVR]: PR target/45263: Don't use r20 around calls of __tablejump_elpm__

2011-04-13 Thread Georg-Johann Lay
PR target/45263 * config/avr/libgcc.S (__do_global_ctors, __do_global_dtors): Don't use r20 around calls of __tablejump_elpm__ http://gcc.gnu.org/viewcvs?view=revision&revision=172384 http://gcc.gnu.org/viewcvs?view=revision&revision=172385 Index: config/avr/libgcc.S ==

Re: RFA: Add makefile for cross-configuration torture test

2011-04-13 Thread Joseph S. Myers
On Wed, 13 Apr 2011, Joern Rennecke wrote: > This is based on config-list.mk in the pr46489-20101227-branch, but with > configurations removed that have been recently removed or obsoleted. On the whole I think the obsolete but not yet removed targets should be included in the list with --enable-

Re: RFA: Add makefile for cross-configuration torture test

2011-04-13 Thread Joern Rennecke
Quoting Rainer Orth : alpha-dec-osf5.1 Would you care to report the kind of breakage you found? This may simply be a case of mips-t* only being buildable native (PR target/3746) and gas/gld not supporting the target. Yes, that's what it is. gcc -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCT

Re: [pph/libcpp] Allow include callback to not read the file (issue4388057)

2011-04-13 Thread Tom Tromey
> "Diego" == Diego Novillo writes: Lawrence> This change is not source compatible with existing code using Lawrence> the callbacks (which may not be in the gcc svn).  Perhaps a new Lawrence> callback is needed? Diego> Well, it only changes the return value for the callback. Existing Diego>

Re: [pph/libcpp] Allow include callback to not read the file (issue4388057)

2011-04-13 Thread Tom Tromey
> "Diego" == Diego Novillo writes: Diego> During pph processing, when we find an included file that we are going Diego> to instantiate from an image, we don't want libcpp to stack and read Diego> it. Diego> I've implemented this by allowing the 'include' callback to return a Diego> boolean v

Added myself to MAINTAINERS

2011-04-13 Thread Georg-Johann Lay
Added myself to MAINTAINERS. Please update your MAINTAINERS file. Johann Index: ChangeLog === --- ChangeLog (Revision 172381) +++ ChangeLog (Revision 172382) @@ -1,3 +1,7 @@ +2011-04-13 Georg-Johann Lay + + * MAINTAINE

Re: RFA: Add makefile for cross-configuration torture test

2011-04-13 Thread Rainer Orth
Joern Rennecke writes: > Global patches that affect various configurations in various ways tend to > break configurations. People usually make an effort with global search and > replace, but all too often some details are overlooked (or simply mistyped). > > This Makefile is supposed to give cov

[PATCH 6/n, i386]: Merge SSE and AVX patterns using "enable" attribute.

2011-04-13 Thread Uros Bizjak
Hello! Attached patches converts "Parallel integral element swizzling" section of sse.md. 2011-04-13 Uros Bizjak * config/i386/sse.md (pinsrbits): Remove. (sse2_packsswb): Merge with *avx_packsswb. (sse2_packssdw): Merge with *avx_packssdw. (sse2_packuswb): Mer

[PATCH][ARM] New testcases for NEON

2011-04-13 Thread Andrew Stubbs
Hi, This old patch has been carried in the CodeSourcery toolchain for some time now. It just adds a few new testcases for vectorization. OK? Andrew 2008-12-03 Daniel Jacobowitz gcc/testsuite/ * gcc.dg/vect/vect-shift-3.c, gcc.dg/vect/vect-shift-4.c: New. * lib/target-supports.exp (chec

[committed] Fix __float128 OpenMP atomics in 32-bit cc1 (PR middle-end/48591)

2011-04-13 Thread Jakub Jelinek
Hi! In 32-bit cc1 on i?86-linux the attached testcases ICE, because built_in_decls[BUILT_IN_VAL_COMPARE_AND_SWAP_16] is NULL. There is no such integer type, but __float128 has that size. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk and 4.6. 2011-04-13

RFA: Add makefile for cross-configuration torture test

2011-04-13 Thread Joern Rennecke
Global patches that affect various configurations in various ways tend to break configurations. People usually make an effort with global search and replace, but all too often some details are overlooked (or simply mistyped). This Makefile is supposed to give coverage of all the main configure t

Re: [trans-mem] PR 47952 Re: weak aliases, .tm_clone_table, and binutils confusion

2011-04-13 Thread Patrick Marlier
About this line of my patch for aliases: DECL_WEAK (tm_alias) = DECL_WEAK (alias->decl); You can reproduce it with the same testcase, looking at these symbols: .weak _ZNSt14_List_iteratorIN4Game12BuildProjectEEC1EPSt15_List_node_base .globl _ZGTtNSt14_List_iteratorIN4Game12BuildProjectEEC1EPSt

Re: [PATCH 3/6] Allow jumps in epilogues

2011-04-13 Thread Bernd Schmidt
The final part, an updated version of the old 004-dw2cfg patch. This does much better placement of remember/restore; in almost all cases the code is identical to what we currently generate, modulo minor differences around the PROLOGUE_END label. I've made it emit queued register saves before PROLOG

Re: [PATCH 3/6] Allow jumps in epilogues

2011-04-13 Thread Bernd Schmidt
The second part is a new patch, which reduces the amount of different code paths we can take in add_fde_cfi, as this was becoming unmanageable. The concept is to first emit just the CFI notes, in all cases. Later, after we're done producing the CFI insns we need, another pass over the rtl adds the

Re: [PATCH 3/6] Allow jumps in epilogues

2011-04-13 Thread Bernd Schmidt
On 04/13/2011 04:14 PM, Bernd Schmidt wrote: > On 04/11/2011 07:10 PM, Richard Henderson wrote: >> Ok. > > Did you receive my reply to this message from earlier today? It doesn't > seem to have made it to gcc-patches yet. Since gcc-patches appears to have dropped the message, I'll resend it in th

Re: [PATCH] PR c++/48574

2011-04-13 Thread Dodji Seketeli
Jason Merrill writes: > On 04/13/2011 10:23 AM, Jakub Jelinek wrote: >> Then >> >> --- gcc/cp/class.c.jj2011-04-12 19:43:49.0 +0200 >> +++ gcc/cp/class.c 2011-04-13 15:23:07.463670993 +0200 >> @@ -5939,6 +5939,7 @@ fixed_type_or_null (tree instance, int * >> itself. *

Re: [PATCH 3/6] Allow jumps in epilogues

2011-04-13 Thread Jakub Jelinek
On Wed, Apr 13, 2011 at 07:44:26AM -0700, Richard Henderson wrote: > Yeah, while I was working on dwarf line numbers recently, I found that > just about the only thing that would produce anything interesting was > a profiled bootstrap. http://gcc.gnu.org/bugzilla/show_bug.cgi?id=48253#c1 is what

Re: [PING] Fix PR46399 - missing mode promotion for libcall args

2011-04-13 Thread Ian Lance Taylor
Andreas Krebbel writes: > On 04/13/2011 03:31 PM, Ian Lance Taylor wrote: >> "Andreas Krebbel" writes: >> >>> This fixes a wrong code generation bug for sw DFP: >>> >>> http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00141.html >> >> Why do we need a new target hook just for libcalls? Why not ju

C++ PATCH for c++/48581 (sfinae failure with ADL in default template arg)

2011-04-13 Thread Jason Merrill
We can't decide that a lookup has failed while we're still in template context, as phase 2 lookup might find it. Tested x86_64-pc-linux-gnu, applying to trunk. commit bf1635b662fb30ac7dc03b292bb52943c4dccc18 Author: Jason Merrill Date: Tue Apr 12 13:59:07 2011 -0400 * pt.c (tsubst_co

[Patch, Fortran] PR 18918: Build + install libcaf_single.a

2011-04-13 Thread Tobias Burnus
Hello all, hi Ralf and Jorge, The attached patch causes libgfortran/ also to build libgfortran/caf/single.c, which will be installed as $PREFIX/.../target-triplet/gcc-version/libcaf_single.a. Build and tested on x86-64-linux. OK for the trunk? * * * As a possible enhancement, one could als

Re: [PATCH 3/6] Allow jumps in epilogues

2011-04-13 Thread Richard Henderson
On 04/13/2011 05:38 AM, Bernd Schmidt wrote: > This bootstraps and tests ok on i686-linux. However, there is work left > to be done. Can I take you up on your offer to work with me on this? > This still requires the i386 output_set_got which I think I can cope > with, but the ia64 backend does a nu

Re: [PATCH] PR c++/48574

2011-04-13 Thread Jason Merrill
On 04/13/2011 10:23 AM, Jakub Jelinek wrote: Then --- gcc/cp/class.c.jj 2011-04-12 19:43:49.0 +0200 +++ gcc/cp/class.c 2011-04-13 15:23:07.463670993 +0200 @@ -5939,6 +5939,7 @@ fixed_type_or_null (tree instance, int * itself. */ if (TREE_CODE (instance) ==

Re: [PATCH] PR c++/48574

2011-04-13 Thread Jakub Jelinek
On Wed, Apr 13, 2011 at 10:17:25AM -0400, Jason Merrill wrote: > On 04/13/2011 09:40 AM, Dodji Seketeli wrote: > >+ if (type_dependent_expression_p (DECL_INITIAL (instance))) > >+{ > >+ /* The initializer is type dependent so we cannot > >+ determin

Re: [PATCH] PR c++/48574

2011-04-13 Thread Jason Merrill
On 04/13/2011 09:40 AM, Dodji Seketeli wrote: + if (type_dependent_expression_p (DECL_INITIAL (instance))) + { + /* The initializer is type dependent so we cannot +determine the dynamic type of this reference. */ + if

Re: [PATCH 3/6] Allow jumps in epilogues

2011-04-13 Thread Bernd Schmidt
On 04/11/2011 07:10 PM, Richard Henderson wrote: > Ok. Did you receive my reply to this message from earlier today? It doesn't seem to have made it to gcc-patches yet. Bernd

Re: [C++ PATCH] Handle correctly ARRAY_REFs from STRING_CST for wchar_t/char{16,32}_t (PR c++/48570)

2011-04-13 Thread Jason Merrill
OK. Jason

Re: [PING] Fix PR46399 - missing mode promotion for libcall args

2011-04-13 Thread Andreas Krebbel
On 04/13/2011 03:31 PM, Ian Lance Taylor wrote: > "Andreas Krebbel" writes: > >> This fixes a wrong code generation bug for sw DFP: >> >> http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00141.html > > Why do we need a new target hook just for libcalls? Why not just use > the existing TARGET_PROMOT

Re: Implement stack arrays even for unknown sizes

2011-04-13 Thread Paul Richard Thomas
Dear Richard and Dominique, > VLAs and malloc based arrays may behave differently with respect to alias > analysis (I'd have to look at some examples).  All effects other than malloc - hmmm, yes. I was forgetting what might happen with inlining. It's not evident, is it? > overhead I would attr

Re: [C++0x] Range-based for statements and ADL

2011-04-13 Thread Jason Merrill
On 04/11/2011 04:50 PM, Rodrigo Rivas wrote: Because the type of the expression must have complete type *only* if it is an array. Actually, if it has class type, it must also have a complete type or the class member lookup is ill-formed. And you can't pass an expression of void type to a fun

[PATCH] PR c++/48574

2011-04-13 Thread Dodji Seketeli
Hello, Consider this short code snippet: struct A { virtual void foo(); }; template void bar(T x) { A &b = *x; b.foo(); } When we look at b.foo() during the parsing (in cp_parser_postfix_expression), 'b.foo()' is (rightfully) considered non-dep

Re: [PING] Fix PR46399 - missing mode promotion for libcall args

2011-04-13 Thread Ian Lance Taylor
"Andreas Krebbel" writes: > This fixes a wrong code generation bug for sw DFP: > > http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00141.html Why do we need a new target hook just for libcalls? Why not just use the existing TARGET_PROMOTE_FUNCTION_MODE hook? We could say that for a libcall functy

Re: [PATCH 1/7] Simple cgraph_node -> cgraph_get_node conversions

2011-04-13 Thread H.J. Lu
On Wed, Apr 6, 2011 at 4:22 PM, Martin Jambor wrote: > Hi, > > this patch changes most of current calls to cgraph_node to calls to > cgraph_get_node.  The function should never return NULL in he contexts > of the callers below and either the probability is so low that it does > not warrant an asse

Re: [google/integration] Enable lightweight debug checks (issue4408041)

2011-04-13 Thread Diego Novillo
On Wed, Apr 13, 2011 at 01:44, Paul Pluzhnikov wrote: > This patch adds lightweight debug checks (if enabled by macros). > > To be applied only to google/integration branch. > > Tested by bootstrapping and running "make check". > > > 2011-04-12  Paul Pluzhnikov   > >        * libstdc++-v3/include/

Re: Tweak Negative() documentation in options.texi

2011-04-13 Thread Joseph S. Myers
On Wed, 13 Apr 2011, Richard Sandiford wrote: > * doc/options.texi (Negative): Explicitly mention that the > Negative chain must be circular. OK. -- Joseph S. Myers jos...@codesourcery.com

Re: Add debugging functions for cp_parser (issue4389045)

2011-04-13 Thread Diego Novillo
On Wed, Apr 13, 2011 at 08:39, Richard Guenther wrote: > On Wed, Apr 13, 2011 at 2:29 PM, Diego Novillo wrote: >> >> I'm not sure if that can be controlled.  When you respond to a hunk on >> Rietveld, it creates the quote automatically.  Maybe there is a way of >> increasing the scope of the quot

Tweak Negative() documentation in options.texi

2011-04-13 Thread Richard Sandiford
I was looking at a backport of the current linux.opt, and got confused by the chain of three Negative()s. I was thinking that saying "A is Negative(B)" was equivalent to saying A == !B. This patch is an attempt to clarify the documentation. Tested with "make doc", "make html" and "make pdf". OK

Re: Add debugging functions for cp_parser (issue4389045)

2011-04-13 Thread Richard Guenther
On Wed, Apr 13, 2011 at 2:29 PM, Diego Novillo wrote: > On Wed, Apr 13, 2011 at 05:25, Richard Guenther > wrote: >> On Wed, Apr 13, 2011 at 2:02 AM,   wrote: >>> One oddity, otherwise LGTM. >>> >>> >>> http://codereview.appspot.com/4389045/diff/1/gcc/cp/parser.c >>> File gcc/cp/parser.c (right):

Re: Add debugging functions for cp_parser (issue4389045)

2011-04-13 Thread Diego Novillo
On Wed, Apr 13, 2011 at 05:25, Richard Guenther wrote: > On Wed, Apr 13, 2011 at 2:02 AM,   wrote: >> One oddity, otherwise LGTM. >> >> >> http://codereview.appspot.com/4389045/diff/1/gcc/cp/parser.c >> File gcc/cp/parser.c (right): >> >> http://codereview.appspot.com/4389045/diff/1/gcc/cp/parser.

RX: Do not use SMOVF insn to move blocks of volatile memory.

2011-04-13 Thread Nick Clifton
Hi Guys, I am checking in the patch below to the 4.5 branch and mainline sources to fix a problem with the RX's SMOVF instruction. This instruction copies blocks of memory, but it always loads and stores aligned 32-bit values. If necessary it will load extra bytes from the beginning or

Re: Implement stack arrays even for unknown sizes

2011-04-13 Thread Richard Guenther
On Wed, Apr 13, 2011 at 12:38 PM, Richard Guenther wrote: > On Wed, Apr 13, 2011 at 11:42 AM, Paul Richard Thomas > wrote: >> Dear Dominique, >> >>> I think it is the automatic array in the subroutine trisolve. Note that the >>> speedup is rather 27->19s and may be darwin specific (slow malloc).

[PING] Fix PR46399 - missing mode promotion for libcall args

2011-04-13 Thread Andreas Krebbel
This fixes a wrong code generation bug for sw DFP: http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00141.html Bye, -Andreas-

Re: [patch i386 mingw g++.dg gcc.dg]: Set -mms-bitfields as default for native windows targets

2011-04-13 Thread Pedro Alves
On Wednesday 13 April 2011 11:43:43, Kai Tietz wrote: > > This is a default ABI change (IIRC, when the option was > > introduced, it was left off as default so to not break the ABI). > > > > Shouldn't we advertise it somewhere? > Yes, I did recently a lot of abi changing for mingw's targets. They

Re: [patch i386 mingw g++.dg gcc.dg]: Set -mms-bitfields as default for native windows targets

2011-04-13 Thread Kai Tietz
2011/4/13 Pedro Alves : > On Tuesday 12 April 2011 19:53:42, Kai Tietz wrote: >> 2011/4/12 Richard Henderson : >> > On 04/12/2011 05:47 AM, Kai Tietz wrote: >> >> ChangeLog gcc/ >> >> >> >>       * config/i386/mingw32.h (TARGET_SUBTARGET_DEFAULT): Add >> >>       MASK_MS_BITFIELD_LAYOUT bit. >> >>

Re: Implement stack arrays even for unknown sizes

2011-04-13 Thread Richard Guenther
On Wed, Apr 13, 2011 at 11:42 AM, Paul Richard Thomas wrote: > Dear Dominique, > >> I think it is the automatic array in the subroutine trisolve. Note that the >> speedup is rather 27->19s and may be darwin specific (slow malloc). > > I saw a speed-up of similar order on FC9/x86_64. > > I strongly

Re: [patch i386 mingw g++.dg gcc.dg]: Set -mms-bitfields as default for native windows targets

2011-04-13 Thread Pedro Alves
On Tuesday 12 April 2011 19:53:42, Kai Tietz wrote: > 2011/4/12 Richard Henderson : > > On 04/12/2011 05:47 AM, Kai Tietz wrote: > >> ChangeLog gcc/ > >> > >> * config/i386/mingw32.h (TARGET_SUBTARGET_DEFAULT): Add > >> MASK_MS_BITFIELD_LAYOUT bit. > >> > >> ChangeLog gcc/testsuite > >>

Re: Implement stack arrays even for unknown sizes

2011-04-13 Thread Paul Richard Thomas
Dear Dominique, > I think it is the automatic array in the subroutine trisolve. Note that the > speedup is rather 27->19s and may be darwin specific (slow malloc). I saw a speed-up of similar order on FC9/x86_64. I strongly doubt that it is anything to do with the automatic array - if it is ther

Re: Add debugging functions for cp_parser (issue4389045)

2011-04-13 Thread Richard Guenther
On Wed, Apr 13, 2011 at 2:02 AM, wrote: > One oddity, otherwise LGTM. > > > http://codereview.appspot.com/4389045/diff/1/gcc/cp/parser.c > File gcc/cp/parser.c (right): > > http://codereview.appspot.com/4389045/diff/1/gcc/cp/parser.c#newcode375 > gcc/cp/parser.c:375: if (flag) > This code will ne

Re: [patch] Split Parse Timevar (issue4378056)

2011-04-13 Thread Richard Guenther
On Tue, Apr 12, 2011 at 9:06 PM, Diego Novillo wrote: > On Tue, Apr 12, 2011 at 14:49, Lawrence Crowl wrote: >> This patch provides more finer, more precise compile time information. >> I sent an advisory mail some time ago, and it was good then.  Please >> confirm for trunk. > > The patch looks

Re: [PATCH] add statistics counting to postreload, copy-rename, and math-opts

2011-04-13 Thread Richard Guenther
On Tue, Apr 12, 2011 at 5:09 PM, Nathan Froyd wrote: > On Tue, Apr 12, 2011 at 04:54:43PM +0200, Richard Guenther wrote: >> On Tue, Apr 12, 2011 at 4:51 PM, Nathan Froyd >> wrote: >> > True, but maybe those testcases should be adjusted--per-pass flags, >> > rather than blindly assuming -O2 inclu

Re: [C++ PATCH] Handle correctly ARRAY_REFs from STRING_CST for wchar_t/char{16,32}_t (PR c++/48570)

2011-04-13 Thread Jakub Jelinek
On Tue, Apr 12, 2011 at 07:13:51PM -0400, Jason Merrill wrote: > On 04/12/2011 01:06 PM, Jakub Jelinek wrote: > >+ : (unsigned)TREE_STRING_LENGTH (ary) > >+ * (TYPE_PRECISION (TREE_TYPE (TREE_TYPE (ary))) > >+ / TYPE_PRECISION (char_type_node))); > > Don't you mean / instead of

Re: [PATCH, ARM] Switch to EABI version 5 for RTEMS

2011-04-13 Thread Sebastian Huber
On 04/06/2011 07:20 PM, Sebastian Huber wrote: > On 06/04/11 18:24, Ralf Corsepius wrote: >> On 04/06/2011 05:20 PM, Sebastian Huber wrote: >> >>> there were several requests for ARM Cortex-M support on RTEMS >>> recently. The >>> first step towards this is a suitable ARM tool chain. I want to us