Skip to content
Permalink
Browse files

build: add new benchmark targets

Adding new build targets: 'bench-addons' & 'bench-addons-clean'. With
these two, it will be easier to manage the dependencies among targets
and easier to build/clean the addons which are being used in
benchmarking.

PR-URL: #20905
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
  • Loading branch information...
kenny-y authored and gabrielschulhof committed Jul 13, 2018
1 parent fe67287 commit 266c1f45a90937f042527d86fb2de3ad08dc0884
Showing with 13 additions and 2 deletions.
  1. +13 −2 Makefile
@@ -141,6 +141,7 @@ clean: ## Remove build artifacts.
$(RM) -r test/tmp*
$(RM) -r test/.tmp*
$(MAKE) test-addons-clean
$(MAKE) bench-addons-clean

.PHONY: distclean
distclean:
@@ -1047,13 +1048,23 @@ ifeq ($(XZ), 0)
endif

.PHONY: bench-all
bench-all:
bench-all: bench-addons-build
@echo "Please use benchmark/run.js or benchmark/compare.js to run the benchmarks."

.PHONY: bench
bench:
bench: bench-addons-build
@echo "Please use benchmark/run.js or benchmark/compare.js to run the benchmarks."

# Build required addons for benchmark before running it.
.PHONY: bench-addons-build
bench-addons-build: benchmark/napi/function_call/build/Release/binding.node \
benchmark/napi/function_args/build/Release/binding.node

.PHONY: bench-addons-clean
bench-addons-clean:
$(RM) -r benchmark/napi/function_call/build
$(RM) -r benchmark/napi/function_args/build

.PHONY: lint-md-clean
lint-md-clean:
$(RM) -r tools/remark-cli/node_modules

0 comments on commit 266c1f4

Please sign in to comment.
You can’t perform that action at this time.