Re: [PR testsuite/47013] Fix SMS testsuite faliures

2011-05-16 Thread Dominique Dhumieres
> Does the attached patch resolve the failure with sms-8.c? Yes. Thanks for the update. Dominique

Re: Patch: New GTY ((atomic)) option

2011-05-16 Thread Nicola Pero
>>> This patch adds a new GTY option, "atomic", which is similar to the >>> identical option you have with Boehm GC >>> and which can be used with pointers to inform the GC/PCH machinery that >>> they point to an area of memory that >> [...] >>> This patch basically implements it, but at this st

Re: Patch: New GTY ((atomic)) option

2011-05-16 Thread Nicola Pero
> Yes why can't you use vec(int,gc)? Can you use that in libiberty ? I thought it was a GCC-only thing ? Thanks

Re: [PATCH: PR target/46975] Replace 32 bit instructions with 16 bit instructions in thumb2

2011-05-16 Thread Carrot Wei
On Fri, Dec 17, 2010 at 8:18 PM, Richard Earnshaw wrote: > > On Thu, 2010-12-16 at 14:45 -0800, Carrot Wei wrote: >> Hi >> >> Compile the following c code with options -march=armv7-a -mthumb -Os >> >> int foo (int s) >> { >>     return s == 1; >> } >> >> GCC 4.6 generates: >> >> : >>    0

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Richard Guenther
On Fri, May 13, 2011 at 7:14 PM, Ian Lance Taylor wrote: > I noticed that buglist.cgi was taking quite a bit of CPU time.  I looked > at some of the long running instances, and they were coming from > searchbots.  I can't think of a good reason for this, so I have > committed this patch to the gcc

Re: Patch: New GTY ((atomic)) option

2011-05-16 Thread Laurynas Biveinis
2011/5/16 Nicola Pero : > 2011-05-16  Nicola Pero   > >        * gengtype.c (walk_type): Implemented "atomic" GTY option. >        * doc/gty.texi (GTY Options): Document "atomic" GTY option. The patch is OK, with difference between "skip" and "atomic" options documented. (Can be done as a follow-u

Re: [PATCH] Fix error: 'previous' may be used uninitialized in this function

2011-05-16 Thread Laurynas Biveinis
> 2011-05-15  Dmitry Gorbachev   > >        * gengtype-state.c (read_state_param_structs): Initialize "previous". The logic in the function is OK, is this to silence some false positive warning? In any case, the patch is OK, I assume you have tested it. Thanks, -- Laurynas

[wwwdocs] Update Fortran section in gcc-4.7/changes.html

2011-05-16 Thread Tobias Burnus
I intent to commit the attached patch soon, unless you have concerns, comments or suggestions. Tobias ? changes.diff Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-4.7/changes.html,v retrieving revision 1.12 diff -u -r1

Re: [patch gimplifier]: Make sure TRUTH_NOT_EXPR has boolean_type_node type and argument

2011-05-16 Thread Richard Guenther
On Sat, May 14, 2011 at 6:14 PM, Eric Botcazou wrote: >> Those issues should be fixed by the attached patch, which relaxes >> strictness of logical operations in tree-cfg.c file. > > Thanks. > >> 2011-05-14  Kai Tietz >> >>         * tree-cfg.c (verify_gimple_assign_unary): Don't enforce >> boolea

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Andrew Haley
On 16/05/11 10:45, Richard Guenther wrote: > On Fri, May 13, 2011 at 7:14 PM, Ian Lance Taylor wrote: >> I noticed that buglist.cgi was taking quite a bit of CPU time. I looked >> at some of the long running instances, and they were coming from >> searchbots. I can't think of a good reason for t

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Michael Matz
Hi, On Mon, 16 May 2011, Andrew Haley wrote: > On 16/05/11 10:45, Richard Guenther wrote: > > On Fri, May 13, 2011 at 7:14 PM, Ian Lance Taylor wrote: > >> I noticed that buglist.cgi was taking quite a bit of CPU time. I looked > >> at some of the long running instances, and they were coming fr

[PATCH] Misc debug info improvements

2011-05-16 Thread Jakub Jelinek
Hi! This patch contains a bunch of smaller debug info improvements: 1) on the typeddwarf.c testcase on x86_64 there were completely needlessly many DW_OP_GNU_entry_value ops emitted, while the corresponding argument registers were still live. Fixed by the cselib_subst_to_values hunk. 2) var

x86 option handling cleanup: -mfpmath=

2011-05-16 Thread Joseph S. Myers
This patch continues cleaning up x86 option handling to use .opt features, by cleaning up the -mfpmath= handling. This is somewhat complicated because of these options being generated by attributes. I exposed a new interface from opts-common.c to be used to convert a string to an enum value as if

[patch ada]: Fix boolean_type_node setup and some cleanup for boolean use

2011-05-16 Thread Kai Tietz
Hello, this fixes the ADA parts of PR middle-end/48989. For Fortran I am still looking, what actual the cause is. ChangeLog 2011-05-16 Kai Tietz PR middle-end/48989 * gcc-interface/trans.c (Exception_Handler_to_gnu_sjlj): Use boolean_false_node instead of integer_zero_n

Re: [PATCH, i386] PR 48743 Correctly detect AMD K6-2+ and K6-3+

2011-05-16 Thread H.J. Lu
On Sun, May 15, 2011 at 10:03 PM, Zuxy Meng wrote: > Misdetected as Athlon by GCC, K6-2+ and K6-3+ are processors that support > extended 3DNow! but don't support extended MMX or CMOV. > > I don't own a K6-2 or Athlon machine. Can anybody have the patch tested? > > 2011-05-16 Zuxy Meng > PR i386/

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Andrew Haley
On 05/16/2011 01:09 PM, Michael Matz wrote: > Hi, > > On Mon, 16 May 2011, Andrew Haley wrote: > >> On 16/05/11 10:45, Richard Guenther wrote: >>> On Fri, May 13, 2011 at 7:14 PM, Ian Lance Taylor wrote: I noticed that buglist.cgi was taking quite a bit of CPU time. I looked at some o

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Andreas Schwab
Andrew Haley writes: > It's not quite the same information, surely. Wouldn't searchers be directed > to an email rather than the bug itself? The mail contains the bugzilla link, so they can easily get there if needed. Andreas. -- Andreas Schwab, sch...@redhat.com GPG Key fingerprint = D4E8 D

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Richard Guenther
On Mon, May 16, 2011 at 3:04 PM, Andrew Haley wrote: > On 05/16/2011 01:09 PM, Michael Matz wrote: >> Hi, >> >> On Mon, 16 May 2011, Andrew Haley wrote: >> >>> On 16/05/11 10:45, Richard Guenther wrote: On Fri, May 13, 2011 at 7:14 PM, Ian Lance Taylor wrote: > I noticed that buglist.cgi

[Patch] fixinclude: shell-quote testing file

2011-05-16 Thread Tristan Gingold
Hi, one of the main VMS include files has a dollar in its name (decc$types.h), which is not shell friendly. As a consequence, when the name is reused by check.tpl to create a baseline file, it must be quoted. This is what is implemented in this patch. Tested by 'make check' in fixincludes/ Ok

Re: [patch gimplifier]: Make sure TRUTH_NOT_EXPR has boolean_type_node type and argument

2011-05-16 Thread Michael Matz
Hi, On Mon, 16 May 2011, Richard Guenther wrote: > We can't use a test for BOOLEAN_TYPE as the middle-end considers a > INTEGER_TYPE with same precision/signedness as compatible and thus may > propagate a variable of INTEGER_TYPE there. I don't understand why > promoting bools to boolean_type

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Andrew Haley
On 05/16/2011 02:10 PM, Richard Guenther wrote: > On Mon, May 16, 2011 at 3:04 PM, Andrew Haley wrote: >> On 05/16/2011 01:09 PM, Michael Matz wrote: >>> Hi, >>> >>> On Mon, 16 May 2011, Andrew Haley wrote: >>> On 16/05/11 10:45, Richard Guenther wrote: > On Fri, May 13, 2011 at 7:14 PM,

[Patch] fixincludes: fix solaris_complex_cxx rule syntax

