Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expose WAMR_BUILD_GC_HEAP_SIZE_DEFAULT as a CMake option #4124

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

swankjesse
Copy link

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
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
Collaborator

@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
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