Skip to content

Commit

Permalink
disable asyncify-advise, its just really spammy
Browse files Browse the repository at this point in the history
  • Loading branch information
justjake committed Aug 31, 2024
1 parent 31aa729 commit 2af655d
Show file tree
Hide file tree
Showing 11 changed files with 1 addition and 11 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,6 @@ CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-extension.js
CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-sourceMapJson.js
CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-wasmOffsetConverter.js
CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-wasmMemory.js
CFLAGS_WASM+=-s ASYNCIFY_ADVISE=1
CFLAGS_WASM+=-O3
CFLAGS_CJS+=-s ENVIRONMENT=node
CFLAGS_MJS+=-s ENVIRONMENT=node
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,6 @@ Variant-specific Emscripten build flags:
"--pre-js $(TEMPLATES)/pre-sourceMapJson.js",
"--pre-js $(TEMPLATES)/pre-wasmOffsetConverter.js",
"--pre-js $(TEMPLATES)/pre-wasmMemory.js",
"-s ASYNCIFY_ADVISE=1",
"-O3"
]
```
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,6 @@ CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-sourceMapJson.js
CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-wasmOffsetConverter.js
CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-wasmMemory.js
CFLAGS_WASM+=-s SINGLE_FILE=1
CFLAGS_WASM+=-s ASYNCIFY_ADVISE=1
CFLAGS_WASM+=-O3


Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,6 @@ Variant-specific Emscripten build flags:
"--pre-js $(TEMPLATES)/pre-wasmOffsetConverter.js",
"--pre-js $(TEMPLATES)/pre-wasmMemory.js",
"-s SINGLE_FILE=1",
"-s ASYNCIFY_ADVISE=1",
"-O3"
]
```
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,6 @@ CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-sourceMapJson.js
CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-wasmOffsetConverter.js
CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-wasmMemory.js
CFLAGS_WASM+=-s SINGLE_FILE=1
CFLAGS_WASM+=-s ASYNCIFY_ADVISE=1
CFLAGS_WASM+=-O3
CFLAGS_CJS+=-s ENVIRONMENT=web,worker,node

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,6 @@ Variant-specific Emscripten build flags:
"--pre-js $(TEMPLATES)/pre-wasmOffsetConverter.js",
"--pre-js $(TEMPLATES)/pre-wasmMemory.js",
"-s SINGLE_FILE=1",
"-s ASYNCIFY_ADVISE=1",
"-O3"
]
```
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,6 @@ CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-sourceMapJson.js
CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-wasmOffsetConverter.js
CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-wasmMemory.js
CFLAGS_WASM+=-s SINGLE_FILE=1
CFLAGS_WASM+=-s ASYNCIFY_ADVISE=1
CFLAGS_WASM+=-O3

CFLAGS_MJS+=-s ENVIRONMENT=node
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,6 @@ Variant-specific Emscripten build flags:
"--pre-js $(TEMPLATES)/pre-wasmOffsetConverter.js",
"--pre-js $(TEMPLATES)/pre-wasmMemory.js",
"-s SINGLE_FILE=1",
"-s ASYNCIFY_ADVISE=1",
"-O3"
]
```
1 change: 0 additions & 1 deletion packages/variant-quickjs-wasmfile-debug-asyncify/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,6 @@ CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-extension.js
CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-sourceMapJson.js
CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-wasmOffsetConverter.js
CFLAGS_WASM+=--pre-js $(TEMPLATES)/pre-wasmMemory.js
CFLAGS_WASM+=-s ASYNCIFY_ADVISE=1
CFLAGS_WASM+=-O3
CFLAGS_CJS+=-s ENVIRONMENT=node
CFLAGS_MJS+=-s ENVIRONMENT=node
Expand Down
1 change: 0 additions & 1 deletion packages/variant-quickjs-wasmfile-debug-asyncify/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,6 @@ Variant-specific Emscripten build flags:
"--pre-js $(TEMPLATES)/pre-sourceMapJson.js",
"--pre-js $(TEMPLATES)/pre-wasmOffsetConverter.js",
"--pre-js $(TEMPLATES)/pre-wasmMemory.js",
"-s ASYNCIFY_ADVISE=1",
"-O3"
]
```
2 changes: 1 addition & 1 deletion scripts/prepareVariants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -223,7 +223,7 @@ function getCflags(targetName: string, variant: BuildVariant) {
flags.push("-g2")
break
case SyncMode.Asyncify:
flags.push("-s ASYNCIFY_ADVISE=1")
// flags.push("-s ASYNCIFY_ADVISE=1")
flags.push(
// # Need to use -O3 - otherwise ASYNCIFY leads to stack overflows (why?)
"-O3",
Expand Down

0 comments on commit 2af655d

Please sign in to comment.