* 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).