Re: Merge C++ conversion into trunk (1/6 - Configury)

2012-08-12 Thread Steven Bosscher
On Mon, Aug 13, 2012 at 2:34 AM, Diego Novillo wrote: > On 12-08-12 16:16 , Steven Bosscher wrote: >> >> On Sun, Aug 12, 2012 at 10:09 PM, Diego Novillo >> wrote: >>> >>> This patch implements the configuration changes needed to >>> bootstrap with a C++ compiler by default. >> >> >> Hi, >> >> Is

Re: [PATCH 1/2] gcc symbol database

2012-08-12 Thread Dodji Seketeli
Hello Yunfeng, Thank you for following up, and sorry for me reviewing your patches so lately. The libcpp changes are coming along nicely, IMHO. I like the fact that they are getting pretty minimal. I just have a few mostly cosmetic comments at this point. [...] > diff -cpr .pc/symdb_enhance_l

Re: [cxx-conversion] Support garbage-collected C++ templates

2012-08-12 Thread Laurynas Biveinis
Diego - >>> * doc/gty.texi: Document support for C++ templates and >>> user-provided markers. >> >> >> The 1st node in this doc file needs s/C/C++/g and perhaps some more >> explanation with an eye on C++. > > > Sorry. I'm not sure what you are trying to say here. I'm referring

Re: Merge C++ conversion into trunk (0/6 - Overview)

2012-08-12 Thread H.J. Lu
On Sun, Aug 12, 2012 at 5:27 PM, Diego Novillo wrote: > On 12-08-12 18:38 , H.J. Lu wrote: >> >> On Sun, Aug 12, 2012 at 3:33 PM, H.J. Lu wrote: >>> >>> On Sun, Aug 12, 2012 at 1:04 PM, Diego Novillo >>> wrote: I will be sending 6 patches that implement all the changes we have bee

Re: Merge C++ conversion into trunk (6/6 - gdb tree macro support)

2012-08-12 Thread Diego Novillo
On 12-08-12 16:40 , Andreas Schwab wrote: Diego Novillo writes: diff --git a/gcc/gdbinit.in b/gcc/gdbinit.in index d1ae46d..858e490 100644 --- a/gcc/gdbinit.in +++ b/gcc/gdbinit.in @@ -182,6 +182,17 @@ document pbm Dump the bitmap that is in $ as a comma-separated list of numbers. end +#

Re: Merge C++ conversion into trunk (1/6 - Configury)

