Re: [PATCH v3 12/18] convert the Go front end to automatic dependencies

2013-09-16 Thread Tom Tromey
Tom I can try a test build using a setting for CC that rejects -c -o. I did this and the build died pretty early on: /home/tromey/Space/Trunk/Git/bin/my-cc -c -DHAVE_CONFIG_H -g -g -I. -I../../gcc/libiberty/../include -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototypes -pedantic

Re: [PATCH, alpha]: Add alpha.o dependecies

2013-09-16 Thread Tom Tromey
Uros == Uros Bizjak ubiz...@gmail.com writes: Uros Attached patch decleres dependecies for config/alpha/alpha.o. Uros 2013-09-16 Uros Bizjak ubiz...@gmail.com Uros * config/alpha/t-alpha: New. Uros * config.gcc (alpha*-*-linux*): Add alpha/t-alpha to tmake_file. Uros

Re: [PATCH v3 05/18] convert the C front end to automatic dependencies

2013-09-16 Thread Tom Tromey
Paolo The series looks okay to me with that change. Two last questions while I'm testing my rebase -- First, do you mind if I resend the whole series? Otherwise I can try to pick out just the patches that have changed, plus the additional patches. Second, should I get a separate ok for the

Re: [patch] Remove unused macro and functions from tree-flow.h and tree-flow-inline.h

2013-09-10 Thread Tom Tromey
Jakub == Jakub Jelinek ja...@redhat.com writes: Jakub On Tue, Sep 10, 2013 at 09:04:41AM -0400, Andrew MacLeod wrote: Really? If it were useful I would have thought they'd be using it now. In any case, it doesn't belong in tree-flow.h.. . shouldn't these functions and macros be in hashtab.h

Re: [PATCH v2 00/18] resurrect automatic dependencies

2013-09-09 Thread Tom Tromey
Tom == Tom Tromey tro...@redhat.com writes: Tom This is version 3 of my series to resurrect automatic dependencies for Tom GCC. Version 2 is here: Tom http://gcc.gnu.org/ml/gcc-patches/2013-07/msg01386.html Tom Ordinarily I would simply ping the existing patches, but Alexandre Tom asked me

Re: v2 of GDB hooks for debugging GCC

