Skip to content

Commit

Permalink
Remove 'check-dg', a wrapper around 'check-local-dg' which was just
Browse files Browse the repository at this point in the history
nuked.

Add a comment that the 'check-lit' rule is really just a legacy of
having two test runners.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@159310 91177308-0d34-0410-b5e6-96231b3b80d8
  • Loading branch information
chandlerc committed Jun 28, 2012
1 parent 01e4a06 commit c1a0af1
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 24 deletions.
13 changes: 1 addition & 12 deletions Makefile.rules
Original file line number Diff line number Diff line change
Expand Up @@ -1964,20 +1964,9 @@ check::
$(EchoCmd) No test directory ; \
fi

# An alias dating from when both lit and DejaGNU test runners were used.
check-lit:: check

check-dg::
$(Verb) if test -d "$(PROJ_OBJ_ROOT)/test" ; then \
if test -f "$(PROJ_OBJ_ROOT)/test/Makefile" ; then \
$(EchoCmd) Running test suite ; \
$(MAKE) -C $(PROJ_OBJ_ROOT)/test check-local-dg ; \
else \
$(EchoCmd) No Makefile in test directory ; \
fi ; \
else \
$(EchoCmd) No test directory ; \
fi

check-all::
$(Verb) if test -d "$(PROJ_OBJ_ROOT)/test" ; then \
if test -f "$(PROJ_OBJ_ROOT)/test/Makefile" ; then \
Expand Down
13 changes: 1 addition & 12 deletions projects/sample/Makefile.llvm.rules
Original file line number Diff line number Diff line change
Expand Up @@ -1866,20 +1866,9 @@ check::
$(EchoCmd) No test directory ; \
fi

# An alias dating from when both lit and DejaGNU test runners were used.
check-lit:: check

check-dg::
$(Verb) if test -d "$(PROJ_OBJ_ROOT)/test" ; then \
if test -f "$(PROJ_OBJ_ROOT)/test/Makefile" ; then \
$(EchoCmd) Running test suite ; \
$(MAKE) -C $(PROJ_OBJ_ROOT)/test check-local-dg ; \
else \
$(EchoCmd) No Makefile in test directory ; \
fi ; \
else \
$(EchoCmd) No test directory ; \
fi

check-all::
$(Verb) if test -d "$(PROJ_OBJ_ROOT)/test" ; then \
if test -f "$(PROJ_OBJ_ROOT)/test/Makefile" ; then \
Expand Down

0 comments on commit c1a0af1

Please sign in to comment.