Skip to content

Do not allocate when cloning ModuleRuntimeInfo#12413

Merged
fitzgen merged 1 commit intobytecodealliance:mainfrom
fitzgen:dont-alloc-when-cloning-module-runtime-info
Jan 23, 2026
Merged

Do not allocate when cloning ModuleRuntimeInfo#12413
fitzgen merged 1 commit intobytecodealliance:mainfrom
fitzgen:dont-alloc-when-cloning-module-runtime-info

Conversation

@fitzgen
Copy link
Copy Markdown
Member

@fitzgen fitzgen commented Jan 23, 2026

We need to clone the info from the allocation request, into the allocated instance, so we should keep it a cheap operation.

@fitzgen fitzgen requested a review from a team as a code owner January 23, 2026 20:26
@fitzgen fitzgen requested review from pchickey and removed request for a team January 23, 2026 20:26
@fitzgen fitzgen enabled auto-merge January 23, 2026 20:52
@fitzgen fitzgen added this pull request to the merge queue Jan 23, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Jan 23, 2026
We need to clone the info from the allocation request, into the allocated
instance, so we should keep it a cheap operation.
@fitzgen fitzgen force-pushed the dont-alloc-when-cloning-module-runtime-info branch from 75526a9 to 220b9a5 Compare January 23, 2026 22:10
@fitzgen fitzgen enabled auto-merge January 23, 2026 22:10
@fitzgen fitzgen added this pull request to the merge queue Jan 23, 2026
Merged via the queue into bytecodealliance:main with commit dd47158 Jan 23, 2026
45 checks passed
@fitzgen fitzgen deleted the dont-alloc-when-cloning-module-runtime-info branch January 23, 2026 22:47
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