2013-08-27 Thread Tom Tromey
David == David Malcolm dmalc...@redhat.com writes: David Is there a precanned event provided by gdb that I can connect to for David when the underlying code has changed and my caches need to be David invalidated? Maybe not :( You could use the exited event as a decent approximation. Also, and

Re: [PATCH v3 12/18] convert the Go front end to automatic dependencies

2013-08-26 Thread Tom Tromey
Ian == Ian Lance Taylor i...@google.com writes: Ian I assume that dropping $(OUTPUT_OPTION) is correct--I haven't looked Ian at the new definition of $(COMPILE). I believe the depcomp script takes care of this. Tom

Re: v2 of GDB hooks for debugging GCC

2013-08-21 Thread Tom Tromey
David == David Malcolm dmalc...@redhat.com writes: Tom Naughty. David We chatted about this at Cauldron; I haven't yet had a chance to David implement the magic bullet approach we discussed there. In the David meantime, is there a API I can call to determine how safe this kludge David is? Not

[PATCH v2 00/18] resurrect automatic dependencies

2013-08-20 Thread Tom Tromey
This is version 3 of my series to resurrect automatic dependencies for GCC. Version 2 is here: http://gcc.gnu.org/ml/gcc-patches/2013-07/msg01386.html Ordinarily I would simply ping the existing patches, but Alexandre asked me to resend the series and CC him. I've rebased the series and

[PATCH v3 02/18] update generated_files

2013-08-20 Thread Tom Tromey
A few generated files were not mentioned in the generated_files variable. This fixes the problem. * Makefile.in (generated_files): Add options.h, target-hooks-def.h, insn-opinit.h, common/common-target-hooks-def.h, pass-instances.def. --- gcc/Makefile.in | 4 +++- 1 file

[PATCH v3 01/18] clean up SHLIB so subshells are not needed

2013-08-20 Thread Tom Tromey
This changes the handling of SHLIB so that it is inlined into DRIVER_DEFINES. This is ok because SHLIB is defined in a Makefile fragment that is included by the generated Makefile. The rationale for this is that it simplifies some .o targets, so that we can share more code. *

[PATCH v3 03/18] move generated_files order-only dependency later

2013-08-20 Thread Tom Tromey
There is an order-only dependency in gcc/Makefile.in that tries to build the generated files before compiling regular objects. However, this appears too early, and so at the time it is seen by make, GCOV_OBJS and GCOV_DUMP_OBJS have not yet been set. This patch fixes the problem and prevents any

[PATCH v3 05/18] convert the C front end to automatic dependencies

2013-08-20 Thread Tom Tromey
This converts the C front end. Note that this fixes a latent bug in gcc/Makefile.in's definition of C_TREE_H. This is needed to avoid breaking this build with this patch. * Makefile.in (C_TREE_H): Reference c/c-tree.h. * Make-lang.in (c/gccspec.o): Remove.

[PATCH v3 04/18] add configury

2013-08-20 Thread Tom Tromey
This adds the configury needed for automatic dependency tracking. It also adds some bits to the Makefile so we can begin converting (removing) explicit dependencies. * Makefile.in (CCDEPMODE, DEPDIR, depcomp, COMPILE.base) (COMPILE, POSTCOMPILE): New variables. (.cc.o

[PATCH v3 06/18] convert the C++ front end to automatic dependencies

2013-08-20 Thread Tom Tromey
This converts the C++ front end. This renames g++spec.o to cp/g++spec.o for uniformity. This lets us remove an explicit rule. This patch does not remove various *_H macros from cp/Make-lang.in. These are still needed by ObjC++. They're removed by a later patch. * Make-lang.in

[PATCH v3 09/18] convert the Java front end to automatic dependencies

2013-08-20 Thread Tom Tromey
This converts the Java front end. We also rename jvspec.o to java/jvspec.o, for uniformity; this lets us remove an explicit rule. * Make-lang.in (jvspec.o): Remove. (CFLAGS-java/jvspec.o): New variable. ($(XGCJ)$(exeext), java_OBJS): Use java/jvspec.o

[PATCH v3 07/18] convert the ObjC front end to automatic dependencies

2013-08-20 Thread Tom Tromey
This converts the ObjC front end. Note that there is a latent possible bug in this code -- both ObjC and ObjC++ define START_HDRS. Whichever is included last, wins; if they are out of sync, then something could break. This possibility is eliminated by this series. * Make-lang.in

[PATCH v3 08/18] convert the ObjC++ front end to automatic dependencies

2013-08-20 Thread Tom Tromey
This converts the ObjC++ front end. Now we can finally remove the *_H macros from cp/Make-lang.in. * Make-lang.in (CXX_TREE_H, CXX_PARSER_H, CXX_PRETTY_PRINT_H): Remove. * Make-lang.in (START_HDRS, cc1objplus-checksum.o) (objcp/objcp-lang.o, objcp/objcp-decl.o

[PATCH v3 10/18] Fix up c-family targets

2013-08-20 Thread Tom Tromey
This removes manual dependencies for the c-family .o files. * Makefile.in (c-family/cppspec.o, c-family/c-common.o) (c-family/c-cppbuiltin.o, c-family/c-dump.o, c-family/c-format.o) (c-family/c-gimplify.o, c-family/c-lex.o, c-family/c-omp.o) (c-family/c-opts.o,

[PATCH v3 12/18] convert the Go front end to automatic dependencies

2013-08-20 Thread Tom Tromey
This converts Go. It renames gospec.o to go/gospec.o, for uniformity and so we can remove an explicit rule. It defines go_OBJS, to conform to the documented Make-lang.in conventions, and to ensure that Go objects are given the correct order-only dependencies on generated files. *

[PATCH v3 13/18] convert LTO to automatic dependencies

2013-08-20 Thread Tom Tromey
This converts LTO. This also fixes a latent buglet in lto/Make-lang.in. lto_OBJS should hold all the objects for a language, but LTO never defined this. * Make-lang.in (LTO_H, LINKER_PLUGIN_API_H, LTO_TREE_H) (lto/lto-lang.o, lto/lto.o, lto/lto-partition.o)

[PATCH v3 11/18] convert the Fortran front end to automatic dependencies

2013-08-20 Thread Tom Tromey
This convert fortran. It renames gfortranspec.o to fortran/gfortranspec.o, for uniformity and to allow removing an explicit rule. * Make-lang.in (fortran_OBJS): Use fortran/gfortranspec.o. (gfortranspec.o): Remove. (CFLAGS-fortran/gfortranspec.o): New variable.

[PATCH v3 16/18] remove last reference to TREE_GIMPLE_H

2013-08-20 Thread Tom Tromey
There is a single reference to TREE_GIMPLE_H in the source tree. Since it is not defined anywhere, we might as well remove the use. * config/i386/t-i386 (i386.o): Don't use TREE_GIMPLE_H. --- gcc/config/i386/t-i386 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v3 17/18] remove last definition of CROSS_FLOAT_H

2013-08-20 Thread Tom Tromey
There is a single definition of CROSS_FLOAT_H in the source. As far as I can tell, this is never used anywhere. So, this patch removes it. * config/mcore/t-mcore (CROSS_FLOAT_H): Remove. --- gcc/config/mcore/t-mcore | 3 --- 1 file changed, 3 deletions(-) diff --git

[PATCH v3 15/18] make out_object_file rule use automatic dependencies

2013-08-20 Thread Tom Tromey
This is a small change to make out_object_file use automatic dependencies. * Makefile.in ($(out_object_file)): Use COMPILE and POSTCOMPILE. --- gcc/Makefile.in | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/gcc/Makefile.in b/gcc/Makefile.in index

[PATCH v3 18/18] remove unused macros

2013-08-20 Thread Tom Tromey
I used this perl script to find unused _H macros in the Makefile. I deleted the definitions it reported and re-ran the script, until there was no more output. The script also makes note of _H variables which are used but never defined. That is how I found the TREE_GIMPLE_H use, fixed earlier in

Re: [patch] [python libstdc++ printers] Fix gdb/15195

2013-08-16 Thread Tom Tromey
Phil == Phil Muldoon pmuld...@redhat.com writes: Phil Anyway, I have regenerated the patch with the fixes requested. Thanks. Phil 2013-08-16 Phil Muldoon pmuld...@redhat.com Phil PR gcc/53477 I think this should say PR libstdc++/53477 Other than this nit, it looks good to me. I can't

Re: [PATCH v2 00/18] resurrect automatic dependencies

2013-08-12 Thread Tom Tromey
Tom == Tom Tromey tro...@redhat.com writes: Tom This is a refresh of my series to resurrect automatic dependency Tom tracking. Ping. Tom

Re: GDB hooks for debugging GCC

2013-08-05 Thread Tom Tromey
David == David Malcolm dmalc...@redhat.com writes: David GDB 7.0 onwards supports hooks written in Python to improve the David quality-of-life within the debugger. The best known are the David pretty-printing hooks [1], which we already use within libstdc++ for David printing better

[PATCH v2 01/18] clean up SHLIB so subshells are not needed

2013-07-29 Thread Tom Tromey
This changes the handling of SHLIB so that it is inlined into DRIVER_DEFINES. This is ok because SHLIB is defined in a Makefile fragment that is included by the generated Makefile. The rationale for this is that it simplifies some .o targets, so that we can share more code. *

[PATCH v2 00/18] resurrect automatic dependencies

2013-07-29 Thread Tom Tromey
This is a refresh of my series to resurrect automatic dependency tracking. v1 was posted here: http://gcc.gnu.org/ml/gcc-patches/2013-07/msg01218.html Compared to v1: * I ran the hanging test using GNU make 3.81 on each revision, as detailed:

[PATCH v2 02/18] update generated_files

2013-07-29 Thread Tom Tromey
A few generated files were not mentioned in the generated_files variable. This fixes the problem. * Makefile.in (generated_files): Add options.h, target-hooks-def.h, insn-opinit.h, common/common-target-hooks-def.h. --- gcc/Makefile.in | 4 +++- 1 file changed, 3

[PATCH v2 06/18] convert the C++ front end to automatic dependencies

2013-07-29 Thread Tom Tromey
This converts the C++ front end. This renames g++spec.o to cp/g++spec.o for uniformity. This lets us remove an explicit rule. This patch does not remove various *_H macros from cp/Make-lang.in. These are still needed by ObjC++. They're removed by a later patch. * Make-lang.in

[PATCH v2 03/18] move generated_files order-only dependency later

2013-07-29 Thread Tom Tromey
There is an order-only dependency in gcc/Makefile.in that tries to build the generated files before compiling regular objects. However, this appears too early, and so at the time it is seen by make, GCOV_OBJS and GCOV_DUMP_OBJS have not yet been set. This patch fixes the problem and prevents any

[PATCH v2 05/18] convert the C front end to automatic dependencies

2013-07-29 Thread Tom Tromey
This converts the C front end. Note that this fixes a latent bug in gcc/Makefile.in's definition of C_TREE_H. This is needed to avoid breaking this build with this patch. * Makefile.in (C_TREE_H): Reference c/c-tree.h. * Make-lang.in (c/gccspec.o): Remove.

[PATCH v2 07/18] convert the ObjC front end to automatic dependencies

2013-07-29 Thread Tom Tromey
This converts the ObjC front end. Note that there is a latent possible bug in this code -- both ObjC and ObjC++ define START_HDRS. Whichever is included last, wins; if they are out of sync, then something could break. This possibility is eliminated by this series. * Make-lang.in

[PATCH v2 08/18] convert the ObjC++ front end to automatic dependencies

2013-07-29 Thread Tom Tromey
This converts the ObjC++ front end. Now we can finally remove the *_H macros from cp/Make-lang.in. * Make-lang.in (CXX_TREE_H, CXX_PARSER_H, CXX_PRETTY_PRINT_H): Remove. * Make-lang.in (START_HDRS, cc1objplus-checksum.o) (objcp/objcp-lang.o, objcp/objcp-decl.o

[PATCH v2 09/18] convert the Java front end to automatic dependencies

2013-07-29 Thread Tom Tromey
This converts the Java front end. We also rename jvspec.o to java/jvspec.o, for uniformity; this lets us remove an explicit rule. * Make-lang.in (jvspec.o): Remove. (CFLAGS-java/jvspec.o): New variable. ($(XGCJ)$(exeext), java_OBJS): Use java/jvspec.o

[PATCH v2 10/18] Fix up c-family targets

2013-07-29 Thread Tom Tromey
This removes manual dependencies for the c-family .o files. * Makefile.in (c-family/cppspec.o, c-family/c-common.o) (c-family/c-cppbuiltin.o, c-family/c-dump.o, c-family/c-format.o) (c-family/c-gimplify.o, c-family/c-lex.o, c-family/c-omp.o) (c-family/c-opts.o,

[PATCH v2 11/18] convert the Fortran front end to automatic dependencies

2013-07-29 Thread Tom Tromey
This convert fortran. It renames gfortranspec.o to fortran/gfortranspec.o, for uniformity and to allow removing an explicit rule. * Make-lang.in (fortran_OBJS): Use fortran/gfortranspec.o. (gfortranspec.o): Remove. (CFLAGS-fortran/gfortranspec.o): New variable.

[PATCH v2 17/18] remove last definition of CROSS_FLOAT_H

2013-07-29 Thread Tom Tromey
There is a single definition of CROSS_FLOAT_H in the source. As far as I can tell, this is never used anywhere. So, this patch removes it. * config/mcore/t-mcore (CROSS_FLOAT_H): Remove. --- gcc/config/mcore/t-mcore | 3 --- 1 file changed, 3 deletions(-) diff --git

[PATCH v2 13/18] convert LTO to automatic dependencies

2013-07-29 Thread Tom Tromey
This converts LTO. This also fixes a latent buglet in lto/Make-lang.in. lto_OBJS should hold all the objects for a language, but LTO never defined this. * Make-lang.in (LTO_H, LINKER_PLUGIN_API_H, LTO_TREE_H) (lto/lto-lang.o, lto/lto.o, lto/lto-partition.o)

[PATCH v2 12/18] convert the Go front end to automatic dependencies

2013-07-29 Thread Tom Tromey
This converts Go. It renames gospec.o to go/gospec.o, for uniformity and so we can remove an explicit rule. It defines go_OBJS, to conform to the documented Make-lang.in conventions, and to ensure that Go objects are given the correct order-only dependencies on generated files. *

[PATCH v2 18/18] remove unused macros

2013-07-29 Thread Tom Tromey
I used this perl script to find unused _H macros in the Makefile. I deleted the definitions it reported and re-ran the script, until there was no more output. The script also makes note of _H variables which are used but never defined. That is how I found the TREE_GIMPLE_H use, fixed earlier in

[PATCH v2 16/18] remove last reference to TREE_GIMPLE_H

2013-07-29 Thread Tom Tromey
There is a single reference to TREE_GIMPLE_H in the source tree. Since it is not defined anywhere, we might as well remove the use. * config/i386/t-i386 (i386.o): Don't use TREE_GIMPLE_H. --- gcc/config/i386/t-i386 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH v2 15/18] make out_object_file rule use automatic dependencies

2013-07-29 Thread Tom Tromey
This is a small change to make out_object_file use automatic dependencies. * Makefile.in ($(out_object_file)): Use COMPILE and POSTCOMPILE. --- gcc/Makefile.in | 12 +++- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/gcc/Makefile.in b/gcc/Makefile.in index

[PATCH v2 04/18] add configury

2013-07-29 Thread Tom Tromey
This adds the configury needed for automatic dependency tracking. It also adds some bits to the Makefile so we can begin converting (removing) explicit dependencies. * Makefile.in (CCDEPMODE, DEPDIR, depcomp, COMPILE.base) (COMPILE, POSTCOMPILE): New variables. (.cc.o

Re: [PATCH v2 08/18] convert the ObjC++ front end to automatic dependencies

2013-07-29 Thread Tom Tromey
Mike == Mike Stump mikest...@comcast.net writes: +objcp/objcp-act.o : objc/objc-act.c +objcp/objc-encoding.o : objc/objc-encoding.c +objcp/objc-map.o : objc/objc-map.c Mike Please, no space before :. Ok with that. I'll make the change, but please note that this is how it was before my

Re: [c++-concepts] Merge from trunk

2013-07-27 Thread Tom Tromey
Gaby == Gabriel Dos Reis g...@axiomatics.org writes: Gaby We will have to watch out for Tom's Makefile rewrite work. FWIW, I've found rebasing this series to be very easy. Due to the way it is split up, only one patch (the big manual dependency removal one) only ever has hard conflicts, and

Re: [PATCH 00/18] resurrect automatic dependency tracking

2013-07-26 Thread Tom Tromey
Joseph == Joseph S Myers jos...@codesourcery.com writes: Joseph We have a reliable reproducer for the bug, at least in the form Joseph in which it appeared with the old patch: Joseph http://gcc.gnu.org/ml/gcc-patches/2008-03/msg01663.html - even Joseph though it was never clear exactly what the

Re: [PATCH 00/18] resurrect automatic dependency tracking

2013-07-26 Thread Tom Tromey
Tom Good idea. I'll dig up make 3.81 and give it a try soon, if not Tom tomorrow, then early next week. I did this today. I downloaded and built GNU make 3.81 and put it in my PATH. Then I did make -j2 builds in the gcc directory of each revision. I also ran touch Makefile.in and then make -j2

[PATCH 03/18] move generated_files order-only dependency later

2013-07-25 Thread Tom Tromey
There is an order-only dependency in gcc/Makefile.in that tries to build the generated files before compiling regular objects. However, this appears too early, and so at the time it is seen by make, GCOV_OBJS and GCOV_DUMP_OBJS have not yet been set. This patch fixes the problem and prevents any

[PATCH 01/18] clean up SHLIB so subshells are not needed

2013-07-25 Thread Tom Tromey
This changes the handling of SHLIB so that it is inlined into DRIVER_DEFINES. This is ok because SHLIB is defined in a Makefile fragment that is included by the generated Makefile. The rationale for this is that it simplifies some .o targets, so that we can share more code. *

[PATCH 02/18] update generated_files

2013-07-25 Thread Tom Tromey
A few generated files were not mentioned in the generated_files variable. This fixes the problem. * Makefile.in (generated_files): Add options.h, target-hooks-def.h, insn-opinit.h, common/common-target-hooks-def.h. --- gcc/Makefile.in | 4 +++- 1 file changed, 3

[PATCH 05/18] convert the C front end to automatic dependencies

2013-07-25 Thread Tom Tromey
This converts the C front end. Note that this fixes a latent bug in gcc/Makefile.in's definition of C_TREE_H. This is needed to avoid breaking this build with this patch. * Makefile.in (C_TREE_H): Reference c/c-tree.h. * Make-lang.in (c/gccspec.o): Remove.

[PATCH 06/18] convert the C++ front end to automatic dependencies

2013-07-25 Thread Tom Tromey
This converts the C++ front end. This renames g++spec.o to cp/g++spec.o for uniformity. This lets us remove an explicit rule. This patch does not remove various *_H macros from cp/Make-lang.in. These are still needed by ObjC++. They're removed by a later patch. * Make-lang.in

[PATCH 08/18] convert the ObjC++ front end to automatic dependencies

2013-07-25 Thread Tom Tromey
This converts the ObjC++ front end. Now we can finally remove the *_H macros from cp/Make-lang.in. * Make-lang.in (CXX_TREE_H, CXX_PARSER_H, CXX_PRETTY_PRINT_H): Remove. * Make-lang.in (START_HDRS, cc1objplus-checksum.o) (objcp/objcp-lang.o, objcp/objcp-decl.o

[PATCH 07/18] convert the ObjC front end to automatic dependencies

2013-07-25 Thread Tom Tromey
This converts the ObjC front end. Note that there is a latent possible bug in this code -- both ObjC and ObjC++ define START_HDRS. Whichever is included last, wins; if they are out of sync, then something could break. This possibility is eliminated by this series. * Make-lang.in

[PATCH 04/18] add configury

2013-07-25 Thread Tom Tromey
This adds the configury needed for automatic dependency tracking. It also adds some bits to the Makefile so we can begin converting (removing) explicit dependencies. * Makefile.in (CCDEPMODE, DEPDIR, depcomp, COMPILE.base) (COMPILE): New variables. (.cc.o .c.o): Use

[PATCH 09/18] convert the Java front end to automatic dependencies

2013-07-25 Thread Tom Tromey
This converts the Java front end. We also rename jvspec.o to java/jvspec.o, for uniformity; this lets us remove an explicit rule. * Make-lang.in (jvspec.o): Remove. (CFLAGS-java/jvspec.o): New variable. ($(XGCJ)$(exeext), java_OBJS): Use java/jvspec.o

[PATCH 10/18] Fix up c-family targets

2013-07-25 Thread Tom Tromey
This removes manual dependencies for the c-family .o files. * Makefile.in (c-family/cppspec.o, c-family/c-common.o) (c-family/c-cppbuiltin.o, c-family/c-dump.o, c-family/c-format.o) (c-family/c-gimplify.o, c-family/c-lex.o, c-family/c-omp.o) (c-family/c-opts.o,

[PATCH 11/18] convert the Fortran front end to automatic dependencies

2013-07-25 Thread Tom Tromey
This convert fortran. It renames gfortranspec.o to fortran/gfortranspec.o, for uniformity and to allow removing an explicit rule. * Make-lang.in (fortran_OBJS): Use fortran/gfortranspec.o. (gfortranspec.o): Remove. (CFLAGS-fortran/gfortranspec.o): New variable.

[PATCH 16/18] remove last reference to TREE_GIMPLE_H

2013-07-25 Thread Tom Tromey
There is a single reference to TREE_GIMPLE_H in the source tree. Since it is not defined anywhere, we might as well remove the use. * config/i386/t-i386 (i386.o): Don't use TREE_GIMPLE_H. --- gcc/config/i386/t-i386 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

[PATCH 12/18] convert the Go front end to automatic dependencies

2013-07-25 Thread Tom Tromey
This converts Go. It renames gospec.o to go/gospec.o, for uniformity and so we can remove an explicit rule. It defines go_OBJS, to conform to the documented Make-lang.in conventions, and to ensure that Go objects are given the correct order-only dependencies on generated files. *

[PATCH 15/18] make out_object_file rule use automatic dependencies

2013-07-25 Thread Tom Tromey
This is a small change to make out_object_file use automatic dependencies. * Makefile.in ($(out_object_file)): Use COMPILE. --- gcc/Makefile.in | 11 ++- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/gcc/Makefile.in b/gcc/Makefile.in index 2da93021..b689442 100644

[PATCH 13/18] convert LTO to automatic dependencies

2013-07-25 Thread Tom Tromey
This converts LTO. This also fixes a latent buglet in lto/Make-lang.in. lto_OBJS should hold all the objects for a language, but LTO never defined this. * Make-lang.in (LTO_H, LINKER_PLUGIN_API_H, LTO_TREE_H) (lto/lto-lang.o, lto/lto.o, lto/lto-partition.o)

[PATCH 18/18] remove unused macros

2013-07-25 Thread Tom Tromey
I used this perl script to find unused _H macros in the Makefile. I deleted the definitions it reported and re-ran the script, until there was no more output. The script also makes note of _H variables which are used but never defined. That is how I found the TREE_GIMPLE_H use, fixed earlier in

[PATCH 17/18] remove last definition of CROSS_FLOAT_H

2013-07-25 Thread Tom Tromey
There is a single definition of CROSS_FLOAT_H in the source. As far as I can tell, this is never used anywhere. So, this patch removes it. * config/mcore/t-mcore (CROSS_FLOAT_H): Remove. --- gcc/config/mcore/t-mcore | 3 --- 1 file changed, 3 deletions(-) diff --git

[PATCH 00/18] resurrect automatic dependency tracking

2013-07-25 Thread Tom Tromey
Somehow patch #0 of my series didn't go out. So, I'm sending it separately. Sorry about that. I don't know what happened, so I wouldn't be totally surprised if it made its way to the list some day :) This patch series resurrects my automatic dependency tracking patch from eons ago. That

Re: [PATCH 00/18] resurrect automatic dependency tracking

2013-07-25 Thread Tom Tromey
Joseph Thus, it would seem appropriate, for each of the 18 successive Joseph states after an initial subsequence of the patches is applied, Joseph to verify that, if you build with make 3.81, touch Makefile.in Joseph and then do make -j2 cc1, it does not produce the hang Joseph previously

Re: RFA: implement C11 _Generic

2013-07-23 Thread Tom Tromey
Joseph == Joseph S Myers jos...@codesourcery.com writes: Joseph I have now revised this patch from a year ago in line with my Joseph understanding of how _Generic ought to handle the various special Joseph cases Thanks for doing this. Joseph + /* The association's type, or NULL_TREE for

Re: [patch] [python libstdc++ printers] Fix gdb/15195

2013-07-23 Thread Tom Tromey
Phil == Phil Muldoon pmuld...@redhat.com writes: Phil On 03/07/13 08:33, Phil Muldoon wrote: This new patch replaces and obsoletes the previous. On further inspection of some other pretty printer related bugs, it seems that all of the printers need to fetch the referenced value where the

Re: [PATCH] Make all usage of input_location be explicit

2013-07-02 Thread Tom Tromey
David == David Malcolm dmalc...@redhat.com writes: David gcc/java/ David 2013-07-02 David Malcolm dmalc...@redhat.com David * class.c (maybe_layout_super_class): Update comment. David * decl.c (java_add_stmt): Remove use of input_filename macro. David * jcf-parse.c (set_source_filename):

Re: [PATCH, libcpp] Do not decrease highest_location if the included file has be included twice.

2013-06-24 Thread Tom Tromey
Dehao == Dehao Chen de...@google.com writes: Dehao Thanks for helping update the patch. The new patch passed all Dehao regression test and can fix the problem in my huge source file. I Dehao added ChangeLog entry to the patch. Could any libcpp maintainers help Dehao check if it is ok for trunk?

Re: [patch] [python libstdc++ printers] Fix gdb/15195

2013-06-13 Thread Tom Tromey
Phil == Phil Muldoon pmuld...@redhat.com writes: Phil Attached is an updated patch correcting the issues that you pointed Phil out. The patch itself looks fine to me, but I don't think I can approve it. Tom

RFA: fix libstdc++ pretty-printer test

2013-06-05 Thread Tom Tromey
Tom Tromey tro...@redhat.com * testsuite/libstdc++-prettyprinters/shared_ptr.cc: Use -O0. diff --git a/libstdc++-v3/testsuite/libstdc++-prettyprinters/shared_ptr.cc b/libstdc++-v3/testsuite/libstdc++-prettyprinters/shared_ptr.cc index 547213f..0e4e6c2 100644 --- a/libstdc++-v3/testsuite

Re: better gdb support for debugging gcc

2013-06-05 Thread Tom Tromey
Mike == Mike Stump mikest...@comcast.net writes: Mike Here is a little one to add support for __FUNCTION__ to gdb so that we Mike can debug gcc… Arguably this should be done in gdb. Could you file a bug for that? Tom

Re: *ping* Re: [PATCH, PR preprocessor/42014] Added LAST_SOURCE_COLUMN in while loop

2013-05-22 Thread Tom Tromey
Shakthi == Shakthi Kannan skan...@redhat.com writes: Shakthi Is the following patch okay for trunk? I still think it needs a test case. I also don't recall -- did you check to see if the column number that is emitted is actually correct? You may want to change the Subject line of your note, as

Re: RFC: PATCH to avoid linking multiple front ends at once with parallel make

2013-05-14 Thread Tom Tromey
Alexandre == Alexandre Oliva aol...@redhat.com writes: Alexandre However, rather than implementing the locking in Makefiles, Alexandre I'm thinking it might be wiser to do so in a script that Alexandre takes the lock name and the command to run while holding the Alexandre lock. It seems to me

Re: [PATCH,RFC] Make libbacktrace more standalone

2013-05-13 Thread Tom Tromey
Alexander == Alexander Monakov amona...@ispras.ru writes: Alexander In GCC's own dwarf2.h as enum tags; dwarf.h uses anonymous enums. I'm curious to know which dwarf2.h you are testing against and/or intend to support. I think there is more than one. Tom

Re: [PATCH, PR preprocessor/42014] Added LAST_SOURCE_COLUMN in while loop

2013-05-10 Thread Tom Tromey
Shakthi == Shakthi Kannan skan...@redhat.com writes: Shakthi 2013-05-10 Shakthi Kannan skan...@redhat.com Shakthi PR preprocessor/42014 Shakthi * gcc/diagnostic.c: Added LAST_SOURCE_COLUMN in while loop. You should mention the function name in there. See the GNU Coding Standards for the

Re: [PATCH] Fix array sizes created by Java FE (PR libgcj/57074)

2013-05-06 Thread Tom Tromey
Jakub == Jakub Jelinek ja...@redhat.com writes: Jakub 2013-05-04 Jakub Jelinek ja...@redhat.com Jakub PR libgcj/57074 Jakub * class.c (emit_symbol_table): Use array type of the Jakub right size for the_syms_decl and its DECL_INITIAL, instead Jakub of symbols_array_type. Set TREE_TYPE

Re: RFC: add some static probes to libstdc++

2013-04-09 Thread Tom Tromey
tested the failing path by using an old sdt.h. This required a number of iterations to ensure that the test failed for the right reasons. Ok? I think it should go on the 4.8 branch as well. Tom 2013-04-09 Tom Tromey tro...@redhat.com * configure, config.h.in: Rebuild

Re: RFC: add some static probes to libstdc++

2013-04-08 Thread Tom Tromey
Jonathan == Jonathan Wakely jwakely@gmail.com writes: Jonathan On 2 April 2013 16:39, Marc Glisse wrote: On Tue, 2 Apr 2013, Jonathan Wakely wrote: Should we update the prerequisites documentation to say that if Systemtap is installed it needs to be at least version X? I thought you

Re: [patch] Unified debug dump function names.

2013-03-25 Thread Tom Tromey
Lawrence == Lawrence Crowl cr...@googlers.com writes: Lawrence This patch is somewhat different from the original plan at Lawrence gcc.gnu.org/wiki/cxx-conversion/debugging-dumps. The reason Lawrence is that gdb has an incomplete implementation of C++ call syntax; Lawrence requiring explicit

Re: [patch] Unified debug dump function names.

2013-03-25 Thread Tom Tromey
Lawrence == Lawrence Crowl cr...@googlers.com writes: Lawrence My model is that I should be able to cut and paste an expression Lawrence from the source to the debugger and have it work. I concede that Lawrence C++ function overload resolution is a hard problem. However, gdb Lawrence has a

Re: [patch] [python libstdc++ printers] Fix gdb/15195

2013-03-21 Thread Tom Tromey
Phil == Phil Muldoon pmuld...@redhat.com writes: Phil 2013-03-21 Phil Muldoon pmuld...@redhat.com Phil PR gdb/15195 I think this should use a full URL. Otherwise it is going to attach the commit to some random GCC bug. Phil * python/libstdcxx/v6/printers.py (StdTuplePrinter): Convert

Re: RFC: add some static probes to libstdc++

2013-03-14 Thread Tom Tromey
Tom 2013-02-27 Tom Tromey tro...@redhat.com Tom* libsupc++/unwind-cxx.h: Include sys/sdt.h if detected. Tom(PROBE2): New macro. Tom* libsupc++/eh_throw.cc (__cxa_throw, __cxa_rethrow): Add probe. Tom* libsupc++/eh_catch.cc (__cxa_begin_catch): Add probe. Tom* configure.ac

Re: [PATCH] Fix libcpp memory leak (PR middle-end/56461)

2013-03-06 Thread Tom Tromey
Jakub == Jakub Jelinek ja...@redhat.com writes: Jakub 2013-02-28 Jakub Jelinek ja...@redhat.com Jakub PR middle-end/56461 Jakub * internal.h (struct cpp_buffer): Add to_free field. Jakub (_cpp_pop_file_buffer): Add third argument. Jakub * files.c (_cpp_stack_file): Set buffer-to_free.

Re: [PATCH] Fix libcpp PCH related memory leaks (PR middle-end/56461)

2013-03-01 Thread Tom Tromey
Jakub == Jakub Jelinek ja...@redhat.com writes: Jakub 2013-03-01 Jakub Jelinek ja...@redhat.com Jakub PR middle-end/56461 Jakub * files.c (_cpp_save_file_entries): Free result at the end. Jakub * pch.c (cpp_string_free): New function. Jakub (cpp_save_state): Use it in htab_create call.

Re: RFC: add some static probes to libstdc++

2013-02-28 Thread Tom Tromey
Dave == Dave Korn dave.korn.cyg...@gmail.com writes: Dave How did it build in the without sys/sdt.h case? Dave Without HAVE_SYS_SDT_H, there's no definition of PROBE2 at all, Dave but you use it unconditionally in eh_{catch,throw}.cc. Am I Dave missing something? Ugh. I must have made some

Re: RFC: add some static probes to libstdc++

2013-02-28 Thread Tom Tromey
to make clean each time. On the plus side, while digging around after being confused by this, I found that I didn't actually need ../config.h -- the code now checks _GLIBCXX_HAVE_SYS_SDT_H instead. Tom 2013-02-27 Tom Tromey tro...@redhat.com * libsupc++/unwind-cxx.h: Include sys/sdt.h

Re: [PATCH] Fix up _cpp_find_file

2013-02-28 Thread Tom Tromey
Jakub == Jakub Jelinek ja...@redhat.com writes: Jakub 2013-02-27 Jakub Jelinek ja...@redhat.com Jakub * files.c (_cpp_find_file): If returning early, before storing Jakub something to *hash_slot and *hash_slot is NULL, call htab_clear_slot Jakub on it. Access *hash_slot using void * type

RFC: add some static probes to libstdc++

2013-02-27 Thread Tom Tromey
the corresponding gdb patches for submission. I've built it both with and without sys/sdt.h, but I haven't yet run the test suite. Tom 2013-02-27 Tom Tromey tro...@redhat.com * libsupc++/unwind-cxx.h: Include ../config.h, sys/sdt.h. (PROBE2): New macro. * libsupc++/eh_throw.cc

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-14 Thread Tom Tromey
Dave == John David Anglin dave.ang...@bell.net writes: Dave I had looked at this a bit in the past. I don't think it's that Dave difficult to add DWARF2 support Dave to GCC on hppa. Although we don't support named sections, we can Dave create named subspaces Dave for the dwarf info. More of

Re: RFA: Update copyright for libcpp

2013-01-14 Thread Tom Tromey
Richard == Richard Sandiford rdsandif...@googlemail.com writes: Richard Here's the result of running the copyright script on libcpp/. Richard It's nice and short, but for consistency with the other patches, Richard I've not listed out every file in the changelog. Richard Tested on

Re: extern C fixes for sunCC

2013-01-03 Thread Tom Tromey
Marc == Marc Glisse marc.gli...@inria.fr writes: Marc libcpp/ Marc * line-map.c (get_combined_adhoc_loc): Cast to extern C type. Yucky. Marc line_map_realloc reallocator Marc - = set-reallocator ? set-reallocator : xrealloc; Marc + = set-reallocator ? set-reallocator Marc +

Re: extern C fixes for sunCC

2013-01-03 Thread Tom Tromey
Marc == Marc Glisse marc.gli...@inria.fr writes: Marc Do you prefer an other approach? No, this is fine, just yucky. Marc Like this? (I'll test if approved, but I am not sure what was wrong Marc with the indentation and parentheses so it may be wrong again) From the node 'Formatting' in the

Re: [PATCH] Allocate extra 16 bytes for -fsanitize=address

2012-12-03 Thread Tom Tromey
Jakub == Jakub Jelinek ja...@redhat.com writes: Jakub 2012-11-23 Jakub Jelinek ja...@redhat.com Jakub PR bootstrap/55380 Jakub * files.c (read_file_guts): Allocate extra 16 bytes instead of Jakub 1 byte at the end of buf. Pass size + 16 instead of size Jakub to _cpp_convert_input. Jakub

Re: [PATCH] Instance information in DWARF discriminators

2012-11-29 Thread Tom Tromey
Thomas == Thomas Quinot qui...@adacore.com writes: Thomas As for the size impact on line_map_ordinary, indeed that's an additional Thomas int here, but I'm not sure we allocate so many objects of that kind that Thomas the increase is significant. Could you please measure it? My guess is that it

<    1   2   3   4   5   6   >