Commit 4f00562
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>1 parent 4c5fc5c commit 4f00562
1 file changed
Lines changed: 13 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -141,6 +141,7 @@ clean: ## Remove build artifacts. | |||
| 141 | 141 | $(RM) -r test/tmp* | |
| 142 | 142 | $(RM) -r test/.tmp* | |
| 143 | 143 | $(MAKE) test-addons-clean | |
| 144 | + $(MAKE) bench-addons-clean | ||
| 144 | 145 | ||
| 145 | 146 | .PHONY: distclean | |
| 146 | 147 | distclean: | |
@@ -1054,13 +1055,23 @@ ifeq ($(XZ), 0) | |||
| 1054 | 1055 | endif | |
| 1055 | 1056 | ||
| 1056 | 1057 | .PHONY: bench-all | |
| 1057 | - bench-all: | ||
| 1058 | + bench-all: bench-addons-build | ||
| 1058 | 1059 | @echo "Please use benchmark/run.js or benchmark/compare.js to run the benchmarks." | |
| 1059 | 1060 | ||
| 1060 | 1061 | .PHONY: bench | |
| 1061 | - bench: | ||
| 1062 | + bench: bench-addons-build | ||
| 1062 | 1063 | @echo "Please use benchmark/run.js or benchmark/compare.js to run the benchmarks." | |
| 1063 | 1064 | ||
| 1065 | + # Build required addons for benchmark before running it. | ||
| 1066 | + .PHONY: bench-addons-build | ||
| 1067 | + bench-addons-build: benchmark/napi/function_call/build/Release/binding.node \ | ||
| 1068 | + benchmark/napi/function_args/build/Release/binding.node | ||
| 1069 | + | ||
| 1070 | + .PHONY: bench-addons-clean | ||
| 1071 | + bench-addons-clean: | ||
| 1072 | + $(RM) -r benchmark/napi/function_call/build | ||
| 1073 | + $(RM) -r benchmark/napi/function_args/build | ||
| 1074 | + | ||
| 1064 | 1075 | .PHONY: lint-md-clean | |
| 1065 | 1076 | lint-md-clean: | |
| 1066 | 1077 | $(RM) -r tools/remark-cli/node_modules | |
0 commit comments