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>
nodejs · Jul 13, 2018 · 4f00562ef093833c54698ea0bc60a5d2522a2b38 · 4f00562
1 parent 4c5fc5c
commit 4f00562ef093833c54698ea0bc60a5d2522a2b38
Unified Split

Showing with 13 additions and 2 deletions.

+13 −2 Makefile
diff --git a/Makefile b/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:
@@ -1054,13 +1055,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