Skip to content

Expose WAMR_BUILD_GC_HEAP_SIZE_DEFAULT as a CMake option#4124

Merged
lum1n0us merged 4 commits intobytecodealliance:mainfrom
swankjesse:WAMR_BUILD_GC_HEAP_SIZE_DEFAULT
Mar 12, 2025
Merged

Expose WAMR_BUILD_GC_HEAP_SIZE_DEFAULT as a CMake option#4124
lum1n0us merged 4 commits intobytecodealliance:mainfrom
swankjesse:WAMR_BUILD_GC_HEAP_SIZE_DEFAULT

Conversation

@swankjesse
Copy link
Copy Markdown

This is wired through to the GC_HEAP_SIZE_DEFAULT constant.

Also honor this value when configuring the engine with the wasm_c_api.

This is wired through to the GC_HEAP_SIZE_DEFAULT constant.

Also honor this value when configuring the engine with the
wasm_c_api.
@swankjesse
Copy link
Copy Markdown
Author

Alternately, it’d be great to expose the gc_heap_size as option in wasm_c_api.h directly. That’d be ideal as it’d give callers the ability to set a heap size at runtime instead of at build time.

@swankjesse swankjesse force-pushed the WAMR_BUILD_GC_HEAP_SIZE_DEFAULT branch from bb949f9 to d609acf Compare March 4, 2025 23:43
Copy link
Copy Markdown
Contributor

@lum1n0us lum1n0us left a comment

Choose a reason for hiding this comment

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

LGTM

@swankjesse swankjesse force-pushed the WAMR_BUILD_GC_HEAP_SIZE_DEFAULT branch from 578ee38 to 9027b2d Compare March 5, 2025 15:41
Copy link
Copy Markdown
Contributor

@TianlongLiang TianlongLiang left a comment

Choose a reason for hiding this comment

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

LGTM

@lum1n0us lum1n0us merged commit c6f9344 into bytecodealliance:main Mar 12, 2025
386 checks passed
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.

4 participants