2011-05-16 Thread Tristan Gingold
Hi, this rule (and only this one) uses the c function of sed. According to man, the syntax should be: [2addr]c\ text (Note the trailing back-slash). But as currently written, there is no backslash in it. As a consequence 'make check' fails at least on Darwin (BSD derived sed comman

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Michael Matz
Hi, On Mon, 16 May 2011, Andrew Haley wrote: > >> It's not quite the same information, surely. Wouldn't searchers be > >> directed to an email rather than the bug itself? > > > > Yes, though there is a link in all mails. > > Right, so we are contemplating a reduction in search quality in exch

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Andrew Haley
On 05/16/2011 02:22 PM, Michael Matz wrote: > Hi, > > On Mon, 16 May 2011, Andrew Haley wrote: > It's not quite the same information, surely. Wouldn't searchers be directed to an email rather than the bug itself? >>> >>> Yes, though there is a link in all mails. >> >> Right, so we are

Re: [Patch] fixinclude: shell-quote testing file

2011-05-16 Thread Andreas Schwab
Tristan Gingold writes: > 2011-05-16 Tristan Gingold > > * check.tpl: Shell-quote testing file. > > diff --git a/fixincludes/check.tpl b/fixincludes/check.tpl > index b239d8b..0f3937a 100644 > --- a/fixincludes/check.tpl > +++ b/fixincludes/check.tpl > @@ -87,7 +87,7 @@ FOR fix =][= >

Re: [patch gimplifier]: Make sure TRUTH_NOT_EXPR has boolean_type_node type and argument

2011-05-16 Thread Richard Guenther
On Mon, May 16, 2011 at 3:17 PM, Michael Matz wrote: > Hi, > > On Mon, 16 May 2011, Richard Guenther wrote: > >> We can't use a test for BOOLEAN_TYPE as the middle-end considers a >> INTEGER_TYPE with same precision/signedness as compatible and thus may >> propagate a variable of INTEGER_TYPE ther

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Michael Matz
Hi, On Mon, 16 May 2011, Andrew Haley wrote: > > It routinely is. bugzilla performance is terrible most of the time > > for me (up to the point of five timeouts in sequence), svn speed is > > mediocre at best, and people with access to gcc.gnu.org often observe > > loads > 25, mostly due to I

Re: [Patch] fixinclude: shell-quote testing file

2011-05-16 Thread Bruce Korb
On Mon, May 16, 2011 at 6:15 AM, Tristan Gingold wrote: > Hi, > > one of the main VMS include files has a dollar in its name (decc$types.h), > which is not shell friendly. > > As a consequence, when the name is reused by check.tpl to create a baseline > file, must be quoted. > Ok for trunk ? >

Re: [Patch] fixinclude: shell-quote testing file

2011-05-16 Thread Tristan Gingold
On May 16, 2011, at 3:26 PM, Andreas Schwab wrote: > Tristan Gingold writes: > >> 2011-05-16 Tristan Gingold >> >> * check.tpl: Shell-quote testing file. >> >> diff --git a/fixincludes/check.tpl b/fixincludes/check.tpl >> index b239d8b..0f3937a 100644 >> --- a/fixincludes/check.tpl >>

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Andrew Haley
On 05/16/2011 02:32 PM, Michael Matz wrote: > > On Mon, 16 May 2011, Andrew Haley wrote: > >>> It routinely is. bugzilla performance is terrible most of the time >>> for me (up to the point of five timeouts in sequence), svn speed is >>> mediocre at best, and people with access to gcc.gnu.org

Re: [Patch] fixinclude: shell-quote testing file

2011-05-16 Thread Tristan Gingold
On May 16, 2011, at 3:33 PM, Bruce Korb wrote: > On Mon, May 16, 2011 at 6:15 AM, Tristan Gingold wrote: >> Hi, >> >> one of the main VMS include files has a dollar in its name (decc$types.h), >> which is not shell friendly. >> >> As a consequence, when the name is reused by check.tpl to crea

Re: [Patch] fixincludes: fix solaris_complex_cxx rule syntax

2011-05-16 Thread Bruce Korb
Hi Tristan, On Mon, May 16, 2011 at 6:20 AM, Tristan Gingold wrote: > Hi, > > this rule (and only this one) uses the c function of sed.  According to man, > the syntax should be: > >     [2addr]c\ >     text > > (Note the trailing back-slash). > But as currently written, there is no backslash in

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Richard Guenther
On Mon, May 16, 2011 at 3:34 PM, Andrew Haley wrote: > On 05/16/2011 02:32 PM, Michael Matz wrote: >> >> On Mon, 16 May 2011, Andrew Haley wrote: >> It routinely is.  bugzilla performance is terrible most of the time for me (up to the point of five timeouts in sequence), svn speed is >>>

Re: [Patch] fixinclude: shell-quote testing file

2011-05-16 Thread Bruce Korb
Hi Tristan, On Mon, May 16, 2011 at 6:39 AM, Tristan Gingold wrote: > Sure.  [If] The patch below is ok for 'make check', is it also ok for you ? Yes, thank you.

[PATCH, i386]: Introduce "z" constraint and merge call patterns

2011-05-16 Thread Uros Bizjak
Hello! Attached patch introduces "z" constraint that matches constant_call_address_operand predicate. Using this constraint, several similar call patterns can be merged into one, substantially lowering the number of similar call patterns. Please note, that the patch also merges SImode and DImode

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Andrew Haley
On 05/16/2011 02:42 PM, Richard Guenther wrote: > On Mon, May 16, 2011 at 3:34 PM, Andrew Haley wrote: >> On 05/16/2011 02:32 PM, Michael Matz wrote: >>> >>> On Mon, 16 May 2011, Andrew Haley wrote: >>> > It routinely is. bugzilla performance is terrible most of the time > for me (up to t

Re: [patch gimplifier]: Make sure TRUTH_NOT_EXPR has boolean_type_node type and argument

2011-05-16 Thread Michael Matz
Hi, On Mon, 16 May 2011, Richard Guenther wrote: > > I think conversion _to_ BOOLEAN_TYPE shouldn't be useless, on the > > grounds that it requires booleanization (at least conceptually), i.e. > > conversion to a set of two values (no matter the precision or size) > > based on the outcome of c

[PATCH][?/n] Cleanup LTO type merging

2011-05-16 Thread Richard Guenther
Hashes of SCC members have been poor since ever as we have to ensure they are the same when the SCC is entered from a different path. But we can do better and mixin SCC member hashes if we can sort the SCC in some way. And we can do so, when sorting after the hash values. The only thing we need

Re: [patch gimplifier]: Make sure TRUTH_NOT_EXPR has boolean_type_node type and argument

2011-05-16 Thread Richard Guenther
On Mon, May 16, 2011 at 3:45 PM, Michael Matz wrote: > Hi, > > On Mon, 16 May 2011, Richard Guenther wrote: > >> > I think conversion _to_ BOOLEAN_TYPE shouldn't be useless, on the >> > grounds that it requires booleanization (at least conceptually), i.e. >> > conversion to a set of two values (no

[PATCH][?/n] Cleanup LTO type merging

2011-05-16 Thread Richard Guenther
The following patch improves hashing types by re-instantiating the patch that makes us visit aggregate target types of pointers and function return and argument types. This halves the collision rate on the type hash table for a linux-kernel build and improves WPA compile-time from 3mins to 1mins

Re: [PATCH] Fix PR46728 (move pow/powi folds to tree phases)

2011-05-16 Thread Richard Guenther
On Fri, May 13, 2011 at 6:00 PM, Nathan Froyd wrote: > On 05/13/2011 11:26 AM, Richard Guenther wrote: >> On Fri, May 13, 2011 at 5:01 PM, Nathan Froyd >> wrote: >>> On 05/13/2011 10:52 AM, William J. Schmidt wrote: This patch addresses PR46728, which notes that pow and powi need to be

Re: [Patch,AVR]: Fix PR45099

2011-05-16 Thread Georg-Johann Lay
Denis Chertykov schrieb: > 2011/5/6 Georg-Johann Lay : >> Nathan Froyd schrieb: >>> On Mon, May 02, 2011 at 05:23:48PM +0200, Georg-Johann Lay wrote: PR45099 is an extension that gives an error when a fixed register is needed to pass a parameter to a function. Because the progra

PATCH to copy_statement_list for c++/47999 (ICE in testsuite on darwin)

2011-05-16 Thread Jason Merrill
On 05/03/2011 02:18 PM, Jason Merrill wrote: I also removed the recursion from copy_statement_list because it would just extra garbage STATEMENT_LISTs since they're already copied by the normal walk_tree. I was wrong about this, the recursion is necessary because tsi_link_after destroys STATEM

[PATCH][?/n] Cleanup LTO type merging

2011-05-16 Thread Richard Guenther
I've seen us merge different named structs which happen to reside on the same variant list. That's bogus, not only because we are adjusting TYPE_MAIN_VARIANT during incremental type-merging and fixup, so computing a persistent hash by looking at it looks fishy as well. Bootstrapped and tested on

Re: [Patch] fixincludes: fix solaris_complex_cxx rule syntax

2011-05-16 Thread Tristan Gingold
On May 16, 2011, at 3:40 PM, Bruce Korb wrote: > Hi Tristan, > > On Mon, May 16, 2011 at 6:20 AM, Tristan Gingold wrote: >> Hi, >> >> this rule (and only this one) uses the c function of sed. According to man, >> the syntax should be: >> >> [2addr]c\ >> text >> >> (Note the trailin

Re: [Patch,AVR]: Fix PR45099

2011-05-16 Thread Denis Chertykov
2011/5/16 Georg-Johann Lay : > Denis Chertykov schrieb: >> 2011/5/6 Georg-Johann Lay : >>> Nathan Froyd schrieb: On Mon, May 02, 2011 at 05:23:48PM +0200, Georg-Johann Lay wrote: > PR45099 is an extension that gives an error when a fixed register is > needed to pass a parameter to a fu

Re: [Patch,AVR]: Fix PR27663

2011-05-16 Thread Georg-Johann Lay
Denis Chertykov schrieb: > 2011/5/11 Georg-Johann Lay : >> Denis Chertykov schrieb: >>> 2011/5/2 Georg-Johann Lay : >>> This is a fix for an optimization flaw when a long value is composed from byte values. For -fsplit-wide-types (which is still default for avr) the code is

[PATCH, MELT] remove useless use of var in gcc/Makefile.in

2011-05-16 Thread piervit
Hello, this patch is for the MELT branch. My GCC contributor number is 634276. In a previous commit MELTHERE_CFLAGS was remove. I could still find a few references to it in the gcc/Makefile.in. I have also removed MELTHERE_CC1_CFLAGS as it takes its value from a subset of MELTHERE_CFLAGS T

Re: [Patch] fixincludes: fix solaris_complex_cxx rule syntax

2011-05-16 Thread Bruce Korb
Hi Tristan, On Mon, May 16, 2011 at 8:27 AM, Tristan Gingold wrote: > Yes, it is ok on GNU/Linux and Solaris. > > Do I really need to test on HP/UX and AIX ?  Won't be easy for me. Solaris and BSD are usually the most unusual, so I'd say go ahead for mainline and see if you get squawks. With se

Re: [Patch] fixincludes: fix solaris_complex_cxx rule syntax

2011-05-16 Thread Tristan Gingold
On May 16, 2011, at 5:45 PM, Bruce Korb wrote: > Hi Tristan, > > On Mon, May 16, 2011 at 8:27 AM, Tristan Gingold wrote: >> Yes, it is ok on GNU/Linux and Solaris. >> >> Do I really need to test on HP/UX and AIX ? Won't be easy for me. > > Solaris and BSD are usually the most unusual, so I'd

Re: [Patch,AVR]: Solve PR42210

2011-05-16 Thread Georg-Johann Lay
Georg-Johann Lay schrieb: > Richard Henderson schrieb: > >> Why are you adding "optimize" to all these insns? None of them will >> be matched unless combine is run, which implies optimization. > > Here is a patch without optimize in the insn conditions. > > The optimize condition is still prese

Re: C6X port 9/11: Allow defining attributes in terms of another

2011-05-16 Thread Bernd Schmidt
On 05/10/2011 03:47 PM, Bernd Schmidt wrote: > I've found it useful to use a construct such as the following: > > (define_attr "units64" > "unknown,d,d_addr,l,m,s,dl,ds,dls,ls" > (const_string "unknown")) > > (define_attr "units64p" > "unknown,d,d_addr,l,m,s,dl,ds,dls,ls" > (attr "units64

Re: [PATCH, MELT] remove useless use of var in gcc/Makefile.in

2011-05-16 Thread Basile Starynkevitch
On Mon, 16 May 2011 17:37:24 +0200 wrote: > this patch is for the MELT branch. > > My GCC contributor number is 634276. [...] > 2011-05-16 Pierre Vittet > > * Makefile.in: remove useless use of MELTHERE_CFLAGS, > remove variable MELTHERE_CC1_CFLAGS, as it takes its arg from >

Re: [Patch,AVR]: Fix PR27663

2011-05-16 Thread Denis Chertykov
2011/5/16 Georg-Johann Lay : > Denis Chertykov schrieb: >> 2011/5/11 Georg-Johann Lay : >>> Denis Chertykov schrieb: 2011/5/2 Georg-Johann Lay : > This is a fix for an optimization flaw when a long value is composed > from byte values. > > For -fsplit-wide-types (which is

Re: [PATCH] Fix PR46728 (move pow/powi folds to tree phases)

2011-05-16 Thread William J. Schmidt
Richi, thank you for the detailed review! I'll plan to move the power-series expansion into the existing IL walk during pass_cse_sincos. As part of this, I'll move tree_expand_builtin_powi and its subfunctions from builtins.c into tree-ssa-math-opts.c. I'll submit this as a separate patch. I wi

[v3] More noexcept bits, , std::pair, ...

2011-05-16 Thread Paolo Carlini
Hi, also added the missing get(&&) overloads for std::array and std::pair. Tested x86_64-linux, committed. Paolo. // 2011-05-16 Paolo Carlini * include/std/utility (get(std::pair<>&&)): Add. * include/bits/stl_pair.h (pair::swap(pair&), swap(pair

Re: [v3] More noexcept bits, , std::pair, ...

2011-05-16 Thread Paolo Carlini
Oops. Paolo. /// 2011-05-16 Paolo Carlini * include/std/utility: Simplify the last commit, the whole std::get code is C++0x only. Index: include/std/utility === --- include/std/utility (revision 17379

Re: [PATCH, i386]: Introduce "z" constraint and merge call patterns

2011-05-16 Thread Uros Bizjak
On Mon, May 16, 2011 at 3:44 PM, Uros Bizjak wrote: > Please note, that the patch also merges SImode and DImode call > patterns, although SImode patterns use "lsm" constraint originally. > The "l" constraint prevents %esp register, but since operand predicate > already rejects %esp hard reg and %

Re: [PATCH][?/n] Cleanup LTO type merging

2011-05-16 Thread Jan Hubicka
> > I've seen us merge different named structs which happen to reside > on the same variant list. That's bogus, not only because we are > adjusting TYPE_MAIN_VARIANT during incremental type-merging and > fixup, so computing a persistent hash by looking at it looks > fishy as well. Hi, as reporte

Fix cppspec.c lang_specific_driver bug (PR 48677)

2011-05-16 Thread Joseph S. Myers
PR 48677 pointed out a mistake in my conversion of lang_specific_driver to use decoded options. This is fixed by this patch. Bootstrapped with no regressions on x86_64-unknown-linux-gnu. Applied to mainline. Will also apply to 4.6 branch to fix the regression there subject to 4.6 branch testing

Re: [C++0x] contiguous bitfields race implementation

2011-05-16 Thread Aldy Hernandez
Bootstrapped without any issues. Running the entire testsuite with --param=allow-store-data-races=0 is still in progress. BTW, no regressions, even running the entire thing at --param=allow-store-data-races=0 to force testing this new bitfield implementation on all tests.

Re: [wwwdocs] Update Fortran section in gcc-4.7/changes.html

2011-05-16 Thread Tobias Burnus
Tobias Burnus wrote: I intent to commit the attached patch soon, unless you have concerns, comments or suggestions. Committed - with the attachment follow up patch, which fixed validators issues, added another item (-fbacktrace) and linkified some more flags. Tobias Index: changes.html =

Re: [PATCH][?/n] Cleanup LTO type merging

2011-05-16 Thread H.J. Lu
On Mon, May 16, 2011 at 11:32 AM, Jan Hubicka wrote: >> >> I've seen us merge different named structs which happen to reside >> on the same variant list.  That's bogus, not only because we are >> adjusting TYPE_MAIN_VARIANT during incremental type-merging and >> fixup, so computing a persistent ha

Re: [Patch] fixincludes: fix solaris_complex_cxx rule syntax

2011-05-16 Thread Ralf Wildenhues
Hello, * Tristan Gingold wrote on Mon, May 16, 2011 at 03:20:19PM CEST: > Ok for trunk ? FWIW this looks good to me, and certainly necessary for Posix and portable sed, but I can't approve it. Thanks, Ralf > fixincludes/ > 2011-05-16 Tristan Gingold > > * inclhack.def (solaris_complex

[4.5/4.6 PATCH, i386]: Some random fixes in FP move patterns

2011-05-16 Thread Uros Bizjak
Hello! This patch backports recent fixes in FP moves from mainline to 4.5 and 4.6 branches. 2011-05-16 Uros Bizjak * config/i386/i386.md (*movxf_internal): Disable CONST_DOUBLE optimization for CM_MEDIUM and CM_LARGE code models. Fix usage of standard_80387_constant_p

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Ian Lance Taylor
Richard Guenther writes: > On Fri, May 13, 2011 at 7:14 PM, Ian Lance Taylor wrote: >> I noticed that buglist.cgi was taking quite a bit of CPU time.  I looked >> at some of the long running instances, and they were coming from >> searchbots.  I can't think of a good reason for this, so I have >

[Patch, Fortran, committed] gfortran.texi: _gfortran_set_options, TR 29113

2011-05-16 Thread Tobias Burnus
Update _gfortran_set_options now that -std=f2008tr is supported for TR 29113, add a "TR 29113 Status" section, and update coarray entry now that multiple images are (experimentally, incomplete) supported. Committed as Rev. 173804. Tobias Index: gcc/fortran/gfortran.texi ==

[Patch, Fortran] PR 48700: memory leak with MOVE_ALLOC

2011-05-16 Thread Janus Weil
Hi all, here is a simple patch which takes care of fixing a memory leak, which occurs when calling MOVE_ALLOC with allocatable scalar arguments. We simply missed to deallocate the second argument 'TO', which is INTENT(OUT), in case it is already allocated. The patch was regtested on x86_64-unknow

Re: C++ PATCH for c++/48969 (infinite template recursion with enum scope)

2011-05-16 Thread Jason Merrill
On 05/13/2011 06:22 PM, Jason Merrill wrote: My initial implementation used a VEC to keep track of current deductions in process, but I switched it to use a hash table instead; the overhead for using a hash table rather than a VEC on the most common case (very low deduction nesting) is small, but

Re: [google] support for building Linux kernel with FDO (issue4523061)

2011-05-16 Thread Xinliang David Li
Agreed. David On Sun, May 15, 2011 at 3:27 AM, Jan Hubicka wrote: >> On Fri, May 13, 2011 at 5:54 AM, Paolo Bonzini wrote: >> > On 05/13/2011 03:03 AM, Rong Xu wrote: >> >> >> >>        * gcc/coverage.c        (revision 173717): set a flag if building >> >> for Linux kernel. >> >>        * gcc/

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Joseph S. Myers
On Mon, 16 May 2011, Ian Lance Taylor wrote: > Richard Guenther writes: > > > On Fri, May 13, 2011 at 7:14 PM, Ian Lance Taylor wrote: > >> I noticed that buglist.cgi was taking quite a bit of CPU time.  I looked > >> at some of the long running instances, and they were coming from > >> searchb

Re: [PATCH][?/n] Cleanup LTO type merging

2011-05-16 Thread H.J. Lu
On Mon, May 16, 2011 at 7:17 AM, Richard Guenther wrote: > > The following patch improves hashing types by re-instantiating the > patch that makes us visit aggregate target types of pointers and > function return and argument types.  This halves the collision > rate on the type hash table for a li

Re: [google] support for building Linux kernel with FDO (issue4523061)

2011-05-16 Thread Andi Kleen
Jan Hubicka writes: > Yep, > I think it does make sense to share the implementation, but we need to find > resonable way to do so. I doubt this will be very popular with the kernel community, which prefers self contained code. Also the interface doesn't change that often or does it? The curren

Re: [google] support for building Linux kernel with FDO (issue4523061)

2011-05-16 Thread Rong Xu
On Mon, May 16, 2011 at 3:38 PM, Andi Kleen wrote: > Jan Hubicka writes: >> Yep, >> I think it does make sense to share the implementation, but we need to find >> resonable way to do so. > > I doubt this will be very popular with the kernel community, which > prefers self contained code. > > Also

Re: [google] support for building Linux kernel with FDO (issue4523061)

2011-05-16 Thread Xinliang David Li
On Mon, May 16, 2011 at 3:38 PM, Andi Kleen wrote: > Jan Hubicka writes: >> Yep, >> I think it does make sense to share the implementation, but we need to find >> resonable way to do so. > > I doubt this will be very popular with the kernel community, which > prefers self contained code. Won't b

Re: [google] support for building Linux kernel with FDO (issue4523061)

2011-05-16 Thread Jan Hubicka
> On Mon, May 16, 2011 at 3:38 PM, Andi Kleen wrote: > > Jan Hubicka writes: > >> Yep, > >> I think it does make sense to share the implementation, but we need to find > >> resonable way to do so. > > > > I doubt this will be very popular with the kernel community, which > > prefers self containe

Re: [google] support for building Linux kernel with FDO (issue4523061)

2011-05-16 Thread Andi Kleen
Rong Xu writes: >> The current kernel code is for gcc 3. That could be simply >> replaced with a modern gcc 4 interface. > > You cannot replace. You have to keep the code for every gcc versions > that being used. I don't think it's a problem to not support gcc 3 gcov/profiling anymore. -Andi -

Re: [google] support for building Linux kernel with FDO (issue4523061)

2011-05-16 Thread Rong Xu
On Sun, May 15, 2011 at 3:27 AM, Jan Hubicka wrote: >> On Fri, May 13, 2011 at 5:54 AM, Paolo Bonzini wrote: >> > On 05/13/2011 03:03 AM, Rong Xu wrote: >> >> >> >>        * gcc/coverage.c        (revision 173717): set a flag if building >> >> for Linux kernel. >> >>        * gcc/tree-profile.c  

Re: [google] support for building Linux kernel with FDO (issue4523061)

2011-05-16 Thread Rong Xu
That will be good. But you never know, we internally have fixed some bugs that filed to us because people use kernel's old gcov code (many versions guarded by ifdef) for their tests. -Rong On Mon, May 16, 2011 at 4:23 PM, Andi Kleen wrote: > Rong Xu writes: > >>> The current kernel code is for

Re: [PATCH, i386] PR 48743 Correctly detect AMD K6-2+ and K6-3+

2011-05-16 Thread Zuxy Meng
2011/5/16 H.J. Lu : > On Sun, May 15, 2011 at 10:03 PM, Zuxy Meng wrote: >> Misdetected as Athlon by GCC, K6-2+ and K6-3+ are processors that support >> extended 3DNow! but don't support extended MMX or CMOV. >> >> I don't own a K6-2 or Athlon machine. Can anybody have the patch tested? >> >> 2011

Re: [PATCH, i386] PR 48743 Correctly detect AMD K6-2+ and K6-3+

2011-05-16 Thread H.J. Lu
On Mon, May 16, 2011 at 7:41 PM, Zuxy Meng wrote: > 2011/5/16 H.J. Lu : >> On Sun, May 15, 2011 at 10:03 PM, Zuxy Meng wrote: >>> Misdetected as Athlon by GCC, K6-2+ and K6-3+ are processors that support >>> extended 3DNow! but don't support extended MMX or CMOV. >>> >>> I don't own a K6-2 or Ath

Re: [PATCH, i386] PR 48743 Correctly detect AMD K6-2+ and K6-3+

2011-05-16 Thread Zuxy Meng
2011/5/17 H.J. Lu : > On Mon, May 16, 2011 at 7:41 PM, Zuxy Meng wrote: >> 2011/5/16 H.J. Lu : >>> On Sun, May 15, 2011 at 10:03 PM, Zuxy Meng wrote: Misdetected as Athlon by GCC, K6-2+ and K6-3+ are processors that support extended 3DNow! but don't support extended MMX or CMOV. >>

Re: Don't let search bots look at buglist.cgi

2011-05-16 Thread Ian Lance Taylor
On Mon, May 16, 2011 at 6:42 AM, Richard Guenther wrote: >>> >>> httpd being in the top-10 always, fiddling with bugzilla URLs? >>> (Note, I don't have access to gcc.gnu.org, I'm relaying info from multiple >>> instances of discussion on #gcc and richi poking on it; that said, it >>> still might n

[PR testsuite/47013] Fix SMS testsuite faliures (re-submission)

2011-05-16 Thread Revital Eres
Hello, Attached is a new version of the patch. Thanks to Dominique Dhumieres for testing on powerpc-apple-darwin9 and x86_64-apple-darwin10. Tested ppc64-redhat-linux on both with -m32,-m64 and SPU. OK for mainline? Thanks, Revital testsuite/Changelog PR rtl-optimization/47013

Re: C6X port 8/11: A new FUNCTION_ARG macro

2011-05-16 Thread Paolo Bonzini
On 05/12/2011 05:40 PM, Bernd Schmidt wrote: + if (targetm.calls.function_arg_round_to_arg_boundary (passed_mode, type)) +round_boundary = boundary; + else +round_boundary = PARM_BOUNDARY; Why add an if, instead of making the new target hook function_arg_round_boundary? The default

Re: Prefixes for libgcc symbols (C6X 9.5/11)

2011-05-16 Thread Paolo Bonzini
On 05/13/2011 03:40 PM, Bernd Schmidt wrote: gcc/ * libgcc2.h (__NW, __NDW): Define using a __gnu_ prefix if LIBGCC2_GNU_PREFIX is defined. (__N): New macro. (__powisf2, __powidf2, __powitf2, __powixf2, __bswapsi2, __bswapdi2, __mulsc3, __muldc3, _