Skip to content

Replace withBuiltinMalloc with noLeakCheck. NFC#26074

Merged
sbc100 merged 1 commit intoemscripten-core:mainfrom
sbc100:withBuiltinMalloc
Jan 10, 2026
Merged

Replace withBuiltinMalloc with noLeakCheck. NFC#26074
sbc100 merged 1 commit intoemscripten-core:mainfrom
sbc100:withBuiltinMalloc

Conversation

@sbc100
Copy link
Copy Markdown
Collaborator

@sbc100 sbc100 commented Jan 9, 2026

This is a simpler and more targeted solution that also happens to work with WASM_ESM_INTEGRATION.

@sbc100 sbc100 requested review from aheejin and kripken January 9, 2026 23:04
@sbc100 sbc100 force-pushed the withBuiltinMalloc branch from e36efc3 to 7dc0963 Compare January 9, 2026 23:06
This is a simpler and more targeted solution that also happens to work
with WASM_ESM_INTEGRATION.
@sbc100 sbc100 force-pushed the withBuiltinMalloc branch from 7dc0963 to 275cf61 Compare January 9, 2026 23:57
Copy link
Copy Markdown
Member

@kripken kripken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@sbc100 sbc100 enabled auto-merge (squash) January 10, 2026 00:04
@sbc100 sbc100 merged commit aa81962 into emscripten-core:main Jan 10, 2026
35 checks passed
@sbc100 sbc100 deleted the withBuiltinMalloc branch January 10, 2026 00:58
inolen pushed a commit to inolen/emscripten that referenced this pull request Feb 13, 2026
…26074)

This is a simpler and more targeted solution that also happens to work
with WASM_ESM_INTEGRATION.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants