Skip to content

Commit

Permalink
NighlytTest: Eliminate test-set and test-finish phony targets, relyin…
Browse files Browse the repository at this point in the history
…g on dependency declaration order is bogus.

llvm-svn: 91031
  • Loading branch information
ddunbar committed Dec 10, 2009
1 parent 51662ef commit 426e538
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
5 changes: 1 addition & 4 deletions Makefile.programs
Original file line number Diff line number Diff line change
Expand Up @@ -738,10 +738,7 @@ endif
# the default target is the test target. Here we dispatch to a specific set of
# tests.
#
.PHONY: test-setup test-finish
test-setup::
test-finish::
test:: test-setup $(PROGRAMS_TO_TEST:%=test.$(TEST).%) test-finish
test:: $(PROGRAMS_TO_TEST:%=test.$(TEST).%)

# AVAILABLE_TESTS - Compute the set of tests available for user help
#
Expand Down
2 changes: 0 additions & 2 deletions TEST.buildrepo.Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,6 @@ DESTDIR := $(BYTECODE_REPOSITORY)/$(subst $(PROGDIR),,$(CURDIR))

.PRECIOUS: $(DESTDIR)/.dir $(DESTDIR)/%.bc

test-setup:: $(DESTDIR)/.dir

# To Make a file up-to-date, just copy it over.
$(PROGRAMS_TO_TEST:%=$(DESTDIR)/%.bc): \
$(DESTDIR)/%.bc: Output/%.llvm.bc
Expand Down

0 comments on commit 426e538

Please sign in to comment.