Skip to content

Commit dd91fa5

Browse files
authored
fix: perf: Use wasm-opt -O3 when making a release build (#870)
1 parent 9222f1d commit dd91fa5

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

runtime/js-compute-runtime/Makefile

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -59,14 +59,14 @@ ifneq ($(DEBUG),false)
5959
CARGO_FLAG :=
6060
OPT_FLAGS += -DDEBUG -DJS_DEBUG -g
6161

62-
# Define an empty WASM_STRIP macro when making a debug build
62+
# Define an empty WASM_OPT macro when making a debug build
6363
WASM_STRIP =
6464
else
6565
MODE := release
6666
CARGO_FLAG := --release
6767

6868
# Strip binaries when making a non-debug build.
69-
WASM_STRIP = wasm-opt --strip-debug -o $1 $1
69+
WASM_OPT = wasm-opt -O3 --strip-debug -o $1 $1
7070
endif
7171

7272
# The path to the wasm-tools executable
@@ -321,7 +321,7 @@ $(OBJ_DIR)/js-compute-runtime.wasm: $(OBJ_DIR)/impl/main.o
321321
$(OBJ_DIR)/js-compute-runtime.wasm: $(OBJ_DIR)/host_interface/component/fastly_world_adapter.o
322322
$(call cmd_format,WASI_LD,$@) PATH="$(FSM_SRC)/scripts:$$PATH" \
323323
$(WASI_CXX) $(LD_FLAGS) $(OPENSSL_LIBS) -o $@ $^
324-
$(call cmd_format,WASM_STRIP,$@) $(call WASM_STRIP,$@)
324+
$(call cmd_format,WASM_OPT,$@) $(call WASM_OPT,$@)
325325
$(call cmd_format,WASM_METADATA,$@) $(call WASM_METADATA,$@)
326326

327327
# Compute runtime component build ##############################################
@@ -341,7 +341,7 @@ $(OBJ_DIR)/js-compute-runtime-component.wasm: $(FSM_SRC)/host_interface/componen
341341
$(OBJ_DIR)/js-compute-runtime-component.wasm: $(OBJ_DIR)/host_interface/component/fastly_world.o
342342
$(call cmd_format,WASI_LD,$@) PATH="$(FSM_SRC)/scripts:$$PATH" \
343343
$(WASI_CXX) $(LD_FLAGS) $(OPENSSL_LIBS) -o $@ $^
344-
$(call cmd_format,WASM_STRIP,$@) $(call WASM_STRIP,$@)
344+
$(call cmd_format,WASM_OPT,$@) $(call WASM_OPT,$@)
345345
$(call cmd_format,WASM_METADATA,$@) $(call WASM_METADATA,$@)
346346

347347
# Shared builtins build ########################################################

0 commit comments

Comments
 (0)