2012-08-12 Thread Diego Novillo
On 12-08-12 16:56 , Gerald Pfeifer wrote: On Sun, 12 Aug 2012, Diego Novillo wrote: ChangeLog 2012-08-12 Diego Novillo * Makefile.in (CXX_FOR_BUILD): Define. (BUILD_CXX_FLAGS): Define (COMPILER_FOR_BUILD): Set to CXX_FOR_BUILD if building with C++. (LINKER_FO

Re: Merge C++ conversion into trunk (1/6 - Configury)

2012-08-12 Thread Diego Novillo
On 12-08-12 16:16 , Steven Bosscher wrote: On Sun, Aug 12, 2012 at 10:09 PM, Diego Novillo wrote: This patch implements the configuration changes needed to bootstrap with a C++ compiler by default. Hi, Is it possible to add -fno-rtti to the default CXX_FLAGS, and remove it if necessary? I

Re: Merge C++ conversion into trunk (0/6 - Overview)

2012-08-12 Thread Diego Novillo
On 12-08-12 16:57 , Marc Glisse wrote: other compiler that managed. IBM and Oracle both fail (the comment is not clear, but I think 12.3 also fails, just not exactly in the same way), and HP and Intel (to mention just a few) are not listed. We should fix/workaround failing host C++ compilers s

Re: Merge C++ conversion into trunk (0/6 - Overview)

2012-08-12 Thread Diego Novillo
On 12-08-12 18:38 , H.J. Lu wrote: On Sun, Aug 12, 2012 at 3:33 PM, H.J. Lu wrote: On Sun, Aug 12, 2012 at 1:04 PM, Diego Novillo wrote: I will be sending 6 patches that implement all the changes we have been making on the cxx-conversion branch. As described in http://gcc.gnu.org/ml/gcc/2012

Re: [PATCH, MIPS] Add 34Kn cpu

2012-08-12 Thread Gerald Pfeifer
On Sat, 4 Aug 2012, Sandra Loosemore wrote: >> And http://gcc.gnu.org/gcc-4.8/changes.html :-) > Eh, this is pretty trivial compared to some of the other MIPS > improvements that are either recently committed or in the pipeline -- > specifically, the microMIPS support Catherine is working on, and

Re: Merge C++ conversion into trunk (0/6 - Overview)

2012-08-12 Thread H.J. Lu
On Sun, Aug 12, 2012 at 3:33 PM, H.J. Lu wrote: > On Sun, Aug 12, 2012 at 1:04 PM, Diego Novillo wrote: >> I will be sending 6 patches that implement all the changes we >> have been making on the cxx-conversion branch. As described in >> http://gcc.gnu.org/ml/gcc/2012-08/msg00015.html, these pat

Re: Merge C++ conversion into trunk (0/6 - Overview)

2012-08-12 Thread H.J. Lu
On Sun, Aug 12, 2012 at 1:04 PM, Diego Novillo wrote: > I will be sending 6 patches that implement all the changes we > have been making on the cxx-conversion branch. As described in > http://gcc.gnu.org/ml/gcc/2012-08/msg00015.html, these patches > change the default bootstrap process so that st

[patch] Reduce memory overhead for large functions

2012-08-12 Thread Steven Bosscher
Hello, This patch tried to use non-clearing memory allocation where possible. This is especially important for very large functions, when arrays of size in the order of n_basic_blocks or num_ssa_names are allocated to hold sparse data sets. For such cases the overhead of memset becomes measurable

Re: Merge C++ conversion into trunk (5/6 - double_int rewrite)

2012-08-12 Thread Marc Glisse
On Sun, 12 Aug 2012, Diego Novillo wrote: This implements the double_int rewrite. See http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00711.html for details. Diego. I am taking it as a chance to ask a couple questions about the coding conventions. 2012-08-12 Lawrence Crowl * hash

Re: [SH] PR 51244 - Add combiner patterns for branch inversion

2012-08-12 Thread Kaz Kojima
Oleg Endo wrote: > This adds some patters that allow the combine pass to catch inverted > compare and branch opportunities better. > > Tested on rev 190273 with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" > > and no new fai

Re: [SH] Minor prefetch insn corrections

2012-08-12 Thread Kaz Kojima
Oleg Endo wrote: > I've noticed that __builtin_prefetch was not working on SH3, although > the pref instruction is available on SH3. Also, it seems that some of > the checks in the prefetch expander are redundant. > Tested on rev 190273 with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \

Re: Merge C++ conversion into trunk (0/6 - Overview)

2012-08-12 Thread Marc Glisse
On Sun, 12 Aug 2012, Diego Novillo wrote: For those who would like to build the conversion, you can either checkout the branch from SVN (svn://gcc.gnu.org/gcc/branches/cxx-conversion) or get the merged trunk I have in the git repo (branch dnovillo/cxx-conversion). The bootstrap changes have alr

Re: Merge C++ conversion into trunk (1/6 - Configury)

2012-08-12 Thread Gerald Pfeifer
On Sun, 12 Aug 2012, Diego Novillo wrote: > ChangeLog > 2012-08-12 Diego Novillo > > * Makefile.in (CXX_FOR_BUILD): Define. > (BUILD_CXX_FLAGS): Define > (COMPILER_FOR_BUILD): Set to CXX_FOR_BUILD if building with C++. > (LINKER_FOR_BUILD): Likewise. > (BUILD_COMP

Re: Merge C++ conversion into trunk (6/6 - gdb tree macro support)

2012-08-12 Thread Andreas Schwab
Diego Novillo writes: > diff --git a/gcc/gdbinit.in b/gcc/gdbinit.in > index d1ae46d..858e490 100644 > --- a/gcc/gdbinit.in > +++ b/gcc/gdbinit.in > @@ -182,6 +182,17 @@ document pbm > Dump the bitmap that is in $ as a comma-separated list of numbers. > end > > +# Define some macros helpful t

Merge C++ conversion into trunk (6/6 - gdb tree macro support)

2012-08-12 Thread Diego Novillo
This implements support for calling tree macros from gdb. See http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00711.html for details. Diego. 2012-08-12 Lawrence Crowl * tree.h (tree_check): New. (TREE_CHECK): Use inline function above instead of __extension__. (tree_no

Re: Merge C++ conversion into trunk (1/6 - Configury)

2012-08-12 Thread Steven Bosscher
On Sun, Aug 12, 2012 at 10:09 PM, Diego Novillo wrote: > This patch implements the configuration changes needed to > bootstrap with a C++ compiler by default. Hi, Is it possible to add -fno-rtti to the default CXX_FLAGS, and remove it if necessary? Ciao! Steven

Merge C++ conversion into trunk (1/6 - Configury)

2012-08-12 Thread Diego Novillo
This patch implements the configuration changes needed to bootstrap with a C++ compiler by default. See http://gcc.gnu.org/ml/gcc-patches/2012-08/msg00711.html for details. Diego. ChangeLog 2012-08-12 Diego Novillo * Makefile.in (CXX_FOR_BUILD): Define. (BUILD_CXX_FLAGS): D

Merge C++ conversion into trunk (0/6 - Overview)

2012-08-12 Thread Diego Novillo
I will be sending 6 patches that implement all the changes we have been making on the cxx-conversion branch. As described in http://gcc.gnu.org/ml/gcc/2012-08/msg00015.html, these patches change the default bootstrap process so that stage 1 always builds with a C++ compiler. Other than the boots

Re: Value type of map need not be default copyable

2012-08-12 Thread François Dumont
On 08/11/2012 03:47 PM, Marc Glisse wrote: On Sat, 11 Aug 2012, François Dumont wrote: Your remark on using std::move rather than std::forward Marc made sens but didn't work. I don't understand why but the new test is showing that std::forward works. If anyone can explain why std::move doe

Re: [C++ Pubnames Patch] Anonymous namespaces enclosed in named namespaces. (issue6343052)

2012-08-12 Thread Jack Howarth
On Sun, Jul 01, 2012 at 09:33:06AM -0500, Gabriel Dos Reis wrote: > On Thu, Jun 28, 2012 at 12:50 PM, Sterling Augustine > wrote: > > The enclosed patch adds a fix for the pubnames anonymous namespaces > > contained > > within named namespaces, and adds an extensive test for the various > > pubn

[v3] improve exception text when threads not enabled

2012-08-12 Thread Jonathan Wakely
This improves the fairly uninformative "Operation not supported" message given when std::thread is used without linking to libpthread. Now you get: terminate called after throwing an instance of 'std::system_error' what(): Enable multithreading to use std::thread: Operation not permitted Abort

[SH, committed]: Fix FPU related test cases

2012-08-12 Thread Oleg Endo
Hello, This fixes some of the FPU related SH target test cases, which are not skipped / not skipped appropriately. Tested with make -k check-gcc RUNTESTFLAGS="sh.exp --target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2e/-mb,/-m2e/-ml,-m3/-mb,-m3/-ml,-m3e/-mb,-m3e/-mb, -m2a/-mb,-m2a-single-only/-mb,-m4/-ml,

[SH, committed]: Fix FPU related test cases

2012-08-12 Thread Oleg Endo
Hello, This fixes some of the FPU related SH target test cases, which are not skipped / not skipped appropriately. Tested with make -k check-gcc RUNTESTFLAGS="sh.exp --target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2e/-mb,/-m2e/-ml,-m3/-mb,-m3/-ml,-m3e/-mb,-m3e/-mb, -m2a/-mb,-m2a-single-only/-mb,-m4/-ml,

Re: PR 54193: raw gimple dump of vec_perm_expr

2012-08-12 Thread Richard Guenther
On Sat, Aug 11, 2012 at 10:37 PM, Marc Glisse wrote: > Hello, > > I'll have to retest this patch tomorrow (although I don't expect the > modified code is ever called), for some reason the testsuite took twice as > long as usual to run and showed some weird stuff tonight. > > There doesn't seem to

[SH] Minor prefetch insn corrections

2012-08-12 Thread Oleg Endo
Hello, I've noticed that __builtin_prefetch was not working on SH3, although the pref instruction is available on SH3. Also, it seems that some of the checks in the prefetch expander are redundant. Tested on rev 190273 with make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/

[SH] PR 51244 - Add combiner patterns for branch inversion

2012-08-12 Thread Oleg Endo
Hello, This adds some patters that allow the combine pass to catch inverted compare and branch opportunities better. Tested on rev 190273 with make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" and no new failures. OK? PS: I'm still

Re: [SH] PR 39423 - Fix ICEs, add more tests

2012-08-12 Thread Kaz Kojima
Oleg Endo wrote: > This patch fixes ICEs such as reported in the PR due to wrong predicate > matching and adds test cases for those. > Tested on rev 190273 with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" > > and no new fail

[SH] PR 39423 - Fix ICEs, add more tests

2012-08-12 Thread Oleg Endo
Hello, This patch fixes ICEs such as reported in the PR due to wrong predicate matching and adds test cases for those. Tested on rev 190273 with make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m4/-ml,-m4/-mb,-m4a/-ml,-m4a/-mb}" and no new failures. OK? Cheers, Oleg

Re: Value type of map need not be default copyable

2012-08-12 Thread Marc Glisse
On Sun, 12 Aug 2012, Jonathan Wakely wrote: On 11 August 2012 14:47, Marc Glisse wrote: What testcase failed? I just tried the 2.cc file you added with the patch, and replacing forward(__k) with move(__k) compiled fine. Shouldn't it be std::move(__k) to disable ADL though? It is written st

Re: Value type of map need not be default copyable

2012-08-12 Thread Jonathan Wakely
On 11 August 2012 14:47, Marc Glisse wrote: > > What testcase failed? I just tried the 2.cc file you added with the patch, > and replacing forward(__k) with move(__k) compiled fine. Shouldn't it be std::move(__k) to disable ADL though?

Re: [Patch, Fortran] PR54221 - mark private module vars/procs as TREE_PUBLIC()=0

2012-08-12 Thread Mikael Morin
On 12/08/2012 10:56, Tobias Burnus wrote: > Build and regtested on x86-64-linux. > OK for the trunk? OK. > 2012-08-11 Tobias Burnus > > PR fortran/54221 > * vect/vect-gems.f90: Don't mark module vars as PRIVATE as > they appear (ninitialized on the RHS.

RE: [PATCH,i386] fma,fma4 and xop flags

2012-08-12 Thread Gopalasubramanian, Ganesh
Thank you Uros, Richard! I will confirm the test results in couple off days. Regards Ganesh -Original Message- From: Uros Bizjak [mailto:ubiz...@gmail.com] Sent: Saturday, August 11, 2012 3:54 AM To: Richard Henderson Cc: Gopalasubramanian, Ganesh; gcc-patches@gcc.gnu.org Subject: Re: [

Re: [Patch,AVR] PR54222: Add fixed point support

2012-08-12 Thread Denis Chertykov
2012/8/11 Georg-Johann Lay : > Weddington, Eric schrieb: >>> >>> From: Georg-Johann Lay >>> >>> >>> The first step would be to bisect and find the patch that lead to >>> PR53923. It was not a change in the avr BE, so the question goes >>> to the authors of the respective patch. >>> >>> Up to now I

[Patch, Fortran] PR54221 - mark private module vars/procs as TREE_PUBLIC()=0

2012-08-12 Thread Tobias Burnus
Since April, gfortran no longer marks module procedures/variables as TREE_PUBLIC, if they are marked as PRIVATE. (-> improved optimization, reduced file size) However, as this PR shows, for "PRIVATE" instead of "PRIVATE :: symbol" this didn't work, which is addressed by the attached patch. I