* [patch] Fix many Makefile dependencies, round 3
@ 2012-05-31 16:48 Steven Bosscher
2012-06-01 9:17 ` Richard Guenther
0 siblings, 1 reply; 2+ messages in thread
From: Steven Bosscher @ 2012-05-31 16:48 UTC (permalink / raw)
To: Richard Guenther; +Cc: GCC Patches
[-- Attachment #1: Type: text/plain, Size: 444 bytes --]
Hello,
A smaller patch this time, but bigger impact: GIMPLE_H does not have
to depend on TARGET_H. Only a few files need TARGET_H as well as
GIMPLE_H, and most of them already included TARGET_H anyway. The ones
that don't, are fixed with this patch.
Bootstrapped&tested on powerpc64-unknown-linux-gnu, and verified that
my contrib/check_makefile_deps.sh output has no new missing
dependencies (and far fewer unneeded deps). OK?
Ciao!
Steven
[-- Attachment #2: fix_Makefile_in_3.diff --]
[-- Type: application/octet-stream, Size: 2687 bytes --]
diff -u Makefile.in Makefile.in
--- Makefile.in (working copy)
+++ Makefile.in (working copy)
@@ -864,7 +864,7 @@
REGSET_H = regset.h $(BITMAP_H) hard-reg-set.h
BASIC_BLOCK_H = basic-block.h $(PREDICT_H) $(VEC_H) $(FUNCTION_H) cfghooks.h
GIMPLE_H = gimple.h gimple.def gsstruct.def pointer-set.h $(VEC_H) \
- vecir.h $(GGC_H) $(BASIC_BLOCK_H) $(TARGET_H) tree-ssa-operands.h \
+ vecir.h $(GGC_H) $(BASIC_BLOCK_H) tree-ssa-operands.h \
tree-ssa-alias.h $(INTERNAL_FN_H)
TRANS_MEM_H = trans-mem.h
GCOV_IO_H = gcov-io.h gcov-iov.h auto-host.h
@@ -2473,7 +2473,7 @@
$(TREE_H) $(TM_H) $(FLAGS_H) $(FUNCTION_H) $(EXCEPT_H) langhooks.h \
$(GGC_H) $(TREE_PASS_H) coretypes.h $(TIMEVAR_H) pointer-set.h \
$(TREE_DUMP_H) $(TREE_INLINE_H) tree-iterator.h toplev.h \
- $(DIAGNOSTIC_CORE_H) $(CFGLOOP_H)
+ $(DIAGNOSTIC_CORE_H) $(TARGET_H) $(CFGLOOP_H)
tree-ssa-loop.o : tree-ssa-loop.c $(CONFIG_H) $(SYSTEM_H) coretypes.h $(TM_H) \
$(TREE_H) $(TM_P_H) $(BASIC_BLOCK_H) output.h \
$(DIAGNOSTIC_H) $(TREE_FLOW_H) $(TREE_DUMP_H) $(TREE_PASS_H) $(TIMEVAR_H) \
@@ -2551,7 +2551,7 @@
$(TM_H) coretypes.h $(TREE_DUMP_H) $(TREE_PASS_H) $(FLAGS_H) \
tree-iterator.h $(BASIC_BLOCK_H) $(GIMPLE_H) $(TREE_INLINE_H) \
$(VEC_H) langhooks.h alloc-pool.h pointer-set.h $(CFGLOOP_H) \
- $(GIMPLE_PRETTY_PRINT_H) $(DIAGNOSTIC_CORE_H)
+ $(TARGET_H) $(GIMPLE_PRETTY_PRINT_H) $(DIAGNOSTIC_CORE_H)
tree-optimize.o : tree-optimize.c $(TREE_FLOW_H) $(CONFIG_H) $(SYSTEM_H) \
$(TREE_H) $(TM_P_H) $(GGC_H) output.h \
$(DIAGNOSTIC_H) $(BASIC_BLOCK_H) $(FLAGS_H) $(TIMEVAR_H) $(TM_H) \
@@ -2938,7 +2938,7 @@
$(GGC_H) $(TIMEVAR_H) debug.h $(TARGET_H) output.h $(GIMPLE_H) \
$(TREE_FLOW_H)
ipa.o : ipa.c $(CONFIG_H) $(SYSTEM_H) coretypes.h $(TM_H) $(CGRAPH_H) \
- $(TREE_PASS_H) $(TIMEVAR_H) $(GIMPLE_H) $(GGC_H) pointer-set.h \
+ $(TREE_PASS_H) $(TIMEVAR_H) $(GIMPLE_H) $(TARGET_H) $(GGC_H) pointer-set.h \
$(IPA_UTILS_H)
ipa-prop.o : ipa-prop.c $(CONFIG_H) $(SYSTEM_H) coretypes.h \
langhooks.h $(GGC_H) $(TARGET_H) $(CGRAPH_H) $(IPA_PROP_H) $(DIAGNOSTIC_H) \
@@ -3127,7 +3127,7 @@
mcf.o : mcf.c profile.h $(CONFIG_H) $(SYSTEM_H) $(TM_H) coretypes.h \
$(BASIC_BLOCK_H) output.h langhooks.h $(GCOV_IO_H) $(TREE_H)
tree-profile.o : tree-profile.c $(CONFIG_H) $(SYSTEM_H) coretypes.h \
- $(TM_H) $(TREE_H) $(FLAGS_H) $(FUNCTION_H) \
+ $(TM_H) $(TARGET_H) $(TREE_H) $(FLAGS_H) $(FUNCTION_H) \
$(BASIC_BLOCK_H) $(DIAGNOSTIC_CORE_H) $(COVERAGE_H) $(TREE_H) value-prof.h $(TREE_DUMP_H) \
$(TREE_PASS_H) $(TREE_FLOW_H) $(TIMEVAR_H) gt-tree-profile.h $(CGRAPH_H)
value-prof.o : value-prof.c $(CONFIG_H) $(SYSTEM_H) coretypes.h $(TM_H) \
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [patch] Fix many Makefile dependencies, round 3
2012-05-31 16:48 [patch] Fix many Makefile dependencies, round 3 Steven Bosscher
@ 2012-06-01 9:17 ` Richard Guenther
0 siblings, 0 replies; 2+ messages in thread
From: Richard Guenther @ 2012-06-01 9:17 UTC (permalink / raw)
To: Steven Bosscher; +Cc: GCC Patches
On Thu, May 31, 2012 at 6:48 PM, Steven Bosscher <stevenb.gcc@gmail.com> wrote:
> Hello,
>
> A smaller patch this time, but bigger impact: GIMPLE_H does not have
> to depend on TARGET_H. Only a few files need TARGET_H as well as
> GIMPLE_H, and most of them already included TARGET_H anyway. The ones
> that don't, are fixed with this patch.
>
> Bootstrapped&tested on powerpc64-unknown-linux-gnu, and verified that
> my contrib/check_makefile_deps.sh output has no new missing
> dependencies (and far fewer unneeded deps). OK?
Ok.
Thanks,
Richard.
> Ciao!
> Steven
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2012-06-01 9:17 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-05-31 16:48 [patch] Fix many Makefile dependencies, round 3 Steven Bosscher
2012-06-01 9:17 ` Richard Guenther
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).