* Reorganise machmode.h headers @ 2016-11-16 16:32 Richard Sandiford 2016-12-20 0:43 ` Jeff Law 0 siblings, 1 reply; 7+ messages in thread From: Richard Sandiford @ 2016-11-16 16:32 UTC (permalink / raw) To: gcc-patches Later patches will make machmode.h rely on wide-int.h and the new poly-int.h, so it needs to appear later in the coretypes.h include list. Previously machmode.h included insn-modes.h, which as well as the main mode enum contains configuration information like MAX_BITSIZE_MODE_ANY_INT. This still needs to come first, since files like wide-int.h depend on the configuration information. Similarly, later patches will make the auto-generated inline mode size functions use poly-int.h, so the patch splits them out into their own header file and includes it after the integer utilities. The patch also makes the generator files include machmode.h via coretypes.h. Previously they did it by more indirect means. Finally, the patch makes wide-int-print.h available via coretypes.h too. There didn't seem to be any reason to force only the print routines to be included directly, and it would be painful to extend that approach to the new polynomial integer classes. Tested on aarch64-linux-gnu and x86_64-linux-gnu. OK to install? Thanks, Richard [ This patch is part of the SVE series posted here: https://gcc.gnu.org/ml/gcc/2016-11/msg00030.html ] gcc/ 2016-11-16 Richard Sandiford <richard.sandiford@arm.com> Alan Hayward <alan.hayward@arm.com> David Sherwood <david.sherwood@arm.com> * Makefile.in (MACHMODE_H): Remove insn-modes.h (CORETYPES_H): New define. (MOSTLYCLEANFILES): Add insn-modes-inline.h. (insn-modes-inline.h, s-modes-inline-h): New rules. (generated_files): Add insn-modes-inline.h. (RTL_BASE_H, TREE_CORE_H): Use CORETYPES_H instead of coretypes.h. (build/gensupport.o, build/print-rtl.o, build/read-md.o): Likewise. (build/read-rtl.o, build/rtl.o, build/vec.o, build/hash-table.o) (build/inchash.o, build/gencondmd.o, build/genattr.o): Likewise. (build/genattr-common.o, build/genattrtab.o, build/genautomata.o) (build/gencheck.o, build/gencodes.o, build/genconditions.o): Likewise. (build/genconfig.o, build/genconstants.o, build/genemit.o): Likewise. (build/genenums.o, build/genextract.o, build/genflags.o): Likewise. (build/gentarget-def.o, build/genmddeps.o, build/genopinit.o) (build/genoutput.o, build/genpeep.o, build/genpreds.o): Likewise. (build/genrecog.o, build/genmddump.o, build/genmatch.o): Likewise. (build/gencfn-macros.o, build/gcov-iov.o): Likewise. * coretypes.h: Include everything up to real.h for generators. Include insn-modes.h first. Include wide-int-print.h after wide-int.h. Include insn-modes-inline.h and then machmode.h. * machmode.h: Don't include insn-modes.h here. * function-tests.c: Remove includes of signop.h, machmode.h, double-int.h and wide-int.h. * rtl.h: Likewise. * gcc-rich-location.c: Remove includes of machmode.h, double-int.h and wide-int.h. * optc-save-gen.awk: Likewise. * gencheck.c (BITS_PER_UNIT): Delete dummy definition. * godump.c: Remove include of wide-int-print.h. * pretty-print.h: Likewise. * wide-int-print.cc: Likewise. * wide-int.cc: Likewise. * hash-map-tests.c: Remove include of signop.h. * hash-set-tests.c: Likewise. * rtl-tests.c: Likewise. * mkconfig.sh: Remove include of machmode.h. * genmodes.c (emit_insn_modes_h): Split emission of inline functions into... (emit_insn_modes_inline_h): ...this new function. Emit the code into an insn-modes-inline.h header file, adding appropriate include guards and end comments. (emit_insn_modes_c_header): Remove include of machmode.h. (emit_min_insn_modes_c_header): Include coretypes.h rather than machmode.h. (main): Handle -i flag and call emit_insn_modes_inline_h when it is passed. diff --git a/gcc/Makefile.in b/gcc/Makefile.in index 7ecd1e4..2daa6a6 100644 --- a/gcc/Makefile.in +++ b/gcc/Makefile.in @@ -896,14 +896,15 @@ COMMON_TARGET_DEF = common/common-target.def target-hooks-macros.h TARGET_H = $(TM_H) target.h $(TARGET_DEF) insn-modes.h insn-codes.h C_TARGET_H = c-family/c-target.h $(C_TARGET_DEF) COMMON_TARGET_H = common/common-target.h $(INPUT_H) $(COMMON_TARGET_DEF) -MACHMODE_H = machmode.h mode-classes.def insn-modes.h +MACHMODE_H = machmode.h mode-classes.def HOOKS_H = hooks.h $(MACHMODE_H) HOSTHOOKS_DEF_H = hosthooks-def.h $(HOOKS_H) LANGHOOKS_DEF_H = langhooks-def.h $(HOOKS_H) TARGET_DEF_H = target-def.h target-hooks-def.h $(HOOKS_H) targhooks.h C_TARGET_DEF_H = c-family/c-target-def.h c-family/c-target-hooks-def.h \ $(TREE_H) $(C_COMMON_H) $(HOOKS_H) common/common-targhooks.h -RTL_BASE_H = coretypes.h rtl.h rtl.def $(MACHMODE_H) reg-notes.def \ +CORETYPES_H = coretypes.h insn-modes.h insn-modes-inline.h +RTL_BASE_H = $(CORETYPES_H) rtl.h rtl.def $(MACHMODE_H) reg-notes.def \ insn-notes.def $(INPUT_H) $(REAL_H) statistics.h $(VEC_H) \ $(FIXED_VALUE_H) alias.h $(HASHTAB_H) FIXED_VALUE_H = fixed-value.h $(MACHMODE_H) double-int.h @@ -914,7 +915,7 @@ BUILTINS_DEF = builtins.def sync-builtins.def omp-builtins.def \ gtm-builtins.def sanitizer.def cilkplus.def cilk-builtins.def INTERNAL_FN_DEF = internal-fn.def INTERNAL_FN_H = internal-fn.h $(INTERNAL_FN_DEF) -TREE_CORE_H = tree-core.h coretypes.h all-tree.def tree.def \ +TREE_CORE_H = tree-core.h $(CORETYPES_H) all-tree.def tree.def \ c-family/c-common.def $(lang_tree_files) $(MACHMODE_H) \ $(BUILTINS_DEF) $(INPUT_H) statistics.h \ $(VEC_H) treestruct.def $(HASHTAB_H) \ @@ -1615,7 +1616,7 @@ MOSTLYCLEANFILES = insn-flags.h insn-config.h insn-codes.h \ insn-attr.h insn-attr-common.h insn-attrtab.c insn-dfatab.c \ insn-latencytab.c insn-opinit.c insn-opinit.h insn-preds.c insn-constants.h \ tm-preds.h tm-constrs.h checksum-options gimple-match.c generic-match.c \ - tree-check.h min-insn-modes.c insn-modes.c insn-modes.h \ + tree-check.h min-insn-modes.c insn-modes.c insn-modes.h insn-modes-inline.h \ genrtl.h gt-*.h gtype-*.h gtype-desc.c gtyp-input.list \ case-cfn-macros.h cfn-operators.pd \ xgcc$(exeext) cpp$(exeext) $(FULL_DRIVER_NAME) \ @@ -2293,6 +2294,7 @@ s-genrtl-h: build/gengenrtl$(build_exeext) insn-modes.c: s-modes; @true insn-modes.h: s-modes-h; @true +insn-modes-inline.h: s-modes-inline-h; @true min-insn-modes.c: s-modes-m; @true s-modes: build/genmodes$(build_exeext) @@ -2305,6 +2307,12 @@ s-modes-h: build/genmodes$(build_exeext) $(SHELL) $(srcdir)/../move-if-change tmp-modes.h insn-modes.h $(STAMP) s-modes-h +s-modes-inline-h: build/genmodes$(build_exeext) + $(RUN_GEN) build/genmodes$(build_exeext) -i > tmp-modes-inline.h + $(SHELL) $(srcdir)/../move-if-change tmp-modes-inline.h \ + insn-modes-inline.h + $(STAMP) s-modes-inline-h + s-modes-m: build/genmodes$(build_exeext) $(RUN_GEN) build/genmodes$(build_exeext) -m > tmp-min-modes.c $(SHELL) $(srcdir)/../move-if-change tmp-min-modes.c min-insn-modes.c @@ -2537,7 +2545,8 @@ s-gtype: build/gengtype$(build_exeext) $(filter-out [%], $(GTFILES)) \ generated_files = config.h tm.h $(TM_P_H) $(TM_H) multilib.h \ $(simple_generated_h) specs.h \ - tree-check.h genrtl.h insn-modes.h tm-preds.h tm-constrs.h \ + tree-check.h genrtl.h insn-modes.h insn-modes-inline.h \ + tm-preds.h tm-constrs.h \ $(ALL_GTFILES_H) gtype-desc.c gtype-desc.h gcov-iov.h \ options.h target-hooks-def.h insn-opinit.h \ common/common-target-hooks-def.h pass-instances.def \ @@ -2564,30 +2573,30 @@ build/version.o: version.c version.h \ # Header dependencies for the programs that generate source code. # These are library modules... build/errors.o : errors.c $(BCONFIG_H) $(SYSTEM_H) errors.h -build/gensupport.o: gensupport.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ - $(GTM_H) $(RTL_BASE_H) $(OBSTACK_H) errors.h $(HASHTAB_H) \ - $(READ_MD_H) $(GENSUPPORT_H) -build/ggc-none.o : ggc-none.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ +build/gensupport.o: gensupport.c $(BCONFIG_H) $(SYSTEM_H) \ + $(CORETYPES_H) $(GTM_H) $(RTL_BASE_H) $(OBSTACK_H) errors.h \ + $(HASHTAB_H) $(READ_MD_H) $(GENSUPPORT_H) +build/ggc-none.o : ggc-none.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ $(GGC_H) build/min-insn-modes.o : min-insn-modes.c $(BCONFIG_H) $(SYSTEM_H) \ $(MACHMODE_H) -build/print-rtl.o: print-rtl.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ +build/print-rtl.o: print-rtl.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ $(GTM_H) $(RTL_BASE_H) -build/read-md.o: read-md.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ +build/read-md.o: read-md.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ $(HASHTAB_H) errors.h $(READ_MD_H) -build/read-rtl.o: read-rtl.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ +build/read-rtl.o: read-rtl.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ $(GTM_H) $(RTL_BASE_H) $(OBSTACK_H) $(HASHTAB_H) $(READ_MD_H) \ $(GENSUPPORT_H) -build/rtl.o: rtl.c $(BCONFIG_H) coretypes.h $(GTM_H) $(SYSTEM_H) \ +build/rtl.o: rtl.c $(BCONFIG_H) $(CORETYPES_H) $(GTM_H) $(SYSTEM_H) \ $(RTL_H) $(GGC_H) errors.h -build/vec.o : vec.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h $(VEC_H) \ - $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) -build/hash-table.o : hash-table.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ - $(HASH_TABLE_H) $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) -build/inchash.o : inchash.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ - $(HASHTAB_H) inchash.h +build/vec.o : vec.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) $(VEC_H) \ + $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) +build/hash-table.o : hash-table.c $(BCONFIG_H) $(SYSTEM_H) \ + $(CORETYPES_H) $(HASH_TABLE_H) $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) +build/inchash.o : inchash.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ + $(HASHTAB_H) inchash.h build/gencondmd.o : build/gencondmd.c $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) insn-constants.h \ + $(CORETYPES_H) $(GTM_H) insn-constants.h \ $(filter-out insn-flags.h, $(RTL_H) $(TM_P_H) $(FUNCTION_H) $(REGS_H) \ $(RECOG_H) output.h $(FLAGS_H) $(RESOURCE_H) toplev.h $(DIAGNOSTIC_CORE_H) reload.h \ $(EXCEPT_H) tm-constrs.h) @@ -2599,39 +2608,39 @@ build/gencondmd.o : \ # ...these are the programs themselves. build/genattr.o : genattr.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) build/genattr-common.o : genattr-common.c $(RTL_BASE_H) $(BCONFIG_H) \ - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) build/genattrtab.o : genattrtab.c $(RTL_BASE_H) $(OBSTACK_H) \ - $(BCONFIG_H) $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(GGC_H) \ + $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(GGC_H) \ $(READ_MD_H) $(GENSUPPORT_H) $(FNMATCH_H) build/genautomata.o : genautomata.c $(RTL_BASE_H) $(OBSTACK_H) \ - $(BCONFIG_H) $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(VEC_H) \ + $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(VEC_H) \ $(HASHTAB_H) $(GENSUPPORT_H) $(FNMATCH_H) build/gencheck.o : gencheck.c all-tree.def $(BCONFIG_H) $(GTM_H) \ - $(SYSTEM_H) coretypes.h tree.def c-family/c-common.def \ + $(SYSTEM_H) $(CORETYPES_H) tree.def c-family/c-common.def \ $(lang_tree_files) gimple.def build/genchecksum.o : genchecksum.c $(BCONFIG_H) $(SYSTEM_H) $(MD5_H) build/gencodes.o : gencodes.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) build/genconditions.o : genconditions.c $(RTL_BASE_H) $(BCONFIG_H) \ - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(HASHTAB_H) $(READ_MD_H) \ - $(GENSUPPORT_H) + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(HASHTAB_H) \ + $(READ_MD_H) $(GENSUPPORT_H) build/genconfig.o : genconfig.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) build/genconstants.o : genconstants.c $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h errors.h $(READ_MD_H) + $(CORETYPES_H) errors.h $(READ_MD_H) build/genemit.o : genemit.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) internal-fn.def + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) internal-fn.def build/genenums.o : genenums.c $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h errors.h $(READ_MD_H) + $(CORETYPES_H) errors.h $(READ_MD_H) build/genextract.o : genextract.c $(RTL_BASE_H) $(BCONFIG_H) \ - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) build/genflags.o : genflags.c $(RTL_BASE_H) $(OBSTACK_H) $(BCONFIG_H) \ - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) build/gentarget-def.o : gentarget-def.c $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) $(RTL_BASE_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) \ - $(HASH_TABLE_H) target-insns.def + $(CORETYPES_H) $(GTM_H) $(RTL_BASE_H) errors.h $(READ_MD_H) \ + $(GENSUPPORT_H) $(HASH_TABLE_H) target-insns.def build/gengenrtl.o : gengenrtl.c $(BCONFIG_H) $(SYSTEM_H) rtl.def # The gengtype generator program is special: Two versions are built. @@ -2667,30 +2676,32 @@ build/gengtype.o: $(BCONFIG_H) CFLAGS-errors.o += -DHOST_GENERATOR_FILE -build/genmddeps.o: genmddeps.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ +build/genmddeps.o: genmddeps.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ errors.h $(READ_MD_H) build/genmodes.o : genmodes.c $(BCONFIG_H) $(SYSTEM_H) errors.h \ $(HASHTAB_H) machmode.def $(extra_modes_file) build/genopinit.o : genopinit.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) optabs.def + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) optabs.def build/genoutput.o : genoutput.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) build/genpeep.o : genpeep.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) toplev.h $(DIAGNOSTIC_CORE_H) + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) toplev.h \ + $(DIAGNOSTIC_CORE_H) build/genpreds.o : genpreds.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) $(OBSTACK_H) + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) $(OBSTACK_H) build/genrecog.o : genrecog.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) \ + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) \ $(HASH_TABLE_H) inchash.h build/genhooks.o : genhooks.c $(TARGET_DEF) $(C_TARGET_DEF) \ $(COMMON_TARGET_DEF) $(BCONFIG_H) $(SYSTEM_H) errors.h build/genmddump.o : genmddump.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) build/genmatch.o : genmatch.c $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h errors.h $(HASH_TABLE_H) hash-map.h $(GGC_H) is-a.h \ + $(CORETYPES_H) errors.h $(HASH_TABLE_H) hash-map.h $(GGC_H) is-a.h \ tree.def builtins.def internal-fn.def build/gencfn-macros.o : gencfn-macros.c $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h errors.h $(HASH_TABLE_H) hash-set.h builtins.def internal-fn.def + $(CORETYPES_H) errors.h $(HASH_TABLE_H) hash-set.h builtins.def \ + internal-fn.def # Compile the programs that generate insn-* from the machine description. # They are compiled with $(COMPILER_FOR_BUILD), and associated libraries, @@ -2788,8 +2799,8 @@ CFLAGS-cppdefault.o += $(PREPROCESSOR_DEFINES) # having an empty command (nothing following the semicolon). # gcov-iov.c is run on the build machine to generate gcov-iov.h from version.c -build/gcov-iov.o: gcov-iov.c $(BCONFIG_H) coretypes.h $(GTM_H) \ - $(SYSTEM_H) coretypes.h $(TM_H) +build/gcov-iov.o: gcov-iov.c $(BCONFIG_H) $(CORETYPES_H) $(GTM_H) \ + $(SYSTEM_H) $(CORETYPES_H) $(TM_H) build/gcov-iov$(build_exeext): build/gcov-iov.o +$(LINKER_FOR_BUILD) $(BUILD_LINKERFLAGS) $(BUILD_LDFLAGS) \ diff --git a/gcc/coretypes.h b/gcc/coretypes.h index 6d78a25..e77c9dc 100644 --- a/gcc/coretypes.h +++ b/gcc/coretypes.h @@ -340,12 +340,17 @@ typedef void (*gt_pointer_operator) (void *, void *); typedef unsigned char uchar; #endif -/* Most host source files will require the following headers. */ -#if !defined (GENERATOR_FILE) && !defined (USED_FOR_TARGET) -#include "machmode.h" +/* Most source files will require the following headers. */ +#if !defined (USED_FOR_TARGET) +#include "insn-modes.h" #include "signop.h" #include "wide-int.h" +#include "wide-int-print.h" +#include "insn-modes-inline.h" +#include "machmode.h" #include "double-int.h" +/* Most host source files will require the following headers. */ +#if !defined (GENERATOR_FILE) #include "real.h" #include "fixed-value.h" #include "hash-table.h" @@ -353,6 +358,7 @@ typedef unsigned char uchar; #include "input.h" #include "is-a.h" #include "memory-block.h" +#endif #endif /* GENERATOR_FILE && !USED_FOR_TARGET */ #endif /* coretypes.h */ diff --git a/gcc/function-tests.c b/gcc/function-tests.c index b0c44cf..c2d5a81 100644 --- a/gcc/function-tests.c +++ b/gcc/function-tests.c @@ -22,7 +22,6 @@ along with GCC; see the file COPYING3. If not see #include "coretypes.h" #include "tm.h" #include "opts.h" -#include "signop.h" #include "hash-set.h" #include "fixed-value.h" #include "alias.h" @@ -38,7 +37,6 @@ along with GCC; see the file COPYING3. If not see #include "vec.h" #include "hashtab.h" #include "hash-set.h" -#include "machmode.h" #include "hard-reg-set.h" #include "input.h" #include "function.h" @@ -56,10 +54,8 @@ along with GCC; see the file COPYING3. If not see #include "gimplify.h" #include "tree-cfg.h" #include "basic-block.h" -#include "double-int.h" #include "alias.h" #include "symtab.h" -#include "wide-int.h" #include "inchash.h" #include "tree.h" #include "fold-const.h" diff --git a/gcc/gcc-rich-location.c b/gcc/gcc-rich-location.c index b8787b4..3408c11 100644 --- a/gcc/gcc-rich-location.c +++ b/gcc/gcc-rich-location.c @@ -23,13 +23,10 @@ along with GCC; see the file COPYING3. If not see #include "tm.h" #include "rtl.h" #include "hash-set.h" -#include "machmode.h" #include "vec.h" -#include "double-int.h" #include "input.h" #include "alias.h" #include "symtab.h" -#include "wide-int.h" #include "inchash.h" #include "tree-core.h" #include "tree.h" diff --git a/gcc/gencheck.c b/gcc/gencheck.c index 23be4a8..5673d18 100644 --- a/gcc/gencheck.c +++ b/gcc/gencheck.c @@ -17,9 +17,6 @@ You should have received a copy of the GNU General Public License along with GCC; see the file COPYING3. If not see <http://www.gnu.org/licenses/>. */ -/* We don't have insn-modes.h, but we include tm.h. */ -#define BITS_PER_UNIT 8 - #include "bconfig.h" #include "system.h" #include "coretypes.h" diff --git a/gcc/genmodes.c b/gcc/genmodes.c index 92ca055..2088939 100644 --- a/gcc/genmodes.c +++ b/gcc/genmodes.c @@ -1204,6 +1204,24 @@ enum machine_mode\n{"); printf ("#define NUM_INT_N_ENTS %d\n", n_int_n_ents); + puts ("\ +\n\ +#endif /* insn-modes.h */"); +} + +static void +emit_insn_modes_inline_h (void) +{ + printf ("/* Generated automatically from machmode.def%s%s\n", + HAVE_EXTRA_MODES ? " and " : "", + EXTRA_MODES_FILE); + + puts ("\ + by genmodes. */\n\ +\n\ +#ifndef GCC_INSN_MODES_INLINE_H\n\ +#define GCC_INSN_MODES_INLINE_H"); + puts ("\n#if !defined (USED_FOR_TARGET) && GCC_VERSION >= 4001\n"); emit_mode_size_inline (); emit_mode_nunits_inline (); @@ -1214,7 +1232,7 @@ enum machine_mode\n{"); puts ("\ \n\ -#endif /* insn-modes.h */"); +#endif /* insn-modes-inline.h */"); } static void @@ -1231,7 +1249,6 @@ emit_insn_modes_c_header (void) #include \"system.h\"\n\ #include \"coretypes.h\"\n\ #include \"tm.h\"\n\ -#include \"machmode.h\"\n\ #include \"real.h\""); } @@ -1247,7 +1264,7 @@ emit_min_insn_modes_c_header (void) \n\ #include \"bconfig.h\"\n\ #include \"system.h\"\n\ -#include \"machmode.h\""); +#include \"coretypes.h\""); } static void @@ -1797,18 +1814,20 @@ emit_min_insn_modes_c (void) int main (int argc, char **argv) { - bool gen_header = false, gen_min = false; + bool gen_header = false, gen_inlines = false, gen_min = false; progname = argv[0]; if (argc == 1) ; else if (argc == 2 && !strcmp (argv[1], "-h")) gen_header = true; + else if (argc == 2 && !strcmp (argv[1], "-i")) + gen_inlines = true; else if (argc == 2 && !strcmp (argv[1], "-m")) gen_min = true; else { - error ("usage: %s [-h|-m] > file", progname); + error ("usage: %s [-h|-i|-m] > file", progname); return FATAL_EXIT_CODE; } @@ -1824,6 +1843,8 @@ main (int argc, char **argv) if (gen_header) emit_insn_modes_h (); + else if (gen_inlines) + emit_insn_modes_inline_h (); else if (gen_min) emit_min_insn_modes_c (); else diff --git a/gcc/godump.c b/gcc/godump.c index 608542c..f3d8710 100644 --- a/gcc/godump.c +++ b/gcc/godump.c @@ -31,7 +31,6 @@ along with GCC; see the file COPYING3. If not see #include "system.h" #include "coretypes.h" #include "tree.h" -#include "wide-int-print.h" #include "diagnostic-core.h" #include "debug.h" #include "stor-layout.h" diff --git a/gcc/hash-map-tests.c b/gcc/hash-map-tests.c index fd1bb95..d06b31a 100644 --- a/gcc/hash-map-tests.c +++ b/gcc/hash-map-tests.c @@ -22,7 +22,6 @@ along with GCC; see the file COPYING3. If not see #include "coretypes.h" #include "tm.h" #include "opts.h" -#include "signop.h" #include "hash-set.h" #include "fixed-value.h" #include "alias.h" diff --git a/gcc/hash-set-tests.c b/gcc/hash-set-tests.c index db408f2..965b409 100644 --- a/gcc/hash-set-tests.c +++ b/gcc/hash-set-tests.c @@ -22,7 +22,6 @@ along with GCC; see the file COPYING3. If not see #include "coretypes.h" #include "tm.h" #include "opts.h" -#include "signop.h" #include "hash-set.h" #include "selftest.h" diff --git a/gcc/machmode.h b/gcc/machmode.h index 3dcadd8..e64c5f6 100644 --- a/gcc/machmode.h +++ b/gcc/machmode.h @@ -20,9 +20,6 @@ along with GCC; see the file COPYING3. If not see #ifndef HAVE_MACHINE_MODES #define HAVE_MACHINE_MODES -/* Make an enum class that gives all the machine modes. */ -#include "insn-modes.h" - /* Get the name of mode MODE as a string. */ extern const char * const mode_name[NUM_MACHINE_MODES]; diff --git a/gcc/mkconfig.sh b/gcc/mkconfig.sh index 67dfac6..a9658f7 100644 --- a/gcc/mkconfig.sh +++ b/gcc/mkconfig.sh @@ -100,9 +100,6 @@ case $output in #if defined IN_GCC && !defined GENERATOR_FILE # include "insn-modes.h" #endif -#if defined IN_GCC && defined GENERATOR_FILE && !defined BITS_PER_UNIT -#include "machmode.h" -#endif EOF ;; esac diff --git a/gcc/optc-save-gen.awk b/gcc/optc-save-gen.awk index 8ce4248..8b534ae 100644 --- a/gcc/optc-save-gen.awk +++ b/gcc/optc-save-gen.awk @@ -41,13 +41,10 @@ print "#include " quote "flags.h" quote print "#include " quote "target.h" quote print "#include " quote "inchash.h" quote print "#include " quote "hash-set.h" quote -print "#include " quote "machmode.h" quote print "#include " quote "vec.h" quote -print "#include " quote "double-int.h" quote print "#include " quote "input.h" quote print "#include " quote "alias.h" quote print "#include " quote "symtab.h" quote -print "#include " quote "wide-int.h" quote print "#include " quote "inchash.h" quote print "#include " quote "tree.h" quote print "#include " quote "fold-const.h" quote diff --git a/gcc/pretty-print.h b/gcc/pretty-print.h index f49f35a..d84e9f4 100644 --- a/gcc/pretty-print.h +++ b/gcc/pretty-print.h @@ -22,7 +22,6 @@ along with GCC; see the file COPYING3. If not see #define GCC_PRETTY_PRINT_H #include "obstack.h" -#include "wide-int-print.h" /* Maximum number of format string arguments. */ #define PP_NL_ARGMAX 30 diff --git a/gcc/rtl-tests.c b/gcc/rtl-tests.c index 228226b..c6cbed8 100644 --- a/gcc/rtl-tests.c +++ b/gcc/rtl-tests.c @@ -22,7 +22,6 @@ along with GCC; see the file COPYING3. If not see #include "coretypes.h" #include "tm.h" #include "opts.h" -#include "signop.h" #include "hash-set.h" #include "fixed-value.h" #include "alias.h" diff --git a/gcc/rtl.h b/gcc/rtl.h index 3a7a007..28bc328 100644 --- a/gcc/rtl.h +++ b/gcc/rtl.h @@ -24,10 +24,6 @@ along with GCC; see the file COPYING3. If not see machmode.h and other files to exist and would not normally have been included by coretypes.h. */ #ifdef GENERATOR_FILE -#include "machmode.h" -#include "signop.h" -#include "wide-int.h" -#include "double-int.h" #include "real.h" #include "fixed-value.h" #include "statistics.h" diff --git a/gcc/wide-int-print.cc b/gcc/wide-int-print.cc index 70f2fb4..40fa488 100644 --- a/gcc/wide-int-print.cc +++ b/gcc/wide-int-print.cc @@ -21,7 +21,6 @@ along with GCC; see the file COPYING3. If not see #include "config.h" #include "system.h" #include "coretypes.h" -#include "wide-int-print.h" /* * public printing routines. diff --git a/gcc/wide-int.cc b/gcc/wide-int.cc index 1a209bb..5856967 100644 --- a/gcc/wide-int.cc +++ b/gcc/wide-int.cc @@ -24,7 +24,6 @@ along with GCC; see the file COPYING3. If not see #include "tm.h" #include "tree.h" #include "selftest.h" -#include "wide-int-print.h" #define HOST_BITS_PER_HALF_WIDE_INT 32 ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Reorganise machmode.h headers 2016-11-16 16:32 Reorganise machmode.h headers Richard Sandiford @ 2016-12-20 0:43 ` Jeff Law 2016-12-22 16:19 ` Richard Sandiford 2017-05-24 15:08 ` Richard Sandiford 0 siblings, 2 replies; 7+ messages in thread From: Jeff Law @ 2016-12-20 0:43 UTC (permalink / raw) To: gcc-patches, richard.sandiford On 11/16/2016 09:32 AM, Richard Sandiford wrote: > Later patches will make machmode.h rely on wide-int.h and the > new poly-int.h, so it needs to appear later in the coretypes.h > include list. > > Previously machmode.h included insn-modes.h, which as well as > the main mode enum contains configuration information like > MAX_BITSIZE_MODE_ANY_INT. This still needs to come first, > since files like wide-int.h depend on the configuration > information. > > Similarly, later patches will make the auto-generated inline > mode size functions use poly-int.h, so the patch splits them > out into their own header file and includes it after the > integer utilities. > > The patch also makes the generator files include machmode.h > via coretypes.h. Previously they did it by more indirect means. > > Finally, the patch makes wide-int-print.h available via coretypes.h > too. There didn't seem to be any reason to force only the print > routines to be included directly, and it would be painful to extend > that approach to the new polynomial integer classes. > > Tested on aarch64-linux-gnu and x86_64-linux-gnu. OK to install? > > Thanks, > Richard > > > [ This patch is part of the SVE series posted here: > https://gcc.gnu.org/ml/gcc/2016-11/msg00030.html ] > > gcc/ > 2016-11-16 Richard Sandiford <richard.sandiford@arm.com> > Alan Hayward <alan.hayward@arm.com> > David Sherwood <david.sherwood@arm.com> > > * Makefile.in (MACHMODE_H): Remove insn-modes.h > (CORETYPES_H): New define. > (MOSTLYCLEANFILES): Add insn-modes-inline.h. > (insn-modes-inline.h, s-modes-inline-h): New rules. > (generated_files): Add insn-modes-inline.h. > (RTL_BASE_H, TREE_CORE_H): Use CORETYPES_H instead of coretypes.h. > (build/gensupport.o, build/print-rtl.o, build/read-md.o): Likewise. > (build/read-rtl.o, build/rtl.o, build/vec.o, build/hash-table.o) > (build/inchash.o, build/gencondmd.o, build/genattr.o): Likewise. > (build/genattr-common.o, build/genattrtab.o, build/genautomata.o) > (build/gencheck.o, build/gencodes.o, build/genconditions.o): Likewise. > (build/genconfig.o, build/genconstants.o, build/genemit.o): Likewise. > (build/genenums.o, build/genextract.o, build/genflags.o): Likewise. > (build/gentarget-def.o, build/genmddeps.o, build/genopinit.o) > (build/genoutput.o, build/genpeep.o, build/genpreds.o): Likewise. > (build/genrecog.o, build/genmddump.o, build/genmatch.o): Likewise. > (build/gencfn-macros.o, build/gcov-iov.o): Likewise. > * coretypes.h: Include everything up to real.h for generators. > Include insn-modes.h first. Include wide-int-print.h after > wide-int.h. Include insn-modes-inline.h and then machmode.h. > * machmode.h: Don't include insn-modes.h here. > * function-tests.c: Remove includes of signop.h, machmode.h, > double-int.h and wide-int.h. > * rtl.h: Likewise. > * gcc-rich-location.c: Remove includes of machmode.h, double-int.h > and wide-int.h. > * optc-save-gen.awk: Likewise. > * gencheck.c (BITS_PER_UNIT): Delete dummy definition. > * godump.c: Remove include of wide-int-print.h. > * pretty-print.h: Likewise. > * wide-int-print.cc: Likewise. > * wide-int.cc: Likewise. > * hash-map-tests.c: Remove include of signop.h. > * hash-set-tests.c: Likewise. > * rtl-tests.c: Likewise. > * mkconfig.sh: Remove include of machmode.h. > * genmodes.c (emit_insn_modes_h): Split emission of inline functions > into... > (emit_insn_modes_inline_h): ...this new function. Emit the code > into an insn-modes-inline.h header file, adding appropriate > include guards and end comments. > (emit_insn_modes_c_header): Remove include of machmode.h. > (emit_min_insn_modes_c_header): Include coretypes.h rather than > machmode.h. > (main): Handle -i flag and call emit_insn_modes_inline_h when > it is passed. So I don't see anything here particularly problematical. My question is whether or not there's anything significant to be gained to moving forward with this kit, assuming the 67 piece kit is not likely to move forward. I do think you'll need some tweaks to the contrib/header-tools which know about the core headers and dependencies. Hopefully what's in there is easy enough to figure out how to twiddle appropriately. Jeff ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Reorganise machmode.h headers 2016-12-20 0:43 ` Jeff Law @ 2016-12-22 16:19 ` Richard Sandiford 2017-05-24 15:08 ` Richard Sandiford 1 sibling, 0 replies; 7+ messages in thread From: Richard Sandiford @ 2016-12-22 16:19 UTC (permalink / raw) To: Jeff Law; +Cc: gcc-patches, richard.sandiford Jeff Law <law@redhat.com> writes: > On 11/16/2016 09:32 AM, Richard Sandiford wrote: >> Later patches will make machmode.h rely on wide-int.h and the >> new poly-int.h, so it needs to appear later in the coretypes.h >> include list. >> >> Previously machmode.h included insn-modes.h, which as well as >> the main mode enum contains configuration information like >> MAX_BITSIZE_MODE_ANY_INT. This still needs to come first, >> since files like wide-int.h depend on the configuration >> information. >> >> Similarly, later patches will make the auto-generated inline >> mode size functions use poly-int.h, so the patch splits them >> out into their own header file and includes it after the >> integer utilities. >> >> The patch also makes the generator files include machmode.h >> via coretypes.h. Previously they did it by more indirect means. >> >> Finally, the patch makes wide-int-print.h available via coretypes.h >> too. There didn't seem to be any reason to force only the print >> routines to be included directly, and it would be painful to extend >> that approach to the new polynomial integer classes. >> >> Tested on aarch64-linux-gnu and x86_64-linux-gnu. OK to install? >> >> Thanks, >> Richard >> >> >> [ This patch is part of the SVE series posted here: >> https://gcc.gnu.org/ml/gcc/2016-11/msg00030.html ] >> >> gcc/ >> 2016-11-16 Richard Sandiford <richard.sandiford@arm.com> >> Alan Hayward <alan.hayward@arm.com> >> David Sherwood <david.sherwood@arm.com> >> >> * Makefile.in (MACHMODE_H): Remove insn-modes.h >> (CORETYPES_H): New define. >> (MOSTLYCLEANFILES): Add insn-modes-inline.h. >> (insn-modes-inline.h, s-modes-inline-h): New rules. >> (generated_files): Add insn-modes-inline.h. >> (RTL_BASE_H, TREE_CORE_H): Use CORETYPES_H instead of coretypes.h. >> (build/gensupport.o, build/print-rtl.o, build/read-md.o): Likewise. >> (build/read-rtl.o, build/rtl.o, build/vec.o, build/hash-table.o) >> (build/inchash.o, build/gencondmd.o, build/genattr.o): Likewise. >> (build/genattr-common.o, build/genattrtab.o, build/genautomata.o) >> (build/gencheck.o, build/gencodes.o, build/genconditions.o): Likewise. >> (build/genconfig.o, build/genconstants.o, build/genemit.o): Likewise. >> (build/genenums.o, build/genextract.o, build/genflags.o): Likewise. >> (build/gentarget-def.o, build/genmddeps.o, build/genopinit.o) >> (build/genoutput.o, build/genpeep.o, build/genpreds.o): Likewise. >> (build/genrecog.o, build/genmddump.o, build/genmatch.o): Likewise. >> (build/gencfn-macros.o, build/gcov-iov.o): Likewise. >> * coretypes.h: Include everything up to real.h for generators. >> Include insn-modes.h first. Include wide-int-print.h after >> wide-int.h. Include insn-modes-inline.h and then machmode.h. >> * machmode.h: Don't include insn-modes.h here. >> * function-tests.c: Remove includes of signop.h, machmode.h, >> double-int.h and wide-int.h. >> * rtl.h: Likewise. >> * gcc-rich-location.c: Remove includes of machmode.h, double-int.h >> and wide-int.h. >> * optc-save-gen.awk: Likewise. >> * gencheck.c (BITS_PER_UNIT): Delete dummy definition. >> * godump.c: Remove include of wide-int-print.h. >> * pretty-print.h: Likewise. >> * wide-int-print.cc: Likewise. >> * wide-int.cc: Likewise. >> * hash-map-tests.c: Remove include of signop.h. >> * hash-set-tests.c: Likewise. >> * rtl-tests.c: Likewise. >> * mkconfig.sh: Remove include of machmode.h. >> * genmodes.c (emit_insn_modes_h): Split emission of inline functions >> into... >> (emit_insn_modes_inline_h): ...this new function. Emit the code >> into an insn-modes-inline.h header file, adding appropriate >> include guards and end comments. >> (emit_insn_modes_c_header): Remove include of machmode.h. >> (emit_min_insn_modes_c_header): Include coretypes.h rather than >> machmode.h. >> (main): Handle -i flag and call emit_insn_modes_inline_h when >> it is passed. > So I don't see anything here particularly problematical. My question is > whether or not there's anything significant to be gained to moving > forward with this kit, assuming the 67 piece kit is not likely to move > forward. Yeah, I agree it's not worth it now that those patches won't go in. > I do think you'll need some tweaks to the contrib/header-tools which > know about the core headers and dependencies. Hopefully what's in there > is easy enough to figure out how to twiddle appropriately. Thanks for the pointer, will make a note to look at that for GCC 8. Richard ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Reorganise machmode.h headers 2016-12-20 0:43 ` Jeff Law 2016-12-22 16:19 ` Richard Sandiford @ 2017-05-24 15:08 ` Richard Sandiford 2017-06-12 6:37 ` Richard Sandiford 2017-06-29 22:39 ` Jeff Law 1 sibling, 2 replies; 7+ messages in thread From: Richard Sandiford @ 2017-05-24 15:08 UTC (permalink / raw) To: Jeff Law; +Cc: gcc-patches Jeff Law <law@redhat.com> writes: > On 11/16/2016 09:32 AM, Richard Sandiford wrote: >> Later patches will make machmode.h rely on wide-int.h and the >> new poly-int.h, so it needs to appear later in the coretypes.h >> include list. >> >> Previously machmode.h included insn-modes.h, which as well as >> the main mode enum contains configuration information like >> MAX_BITSIZE_MODE_ANY_INT. This still needs to come first, >> since files like wide-int.h depend on the configuration >> information. >> >> Similarly, later patches will make the auto-generated inline >> mode size functions use poly-int.h, so the patch splits them >> out into their own header file and includes it after the >> integer utilities. >> >> The patch also makes the generator files include machmode.h >> via coretypes.h. Previously they did it by more indirect means. >> >> Finally, the patch makes wide-int-print.h available via coretypes.h >> too. There didn't seem to be any reason to force only the print >> routines to be included directly, and it would be painful to extend >> that approach to the new polynomial integer classes. >> >> Tested on aarch64-linux-gnu and x86_64-linux-gnu. OK to install? >> >> Thanks, >> Richard >> >> >> [ This patch is part of the SVE series posted here: >> https://gcc.gnu.org/ml/gcc/2016-11/msg00030.html ] >> >> gcc/ >> 2016-11-16 Richard Sandiford <richard.sandiford@arm.com> >> Alan Hayward <alan.hayward@arm.com> >> David Sherwood <david.sherwood@arm.com> >> >> * Makefile.in (MACHMODE_H): Remove insn-modes.h >> (CORETYPES_H): New define. >> (MOSTLYCLEANFILES): Add insn-modes-inline.h. >> (insn-modes-inline.h, s-modes-inline-h): New rules. >> (generated_files): Add insn-modes-inline.h. >> (RTL_BASE_H, TREE_CORE_H): Use CORETYPES_H instead of coretypes.h. >> (build/gensupport.o, build/print-rtl.o, build/read-md.o): Likewise. >> (build/read-rtl.o, build/rtl.o, build/vec.o, build/hash-table.o) >> (build/inchash.o, build/gencondmd.o, build/genattr.o): Likewise. >> (build/genattr-common.o, build/genattrtab.o, build/genautomata.o) >> (build/gencheck.o, build/gencodes.o, build/genconditions.o): Likewise. >> (build/genconfig.o, build/genconstants.o, build/genemit.o): Likewise. >> (build/genenums.o, build/genextract.o, build/genflags.o): Likewise. >> (build/gentarget-def.o, build/genmddeps.o, build/genopinit.o) >> (build/genoutput.o, build/genpeep.o, build/genpreds.o): Likewise. >> (build/genrecog.o, build/genmddump.o, build/genmatch.o): Likewise. >> (build/gencfn-macros.o, build/gcov-iov.o): Likewise. >> * coretypes.h: Include everything up to real.h for generators. >> Include insn-modes.h first. Include wide-int-print.h after >> wide-int.h. Include insn-modes-inline.h and then machmode.h. >> * machmode.h: Don't include insn-modes.h here. >> * function-tests.c: Remove includes of signop.h, machmode.h, >> double-int.h and wide-int.h. >> * rtl.h: Likewise. >> * gcc-rich-location.c: Remove includes of machmode.h, double-int.h >> and wide-int.h. >> * optc-save-gen.awk: Likewise. >> * gencheck.c (BITS_PER_UNIT): Delete dummy definition. >> * godump.c: Remove include of wide-int-print.h. >> * pretty-print.h: Likewise. >> * wide-int-print.cc: Likewise. >> * wide-int.cc: Likewise. >> * hash-map-tests.c: Remove include of signop.h. >> * hash-set-tests.c: Likewise. >> * rtl-tests.c: Likewise. >> * mkconfig.sh: Remove include of machmode.h. >> * genmodes.c (emit_insn_modes_h): Split emission of inline functions >> into... >> (emit_insn_modes_inline_h): ...this new function. Emit the code >> into an insn-modes-inline.h header file, adding appropriate >> include guards and end comments. >> (emit_insn_modes_c_header): Remove include of machmode.h. >> (emit_min_insn_modes_c_header): Include coretypes.h rather than >> machmode.h. >> (main): Handle -i flag and call emit_insn_modes_inline_h when >> it is passed. > So I don't see anything here particularly problematical. My question is > whether or not there's anything significant to be gained to moving > forward with this kit, assuming the 67 piece kit is not likely to move > forward. > > I do think you'll need some tweaks to the contrib/header-tools which > know about the core headers and dependencies. Hopefully what's in there > is easy enough to figure out how to twiddle appropriately. OK, thanks for the pointer. I think this patch should do that. gcc-order-headers seems to have bitrotted a bit, since it gives: Traceback (most recent call last): File "../contrib/header-tools/gcc-order-headers", line 267, in <module> process_known_dups () File "../contrib/header-tools/gcc-order-headers", line 101, in process_known_dups if dups[i] and "rtl.h" in dups[i]: KeyError: 'dumpfile.h' But the change itself looks obvious. Tested on aarch64-linux-gnu and x86_64-linux-gnu. Also tested by compiling one target for each cpu directory. OK to install? Thanks, Richard [Based on the code ARM contributed in branches/ARM/sve-branch@242100] 2016-11-16 Richard Sandiford <richard.sandiford@linaro.org> Alan Hayward <alan.hayward@arm.com> David Sherwood <david.sherwood@arm.com> contrib/header-tools/ * graph-header-logs (ignore): Update coretypes.h header list. gcc/ * Makefile.in (MACHMODE_H): Remove insn-modes.h (CORETYPES_H): New define. (MOSTLYCLEANFILES): Add insn-modes-inline.h. (insn-modes-inline.h, s-modes-inline-h): New rules. (generated_files): Add insn-modes-inline.h. (RTL_BASE_H, TREE_CORE_H): Use CORETYPES_H instead of coretypes.h. (build/gensupport.o, build/ggc-none.o, build/print-rtl.o): Likewise. (build/read-md.o, build/read-rtl.o, build/rtl.o): Likewise. (build/vec.o, build/hash-table.o, build/inchash.o): Likewise. (build/gencondmd.o, build/genattr.o, build/genattr-common.o): Likewise. (build/genattrtab.o, build/genautomata.o, build/gencheck.o): Likewise. (build/gencodes.o, build/genconditions.o): Likewise. (build/genconfig.o, build/genconstants.o, build/genemit.o): Likewise. (build/genenums.o, build/genextract.o, build/genflags.o): Likewise. (build/gentarget-def.o, build/genmddeps.o, build/genopinit.o) (build/genoutput.o, build/genpeep.o, build/genpreds.o): Likewise. (build/genrecog.o, build/genmddump.o, build/genmatch.o): Likewise. (build/gencfn-macros.o, build/gcov-iov.o): Likewise. * coretypes.h: Include everything up to real.h for generators. Include insn-modes.h first. Include wide-int-print.h after wide-int.h. Include insn-modes-inline.h and then machmode.h. * machmode.h: Don't include insn-modes.h here. * function-tests.c: Remove includes of signop.h, machmode.h, double-int.h and wide-int.h. * rtl.h: Likewise. * gcc-rich-location.c: Remove includes of machmode.h, double-int.h and wide-int.h. * optc-save-gen.awk: Likewise. * gencheck.c (BITS_PER_UNIT): Delete dummy definition. * godump.c: Remove include of wide-int-print.h. * pretty-print.h: Likewise. * wide-int-print.cc: Likewise. * wide-int.cc: Likewise. * hash-map-tests.c: Remove include of signop.h. * hash-set-tests.c: Likewise. * rtl-tests.c: Likewise. * mkconfig.sh: Remove include of machmode.h. * genmodes.c (emit_insn_modes_h): Split emission of inline functions into... (emit_insn_modes_inline_h): ...this new function. Emit the code into an insn-modes-inline.h header file, adding appropriate include guards and end comments. (emit_insn_modes_c_header): Remove include of machmode.h. (emit_min_insn_modes_c_header): Include coretypes.h rather than machmode.h. (main): Handle -i flag and call emit_insn_modes_inline_h when it is passed. Index: contrib/header-tools/graph-header-logs =================================================================== --- contrib/header-tools/graph-header-logs 2015-11-11 15:40:07.000000000 +0000 +++ contrib/header-tools/graph-header-logs 2017-05-24 14:37:44.737223605 +0100 @@ -25,9 +25,12 @@ depstring = ("In file included from", " # indentation indicates nesting levels of included files ignore = [ "coretypes_h", - "machmode_h", + "insn_modes_h", "signop_h", "wide_int_h", + "wide_int_print_h", + "insn_modes_inline_h", + "machmode_h", "double_int_h", "real_h", "fixed_value_h", Index: gcc/Makefile.in =================================================================== --- gcc/Makefile.in 2017-05-23 19:29:59.645724759 +0100 +++ gcc/Makefile.in 2017-05-24 14:37:44.737223605 +0100 @@ -900,14 +900,15 @@ COMMON_TARGET_DEF = common/common-target TARGET_H = $(TM_H) target.h $(TARGET_DEF) insn-modes.h insn-codes.h C_TARGET_H = c-family/c-target.h $(C_TARGET_DEF) COMMON_TARGET_H = common/common-target.h $(INPUT_H) $(COMMON_TARGET_DEF) -MACHMODE_H = machmode.h mode-classes.def insn-modes.h +MACHMODE_H = machmode.h mode-classes.def HOOKS_H = hooks.h $(MACHMODE_H) HOSTHOOKS_DEF_H = hosthooks-def.h $(HOOKS_H) LANGHOOKS_DEF_H = langhooks-def.h $(HOOKS_H) TARGET_DEF_H = target-def.h target-hooks-def.h $(HOOKS_H) targhooks.h C_TARGET_DEF_H = c-family/c-target-def.h c-family/c-target-hooks-def.h \ $(TREE_H) $(C_COMMON_H) $(HOOKS_H) common/common-targhooks.h -RTL_BASE_H = coretypes.h rtl.h rtl.def $(MACHMODE_H) reg-notes.def \ +CORETYPES_H = coretypes.h insn-modes.h insn-modes-inline.h +RTL_BASE_H = $(CORETYPES_H) rtl.h rtl.def $(MACHMODE_H) reg-notes.def \ insn-notes.def $(INPUT_H) $(REAL_H) statistics.h $(VEC_H) \ $(FIXED_VALUE_H) alias.h $(HASHTAB_H) FIXED_VALUE_H = fixed-value.h $(MACHMODE_H) double-int.h @@ -919,7 +920,7 @@ BUILTINS_DEF = builtins.def sync-builtin hsa-builtins.def INTERNAL_FN_DEF = internal-fn.def INTERNAL_FN_H = internal-fn.h $(INTERNAL_FN_DEF) -TREE_CORE_H = tree-core.h coretypes.h all-tree.def tree.def \ +TREE_CORE_H = tree-core.h $(CORETYPES_H) all-tree.def tree.def \ c-family/c-common.def $(lang_tree_files) $(MACHMODE_H) \ $(BUILTINS_DEF) $(INPUT_H) statistics.h \ $(VEC_H) treestruct.def $(HASHTAB_H) \ @@ -1631,7 +1632,7 @@ MOSTLYCLEANFILES = insn-flags.h insn-con insn-attr.h insn-attr-common.h insn-attrtab.c insn-dfatab.c \ insn-latencytab.c insn-opinit.c insn-opinit.h insn-preds.c insn-constants.h \ tm-preds.h tm-constrs.h checksum-options gimple-match.c generic-match.c \ - tree-check.h min-insn-modes.c insn-modes.c insn-modes.h \ + tree-check.h min-insn-modes.c insn-modes.c insn-modes.h insn-modes-inline.h \ genrtl.h gt-*.h gtype-*.h gtype-desc.c gtyp-input.list \ case-cfn-macros.h cfn-operators.pd \ xgcc$(exeext) cpp$(exeext) $(FULL_DRIVER_NAME) \ @@ -2317,6 +2318,7 @@ s-genrtl-h: build/gengenrtl$(build_exeex insn-modes.c: s-modes; @true insn-modes.h: s-modes-h; @true +insn-modes-inline.h: s-modes-inline-h; @true min-insn-modes.c: s-modes-m; @true s-modes: build/genmodes$(build_exeext) @@ -2329,6 +2331,12 @@ s-modes-h: build/genmodes$(build_exeext) $(SHELL) $(srcdir)/../move-if-change tmp-modes.h insn-modes.h $(STAMP) s-modes-h +s-modes-inline-h: build/genmodes$(build_exeext) + $(RUN_GEN) build/genmodes$(build_exeext) -i > tmp-modes-inline.h + $(SHELL) $(srcdir)/../move-if-change tmp-modes-inline.h \ + insn-modes-inline.h + $(STAMP) s-modes-inline-h + s-modes-m: build/genmodes$(build_exeext) $(RUN_GEN) build/genmodes$(build_exeext) -m > tmp-min-modes.c $(SHELL) $(srcdir)/../move-if-change tmp-min-modes.c min-insn-modes.c @@ -2565,7 +2573,8 @@ s-gtype: build/gengtype$(build_exeext) $ generated_files = config.h tm.h $(TM_P_H) $(TM_H) multilib.h \ $(simple_generated_h) specs.h \ - tree-check.h genrtl.h insn-modes.h tm-preds.h tm-constrs.h \ + tree-check.h genrtl.h insn-modes.h insn-modes-inline.h \ + tm-preds.h tm-constrs.h \ $(ALL_GTFILES_H) gtype-desc.c gtype-desc.h gcov-iov.h \ options.h target-hooks-def.h insn-opinit.h \ common/common-target-hooks-def.h pass-instances.def \ @@ -2592,30 +2601,30 @@ build/version.o: version.c version.h \ # Header dependencies for the programs that generate source code. # These are library modules... build/errors.o : errors.c $(BCONFIG_H) $(SYSTEM_H) errors.h -build/gensupport.o: gensupport.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ - $(GTM_H) $(RTL_BASE_H) $(OBSTACK_H) errors.h $(HASHTAB_H) \ - $(READ_MD_H) $(GENSUPPORT_H) -build/ggc-none.o : ggc-none.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ +build/gensupport.o: gensupport.c $(BCONFIG_H) $(SYSTEM_H) \ + $(CORETYPES_H) $(GTM_H) $(RTL_BASE_H) $(OBSTACK_H) errors.h \ + $(HASHTAB_H) $(READ_MD_H) $(GENSUPPORT_H) +build/ggc-none.o : ggc-none.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ $(GGC_H) build/min-insn-modes.o : min-insn-modes.c $(BCONFIG_H) $(SYSTEM_H) \ $(MACHMODE_H) -build/print-rtl.o: print-rtl.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ +build/print-rtl.o: print-rtl.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ $(GTM_H) $(RTL_BASE_H) -build/read-md.o: read-md.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ +build/read-md.o: read-md.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ $(HASHTAB_H) errors.h $(READ_MD_H) -build/read-rtl.o: read-rtl.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ +build/read-rtl.o: read-rtl.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ $(GTM_H) $(RTL_BASE_H) $(OBSTACK_H) $(HASHTAB_H) $(READ_MD_H) \ $(GENSUPPORT_H) -build/rtl.o: rtl.c $(BCONFIG_H) coretypes.h $(GTM_H) $(SYSTEM_H) \ +build/rtl.o: rtl.c $(BCONFIG_H) $(CORETYPES_H) $(GTM_H) $(SYSTEM_H) \ $(RTL_H) $(GGC_H) errors.h -build/vec.o : vec.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h $(VEC_H) \ - $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) -build/hash-table.o : hash-table.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ - $(HASH_TABLE_H) $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) -build/inchash.o : inchash.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ - $(HASHTAB_H) inchash.h +build/vec.o : vec.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) $(VEC_H) \ + $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) +build/hash-table.o : hash-table.c $(BCONFIG_H) $(SYSTEM_H) \ + $(CORETYPES_H) $(HASH_TABLE_H) $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) +build/inchash.o : inchash.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ + $(HASHTAB_H) inchash.h build/gencondmd.o : build/gencondmd.c $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) insn-constants.h \ + $(CORETYPES_H) $(GTM_H) insn-constants.h \ $(filter-out insn-flags.h, $(RTL_H) $(TM_P_H) $(FUNCTION_H) $(REGS_H) \ $(RECOG_H) output.h $(FLAGS_H) $(RESOURCE_H) toplev.h $(DIAGNOSTIC_CORE_H) reload.h \ $(EXCEPT_H) tm-constrs.h) @@ -2627,39 +2636,39 @@ build/gencondmd.o : \ # ...these are the programs themselves. build/genattr.o : genattr.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) build/genattr-common.o : genattr-common.c $(RTL_BASE_H) $(BCONFIG_H) \ - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) build/genattrtab.o : genattrtab.c $(RTL_BASE_H) $(OBSTACK_H) \ - $(BCONFIG_H) $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(GGC_H) \ + $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(GGC_H) \ $(READ_MD_H) $(GENSUPPORT_H) $(FNMATCH_H) build/genautomata.o : genautomata.c $(RTL_BASE_H) $(OBSTACK_H) \ - $(BCONFIG_H) $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(VEC_H) \ + $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(VEC_H) \ $(HASHTAB_H) $(GENSUPPORT_H) $(FNMATCH_H) build/gencheck.o : gencheck.c all-tree.def $(BCONFIG_H) $(GTM_H) \ - $(SYSTEM_H) coretypes.h tree.def c-family/c-common.def \ + $(SYSTEM_H) $(CORETYPES_H) tree.def c-family/c-common.def \ $(lang_tree_files) gimple.def build/genchecksum.o : genchecksum.c $(BCONFIG_H) $(SYSTEM_H) $(MD5_H) build/gencodes.o : gencodes.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) build/genconditions.o : genconditions.c $(RTL_BASE_H) $(BCONFIG_H) \ - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(HASHTAB_H) $(READ_MD_H) \ - $(GENSUPPORT_H) + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(HASHTAB_H) \ + $(READ_MD_H) $(GENSUPPORT_H) build/genconfig.o : genconfig.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) build/genconstants.o : genconstants.c $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h errors.h $(READ_MD_H) + $(CORETYPES_H) errors.h $(READ_MD_H) build/genemit.o : genemit.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) internal-fn.def + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) internal-fn.def build/genenums.o : genenums.c $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h errors.h $(READ_MD_H) + $(CORETYPES_H) errors.h $(READ_MD_H) build/genextract.o : genextract.c $(RTL_BASE_H) $(BCONFIG_H) \ - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) build/genflags.o : genflags.c $(RTL_BASE_H) $(OBSTACK_H) $(BCONFIG_H) \ - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) build/gentarget-def.o : gentarget-def.c $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) $(RTL_BASE_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) \ - $(HASH_TABLE_H) target-insns.def + $(CORETYPES_H) $(GTM_H) $(RTL_BASE_H) errors.h $(READ_MD_H) \ + $(GENSUPPORT_H) $(HASH_TABLE_H) target-insns.def build/gengenrtl.o : gengenrtl.c $(BCONFIG_H) $(SYSTEM_H) rtl.def # The gengtype generator program is special: Two versions are built. @@ -2695,30 +2704,32 @@ build/gengtype.o: $(BCONFIG_H) CFLAGS-errors.o += -DHOST_GENERATOR_FILE -build/genmddeps.o: genmddeps.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ +build/genmddeps.o: genmddeps.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ errors.h $(READ_MD_H) build/genmodes.o : genmodes.c $(BCONFIG_H) $(SYSTEM_H) errors.h \ $(HASHTAB_H) machmode.def $(extra_modes_file) build/genopinit.o : genopinit.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) optabs.def + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) optabs.def build/genoutput.o : genoutput.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) build/genpeep.o : genpeep.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) toplev.h $(DIAGNOSTIC_CORE_H) + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) toplev.h \ + $(DIAGNOSTIC_CORE_H) build/genpreds.o : genpreds.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) $(OBSTACK_H) + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) $(OBSTACK_H) build/genrecog.o : genrecog.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) \ + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) \ $(HASH_TABLE_H) inchash.h build/genhooks.o : genhooks.c $(TARGET_DEF) $(C_TARGET_DEF) \ $(COMMON_TARGET_DEF) $(BCONFIG_H) $(SYSTEM_H) errors.h build/genmddump.o : genmddump.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) build/genmatch.o : genmatch.c $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h errors.h $(HASH_TABLE_H) hash-map.h $(GGC_H) is-a.h \ + $(CORETYPES_H) errors.h $(HASH_TABLE_H) hash-map.h $(GGC_H) is-a.h \ tree.def builtins.def internal-fn.def build/gencfn-macros.o : gencfn-macros.c $(BCONFIG_H) $(SYSTEM_H) \ - coretypes.h errors.h $(HASH_TABLE_H) hash-set.h builtins.def internal-fn.def + $(CORETYPES_H) errors.h $(HASH_TABLE_H) hash-set.h builtins.def \ + internal-fn.def # Compile the programs that generate insn-* from the machine description. # They are compiled with $(COMPILER_FOR_BUILD), and associated libraries, @@ -2816,8 +2827,8 @@ CFLAGS-cppdefault.o += $(PREPROCESSOR_DE # having an empty command (nothing following the semicolon). # gcov-iov.c is run on the build machine to generate gcov-iov.h from version.c -build/gcov-iov.o: gcov-iov.c $(BCONFIG_H) coretypes.h $(GTM_H) \ - $(SYSTEM_H) coretypes.h $(TM_H) +build/gcov-iov.o: gcov-iov.c $(BCONFIG_H) $(CORETYPES_H) $(GTM_H) \ + $(SYSTEM_H) $(CORETYPES_H) $(TM_H) build/gcov-iov$(build_exeext): build/gcov-iov.o +$(LINKER_FOR_BUILD) $(BUILD_LINKERFLAGS) $(BUILD_LDFLAGS) \ Index: gcc/coretypes.h =================================================================== --- gcc/coretypes.h 2017-05-18 07:51:11.876801735 +0100 +++ gcc/coretypes.h 2017-05-24 14:37:44.737223605 +0100 @@ -358,12 +358,17 @@ typedef void (*gt_pointer_operator) (voi typedef unsigned char uchar; #endif -/* Most host source files will require the following headers. */ -#if !defined (GENERATOR_FILE) && !defined (USED_FOR_TARGET) -#include "machmode.h" +/* Most source files will require the following headers. */ +#if !defined (USED_FOR_TARGET) +#include "insn-modes.h" #include "signop.h" #include "wide-int.h" +#include "wide-int-print.h" +#include "insn-modes-inline.h" +#include "machmode.h" #include "double-int.h" +/* Most host source files will require the following headers. */ +#if !defined (GENERATOR_FILE) #include "real.h" #include "fixed-value.h" #include "hash-table.h" @@ -372,6 +377,7 @@ typedef unsigned char uchar; #include "is-a.h" #include "memory-block.h" #include "dumpfile.h" +#endif #endif /* GENERATOR_FILE && !USED_FOR_TARGET */ #endif /* coretypes.h */ Index: gcc/machmode.h =================================================================== --- gcc/machmode.h 2017-02-23 19:54:03.000000000 +0000 +++ gcc/machmode.h 2017-05-24 14:37:44.739230513 +0100 @@ -20,9 +20,6 @@ Software Foundation; either version 3, o #ifndef HAVE_MACHINE_MODES #define HAVE_MACHINE_MODES -/* Make an enum class that gives all the machine modes. */ -#include "insn-modes.h" - /* Get the name of mode MODE as a string. */ extern const char * const mode_name[NUM_MACHINE_MODES]; Index: gcc/function-tests.c =================================================================== --- gcc/function-tests.c 2017-02-23 19:54:03.000000000 +0000 +++ gcc/function-tests.c 2017-05-24 14:37:44.738227059 +0100 @@ -22,7 +22,6 @@ Software Foundation; either version 3, o #include "coretypes.h" #include "tm.h" #include "opts.h" -#include "signop.h" #include "hash-set.h" #include "fixed-value.h" #include "alias.h" @@ -38,7 +37,6 @@ Software Foundation; either version 3, o #include "vec.h" #include "hashtab.h" #include "hash-set.h" -#include "machmode.h" #include "hard-reg-set.h" #include "input.h" #include "function.h" @@ -56,10 +54,8 @@ Software Foundation; either version 3, o #include "gimplify.h" #include "tree-cfg.h" #include "basic-block.h" -#include "double-int.h" #include "alias.h" #include "symtab.h" -#include "wide-int.h" #include "inchash.h" #include "tree.h" #include "fold-const.h" Index: gcc/rtl.h =================================================================== --- gcc/rtl.h 2017-03-28 16:19:28.000000000 +0100 +++ gcc/rtl.h 2017-05-24 14:37:44.739230513 +0100 @@ -24,10 +24,6 @@ #define GCC_RTL_H machmode.h and other files to exist and would not normally have been included by coretypes.h. */ #ifdef GENERATOR_FILE -#include "machmode.h" -#include "signop.h" -#include "wide-int.h" -#include "double-int.h" #include "real.h" #include "fixed-value.h" #include "statistics.h" Index: gcc/gcc-rich-location.c =================================================================== --- gcc/gcc-rich-location.c 2017-02-23 19:54:12.000000000 +0000 +++ gcc/gcc-rich-location.c 2017-05-24 14:37:44.738227059 +0100 @@ -23,13 +23,10 @@ Software Foundation; either version 3, o #include "tm.h" #include "rtl.h" #include "hash-set.h" -#include "machmode.h" #include "vec.h" -#include "double-int.h" #include "input.h" #include "alias.h" #include "symtab.h" -#include "wide-int.h" #include "inchash.h" #include "tree-core.h" #include "tree.h" Index: gcc/optc-save-gen.awk =================================================================== --- gcc/optc-save-gen.awk 2017-02-23 19:54:15.000000000 +0000 +++ gcc/optc-save-gen.awk 2017-05-24 14:37:44.739230513 +0100 @@ -41,13 +41,10 @@ print "#include " quote "flags.h" quote print "#include " quote "target.h" quote print "#include " quote "inchash.h" quote print "#include " quote "hash-set.h" quote -print "#include " quote "machmode.h" quote print "#include " quote "vec.h" quote -print "#include " quote "double-int.h" quote print "#include " quote "input.h" quote print "#include " quote "alias.h" quote print "#include " quote "symtab.h" quote -print "#include " quote "wide-int.h" quote print "#include " quote "inchash.h" quote print "#include " quote "tree.h" quote print "#include " quote "fold-const.h" quote Index: gcc/gencheck.c =================================================================== --- gcc/gencheck.c 2017-02-23 19:54:03.000000000 +0000 +++ gcc/gencheck.c 2017-05-24 14:37:44.738227059 +0100 @@ -17,9 +17,6 @@ Software Foundation; either version 3, o along with GCC; see the file COPYING3. If not see <http://www.gnu.org/licenses/>. */ -/* We don't have insn-modes.h, but we include tm.h. */ -#define BITS_PER_UNIT 8 - #include "bconfig.h" #include "system.h" #include "coretypes.h" Index: gcc/godump.c =================================================================== --- gcc/godump.c 2017-02-23 19:54:03.000000000 +0000 +++ gcc/godump.c 2017-05-24 14:37:44.738227059 +0100 @@ -31,7 +31,6 @@ Software Foundation; either version 3, o #include "system.h" #include "coretypes.h" #include "tree.h" -#include "wide-int-print.h" #include "diagnostic-core.h" #include "debug.h" #include "stor-layout.h" Index: gcc/pretty-print.h =================================================================== --- gcc/pretty-print.h 2017-02-23 19:54:04.000000000 +0000 +++ gcc/pretty-print.h 2017-05-24 14:37:44.739230513 +0100 @@ -22,7 +22,6 @@ Software Foundation; either version 3, o #define GCC_PRETTY_PRINT_H #include "obstack.h" -#include "wide-int-print.h" /* Maximum number of format string arguments. */ #define PP_NL_ARGMAX 30 Index: gcc/wide-int-print.cc =================================================================== --- gcc/wide-int-print.cc 2017-02-23 19:54:03.000000000 +0000 +++ gcc/wide-int-print.cc 2017-05-24 14:37:44.740233967 +0100 @@ -21,7 +21,6 @@ along with GCC; see the file COPYING3. #include "config.h" #include "system.h" #include "coretypes.h" -#include "wide-int-print.h" /* * public printing routines. Index: gcc/wide-int.cc =================================================================== --- gcc/wide-int.cc 2017-02-23 19:54:04.000000000 +0000 +++ gcc/wide-int.cc 2017-05-24 14:37:44.740233967 +0100 @@ -24,7 +24,6 @@ along with GCC; see the file COPYING3. #include "tm.h" #include "tree.h" #include "selftest.h" -#include "wide-int-print.h" #define HOST_BITS_PER_HALF_WIDE_INT 32 Index: gcc/hash-map-tests.c =================================================================== --- gcc/hash-map-tests.c 2017-02-23 19:54:12.000000000 +0000 +++ gcc/hash-map-tests.c 2017-05-24 14:37:44.738227059 +0100 @@ -22,7 +22,6 @@ Software Foundation; either version 3, o #include "coretypes.h" #include "tm.h" #include "opts.h" -#include "signop.h" #include "hash-set.h" #include "fixed-value.h" #include "alias.h" Index: gcc/hash-set-tests.c =================================================================== --- gcc/hash-set-tests.c 2017-02-23 19:54:15.000000000 +0000 +++ gcc/hash-set-tests.c 2017-05-24 14:37:44.739230513 +0100 @@ -22,7 +22,6 @@ Software Foundation; either version 3, o #include "coretypes.h" #include "tm.h" #include "opts.h" -#include "signop.h" #include "hash-set.h" #include "selftest.h" Index: gcc/rtl-tests.c =================================================================== --- gcc/rtl-tests.c 2017-02-23 19:54:15.000000000 +0000 +++ gcc/rtl-tests.c 2017-05-24 14:37:44.739230513 +0100 @@ -22,7 +22,6 @@ Software Foundation; either version 3, o #include "coretypes.h" #include "tm.h" #include "opts.h" -#include "signop.h" #include "hash-set.h" #include "fixed-value.h" #include "alias.h" Index: gcc/mkconfig.sh =================================================================== --- gcc/mkconfig.sh 2017-02-23 19:54:03.000000000 +0000 +++ gcc/mkconfig.sh 2017-05-24 14:37:44.739230513 +0100 @@ -100,9 +100,6 @@ case $output in #if defined IN_GCC && !defined GENERATOR_FILE # include "insn-modes.h" #endif -#if defined IN_GCC && defined GENERATOR_FILE && !defined BITS_PER_UNIT -#include "machmode.h" -#endif EOF ;; esac Index: gcc/genmodes.c =================================================================== --- gcc/genmodes.c 2017-03-28 16:19:28.000000000 +0100 +++ gcc/genmodes.c 2017-05-24 14:37:44.738227059 +0100 @@ -1204,6 +1204,24 @@ enum machine_mode\n{"); printf ("#define NUM_INT_N_ENTS %d\n", n_int_n_ents); + puts ("\ +\n\ +#endif /* insn-modes.h */"); +} + +static void +emit_insn_modes_inline_h (void) +{ + printf ("/* Generated automatically from machmode.def%s%s\n", + HAVE_EXTRA_MODES ? " and " : "", + EXTRA_MODES_FILE); + + puts ("\ + by genmodes. */\n\ +\n\ +#ifndef GCC_INSN_MODES_INLINE_H\n\ +#define GCC_INSN_MODES_INLINE_H"); + puts ("\n#if !defined (USED_FOR_TARGET) && GCC_VERSION >= 4001\n"); emit_mode_size_inline (); emit_mode_nunits_inline (); @@ -1214,7 +1232,7 @@ enum machine_mode\n{"); puts ("\ \n\ -#endif /* insn-modes.h */"); +#endif /* insn-modes-inline.h */"); } static void @@ -1231,7 +1249,6 @@ emit_insn_modes_c_header (void) #include \"system.h\"\n\ #include \"coretypes.h\"\n\ #include \"tm.h\"\n\ -#include \"machmode.h\"\n\ #include \"real.h\""); } @@ -1247,7 +1264,7 @@ emit_min_insn_modes_c_header (void) \n\ #include \"bconfig.h\"\n\ #include \"system.h\"\n\ -#include \"machmode.h\""); +#include \"coretypes.h\""); } static void @@ -1799,18 +1816,20 @@ emit_min_insn_modes_c (void) int main (int argc, char **argv) { - bool gen_header = false, gen_min = false; + bool gen_header = false, gen_inlines = false, gen_min = false; progname = argv[0]; if (argc == 1) ; else if (argc == 2 && !strcmp (argv[1], "-h")) gen_header = true; + else if (argc == 2 && !strcmp (argv[1], "-i")) + gen_inlines = true; else if (argc == 2 && !strcmp (argv[1], "-m")) gen_min = true; else { - error ("usage: %s [-h|-m] > file", progname); + error ("usage: %s [-h|-i|-m] > file", progname); return FATAL_EXIT_CODE; } @@ -1826,6 +1845,8 @@ main (int argc, char **argv) if (gen_header) emit_insn_modes_h (); + else if (gen_inlines) + emit_insn_modes_inline_h (); else if (gen_min) emit_min_insn_modes_c (); else ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Reorganise machmode.h headers 2017-05-24 15:08 ` Richard Sandiford @ 2017-06-12 6:37 ` Richard Sandiford 2017-06-22 11:34 ` Richard Sandiford 2017-06-29 22:39 ` Jeff Law 1 sibling, 1 reply; 7+ messages in thread From: Richard Sandiford @ 2017-06-12 6:37 UTC (permalink / raw) To: Jeff Law; +Cc: gcc-patches Ping Richard Sandiford <richard.sandiford@linaro.org> writes: > Jeff Law <law@redhat.com> writes: >> On 11/16/2016 09:32 AM, Richard Sandiford wrote: >>> Later patches will make machmode.h rely on wide-int.h and the >>> new poly-int.h, so it needs to appear later in the coretypes.h >>> include list. >>> >>> Previously machmode.h included insn-modes.h, which as well as >>> the main mode enum contains configuration information like >>> MAX_BITSIZE_MODE_ANY_INT. This still needs to come first, >>> since files like wide-int.h depend on the configuration >>> information. >>> >>> Similarly, later patches will make the auto-generated inline >>> mode size functions use poly-int.h, so the patch splits them >>> out into their own header file and includes it after the >>> integer utilities. >>> >>> The patch also makes the generator files include machmode.h >>> via coretypes.h. Previously they did it by more indirect means. >>> >>> Finally, the patch makes wide-int-print.h available via coretypes.h >>> too. There didn't seem to be any reason to force only the print >>> routines to be included directly, and it would be painful to extend >>> that approach to the new polynomial integer classes. >>> >>> Tested on aarch64-linux-gnu and x86_64-linux-gnu. OK to install? >>> >>> Thanks, >>> Richard >>> >>> >>> [ This patch is part of the SVE series posted here: >>> https://gcc.gnu.org/ml/gcc/2016-11/msg00030.html ] >>> >>> gcc/ >>> 2016-11-16 Richard Sandiford <richard.sandiford@arm.com> >>> Alan Hayward <alan.hayward@arm.com> >>> David Sherwood <david.sherwood@arm.com> >>> >>> * Makefile.in (MACHMODE_H): Remove insn-modes.h >>> (CORETYPES_H): New define. >>> (MOSTLYCLEANFILES): Add insn-modes-inline.h. >>> (insn-modes-inline.h, s-modes-inline-h): New rules. >>> (generated_files): Add insn-modes-inline.h. >>> (RTL_BASE_H, TREE_CORE_H): Use CORETYPES_H instead of coretypes.h. >>> (build/gensupport.o, build/print-rtl.o, build/read-md.o): Likewise. >>> (build/read-rtl.o, build/rtl.o, build/vec.o, build/hash-table.o) >>> (build/inchash.o, build/gencondmd.o, build/genattr.o): Likewise. >>> (build/genattr-common.o, build/genattrtab.o, build/genautomata.o) >>> (build/gencheck.o, build/gencodes.o, build/genconditions.o): Likewise. >>> (build/genconfig.o, build/genconstants.o, build/genemit.o): Likewise. >>> (build/genenums.o, build/genextract.o, build/genflags.o): Likewise. >>> (build/gentarget-def.o, build/genmddeps.o, build/genopinit.o) >>> (build/genoutput.o, build/genpeep.o, build/genpreds.o): Likewise. >>> (build/genrecog.o, build/genmddump.o, build/genmatch.o): Likewise. >>> (build/gencfn-macros.o, build/gcov-iov.o): Likewise. >>> * coretypes.h: Include everything up to real.h for generators. >>> Include insn-modes.h first. Include wide-int-print.h after >>> wide-int.h. Include insn-modes-inline.h and then machmode.h. >>> * machmode.h: Don't include insn-modes.h here. >>> * function-tests.c: Remove includes of signop.h, machmode.h, >>> double-int.h and wide-int.h. >>> * rtl.h: Likewise. >>> * gcc-rich-location.c: Remove includes of machmode.h, double-int.h >>> and wide-int.h. >>> * optc-save-gen.awk: Likewise. >>> * gencheck.c (BITS_PER_UNIT): Delete dummy definition. >>> * godump.c: Remove include of wide-int-print.h. >>> * pretty-print.h: Likewise. >>> * wide-int-print.cc: Likewise. >>> * wide-int.cc: Likewise. >>> * hash-map-tests.c: Remove include of signop.h. >>> * hash-set-tests.c: Likewise. >>> * rtl-tests.c: Likewise. >>> * mkconfig.sh: Remove include of machmode.h. >>> * genmodes.c (emit_insn_modes_h): Split emission of inline functions >>> into... >>> (emit_insn_modes_inline_h): ...this new function. Emit the code >>> into an insn-modes-inline.h header file, adding appropriate >>> include guards and end comments. >>> (emit_insn_modes_c_header): Remove include of machmode.h. >>> (emit_min_insn_modes_c_header): Include coretypes.h rather than >>> machmode.h. >>> (main): Handle -i flag and call emit_insn_modes_inline_h when >>> it is passed. >> So I don't see anything here particularly problematical. My question is >> whether or not there's anything significant to be gained to moving >> forward with this kit, assuming the 67 piece kit is not likely to move >> forward. >> >> I do think you'll need some tweaks to the contrib/header-tools which >> know about the core headers and dependencies. Hopefully what's in there >> is easy enough to figure out how to twiddle appropriately. > > OK, thanks for the pointer. I think this patch should do that. > > gcc-order-headers seems to have bitrotted a bit, since it gives: > > Traceback (most recent call last): > File "../contrib/header-tools/gcc-order-headers", line 267, in <module> > process_known_dups () > File "../contrib/header-tools/gcc-order-headers", line 101, in process_known_dups > if dups[i] and "rtl.h" in dups[i]: > KeyError: 'dumpfile.h' > > But the change itself looks obvious. > > Tested on aarch64-linux-gnu and x86_64-linux-gnu. Also tested by > compiling one target for each cpu directory. OK to install? > > Thanks, > Richard > > > [Based on the code ARM contributed in branches/ARM/sve-branch@242100] > > 2016-11-16 Richard Sandiford <richard.sandiford@linaro.org> > Alan Hayward <alan.hayward@arm.com> > David Sherwood <david.sherwood@arm.com> > > contrib/header-tools/ > * graph-header-logs (ignore): Update coretypes.h header list. > > gcc/ > * Makefile.in (MACHMODE_H): Remove insn-modes.h > (CORETYPES_H): New define. > (MOSTLYCLEANFILES): Add insn-modes-inline.h. > (insn-modes-inline.h, s-modes-inline-h): New rules. > (generated_files): Add insn-modes-inline.h. > (RTL_BASE_H, TREE_CORE_H): Use CORETYPES_H instead of coretypes.h. > (build/gensupport.o, build/ggc-none.o, build/print-rtl.o): Likewise. > (build/read-md.o, build/read-rtl.o, build/rtl.o): Likewise. > (build/vec.o, build/hash-table.o, build/inchash.o): Likewise. > (build/gencondmd.o, build/genattr.o, build/genattr-common.o): Likewise. > (build/genattrtab.o, build/genautomata.o, build/gencheck.o): Likewise. > (build/gencodes.o, build/genconditions.o): Likewise. > (build/genconfig.o, build/genconstants.o, build/genemit.o): Likewise. > (build/genenums.o, build/genextract.o, build/genflags.o): Likewise. > (build/gentarget-def.o, build/genmddeps.o, build/genopinit.o) > (build/genoutput.o, build/genpeep.o, build/genpreds.o): Likewise. > (build/genrecog.o, build/genmddump.o, build/genmatch.o): Likewise. > (build/gencfn-macros.o, build/gcov-iov.o): Likewise. > * coretypes.h: Include everything up to real.h for generators. > Include insn-modes.h first. Include wide-int-print.h after > wide-int.h. Include insn-modes-inline.h and then machmode.h. > * machmode.h: Don't include insn-modes.h here. > * function-tests.c: Remove includes of signop.h, machmode.h, > double-int.h and wide-int.h. > * rtl.h: Likewise. > * gcc-rich-location.c: Remove includes of machmode.h, double-int.h > and wide-int.h. > * optc-save-gen.awk: Likewise. > * gencheck.c (BITS_PER_UNIT): Delete dummy definition. > * godump.c: Remove include of wide-int-print.h. > * pretty-print.h: Likewise. > * wide-int-print.cc: Likewise. > * wide-int.cc: Likewise. > * hash-map-tests.c: Remove include of signop.h. > * hash-set-tests.c: Likewise. > * rtl-tests.c: Likewise. > * mkconfig.sh: Remove include of machmode.h. > * genmodes.c (emit_insn_modes_h): Split emission of inline functions > into... > (emit_insn_modes_inline_h): ...this new function. Emit the code > into an insn-modes-inline.h header file, adding appropriate > include guards and end comments. > (emit_insn_modes_c_header): Remove include of machmode.h. > (emit_min_insn_modes_c_header): Include coretypes.h rather than > machmode.h. > (main): Handle -i flag and call emit_insn_modes_inline_h when > it is passed. > > Index: contrib/header-tools/graph-header-logs > =================================================================== > --- contrib/header-tools/graph-header-logs 2015-11-11 15:40:07.000000000 +0000 > +++ contrib/header-tools/graph-header-logs 2017-05-24 14:37:44.737223605 +0100 > @@ -25,9 +25,12 @@ depstring = ("In file included from", " > > # indentation indicates nesting levels of included files > ignore = [ "coretypes_h", > - "machmode_h", > + "insn_modes_h", > "signop_h", > "wide_int_h", > + "wide_int_print_h", > + "insn_modes_inline_h", > + "machmode_h", > "double_int_h", > "real_h", > "fixed_value_h", > Index: gcc/Makefile.in > =================================================================== > --- gcc/Makefile.in 2017-05-23 19:29:59.645724759 +0100 > +++ gcc/Makefile.in 2017-05-24 14:37:44.737223605 +0100 > @@ -900,14 +900,15 @@ COMMON_TARGET_DEF = common/common-target > TARGET_H = $(TM_H) target.h $(TARGET_DEF) insn-modes.h insn-codes.h > C_TARGET_H = c-family/c-target.h $(C_TARGET_DEF) > COMMON_TARGET_H = common/common-target.h $(INPUT_H) $(COMMON_TARGET_DEF) > -MACHMODE_H = machmode.h mode-classes.def insn-modes.h > +MACHMODE_H = machmode.h mode-classes.def > HOOKS_H = hooks.h $(MACHMODE_H) > HOSTHOOKS_DEF_H = hosthooks-def.h $(HOOKS_H) > LANGHOOKS_DEF_H = langhooks-def.h $(HOOKS_H) > TARGET_DEF_H = target-def.h target-hooks-def.h $(HOOKS_H) targhooks.h > C_TARGET_DEF_H = c-family/c-target-def.h c-family/c-target-hooks-def.h \ > $(TREE_H) $(C_COMMON_H) $(HOOKS_H) common/common-targhooks.h > -RTL_BASE_H = coretypes.h rtl.h rtl.def $(MACHMODE_H) reg-notes.def \ > +CORETYPES_H = coretypes.h insn-modes.h insn-modes-inline.h > +RTL_BASE_H = $(CORETYPES_H) rtl.h rtl.def $(MACHMODE_H) reg-notes.def \ > insn-notes.def $(INPUT_H) $(REAL_H) statistics.h $(VEC_H) \ > $(FIXED_VALUE_H) alias.h $(HASHTAB_H) > FIXED_VALUE_H = fixed-value.h $(MACHMODE_H) double-int.h > @@ -919,7 +920,7 @@ BUILTINS_DEF = builtins.def sync-builtin > hsa-builtins.def > INTERNAL_FN_DEF = internal-fn.def > INTERNAL_FN_H = internal-fn.h $(INTERNAL_FN_DEF) > -TREE_CORE_H = tree-core.h coretypes.h all-tree.def tree.def \ > +TREE_CORE_H = tree-core.h $(CORETYPES_H) all-tree.def tree.def \ > c-family/c-common.def $(lang_tree_files) $(MACHMODE_H) \ > $(BUILTINS_DEF) $(INPUT_H) statistics.h \ > $(VEC_H) treestruct.def $(HASHTAB_H) \ > @@ -1631,7 +1632,7 @@ MOSTLYCLEANFILES = insn-flags.h insn-con > insn-attr.h insn-attr-common.h insn-attrtab.c insn-dfatab.c \ > insn-latencytab.c insn-opinit.c insn-opinit.h insn-preds.c insn-constants.h \ > tm-preds.h tm-constrs.h checksum-options gimple-match.c generic-match.c \ > - tree-check.h min-insn-modes.c insn-modes.c insn-modes.h \ > + tree-check.h min-insn-modes.c insn-modes.c insn-modes.h insn-modes-inline.h \ > genrtl.h gt-*.h gtype-*.h gtype-desc.c gtyp-input.list \ > case-cfn-macros.h cfn-operators.pd \ > xgcc$(exeext) cpp$(exeext) $(FULL_DRIVER_NAME) \ > @@ -2317,6 +2318,7 @@ s-genrtl-h: build/gengenrtl$(build_exeex > > insn-modes.c: s-modes; @true > insn-modes.h: s-modes-h; @true > +insn-modes-inline.h: s-modes-inline-h; @true > min-insn-modes.c: s-modes-m; @true > > s-modes: build/genmodes$(build_exeext) > @@ -2329,6 +2331,12 @@ s-modes-h: build/genmodes$(build_exeext) > $(SHELL) $(srcdir)/../move-if-change tmp-modes.h insn-modes.h > $(STAMP) s-modes-h > > +s-modes-inline-h: build/genmodes$(build_exeext) > + $(RUN_GEN) build/genmodes$(build_exeext) -i > tmp-modes-inline.h > + $(SHELL) $(srcdir)/../move-if-change tmp-modes-inline.h \ > + insn-modes-inline.h > + $(STAMP) s-modes-inline-h > + > s-modes-m: build/genmodes$(build_exeext) > $(RUN_GEN) build/genmodes$(build_exeext) -m > tmp-min-modes.c > $(SHELL) $(srcdir)/../move-if-change tmp-min-modes.c min-insn-modes.c > @@ -2565,7 +2573,8 @@ s-gtype: build/gengtype$(build_exeext) $ > > generated_files = config.h tm.h $(TM_P_H) $(TM_H) multilib.h \ > $(simple_generated_h) specs.h \ > - tree-check.h genrtl.h insn-modes.h tm-preds.h tm-constrs.h \ > + tree-check.h genrtl.h insn-modes.h insn-modes-inline.h \ > + tm-preds.h tm-constrs.h \ > $(ALL_GTFILES_H) gtype-desc.c gtype-desc.h gcov-iov.h \ > options.h target-hooks-def.h insn-opinit.h \ > common/common-target-hooks-def.h pass-instances.def \ > @@ -2592,30 +2601,30 @@ build/version.o: version.c version.h \ > # Header dependencies for the programs that generate source code. > # These are library modules... > build/errors.o : errors.c $(BCONFIG_H) $(SYSTEM_H) errors.h > -build/gensupport.o: gensupport.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ > - $(GTM_H) $(RTL_BASE_H) $(OBSTACK_H) errors.h $(HASHTAB_H) \ > - $(READ_MD_H) $(GENSUPPORT_H) > -build/ggc-none.o : ggc-none.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ > +build/gensupport.o: gensupport.c $(BCONFIG_H) $(SYSTEM_H) \ > + $(CORETYPES_H) $(GTM_H) $(RTL_BASE_H) $(OBSTACK_H) errors.h \ > + $(HASHTAB_H) $(READ_MD_H) $(GENSUPPORT_H) > +build/ggc-none.o : ggc-none.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ > $(GGC_H) > build/min-insn-modes.o : min-insn-modes.c $(BCONFIG_H) $(SYSTEM_H) \ > $(MACHMODE_H) > -build/print-rtl.o: print-rtl.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ > +build/print-rtl.o: print-rtl.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ > $(GTM_H) $(RTL_BASE_H) > -build/read-md.o: read-md.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ > +build/read-md.o: read-md.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ > $(HASHTAB_H) errors.h $(READ_MD_H) > -build/read-rtl.o: read-rtl.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ > +build/read-rtl.o: read-rtl.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ > $(GTM_H) $(RTL_BASE_H) $(OBSTACK_H) $(HASHTAB_H) $(READ_MD_H) \ > $(GENSUPPORT_H) > -build/rtl.o: rtl.c $(BCONFIG_H) coretypes.h $(GTM_H) $(SYSTEM_H) \ > +build/rtl.o: rtl.c $(BCONFIG_H) $(CORETYPES_H) $(GTM_H) $(SYSTEM_H) \ > $(RTL_H) $(GGC_H) errors.h > -build/vec.o : vec.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h $(VEC_H) \ > - $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) > -build/hash-table.o : hash-table.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ > - $(HASH_TABLE_H) $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) > -build/inchash.o : inchash.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ > - $(HASHTAB_H) inchash.h > +build/vec.o : vec.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) $(VEC_H) \ > + $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) > +build/hash-table.o : hash-table.c $(BCONFIG_H) $(SYSTEM_H) \ > + $(CORETYPES_H) $(HASH_TABLE_H) $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) > +build/inchash.o : inchash.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ > + $(HASHTAB_H) inchash.h > build/gencondmd.o : build/gencondmd.c $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h $(GTM_H) insn-constants.h \ > + $(CORETYPES_H) $(GTM_H) insn-constants.h \ > $(filter-out insn-flags.h, $(RTL_H) $(TM_P_H) $(FUNCTION_H) $(REGS_H) \ > $(RECOG_H) output.h $(FLAGS_H) $(RESOURCE_H) toplev.h $(DIAGNOSTIC_CORE_H) reload.h \ > $(EXCEPT_H) tm-constrs.h) > @@ -2627,39 +2636,39 @@ build/gencondmd.o : \ > > # ...these are the programs themselves. > build/genattr.o : genattr.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) > + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) > build/genattr-common.o : genattr-common.c $(RTL_BASE_H) $(BCONFIG_H) \ > - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) > + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) > build/genattrtab.o : genattrtab.c $(RTL_BASE_H) $(OBSTACK_H) \ > - $(BCONFIG_H) $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(GGC_H) \ > + $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(GGC_H) \ > $(READ_MD_H) $(GENSUPPORT_H) $(FNMATCH_H) > build/genautomata.o : genautomata.c $(RTL_BASE_H) $(OBSTACK_H) \ > - $(BCONFIG_H) $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(VEC_H) \ > + $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(VEC_H) \ > $(HASHTAB_H) $(GENSUPPORT_H) $(FNMATCH_H) > build/gencheck.o : gencheck.c all-tree.def $(BCONFIG_H) $(GTM_H) \ > - $(SYSTEM_H) coretypes.h tree.def c-family/c-common.def \ > + $(SYSTEM_H) $(CORETYPES_H) tree.def c-family/c-common.def \ > $(lang_tree_files) gimple.def > build/genchecksum.o : genchecksum.c $(BCONFIG_H) $(SYSTEM_H) $(MD5_H) > build/gencodes.o : gencodes.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) > + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) > build/genconditions.o : genconditions.c $(RTL_BASE_H) $(BCONFIG_H) \ > - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(HASHTAB_H) $(READ_MD_H) \ > - $(GENSUPPORT_H) > + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(HASHTAB_H) \ > + $(READ_MD_H) $(GENSUPPORT_H) > build/genconfig.o : genconfig.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) > + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) > build/genconstants.o : genconstants.c $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h errors.h $(READ_MD_H) > + $(CORETYPES_H) errors.h $(READ_MD_H) > build/genemit.o : genemit.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) internal-fn.def > + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) internal-fn.def > build/genenums.o : genenums.c $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h errors.h $(READ_MD_H) > + $(CORETYPES_H) errors.h $(READ_MD_H) > build/genextract.o : genextract.c $(RTL_BASE_H) $(BCONFIG_H) \ > - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) > + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) > build/genflags.o : genflags.c $(RTL_BASE_H) $(OBSTACK_H) $(BCONFIG_H) \ > - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) > + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) > build/gentarget-def.o : gentarget-def.c $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h $(GTM_H) $(RTL_BASE_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) \ > - $(HASH_TABLE_H) target-insns.def > + $(CORETYPES_H) $(GTM_H) $(RTL_BASE_H) errors.h $(READ_MD_H) \ > + $(GENSUPPORT_H) $(HASH_TABLE_H) target-insns.def > build/gengenrtl.o : gengenrtl.c $(BCONFIG_H) $(SYSTEM_H) rtl.def > > # The gengtype generator program is special: Two versions are built. > @@ -2695,30 +2704,32 @@ build/gengtype.o: $(BCONFIG_H) > > CFLAGS-errors.o += -DHOST_GENERATOR_FILE > > -build/genmddeps.o: genmddeps.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ > +build/genmddeps.o: genmddeps.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ > errors.h $(READ_MD_H) > build/genmodes.o : genmodes.c $(BCONFIG_H) $(SYSTEM_H) errors.h \ > $(HASHTAB_H) machmode.def $(extra_modes_file) > build/genopinit.o : genopinit.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) optabs.def > + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) optabs.def > build/genoutput.o : genoutput.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) > + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) > build/genpeep.o : genpeep.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) toplev.h $(DIAGNOSTIC_CORE_H) > + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) toplev.h \ > + $(DIAGNOSTIC_CORE_H) > build/genpreds.o : genpreds.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) $(OBSTACK_H) > + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) $(OBSTACK_H) > build/genrecog.o : genrecog.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) \ > + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) \ > $(HASH_TABLE_H) inchash.h > build/genhooks.o : genhooks.c $(TARGET_DEF) $(C_TARGET_DEF) \ > $(COMMON_TARGET_DEF) $(BCONFIG_H) $(SYSTEM_H) errors.h > build/genmddump.o : genmddump.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) > + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) > build/genmatch.o : genmatch.c $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h errors.h $(HASH_TABLE_H) hash-map.h $(GGC_H) is-a.h \ > + $(CORETYPES_H) errors.h $(HASH_TABLE_H) hash-map.h $(GGC_H) is-a.h \ > tree.def builtins.def internal-fn.def > build/gencfn-macros.o : gencfn-macros.c $(BCONFIG_H) $(SYSTEM_H) \ > - coretypes.h errors.h $(HASH_TABLE_H) hash-set.h builtins.def internal-fn.def > + $(CORETYPES_H) errors.h $(HASH_TABLE_H) hash-set.h builtins.def \ > + internal-fn.def > > # Compile the programs that generate insn-* from the machine description. > # They are compiled with $(COMPILER_FOR_BUILD), and associated libraries, > @@ -2816,8 +2827,8 @@ CFLAGS-cppdefault.o += $(PREPROCESSOR_DE > # having an empty command (nothing following the semicolon). > > # gcov-iov.c is run on the build machine to generate gcov-iov.h from version.c > -build/gcov-iov.o: gcov-iov.c $(BCONFIG_H) coretypes.h $(GTM_H) \ > - $(SYSTEM_H) coretypes.h $(TM_H) > +build/gcov-iov.o: gcov-iov.c $(BCONFIG_H) $(CORETYPES_H) $(GTM_H) \ > + $(SYSTEM_H) $(CORETYPES_H) $(TM_H) > > build/gcov-iov$(build_exeext): build/gcov-iov.o > +$(LINKER_FOR_BUILD) $(BUILD_LINKERFLAGS) $(BUILD_LDFLAGS) \ > Index: gcc/coretypes.h > =================================================================== > --- gcc/coretypes.h 2017-05-18 07:51:11.876801735 +0100 > +++ gcc/coretypes.h 2017-05-24 14:37:44.737223605 +0100 > @@ -358,12 +358,17 @@ typedef void (*gt_pointer_operator) (voi > typedef unsigned char uchar; > #endif > > -/* Most host source files will require the following headers. */ > -#if !defined (GENERATOR_FILE) && !defined (USED_FOR_TARGET) > -#include "machmode.h" > +/* Most source files will require the following headers. */ > +#if !defined (USED_FOR_TARGET) > +#include "insn-modes.h" > #include "signop.h" > #include "wide-int.h" > +#include "wide-int-print.h" > +#include "insn-modes-inline.h" > +#include "machmode.h" > #include "double-int.h" > +/* Most host source files will require the following headers. */ > +#if !defined (GENERATOR_FILE) > #include "real.h" > #include "fixed-value.h" > #include "hash-table.h" > @@ -372,6 +377,7 @@ typedef unsigned char uchar; > #include "is-a.h" > #include "memory-block.h" > #include "dumpfile.h" > +#endif > #endif /* GENERATOR_FILE && !USED_FOR_TARGET */ > > #endif /* coretypes.h */ > Index: gcc/machmode.h > =================================================================== > --- gcc/machmode.h 2017-02-23 19:54:03.000000000 +0000 > +++ gcc/machmode.h 2017-05-24 14:37:44.739230513 +0100 > @@ -20,9 +20,6 @@ Software Foundation; either version 3, o > #ifndef HAVE_MACHINE_MODES > #define HAVE_MACHINE_MODES > > -/* Make an enum class that gives all the machine modes. */ > -#include "insn-modes.h" > - > /* Get the name of mode MODE as a string. */ > > extern const char * const mode_name[NUM_MACHINE_MODES]; > Index: gcc/function-tests.c > =================================================================== > --- gcc/function-tests.c 2017-02-23 19:54:03.000000000 +0000 > +++ gcc/function-tests.c 2017-05-24 14:37:44.738227059 +0100 > @@ -22,7 +22,6 @@ Software Foundation; either version 3, o > #include "coretypes.h" > #include "tm.h" > #include "opts.h" > -#include "signop.h" > #include "hash-set.h" > #include "fixed-value.h" > #include "alias.h" > @@ -38,7 +37,6 @@ Software Foundation; either version 3, o > #include "vec.h" > #include "hashtab.h" > #include "hash-set.h" > -#include "machmode.h" > #include "hard-reg-set.h" > #include "input.h" > #include "function.h" > @@ -56,10 +54,8 @@ Software Foundation; either version 3, o > #include "gimplify.h" > #include "tree-cfg.h" > #include "basic-block.h" > -#include "double-int.h" > #include "alias.h" > #include "symtab.h" > -#include "wide-int.h" > #include "inchash.h" > #include "tree.h" > #include "fold-const.h" > Index: gcc/rtl.h > =================================================================== > --- gcc/rtl.h 2017-03-28 16:19:28.000000000 +0100 > +++ gcc/rtl.h 2017-05-24 14:37:44.739230513 +0100 > @@ -24,10 +24,6 @@ #define GCC_RTL_H > machmode.h and other files to exist and would not normally have been > included by coretypes.h. */ > #ifdef GENERATOR_FILE > -#include "machmode.h" > -#include "signop.h" > -#include "wide-int.h" > -#include "double-int.h" > #include "real.h" > #include "fixed-value.h" > #include "statistics.h" > Index: gcc/gcc-rich-location.c > =================================================================== > --- gcc/gcc-rich-location.c 2017-02-23 19:54:12.000000000 +0000 > +++ gcc/gcc-rich-location.c 2017-05-24 14:37:44.738227059 +0100 > @@ -23,13 +23,10 @@ Software Foundation; either version 3, o > #include "tm.h" > #include "rtl.h" > #include "hash-set.h" > -#include "machmode.h" > #include "vec.h" > -#include "double-int.h" > #include "input.h" > #include "alias.h" > #include "symtab.h" > -#include "wide-int.h" > #include "inchash.h" > #include "tree-core.h" > #include "tree.h" > Index: gcc/optc-save-gen.awk > =================================================================== > --- gcc/optc-save-gen.awk 2017-02-23 19:54:15.000000000 +0000 > +++ gcc/optc-save-gen.awk 2017-05-24 14:37:44.739230513 +0100 > @@ -41,13 +41,10 @@ print "#include " quote "flags.h" quote > print "#include " quote "target.h" quote > print "#include " quote "inchash.h" quote > print "#include " quote "hash-set.h" quote > -print "#include " quote "machmode.h" quote > print "#include " quote "vec.h" quote > -print "#include " quote "double-int.h" quote > print "#include " quote "input.h" quote > print "#include " quote "alias.h" quote > print "#include " quote "symtab.h" quote > -print "#include " quote "wide-int.h" quote > print "#include " quote "inchash.h" quote > print "#include " quote "tree.h" quote > print "#include " quote "fold-const.h" quote > Index: gcc/gencheck.c > =================================================================== > --- gcc/gencheck.c 2017-02-23 19:54:03.000000000 +0000 > +++ gcc/gencheck.c 2017-05-24 14:37:44.738227059 +0100 > @@ -17,9 +17,6 @@ Software Foundation; either version 3, o > along with GCC; see the file COPYING3. If not see > <http://www.gnu.org/licenses/>. */ > > -/* We don't have insn-modes.h, but we include tm.h. */ > -#define BITS_PER_UNIT 8 > - > #include "bconfig.h" > #include "system.h" > #include "coretypes.h" > Index: gcc/godump.c > =================================================================== > --- gcc/godump.c 2017-02-23 19:54:03.000000000 +0000 > +++ gcc/godump.c 2017-05-24 14:37:44.738227059 +0100 > @@ -31,7 +31,6 @@ Software Foundation; either version 3, o > #include "system.h" > #include "coretypes.h" > #include "tree.h" > -#include "wide-int-print.h" > #include "diagnostic-core.h" > #include "debug.h" > #include "stor-layout.h" > Index: gcc/pretty-print.h > =================================================================== > --- gcc/pretty-print.h 2017-02-23 19:54:04.000000000 +0000 > +++ gcc/pretty-print.h 2017-05-24 14:37:44.739230513 +0100 > @@ -22,7 +22,6 @@ Software Foundation; either version 3, o > #define GCC_PRETTY_PRINT_H > > #include "obstack.h" > -#include "wide-int-print.h" > > /* Maximum number of format string arguments. */ > #define PP_NL_ARGMAX 30 > Index: gcc/wide-int-print.cc > =================================================================== > --- gcc/wide-int-print.cc 2017-02-23 19:54:03.000000000 +0000 > +++ gcc/wide-int-print.cc 2017-05-24 14:37:44.740233967 +0100 > @@ -21,7 +21,6 @@ along with GCC; see the file COPYING3. > #include "config.h" > #include "system.h" > #include "coretypes.h" > -#include "wide-int-print.h" > > /* > * public printing routines. > Index: gcc/wide-int.cc > =================================================================== > --- gcc/wide-int.cc 2017-02-23 19:54:04.000000000 +0000 > +++ gcc/wide-int.cc 2017-05-24 14:37:44.740233967 +0100 > @@ -24,7 +24,6 @@ along with GCC; see the file COPYING3. > #include "tm.h" > #include "tree.h" > #include "selftest.h" > -#include "wide-int-print.h" > > > #define HOST_BITS_PER_HALF_WIDE_INT 32 > Index: gcc/hash-map-tests.c > =================================================================== > --- gcc/hash-map-tests.c 2017-02-23 19:54:12.000000000 +0000 > +++ gcc/hash-map-tests.c 2017-05-24 14:37:44.738227059 +0100 > @@ -22,7 +22,6 @@ Software Foundation; either version 3, o > #include "coretypes.h" > #include "tm.h" > #include "opts.h" > -#include "signop.h" > #include "hash-set.h" > #include "fixed-value.h" > #include "alias.h" > Index: gcc/hash-set-tests.c > =================================================================== > --- gcc/hash-set-tests.c 2017-02-23 19:54:15.000000000 +0000 > +++ gcc/hash-set-tests.c 2017-05-24 14:37:44.739230513 +0100 > @@ -22,7 +22,6 @@ Software Foundation; either version 3, o > #include "coretypes.h" > #include "tm.h" > #include "opts.h" > -#include "signop.h" > #include "hash-set.h" > #include "selftest.h" > > Index: gcc/rtl-tests.c > =================================================================== > --- gcc/rtl-tests.c 2017-02-23 19:54:15.000000000 +0000 > +++ gcc/rtl-tests.c 2017-05-24 14:37:44.739230513 +0100 > @@ -22,7 +22,6 @@ Software Foundation; either version 3, o > #include "coretypes.h" > #include "tm.h" > #include "opts.h" > -#include "signop.h" > #include "hash-set.h" > #include "fixed-value.h" > #include "alias.h" > Index: gcc/mkconfig.sh > =================================================================== > --- gcc/mkconfig.sh 2017-02-23 19:54:03.000000000 +0000 > +++ gcc/mkconfig.sh 2017-05-24 14:37:44.739230513 +0100 > @@ -100,9 +100,6 @@ case $output in > #if defined IN_GCC && !defined GENERATOR_FILE > # include "insn-modes.h" > #endif > -#if defined IN_GCC && defined GENERATOR_FILE && !defined BITS_PER_UNIT > -#include "machmode.h" > -#endif > EOF > ;; > esac > Index: gcc/genmodes.c > =================================================================== > --- gcc/genmodes.c 2017-03-28 16:19:28.000000000 +0100 > +++ gcc/genmodes.c 2017-05-24 14:37:44.738227059 +0100 > @@ -1204,6 +1204,24 @@ enum machine_mode\n{"); > > printf ("#define NUM_INT_N_ENTS %d\n", n_int_n_ents); > > + puts ("\ > +\n\ > +#endif /* insn-modes.h */"); > +} > + > +static void > +emit_insn_modes_inline_h (void) > +{ > + printf ("/* Generated automatically from machmode.def%s%s\n", > + HAVE_EXTRA_MODES ? " and " : "", > + EXTRA_MODES_FILE); > + > + puts ("\ > + by genmodes. */\n\ > +\n\ > +#ifndef GCC_INSN_MODES_INLINE_H\n\ > +#define GCC_INSN_MODES_INLINE_H"); > + > puts ("\n#if !defined (USED_FOR_TARGET) && GCC_VERSION >= 4001\n"); > emit_mode_size_inline (); > emit_mode_nunits_inline (); > @@ -1214,7 +1232,7 @@ enum machine_mode\n{"); > > puts ("\ > \n\ > -#endif /* insn-modes.h */"); > +#endif /* insn-modes-inline.h */"); > } > > static void > @@ -1231,7 +1249,6 @@ emit_insn_modes_c_header (void) > #include \"system.h\"\n\ > #include \"coretypes.h\"\n\ > #include \"tm.h\"\n\ > -#include \"machmode.h\"\n\ > #include \"real.h\""); > } > > @@ -1247,7 +1264,7 @@ emit_min_insn_modes_c_header (void) > \n\ > #include \"bconfig.h\"\n\ > #include \"system.h\"\n\ > -#include \"machmode.h\""); > +#include \"coretypes.h\""); > } > > static void > @@ -1799,18 +1816,20 @@ emit_min_insn_modes_c (void) > int > main (int argc, char **argv) > { > - bool gen_header = false, gen_min = false; > + bool gen_header = false, gen_inlines = false, gen_min = false; > progname = argv[0]; > > if (argc == 1) > ; > else if (argc == 2 && !strcmp (argv[1], "-h")) > gen_header = true; > + else if (argc == 2 && !strcmp (argv[1], "-i")) > + gen_inlines = true; > else if (argc == 2 && !strcmp (argv[1], "-m")) > gen_min = true; > else > { > - error ("usage: %s [-h|-m] > file", progname); > + error ("usage: %s [-h|-i|-m] > file", progname); > return FATAL_EXIT_CODE; > } > > @@ -1826,6 +1845,8 @@ main (int argc, char **argv) > > if (gen_header) > emit_insn_modes_h (); > + else if (gen_inlines) > + emit_insn_modes_inline_h (); > else if (gen_min) > emit_min_insn_modes_c (); > else ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Reorganise machmode.h headers 2017-06-12 6:37 ` Richard Sandiford @ 2017-06-22 11:34 ` Richard Sandiford 0 siblings, 0 replies; 7+ messages in thread From: Richard Sandiford @ 2017-06-22 11:34 UTC (permalink / raw) To: Jeff Law; +Cc: gcc-patches Ping*2 Richard Sandiford <richard.sandiford@linaro.org> writes: > Ping > > Richard Sandiford <richard.sandiford@linaro.org> writes: >> Jeff Law <law@redhat.com> writes: >>> On 11/16/2016 09:32 AM, Richard Sandiford wrote: >>>> Later patches will make machmode.h rely on wide-int.h and the >>>> new poly-int.h, so it needs to appear later in the coretypes.h >>>> include list. >>>> >>>> Previously machmode.h included insn-modes.h, which as well as >>>> the main mode enum contains configuration information like >>>> MAX_BITSIZE_MODE_ANY_INT. This still needs to come first, >>>> since files like wide-int.h depend on the configuration >>>> information. >>>> >>>> Similarly, later patches will make the auto-generated inline >>>> mode size functions use poly-int.h, so the patch splits them >>>> out into their own header file and includes it after the >>>> integer utilities. >>>> >>>> The patch also makes the generator files include machmode.h >>>> via coretypes.h. Previously they did it by more indirect means. >>>> >>>> Finally, the patch makes wide-int-print.h available via coretypes.h >>>> too. There didn't seem to be any reason to force only the print >>>> routines to be included directly, and it would be painful to extend >>>> that approach to the new polynomial integer classes. >>>> >>>> Tested on aarch64-linux-gnu and x86_64-linux-gnu. OK to install? >>>> >>>> Thanks, >>>> Richard >>>> >>>> >>>> [ This patch is part of the SVE series posted here: >>>> https://gcc.gnu.org/ml/gcc/2016-11/msg00030.html ] >>>> >>>> gcc/ >>>> 2016-11-16 Richard Sandiford <richard.sandiford@arm.com> >>>> Alan Hayward <alan.hayward@arm.com> >>>> David Sherwood <david.sherwood@arm.com> >>>> >>>> * Makefile.in (MACHMODE_H): Remove insn-modes.h >>>> (CORETYPES_H): New define. >>>> (MOSTLYCLEANFILES): Add insn-modes-inline.h. >>>> (insn-modes-inline.h, s-modes-inline-h): New rules. >>>> (generated_files): Add insn-modes-inline.h. >>>> (RTL_BASE_H, TREE_CORE_H): Use CORETYPES_H instead of coretypes.h. >>>> (build/gensupport.o, build/print-rtl.o, build/read-md.o): Likewise. >>>> (build/read-rtl.o, build/rtl.o, build/vec.o, build/hash-table.o) >>>> (build/inchash.o, build/gencondmd.o, build/genattr.o): Likewise. >>>> (build/genattr-common.o, build/genattrtab.o, build/genautomata.o) >>>> (build/gencheck.o, build/gencodes.o, build/genconditions.o): Likewise. >>>> (build/genconfig.o, build/genconstants.o, build/genemit.o): Likewise. >>>> (build/genenums.o, build/genextract.o, build/genflags.o): Likewise. >>>> (build/gentarget-def.o, build/genmddeps.o, build/genopinit.o) >>>> (build/genoutput.o, build/genpeep.o, build/genpreds.o): Likewise. >>>> (build/genrecog.o, build/genmddump.o, build/genmatch.o): Likewise. >>>> (build/gencfn-macros.o, build/gcov-iov.o): Likewise. >>>> * coretypes.h: Include everything up to real.h for generators. >>>> Include insn-modes.h first. Include wide-int-print.h after >>>> wide-int.h. Include insn-modes-inline.h and then machmode.h. >>>> * machmode.h: Don't include insn-modes.h here. >>>> * function-tests.c: Remove includes of signop.h, machmode.h, >>>> double-int.h and wide-int.h. >>>> * rtl.h: Likewise. >>>> * gcc-rich-location.c: Remove includes of machmode.h, double-int.h >>>> and wide-int.h. >>>> * optc-save-gen.awk: Likewise. >>>> * gencheck.c (BITS_PER_UNIT): Delete dummy definition. >>>> * godump.c: Remove include of wide-int-print.h. >>>> * pretty-print.h: Likewise. >>>> * wide-int-print.cc: Likewise. >>>> * wide-int.cc: Likewise. >>>> * hash-map-tests.c: Remove include of signop.h. >>>> * hash-set-tests.c: Likewise. >>>> * rtl-tests.c: Likewise. >>>> * mkconfig.sh: Remove include of machmode.h. >>>> * genmodes.c (emit_insn_modes_h): Split emission of inline functions >>>> into... >>>> (emit_insn_modes_inline_h): ...this new function. Emit the code >>>> into an insn-modes-inline.h header file, adding appropriate >>>> include guards and end comments. >>>> (emit_insn_modes_c_header): Remove include of machmode.h. >>>> (emit_min_insn_modes_c_header): Include coretypes.h rather than >>>> machmode.h. >>>> (main): Handle -i flag and call emit_insn_modes_inline_h when >>>> it is passed. >>> So I don't see anything here particularly problematical. My question is >>> whether or not there's anything significant to be gained to moving >>> forward with this kit, assuming the 67 piece kit is not likely to move >>> forward. >>> >>> I do think you'll need some tweaks to the contrib/header-tools which >>> know about the core headers and dependencies. Hopefully what's in there >>> is easy enough to figure out how to twiddle appropriately. >> >> OK, thanks for the pointer. I think this patch should do that. >> >> gcc-order-headers seems to have bitrotted a bit, since it gives: >> >> Traceback (most recent call last): >> File "../contrib/header-tools/gcc-order-headers", line 267, in <module> >> process_known_dups () >> File "../contrib/header-tools/gcc-order-headers", line 101, in > process_known_dups >> if dups[i] and "rtl.h" in dups[i]: >> KeyError: 'dumpfile.h' >> >> But the change itself looks obvious. >> >> Tested on aarch64-linux-gnu and x86_64-linux-gnu. Also tested by >> compiling one target for each cpu directory. OK to install? >> >> Thanks, >> Richard >> >> >> [Based on the code ARM contributed in branches/ARM/sve-branch@242100] >> >> 2016-11-16 Richard Sandiford <richard.sandiford@linaro.org> >> Alan Hayward <alan.hayward@arm.com> >> David Sherwood <david.sherwood@arm.com> >> >> contrib/header-tools/ >> * graph-header-logs (ignore): Update coretypes.h header list. >> >> gcc/ >> * Makefile.in (MACHMODE_H): Remove insn-modes.h >> (CORETYPES_H): New define. >> (MOSTLYCLEANFILES): Add insn-modes-inline.h. >> (insn-modes-inline.h, s-modes-inline-h): New rules. >> (generated_files): Add insn-modes-inline.h. >> (RTL_BASE_H, TREE_CORE_H): Use CORETYPES_H instead of coretypes.h. >> (build/gensupport.o, build/ggc-none.o, build/print-rtl.o): Likewise. >> (build/read-md.o, build/read-rtl.o, build/rtl.o): Likewise. >> (build/vec.o, build/hash-table.o, build/inchash.o): Likewise. >> (build/gencondmd.o, build/genattr.o, build/genattr-common.o): Likewise. >> (build/genattrtab.o, build/genautomata.o, build/gencheck.o): Likewise. >> (build/gencodes.o, build/genconditions.o): Likewise. >> (build/genconfig.o, build/genconstants.o, build/genemit.o): Likewise. >> (build/genenums.o, build/genextract.o, build/genflags.o): Likewise. >> (build/gentarget-def.o, build/genmddeps.o, build/genopinit.o) >> (build/genoutput.o, build/genpeep.o, build/genpreds.o): Likewise. >> (build/genrecog.o, build/genmddump.o, build/genmatch.o): Likewise. >> (build/gencfn-macros.o, build/gcov-iov.o): Likewise. >> * coretypes.h: Include everything up to real.h for generators. >> Include insn-modes.h first. Include wide-int-print.h after >> wide-int.h. Include insn-modes-inline.h and then machmode.h. >> * machmode.h: Don't include insn-modes.h here. >> * function-tests.c: Remove includes of signop.h, machmode.h, >> double-int.h and wide-int.h. >> * rtl.h: Likewise. >> * gcc-rich-location.c: Remove includes of machmode.h, double-int.h >> and wide-int.h. >> * optc-save-gen.awk: Likewise. >> * gencheck.c (BITS_PER_UNIT): Delete dummy definition. >> * godump.c: Remove include of wide-int-print.h. >> * pretty-print.h: Likewise. >> * wide-int-print.cc: Likewise. >> * wide-int.cc: Likewise. >> * hash-map-tests.c: Remove include of signop.h. >> * hash-set-tests.c: Likewise. >> * rtl-tests.c: Likewise. >> * mkconfig.sh: Remove include of machmode.h. >> * genmodes.c (emit_insn_modes_h): Split emission of inline functions >> into... >> (emit_insn_modes_inline_h): ...this new function. Emit the code >> into an insn-modes-inline.h header file, adding appropriate >> include guards and end comments. >> (emit_insn_modes_c_header): Remove include of machmode.h. >> (emit_min_insn_modes_c_header): Include coretypes.h rather than >> machmode.h. >> (main): Handle -i flag and call emit_insn_modes_inline_h when >> it is passed. >> >> Index: contrib/header-tools/graph-header-logs >> =================================================================== >> --- contrib/header-tools/graph-header-logs 2015-11-11 > 15:40:07.000000000 +0000 >> +++ contrib/header-tools/graph-header-logs 2017-05-24 > 14:37:44.737223605 +0100 >> @@ -25,9 +25,12 @@ depstring = ("In file included from", " >> >> # indentation indicates nesting levels of included files >> ignore = [ "coretypes_h", >> - "machmode_h", >> + "insn_modes_h", >> "signop_h", >> "wide_int_h", >> + "wide_int_print_h", >> + "insn_modes_inline_h", >> + "machmode_h", >> "double_int_h", >> "real_h", >> "fixed_value_h", >> Index: gcc/Makefile.in >> =================================================================== >> --- gcc/Makefile.in 2017-05-23 19:29:59.645724759 +0100 >> +++ gcc/Makefile.in 2017-05-24 14:37:44.737223605 +0100 >> @@ -900,14 +900,15 @@ COMMON_TARGET_DEF = common/common-target >> TARGET_H = $(TM_H) target.h $(TARGET_DEF) insn-modes.h insn-codes.h >> C_TARGET_H = c-family/c-target.h $(C_TARGET_DEF) >> COMMON_TARGET_H = common/common-target.h $(INPUT_H) $(COMMON_TARGET_DEF) >> -MACHMODE_H = machmode.h mode-classes.def insn-modes.h >> +MACHMODE_H = machmode.h mode-classes.def >> HOOKS_H = hooks.h $(MACHMODE_H) >> HOSTHOOKS_DEF_H = hosthooks-def.h $(HOOKS_H) >> LANGHOOKS_DEF_H = langhooks-def.h $(HOOKS_H) >> TARGET_DEF_H = target-def.h target-hooks-def.h $(HOOKS_H) targhooks.h >> C_TARGET_DEF_H = c-family/c-target-def.h c-family/c-target-hooks-def.h \ >> $(TREE_H) $(C_COMMON_H) $(HOOKS_H) common/common-targhooks.h >> -RTL_BASE_H = coretypes.h rtl.h rtl.def $(MACHMODE_H) reg-notes.def \ >> +CORETYPES_H = coretypes.h insn-modes.h insn-modes-inline.h >> +RTL_BASE_H = $(CORETYPES_H) rtl.h rtl.def $(MACHMODE_H) reg-notes.def \ >> insn-notes.def $(INPUT_H) $(REAL_H) statistics.h $(VEC_H) \ >> $(FIXED_VALUE_H) alias.h $(HASHTAB_H) >> FIXED_VALUE_H = fixed-value.h $(MACHMODE_H) double-int.h >> @@ -919,7 +920,7 @@ BUILTINS_DEF = builtins.def sync-builtin >> hsa-builtins.def >> INTERNAL_FN_DEF = internal-fn.def >> INTERNAL_FN_H = internal-fn.h $(INTERNAL_FN_DEF) >> -TREE_CORE_H = tree-core.h coretypes.h all-tree.def tree.def \ >> +TREE_CORE_H = tree-core.h $(CORETYPES_H) all-tree.def tree.def \ >> c-family/c-common.def $(lang_tree_files) $(MACHMODE_H) \ >> $(BUILTINS_DEF) $(INPUT_H) statistics.h \ >> $(VEC_H) treestruct.def $(HASHTAB_H) \ >> @@ -1631,7 +1632,7 @@ MOSTLYCLEANFILES = insn-flags.h insn-con >> insn-attr.h insn-attr-common.h insn-attrtab.c insn-dfatab.c \ >> insn-latencytab.c insn-opinit.c insn-opinit.h insn-preds.c > insn-constants.h \ >> tm-preds.h tm-constrs.h checksum-options gimple-match.c generic-match.c \ >> - tree-check.h min-insn-modes.c insn-modes.c insn-modes.h \ >> + tree-check.h min-insn-modes.c insn-modes.c insn-modes.h > insn-modes-inline.h \ >> genrtl.h gt-*.h gtype-*.h gtype-desc.c gtyp-input.list \ >> case-cfn-macros.h cfn-operators.pd \ >> xgcc$(exeext) cpp$(exeext) $(FULL_DRIVER_NAME) \ >> @@ -2317,6 +2318,7 @@ s-genrtl-h: build/gengenrtl$(build_exeex >> >> insn-modes.c: s-modes; @true >> insn-modes.h: s-modes-h; @true >> +insn-modes-inline.h: s-modes-inline-h; @true >> min-insn-modes.c: s-modes-m; @true >> >> s-modes: build/genmodes$(build_exeext) >> @@ -2329,6 +2331,12 @@ s-modes-h: build/genmodes$(build_exeext) >> $(SHELL) $(srcdir)/../move-if-change tmp-modes.h insn-modes.h >> $(STAMP) s-modes-h >> >> +s-modes-inline-h: build/genmodes$(build_exeext) >> + $(RUN_GEN) build/genmodes$(build_exeext) -i > tmp-modes-inline.h >> + $(SHELL) $(srcdir)/../move-if-change tmp-modes-inline.h \ >> + insn-modes-inline.h >> + $(STAMP) s-modes-inline-h >> + >> s-modes-m: build/genmodes$(build_exeext) >> $(RUN_GEN) build/genmodes$(build_exeext) -m > tmp-min-modes.c >> $(SHELL) $(srcdir)/../move-if-change tmp-min-modes.c min-insn-modes.c >> @@ -2565,7 +2573,8 @@ s-gtype: build/gengtype$(build_exeext) $ >> >> generated_files = config.h tm.h $(TM_P_H) $(TM_H) multilib.h \ >> $(simple_generated_h) specs.h \ >> - tree-check.h genrtl.h insn-modes.h tm-preds.h tm-constrs.h \ >> + tree-check.h genrtl.h insn-modes.h insn-modes-inline.h \ >> + tm-preds.h tm-constrs.h \ >> $(ALL_GTFILES_H) gtype-desc.c gtype-desc.h gcov-iov.h \ >> options.h target-hooks-def.h insn-opinit.h \ >> common/common-target-hooks-def.h pass-instances.def \ >> @@ -2592,30 +2601,30 @@ build/version.o: version.c version.h \ >> # Header dependencies for the programs that generate source code. >> # These are library modules... >> build/errors.o : errors.c $(BCONFIG_H) $(SYSTEM_H) errors.h >> -build/gensupport.o: gensupport.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h > \ >> - $(GTM_H) $(RTL_BASE_H) $(OBSTACK_H) errors.h $(HASHTAB_H) \ >> - $(READ_MD_H) $(GENSUPPORT_H) >> -build/ggc-none.o : ggc-none.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ >> +build/gensupport.o: gensupport.c $(BCONFIG_H) $(SYSTEM_H) \ >> + $(CORETYPES_H) $(GTM_H) $(RTL_BASE_H) $(OBSTACK_H) errors.h \ >> + $(HASHTAB_H) $(READ_MD_H) $(GENSUPPORT_H) >> +build/ggc-none.o : ggc-none.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) > \ >> $(GGC_H) >> build/min-insn-modes.o : min-insn-modes.c $(BCONFIG_H) $(SYSTEM_H) \ >> $(MACHMODE_H) >> -build/print-rtl.o: print-rtl.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ >> +build/print-rtl.o: print-rtl.c $(BCONFIG_H) $(SYSTEM_H) > $(CORETYPES_H) \ >> $(GTM_H) $(RTL_BASE_H) >> -build/read-md.o: read-md.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ >> +build/read-md.o: read-md.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ >> $(HASHTAB_H) errors.h $(READ_MD_H) >> -build/read-rtl.o: read-rtl.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ >> +build/read-rtl.o: read-rtl.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ >> $(GTM_H) $(RTL_BASE_H) $(OBSTACK_H) $(HASHTAB_H) $(READ_MD_H) \ >> $(GENSUPPORT_H) >> -build/rtl.o: rtl.c $(BCONFIG_H) coretypes.h $(GTM_H) $(SYSTEM_H) \ >> +build/rtl.o: rtl.c $(BCONFIG_H) $(CORETYPES_H) $(GTM_H) $(SYSTEM_H) \ >> $(RTL_H) $(GGC_H) errors.h >> -build/vec.o : vec.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h $(VEC_H) \ >> - $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) >> -build/hash-table.o : hash-table.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ >> - $(HASH_TABLE_H) $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) >> -build/inchash.o : inchash.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ >> - $(HASHTAB_H) inchash.h >> +build/vec.o : vec.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) $(VEC_H) \ >> + $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) >> +build/hash-table.o : hash-table.c $(BCONFIG_H) $(SYSTEM_H) \ >> + $(CORETYPES_H) $(HASH_TABLE_H) $(GGC_H) toplev.h $(DIAGNOSTIC_CORE_H) >> +build/inchash.o : inchash.c $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) \ >> + $(HASHTAB_H) inchash.h >> build/gencondmd.o : build/gencondmd.c $(BCONFIG_H) $(SYSTEM_H) \ >> - coretypes.h $(GTM_H) insn-constants.h \ >> + $(CORETYPES_H) $(GTM_H) insn-constants.h \ >> $(filter-out insn-flags.h, $(RTL_H) $(TM_P_H) $(FUNCTION_H) $(REGS_H) \ >> $(RECOG_H) output.h $(FLAGS_H) $(RESOURCE_H) toplev.h > $(DIAGNOSTIC_CORE_H) reload.h \ >> $(EXCEPT_H) tm-constrs.h) >> @@ -2627,39 +2636,39 @@ build/gencondmd.o : \ >> >> # ...these are the programs themselves. >> build/genattr.o : genattr.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ >> - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) >> + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) >> build/genattr-common.o : genattr-common.c $(RTL_BASE_H) $(BCONFIG_H) \ >> - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) >> + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) >> build/genattrtab.o : genattrtab.c $(RTL_BASE_H) $(OBSTACK_H) \ >> - $(BCONFIG_H) $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(GGC_H) \ >> + $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(GGC_H) \ >> $(READ_MD_H) $(GENSUPPORT_H) $(FNMATCH_H) >> build/genautomata.o : genautomata.c $(RTL_BASE_H) $(OBSTACK_H) \ >> - $(BCONFIG_H) $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(VEC_H) \ >> + $(BCONFIG_H) $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(VEC_H) \ >> $(HASHTAB_H) $(GENSUPPORT_H) $(FNMATCH_H) >> build/gencheck.o : gencheck.c all-tree.def $(BCONFIG_H) $(GTM_H) \ >> - $(SYSTEM_H) coretypes.h tree.def c-family/c-common.def \ >> + $(SYSTEM_H) $(CORETYPES_H) tree.def c-family/c-common.def \ >> $(lang_tree_files) gimple.def >> build/genchecksum.o : genchecksum.c $(BCONFIG_H) $(SYSTEM_H) $(MD5_H) >> build/gencodes.o : gencodes.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ >> - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) >> + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) >> build/genconditions.o : genconditions.c $(RTL_BASE_H) $(BCONFIG_H) \ >> - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(HASHTAB_H) $(READ_MD_H) > \ >> - $(GENSUPPORT_H) >> + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(HASHTAB_H) \ >> + $(READ_MD_H) $(GENSUPPORT_H) >> build/genconfig.o : genconfig.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) > \ >> - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) >> + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) >> build/genconstants.o : genconstants.c $(BCONFIG_H) $(SYSTEM_H) \ >> - coretypes.h errors.h $(READ_MD_H) >> + $(CORETYPES_H) errors.h $(READ_MD_H) >> build/genemit.o : genemit.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ >> - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) internal-fn.def >> + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) > internal-fn.def >> build/genenums.o : genenums.c $(BCONFIG_H) $(SYSTEM_H) \ >> - coretypes.h errors.h $(READ_MD_H) >> + $(CORETYPES_H) errors.h $(READ_MD_H) >> build/genextract.o : genextract.c $(RTL_BASE_H) $(BCONFIG_H) \ >> - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) >> + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) >> build/genflags.o : genflags.c $(RTL_BASE_H) $(OBSTACK_H) $(BCONFIG_H) > \ >> - $(SYSTEM_H) coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) >> + $(SYSTEM_H) $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) >> build/gentarget-def.o : gentarget-def.c $(BCONFIG_H) $(SYSTEM_H) \ >> - coretypes.h $(GTM_H) $(RTL_BASE_H) errors.h $(READ_MD_H) > $(GENSUPPORT_H) \ >> - $(HASH_TABLE_H) target-insns.def >> + $(CORETYPES_H) $(GTM_H) $(RTL_BASE_H) errors.h $(READ_MD_H) \ >> + $(GENSUPPORT_H) $(HASH_TABLE_H) target-insns.def >> build/gengenrtl.o : gengenrtl.c $(BCONFIG_H) $(SYSTEM_H) rtl.def >> >> # The gengtype generator program is special: Two versions are built. >> @@ -2695,30 +2704,32 @@ build/gengtype.o: $(BCONFIG_H) >> >> CFLAGS-errors.o += -DHOST_GENERATOR_FILE >> >> -build/genmddeps.o: genmddeps.c $(BCONFIG_H) $(SYSTEM_H) coretypes.h \ >> +build/genmddeps.o: genmddeps.c $(BCONFIG_H) $(SYSTEM_H) > $(CORETYPES_H) \ >> errors.h $(READ_MD_H) >> build/genmodes.o : genmodes.c $(BCONFIG_H) $(SYSTEM_H) errors.h \ >> $(HASHTAB_H) machmode.def $(extra_modes_file) >> build/genopinit.o : genopinit.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) > \ >> - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) optabs.def >> + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) optabs.def >> build/genoutput.o : genoutput.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) > \ >> - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) >> + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) >> build/genpeep.o : genpeep.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ >> - coretypes.h $(GTM_H) errors.h $(GENSUPPORT_H) toplev.h $(DIAGNOSTIC_CORE_H) >> + $(CORETYPES_H) $(GTM_H) errors.h $(GENSUPPORT_H) toplev.h \ >> + $(DIAGNOSTIC_CORE_H) >> build/genpreds.o : genpreds.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ >> - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) $(OBSTACK_H) >> + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) $(OBSTACK_H) >> build/genrecog.o : genrecog.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) \ >> - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) \ >> + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) \ >> $(HASH_TABLE_H) inchash.h >> build/genhooks.o : genhooks.c $(TARGET_DEF) $(C_TARGET_DEF) \ >> $(COMMON_TARGET_DEF) $(BCONFIG_H) $(SYSTEM_H) errors.h >> build/genmddump.o : genmddump.c $(RTL_BASE_H) $(BCONFIG_H) $(SYSTEM_H) > \ >> - coretypes.h $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) >> + $(CORETYPES_H) $(GTM_H) errors.h $(READ_MD_H) $(GENSUPPORT_H) >> build/genmatch.o : genmatch.c $(BCONFIG_H) $(SYSTEM_H) \ >> - coretypes.h errors.h $(HASH_TABLE_H) hash-map.h $(GGC_H) is-a.h \ >> + $(CORETYPES_H) errors.h $(HASH_TABLE_H) hash-map.h $(GGC_H) is-a.h \ >> tree.def builtins.def internal-fn.def >> build/gencfn-macros.o : gencfn-macros.c $(BCONFIG_H) $(SYSTEM_H) \ >> - coretypes.h errors.h $(HASH_TABLE_H) hash-set.h builtins.def > internal-fn.def >> + $(CORETYPES_H) errors.h $(HASH_TABLE_H) hash-set.h builtins.def \ >> + internal-fn.def >> >> # Compile the programs that generate insn-* from the machine description. >> # They are compiled with $(COMPILER_FOR_BUILD), and associated libraries, >> @@ -2816,8 +2827,8 @@ CFLAGS-cppdefault.o += $(PREPROCESSOR_DE >> # having an empty command (nothing following the semicolon). >> >> # gcov-iov.c is run on the build machine to generate gcov-iov.h from > version.c >> -build/gcov-iov.o: gcov-iov.c $(BCONFIG_H) coretypes.h $(GTM_H) \ >> - $(SYSTEM_H) coretypes.h $(TM_H) >> +build/gcov-iov.o: gcov-iov.c $(BCONFIG_H) $(CORETYPES_H) $(GTM_H) \ >> + $(SYSTEM_H) $(CORETYPES_H) $(TM_H) >> >> build/gcov-iov$(build_exeext): build/gcov-iov.o >> +$(LINKER_FOR_BUILD) $(BUILD_LINKERFLAGS) $(BUILD_LDFLAGS) \ >> Index: gcc/coretypes.h >> =================================================================== >> --- gcc/coretypes.h 2017-05-18 07:51:11.876801735 +0100 >> +++ gcc/coretypes.h 2017-05-24 14:37:44.737223605 +0100 >> @@ -358,12 +358,17 @@ typedef void (*gt_pointer_operator) (voi >> typedef unsigned char uchar; >> #endif >> >> -/* Most host source files will require the following headers. */ >> -#if !defined (GENERATOR_FILE) && !defined (USED_FOR_TARGET) >> -#include "machmode.h" >> +/* Most source files will require the following headers. */ >> +#if !defined (USED_FOR_TARGET) >> +#include "insn-modes.h" >> #include "signop.h" >> #include "wide-int.h" >> +#include "wide-int-print.h" >> +#include "insn-modes-inline.h" >> +#include "machmode.h" >> #include "double-int.h" >> +/* Most host source files will require the following headers. */ >> +#if !defined (GENERATOR_FILE) >> #include "real.h" >> #include "fixed-value.h" >> #include "hash-table.h" >> @@ -372,6 +377,7 @@ typedef unsigned char uchar; >> #include "is-a.h" >> #include "memory-block.h" >> #include "dumpfile.h" >> +#endif >> #endif /* GENERATOR_FILE && !USED_FOR_TARGET */ >> >> #endif /* coretypes.h */ >> Index: gcc/machmode.h >> =================================================================== >> --- gcc/machmode.h 2017-02-23 19:54:03.000000000 +0000 >> +++ gcc/machmode.h 2017-05-24 14:37:44.739230513 +0100 >> @@ -20,9 +20,6 @@ Software Foundation; either version 3, o >> #ifndef HAVE_MACHINE_MODES >> #define HAVE_MACHINE_MODES >> >> -/* Make an enum class that gives all the machine modes. */ >> -#include "insn-modes.h" >> - >> /* Get the name of mode MODE as a string. */ >> >> extern const char * const mode_name[NUM_MACHINE_MODES]; >> Index: gcc/function-tests.c >> =================================================================== >> --- gcc/function-tests.c 2017-02-23 19:54:03.000000000 +0000 >> +++ gcc/function-tests.c 2017-05-24 14:37:44.738227059 +0100 >> @@ -22,7 +22,6 @@ Software Foundation; either version 3, o >> #include "coretypes.h" >> #include "tm.h" >> #include "opts.h" >> -#include "signop.h" >> #include "hash-set.h" >> #include "fixed-value.h" >> #include "alias.h" >> @@ -38,7 +37,6 @@ Software Foundation; either version 3, o >> #include "vec.h" >> #include "hashtab.h" >> #include "hash-set.h" >> -#include "machmode.h" >> #include "hard-reg-set.h" >> #include "input.h" >> #include "function.h" >> @@ -56,10 +54,8 @@ Software Foundation; either version 3, o >> #include "gimplify.h" >> #include "tree-cfg.h" >> #include "basic-block.h" >> -#include "double-int.h" >> #include "alias.h" >> #include "symtab.h" >> -#include "wide-int.h" >> #include "inchash.h" >> #include "tree.h" >> #include "fold-const.h" >> Index: gcc/rtl.h >> =================================================================== >> --- gcc/rtl.h 2017-03-28 16:19:28.000000000 +0100 >> +++ gcc/rtl.h 2017-05-24 14:37:44.739230513 +0100 >> @@ -24,10 +24,6 @@ #define GCC_RTL_H >> machmode.h and other files to exist and would not normally have been >> included by coretypes.h. */ >> #ifdef GENERATOR_FILE >> -#include "machmode.h" >> -#include "signop.h" >> -#include "wide-int.h" >> -#include "double-int.h" >> #include "real.h" >> #include "fixed-value.h" >> #include "statistics.h" >> Index: gcc/gcc-rich-location.c >> =================================================================== >> --- gcc/gcc-rich-location.c 2017-02-23 19:54:12.000000000 +0000 >> +++ gcc/gcc-rich-location.c 2017-05-24 14:37:44.738227059 +0100 >> @@ -23,13 +23,10 @@ Software Foundation; either version 3, o >> #include "tm.h" >> #include "rtl.h" >> #include "hash-set.h" >> -#include "machmode.h" >> #include "vec.h" >> -#include "double-int.h" >> #include "input.h" >> #include "alias.h" >> #include "symtab.h" >> -#include "wide-int.h" >> #include "inchash.h" >> #include "tree-core.h" >> #include "tree.h" >> Index: gcc/optc-save-gen.awk >> =================================================================== >> --- gcc/optc-save-gen.awk 2017-02-23 19:54:15.000000000 +0000 >> +++ gcc/optc-save-gen.awk 2017-05-24 14:37:44.739230513 +0100 >> @@ -41,13 +41,10 @@ print "#include " quote "flags.h" quote >> print "#include " quote "target.h" quote >> print "#include " quote "inchash.h" quote >> print "#include " quote "hash-set.h" quote >> -print "#include " quote "machmode.h" quote >> print "#include " quote "vec.h" quote >> -print "#include " quote "double-int.h" quote >> print "#include " quote "input.h" quote >> print "#include " quote "alias.h" quote >> print "#include " quote "symtab.h" quote >> -print "#include " quote "wide-int.h" quote >> print "#include " quote "inchash.h" quote >> print "#include " quote "tree.h" quote >> print "#include " quote "fold-const.h" quote >> Index: gcc/gencheck.c >> =================================================================== >> --- gcc/gencheck.c 2017-02-23 19:54:03.000000000 +0000 >> +++ gcc/gencheck.c 2017-05-24 14:37:44.738227059 +0100 >> @@ -17,9 +17,6 @@ Software Foundation; either version 3, o >> along with GCC; see the file COPYING3. If not see >> <http://www.gnu.org/licenses/>. */ >> >> -/* We don't have insn-modes.h, but we include tm.h. */ >> -#define BITS_PER_UNIT 8 >> - >> #include "bconfig.h" >> #include "system.h" >> #include "coretypes.h" >> Index: gcc/godump.c >> =================================================================== >> --- gcc/godump.c 2017-02-23 19:54:03.000000000 +0000 >> +++ gcc/godump.c 2017-05-24 14:37:44.738227059 +0100 >> @@ -31,7 +31,6 @@ Software Foundation; either version 3, o >> #include "system.h" >> #include "coretypes.h" >> #include "tree.h" >> -#include "wide-int-print.h" >> #include "diagnostic-core.h" >> #include "debug.h" >> #include "stor-layout.h" >> Index: gcc/pretty-print.h >> =================================================================== >> --- gcc/pretty-print.h 2017-02-23 19:54:04.000000000 +0000 >> +++ gcc/pretty-print.h 2017-05-24 14:37:44.739230513 +0100 >> @@ -22,7 +22,6 @@ Software Foundation; either version 3, o >> #define GCC_PRETTY_PRINT_H >> >> #include "obstack.h" >> -#include "wide-int-print.h" >> >> /* Maximum number of format string arguments. */ >> #define PP_NL_ARGMAX 30 >> Index: gcc/wide-int-print.cc >> =================================================================== >> --- gcc/wide-int-print.cc 2017-02-23 19:54:03.000000000 +0000 >> +++ gcc/wide-int-print.cc 2017-05-24 14:37:44.740233967 +0100 >> @@ -21,7 +21,6 @@ along with GCC; see the file COPYING3. >> #include "config.h" >> #include "system.h" >> #include "coretypes.h" >> -#include "wide-int-print.h" >> >> /* >> * public printing routines. >> Index: gcc/wide-int.cc >> =================================================================== >> --- gcc/wide-int.cc 2017-02-23 19:54:04.000000000 +0000 >> +++ gcc/wide-int.cc 2017-05-24 14:37:44.740233967 +0100 >> @@ -24,7 +24,6 @@ along with GCC; see the file COPYING3. >> #include "tm.h" >> #include "tree.h" >> #include "selftest.h" >> -#include "wide-int-print.h" >> >> >> #define HOST_BITS_PER_HALF_WIDE_INT 32 >> Index: gcc/hash-map-tests.c >> =================================================================== >> --- gcc/hash-map-tests.c 2017-02-23 19:54:12.000000000 +0000 >> +++ gcc/hash-map-tests.c 2017-05-24 14:37:44.738227059 +0100 >> @@ -22,7 +22,6 @@ Software Foundation; either version 3, o >> #include "coretypes.h" >> #include "tm.h" >> #include "opts.h" >> -#include "signop.h" >> #include "hash-set.h" >> #include "fixed-value.h" >> #include "alias.h" >> Index: gcc/hash-set-tests.c >> =================================================================== >> --- gcc/hash-set-tests.c 2017-02-23 19:54:15.000000000 +0000 >> +++ gcc/hash-set-tests.c 2017-05-24 14:37:44.739230513 +0100 >> @@ -22,7 +22,6 @@ Software Foundation; either version 3, o >> #include "coretypes.h" >> #include "tm.h" >> #include "opts.h" >> -#include "signop.h" >> #include "hash-set.h" >> #include "selftest.h" >> >> Index: gcc/rtl-tests.c >> =================================================================== >> --- gcc/rtl-tests.c 2017-02-23 19:54:15.000000000 +0000 >> +++ gcc/rtl-tests.c 2017-05-24 14:37:44.739230513 +0100 >> @@ -22,7 +22,6 @@ Software Foundation; either version 3, o >> #include "coretypes.h" >> #include "tm.h" >> #include "opts.h" >> -#include "signop.h" >> #include "hash-set.h" >> #include "fixed-value.h" >> #include "alias.h" >> Index: gcc/mkconfig.sh >> =================================================================== >> --- gcc/mkconfig.sh 2017-02-23 19:54:03.000000000 +0000 >> +++ gcc/mkconfig.sh 2017-05-24 14:37:44.739230513 +0100 >> @@ -100,9 +100,6 @@ case $output in >> #if defined IN_GCC && !defined GENERATOR_FILE >> # include "insn-modes.h" >> #endif >> -#if defined IN_GCC && defined GENERATOR_FILE && !defined BITS_PER_UNIT >> -#include "machmode.h" >> -#endif >> EOF >> ;; >> esac >> Index: gcc/genmodes.c >> =================================================================== >> --- gcc/genmodes.c 2017-03-28 16:19:28.000000000 +0100 >> +++ gcc/genmodes.c 2017-05-24 14:37:44.738227059 +0100 >> @@ -1204,6 +1204,24 @@ enum machine_mode\n{"); >> >> printf ("#define NUM_INT_N_ENTS %d\n", n_int_n_ents); >> >> + puts ("\ >> +\n\ >> +#endif /* insn-modes.h */"); >> +} >> + >> +static void >> +emit_insn_modes_inline_h (void) >> +{ >> + printf ("/* Generated automatically from machmode.def%s%s\n", >> + HAVE_EXTRA_MODES ? " and " : "", >> + EXTRA_MODES_FILE); >> + >> + puts ("\ >> + by genmodes. */\n\ >> +\n\ >> +#ifndef GCC_INSN_MODES_INLINE_H\n\ >> +#define GCC_INSN_MODES_INLINE_H"); >> + >> puts ("\n#if !defined (USED_FOR_TARGET) && GCC_VERSION >= 4001\n"); >> emit_mode_size_inline (); >> emit_mode_nunits_inline (); >> @@ -1214,7 +1232,7 @@ enum machine_mode\n{"); >> >> puts ("\ >> \n\ >> -#endif /* insn-modes.h */"); >> +#endif /* insn-modes-inline.h */"); >> } >> >> static void >> @@ -1231,7 +1249,6 @@ emit_insn_modes_c_header (void) >> #include \"system.h\"\n\ >> #include \"coretypes.h\"\n\ >> #include \"tm.h\"\n\ >> -#include \"machmode.h\"\n\ >> #include \"real.h\""); >> } >> >> @@ -1247,7 +1264,7 @@ emit_min_insn_modes_c_header (void) >> \n\ >> #include \"bconfig.h\"\n\ >> #include \"system.h\"\n\ >> -#include \"machmode.h\""); >> +#include \"coretypes.h\""); >> } >> >> static void >> @@ -1799,18 +1816,20 @@ emit_min_insn_modes_c (void) >> int >> main (int argc, char **argv) >> { >> - bool gen_header = false, gen_min = false; >> + bool gen_header = false, gen_inlines = false, gen_min = false; >> progname = argv[0]; >> >> if (argc == 1) >> ; >> else if (argc == 2 && !strcmp (argv[1], "-h")) >> gen_header = true; >> + else if (argc == 2 && !strcmp (argv[1], "-i")) >> + gen_inlines = true; >> else if (argc == 2 && !strcmp (argv[1], "-m")) >> gen_min = true; >> else >> { >> - error ("usage: %s [-h|-m] > file", progname); >> + error ("usage: %s [-h|-i|-m] > file", progname); >> return FATAL_EXIT_CODE; >> } >> >> @@ -1826,6 +1845,8 @@ main (int argc, char **argv) >> >> if (gen_header) >> emit_insn_modes_h (); >> + else if (gen_inlines) >> + emit_insn_modes_inline_h (); >> else if (gen_min) >> emit_min_insn_modes_c (); >> else ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: Reorganise machmode.h headers 2017-05-24 15:08 ` Richard Sandiford 2017-06-12 6:37 ` Richard Sandiford @ 2017-06-29 22:39 ` Jeff Law 1 sibling, 0 replies; 7+ messages in thread From: Jeff Law @ 2017-06-29 22:39 UTC (permalink / raw) To: gcc-patches, richard.sandiford On 05/24/2017 08:54 AM, Richard Sandiford wrote: > Jeff Law <law@redhat.com> writes: >> On 11/16/2016 09:32 AM, Richard Sandiford wrote: >>> Later patches will make machmode.h rely on wide-int.h and the >>> new poly-int.h, so it needs to appear later in the coretypes.h >>> include list. >>> >>> Previously machmode.h included insn-modes.h, which as well as >>> the main mode enum contains configuration information like >>> MAX_BITSIZE_MODE_ANY_INT. This still needs to come first, >>> since files like wide-int.h depend on the configuration >>> information. >>> >>> Similarly, later patches will make the auto-generated inline >>> mode size functions use poly-int.h, so the patch splits them >>> out into their own header file and includes it after the >>> integer utilities. >>> >>> The patch also makes the generator files include machmode.h >>> via coretypes.h. Previously they did it by more indirect means. >>> >>> Finally, the patch makes wide-int-print.h available via coretypes.h >>> too. There didn't seem to be any reason to force only the print >>> routines to be included directly, and it would be painful to extend >>> that approach to the new polynomial integer classes. >>> >>> Tested on aarch64-linux-gnu and x86_64-linux-gnu. OK to install? >>> >>> Thanks, >>> Richard >>> >>> >>> [ This patch is part of the SVE series posted here: >>> https://gcc.gnu.org/ml/gcc/2016-11/msg00030.html ] >>> >>> gcc/ >>> 2016-11-16 Richard Sandiford <richard.sandiford@arm.com> >>> Alan Hayward <alan.hayward@arm.com> >>> David Sherwood <david.sherwood@arm.com> >>> >>> * Makefile.in (MACHMODE_H): Remove insn-modes.h >>> (CORETYPES_H): New define. >>> (MOSTLYCLEANFILES): Add insn-modes-inline.h. >>> (insn-modes-inline.h, s-modes-inline-h): New rules. >>> (generated_files): Add insn-modes-inline.h. >>> (RTL_BASE_H, TREE_CORE_H): Use CORETYPES_H instead of coretypes.h. >>> (build/gensupport.o, build/print-rtl.o, build/read-md.o): Likewise. >>> (build/read-rtl.o, build/rtl.o, build/vec.o, build/hash-table.o) >>> (build/inchash.o, build/gencondmd.o, build/genattr.o): Likewise. >>> (build/genattr-common.o, build/genattrtab.o, build/genautomata.o) >>> (build/gencheck.o, build/gencodes.o, build/genconditions.o): Likewise. >>> (build/genconfig.o, build/genconstants.o, build/genemit.o): Likewise. >>> (build/genenums.o, build/genextract.o, build/genflags.o): Likewise. >>> (build/gentarget-def.o, build/genmddeps.o, build/genopinit.o) >>> (build/genoutput.o, build/genpeep.o, build/genpreds.o): Likewise. >>> (build/genrecog.o, build/genmddump.o, build/genmatch.o): Likewise. >>> (build/gencfn-macros.o, build/gcov-iov.o): Likewise. >>> * coretypes.h: Include everything up to real.h for generators. >>> Include insn-modes.h first. Include wide-int-print.h after >>> wide-int.h. Include insn-modes-inline.h and then machmode.h. >>> * machmode.h: Don't include insn-modes.h here. >>> * function-tests.c: Remove includes of signop.h, machmode.h, >>> double-int.h and wide-int.h. >>> * rtl.h: Likewise. >>> * gcc-rich-location.c: Remove includes of machmode.h, double-int.h >>> and wide-int.h. >>> * optc-save-gen.awk: Likewise. >>> * gencheck.c (BITS_PER_UNIT): Delete dummy definition. >>> * godump.c: Remove include of wide-int-print.h. >>> * pretty-print.h: Likewise. >>> * wide-int-print.cc: Likewise. >>> * wide-int.cc: Likewise. >>> * hash-map-tests.c: Remove include of signop.h. >>> * hash-set-tests.c: Likewise. >>> * rtl-tests.c: Likewise. >>> * mkconfig.sh: Remove include of machmode.h. >>> * genmodes.c (emit_insn_modes_h): Split emission of inline functions >>> into... >>> (emit_insn_modes_inline_h): ...this new function. Emit the code >>> into an insn-modes-inline.h header file, adding appropriate >>> include guards and end comments. >>> (emit_insn_modes_c_header): Remove include of machmode.h. >>> (emit_min_insn_modes_c_header): Include coretypes.h rather than >>> machmode.h. >>> (main): Handle -i flag and call emit_insn_modes_inline_h when >>> it is passed. >> So I don't see anything here particularly problematical. My question is >> whether or not there's anything significant to be gained to moving >> forward with this kit, assuming the 67 piece kit is not likely to move >> forward. >> >> I do think you'll need some tweaks to the contrib/header-tools which >> know about the core headers and dependencies. Hopefully what's in there >> is easy enough to figure out how to twiddle appropriately. > > OK, thanks for the pointer. I think this patch should do that. > > gcc-order-headers seems to have bitrotted a bit, since it gives: > > Traceback (most recent call last): > File "../contrib/header-tools/gcc-order-headers", line 267, in <module> > process_known_dups () > File "../contrib/header-tools/gcc-order-headers", line 101, in process_known_dups > if dups[i] and "rtl.h" in dups[i]: > KeyError: 'dumpfile.h' > > But the change itself looks obvious. > > Tested on aarch64-linux-gnu and x86_64-linux-gnu. Also tested by > compiling one target for each cpu directory. OK to install? > > Thanks, > Richard > > > [Based on the code ARM contributed in branches/ARM/sve-branch@242100] > > 2016-11-16 Richard Sandiford <richard.sandiford@linaro.org> > Alan Hayward <alan.hayward@arm.com> > David Sherwood <david.sherwood@arm.com> > > contrib/header-tools/ > * graph-header-logs (ignore): Update coretypes.h header list. > > gcc/ > * Makefile.in (MACHMODE_H): Remove insn-modes.h > (CORETYPES_H): New define. > (MOSTLYCLEANFILES): Add insn-modes-inline.h. > (insn-modes-inline.h, s-modes-inline-h): New rules. > (generated_files): Add insn-modes-inline.h. > (RTL_BASE_H, TREE_CORE_H): Use CORETYPES_H instead of coretypes.h. > (build/gensupport.o, build/ggc-none.o, build/print-rtl.o): Likewise. > (build/read-md.o, build/read-rtl.o, build/rtl.o): Likewise. > (build/vec.o, build/hash-table.o, build/inchash.o): Likewise. > (build/gencondmd.o, build/genattr.o, build/genattr-common.o): Likewise. > (build/genattrtab.o, build/genautomata.o, build/gencheck.o): Likewise. > (build/gencodes.o, build/genconditions.o): Likewise. > (build/genconfig.o, build/genconstants.o, build/genemit.o): Likewise. > (build/genenums.o, build/genextract.o, build/genflags.o): Likewise. > (build/gentarget-def.o, build/genmddeps.o, build/genopinit.o) > (build/genoutput.o, build/genpeep.o, build/genpreds.o): Likewise. > (build/genrecog.o, build/genmddump.o, build/genmatch.o): Likewise. > (build/gencfn-macros.o, build/gcov-iov.o): Likewise. > * coretypes.h: Include everything up to real.h for generators. > Include insn-modes.h first. Include wide-int-print.h after > wide-int.h. Include insn-modes-inline.h and then machmode.h. > * machmode.h: Don't include insn-modes.h here. > * function-tests.c: Remove includes of signop.h, machmode.h, > double-int.h and wide-int.h. > * rtl.h: Likewise. > * gcc-rich-location.c: Remove includes of machmode.h, double-int.h > and wide-int.h. > * optc-save-gen.awk: Likewise. > * gencheck.c (BITS_PER_UNIT): Delete dummy definition. > * godump.c: Remove include of wide-int-print.h. > * pretty-print.h: Likewise. > * wide-int-print.cc: Likewise. > * wide-int.cc: Likewise. > * hash-map-tests.c: Remove include of signop.h. > * hash-set-tests.c: Likewise. > * rtl-tests.c: Likewise. > * mkconfig.sh: Remove include of machmode.h. > * genmodes.c (emit_insn_modes_h): Split emission of inline functions > into... > (emit_insn_modes_inline_h): ...this new function. Emit the code > into an insn-modes-inline.h header file, adding appropriate > include guards and end comments. > (emit_insn_modes_c_header): Remove include of machmode.h. > (emit_min_insn_modes_c_header): Include coretypes.h rather than > machmode.h. > (main): Handle -i flag and call emit_insn_modes_inline_h when > it is passed. OK. Jeff ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2017-06-29 22:39 UTC | newest] Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2016-11-16 16:32 Reorganise machmode.h headers Richard Sandiford 2016-12-20 0:43 ` Jeff Law 2016-12-22 16:19 ` Richard Sandiford 2017-05-24 15:08 ` Richard Sandiford 2017-06-12 6:37 ` Richard Sandiford 2017-06-22 11:34 ` Richard Sandiford 2017-06-29 22:39 ` Jeff Law
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).