Re: [Google Debugfission] Add pubnames and pubtypes to google/main (issue5489074)

2011-12-17 Thread Cary Coutant
> 2011-12-17   Sterling Augustine   > >        * gcc/dwarf2out.c (is_cu_die, is_namespace_die, is_class_die): New >        functions. >        (add_pubname): Call is_namespace_die, is_cu_die, and is_class_die in >        conditional. >        (add_enumerator_pubname): New function. >        (add_pu

Re: [PATCH SMS 1/2, RFC] Support traversing PS in reverse order

2011-12-17 Thread Revital Eres
Hello, >> This patch support the estimation of register pressure in SMS. >> Although GCC is in stage 3 I would appreciate comments on it. >> Thanks to Richard and Ayal for discussing the implementation and their >> insights. >> >> This part of the patch enables iterating on the partial schedule i

Re: [PATCH] PowerPC section type conflict

2011-12-17 Thread Chung-Lin Tang
On 2011/12/17 06:21 AM, Richard Henderson wrote: > On 12/16/2011 03:16 AM, Chung-Lin Tang wrote: >> Hi, under powerpc targets, using -mrelocatable under some cases triggers >> an error during final assembly generation: rs6000_assemble_integer() >> calls varasm.c:unlikely_text_section_p(), and the c

[PATCH SMS 2/2, RFC] Register pressure estimation for the partial schedule (re-submission)

2011-12-17 Thread Revital Eres
Hello, The attached patch is a resubmission following comments made by Ayal and Richard. Tested and bootstrap with the other patches in the series on ppc64-redhat-linux, enabling SMS on loops with SC 1. Comments are welcome. Thanks, Revital 2011-12-18 Richard Sandiford R

[v3] fix typos in docs

2011-12-17 Thread Jonathan Wakely
* doc/xml/manual/iterators.xml: Replace "sect1" with "section". * doc/xml/manual/algorithms.xml: Likewise. * doc/html/manual/iterators.html: Likewise. * doc/html/manual/algorithms.html: Likewise. Fixes some errors probably introduced by s/section/sect1/ Rather than

C++ PATCH for c++/51588 (ICE with invalid pointer-to-member)

2011-12-17 Thread Jason Merrill
In C++11 an enum can be used in a nested-name-specifier, but it still isn't valid for a pointer-to-member. Tested x86_64-pc-linux-gnu, applying to trunk. commit 80c46fcfb6ea29a6566b285cf94e519ec414d2e9 Author: Jason Merrill Date: Sat Dec 17 15:39:56 2011 -0500 PR c++/51588 * parser

Re: [patch committed SH] Add atomic patterns

2011-12-17 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch should fix the align 2 issues mentioned before and > also fixes the ior fetchop_name. It should be "or" instead of "ior". You are right about that nop shouldn't be inserted after write-back. Thanks for pointing out my thinko. Your patch doesn't work because

Re: [google][4.6]Compiler Directed Multiversioning with new -mvarch option (issue 5490054)

2011-12-17 Thread Sriraman Tallam
Committed to google 4_6 branch. Thanks, -Sri. On Sat, Dec 17, 2011 at 12:25 AM, Xinliang David Li wrote: > ok for google branches. > > David > > On Fri, Dec 16, 2011 at 2:05 PM,   wrote: >> I have uploaded a new patch set with all the mentioned changes made. If >> a function has the target attri

Re: [patch i386][google][4.6]Add new target builtins to check for corei7 and amdfam10 (issue5495075)

2011-12-17 Thread Sriraman Tallam
Checked in. Thanks, -Sri. On Sat, Dec 17, 2011 at 12:21 AM, Xinliang David Li wrote: > ok for google branches. > > David > > On Fri, Dec 16, 2011 at 7:54 PM, Sriraman Tallam wrote: >> Add new target builtins __builtin_cpu_is_intel_corei7 and >> __builtin_cpu_is_amdfam10. >> >>        * config/i

Re: C++ PATCH for c++/51587 (ICE with struct/enum conflict)

2011-12-17 Thread Fabien Chêne
2011/12/17 Jason Merrill : > This testcase was aborting when we tried to check ENUM_UNDERLYING_TYPE on a > non-enum.  Fixed by checking the tree code first. Ahem ... thank you ! -- Fabien

Re: [patch committed SH] Add atomic patterns

2011-12-17 Thread Richard Henderson
On 12/17/2011 12:13 PM, Oleg Endo wrote: > @@ -90,13 +90,13 @@ >return \"\\ > mova\\t1f, r0\\n\\ > \\t\\t%2, %4\\n\\ > +\\t.align\\t2\\n\\ The other thing that should be remembered is that inside a { } block you don't need to double-quote all the \'s. r~

Re: [google] Add support for delete operator that takes the size of the object as a parameter

2011-12-17 Thread Xinliang David Li
ok. David On Sat, Dec 17, 2011 at 11:44 AM, Easwaran Raman wrote: > Based on Paolo's comments I am dropping the changes to > baseline_symbols.txt. As far as minor version, I am bumping it up to > 18. Assuming that this patch will be able to go in gcc 4.8 (with minor > version 18), I want to keep

Re: [patch committed SH] Add atomic patterns

2011-12-17 Thread Oleg Endo
The attached patch should fix the align 2 issues mentioned before and also fixes the ior fetchop_name. It should be "or" instead of "ior". I'm not sure whether it requires re-testing. Just in case now running: make -k -check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml/-msoft-atomic, -m2/-mb/-m

Re: [google] Add support for delete operator that takes the size of the object as a parameter

2011-12-17 Thread Easwaran Raman
Based on Paolo's comments I am dropping the changes to baseline_symbols.txt. As far as minor version, I am bumping it up to 18. Assuming that this patch will be able to go in gcc 4.8 (with minor version 18), I want to keep it at the same version in google/main and google/gcc-4_6. Bootstraps and no

[Google Debugfission] Add pubnames and pubtypes to google/main (issue5489074)

2011-12-17 Thread Sterling Augustine
Enclosed for review and inclusion on the google/main and google/4_6 trees is the first patch for the debugfission project. It includes producing a complete and correct set of .debug_pubnames and .debug_pubtypes, as well as switching the default to generate .debug_pubnames and .debug_pubtypes. It

C++ PATCH for c++/51587 (ICE with struct/enum conflict)

2011-12-17 Thread Jason Merrill
This testcase was aborting when we tried to check ENUM_UNDERLYING_TYPE on a non-enum. Fixed by checking the tree code first. Tested x86_64-pc-linux-gnu, applying to trunk. commit b01c647f6754097ac85e4ca1d7c80c1da8edc660 Author: Jason Merrill Date: Sat Dec 17 09:30:43 2011 -0500 PR c++/

Re: [build] Fix bootstrap/51072: libitm not disabled without c++

2011-12-17 Thread Andreas Schwab
Richard Henderson writes: > There's no "good" place for this. The description in Makefile.def that > libitm uses c++ is only used insofar as the dependencies for libitm -- it's > built after libstdc++ as the language support library. If we put this into > the toplevel configure.ac directly,

Re: [patch committed SH] Add atomic patterns

2011-12-17 Thread Oleg Endo
On Mon, 2011-12-05 at 08:07 +0900, Kaz Kojima wrote: > + return \"\\ > +mova\\t1f, r0\\n\\ > +\\t\\t%2, %4\\n\\ > +\\tmov\\tr15, r1\\n\\ > +\\tmov\\t#(0f-1f), r15\\n\\ > +0:\\tmov.\\t@%1, %0\\n\\ > +\\tcmp/eq\\t%0, %4\\n\\ > +\\tbf\\t1f\\n\\ > +\\tmov.\\t%3, @%1\\n\\ > +\\t.align\\t2\\n\\ > +1:\\

Re: [build] Fix bootstrap/51072: libitm not disabled without c++

2011-12-17 Thread Richard Henderson
On 12/17/2011 06:19 AM, Eric Botcazou wrote: >> Putting into gcc/cp/config-lang.in is a layering violation, it's true. But >> until there's another instance that needs handling, it seems premature to >> build infrastructure to handle this. And it's only one line after all... > > I don't know exa

Re: [build] Fix bootstrap/51072: libitm not disabled without c++

2011-12-17 Thread H.J. Lu
On Sat, Dec 17, 2011 at 6:19 AM, Eric Botcazou wrote: >> Putting into gcc/cp/config-lang.in is a layering violation, it's true.  But >> until there's another instance that needs handling, it seems premature to >> build infrastructure to handle this.  And it's only one line after all... > > I don't

Re: [build] Fix bootstrap/51072: libitm not disabled without c++

2011-12-17 Thread Eric Botcazou
> Putting into gcc/cp/config-lang.in is a layering violation, it's true. But > until there's another instance that needs handling, it seems premature to > build infrastructure to handle this. And it's only one line after all... I don't know exactly why, but this breaks bootstrap on some machines

Re: Ignore thunks in ThreadSanitizer pass (issue 5492055)

2011-12-17 Thread Diego Novillo
On Sat, Dec 17, 2011 at 06:53, wrote: > Yes, for that particular function on Linux/amd64 I see > artificial+ignored and is_thunk==0. > > So is it LGTY? Yes. Diego.

[committed] Fix some MIPS entries in libgcc/config.host

2011-12-17 Thread Richard Sandiford
libgcc wasn't building the soft-float routines for mips-sde-elf, because the config.host stanza removed t-mips from tmake_file. Tested on mips-sde-elf and applied. I should have noticed this when the patches were posted for review, sorry. Richard libgcc/ * config.host (mips*-sde-elf*,

Invalid hard-reg decomposition in lower-subreg

2011-12-17 Thread Richard Sandiford
lower-subreg.c:can_decompose_p uses the following condition to test whether a multiword hard register can be decomposed into words: return (validate_subreg (word_mode, GET_MODE (x), x, UNITS_PER_WORD) && HARD_REGNO_MODE_OK (regno, word_mode)); This doesn't work reliably on

Ping: Out-of-order update of new_spill_reg_store[]

2011-12-17 Thread Richard Sandiford
Ping for this reload patch: http://gcc.gnu.org/ml/gcc-patches/2011-12/msg00266.html or perhaps the original: http://gcc.gnu.org/ml/gcc-patches/2011-10/msg00663.html which fixes some wrong-code regressions on mips64-linux-gnu. Richard

Ping: Fixing liveness information during prepare_shrink_wrap

2011-12-17 Thread Richard Sandiford
Ping for this prepare_shrink_wrap patch: http://gcc.gnu.org/ml/gcc-patches/2011-12/msg00274.html It fixes some wrong-code regressions on MIPS16. Richard

Re: [PATCH] Add Octeon2 indexed load instruction support (and also DSP64 LDX support)

2011-12-17 Thread Richard Sandiford
Andrew Pinski writes: > Index: testsuite/gcc.target/mips/octeon2-lx-1.c > === > --- testsuite/gcc.target/mips/octeon2-lx-1.c (revision 0) > +++ testsuite/gcc.target/mips/octeon2-lx-1.c (revision 0) > @@ -0,0 +1,18 @@ > +/* { dg-do c

Re: [patch] Fix cygwin ada install [was Re: Yet another issue with gcc current trunk with ada on cygwin]

2011-12-17 Thread Eric Botcazou
> I am happily using this patch and await this to be committed. Only after the import library issue is addressed. What do the other libraries bundled with GCC do here? -- Eric Botcazou

Re: [patch] Fix cygwin ada install [was Re: Yet another issue with gcc current trunk with ada on cygwin]

2011-12-17 Thread Christian Joensson
On 8 December 2011 14:02, Dave Korn wrote: > >    Hi again guys, > >  After the previous patch, there's still another bug remaining in the Ada > makefile, relating to how it builds and installs the gnat/gnarl shared > libraries. > >  Windows doesn't have any concept of an rpath in executables, nor

Re: PATCH: Add DEFAULT_LIBC=LIBC_BIONIC and ANDROID_DEFAULT=1 to *-android-*

2011-12-17 Thread Mike Stump
On Dec 16, 2011, at 5:43 PM, H.J. Lu wrote: > When I configure GCC with i686-linux-android target, don't I get > linux as vendor and android OS? No: $ ./config.sub i686-linux-android i686-pc-linux-android pc is the vendor, though, I'd claim this is a misnomer and wrong. A cell phone isn't PC c

Re: PATCH: Add DEFAULT_LIBC=LIBC_BIONIC and ANDROID_DEFAULT=1 to *-android-*

2011-12-17 Thread Mike Stump
On Dec 16, 2011, at 3:27 PM, Joseph S. Myers wrote: > I think using the "company" part of the target triplet to mean anything > should be avoided; I disagree. Think of the string as a key that subscripts a database which has precooked answers that are used instead of probing for the answers whi

Re: [google][4.6]Compiler Directed Multiversioning with new -mvarch option (issue 5490054)

2011-12-17 Thread Xinliang David Li
ok for google branches. David On Fri, Dec 16, 2011 at 2:05 PM, wrote: > I have uploaded a new patch set with all the mentioned changes made. If > a function has the target attribute it will not be touched by the > autoclone pass. > > Also fixed some test cases which were broken because the clon

Re: [patch i386][google][4.6]Add new target builtins to check for corei7 and amdfam10 (issue5495075)

2011-12-17 Thread Xinliang David Li
ok for google branches. David On Fri, Dec 16, 2011 at 7:54 PM, Sriraman Tallam wrote: > Add new target builtins __builtin_cpu_is_intel_corei7 and > __builtin_cpu_is_amdfam10. > >        * config/i386/i386-cpuinfo.c (__processor_model): Add new members >        __cpu_is_intel_corei7 and __cpu_is_