Skip to content

Build with v8_typed_array_max_size_in_heap=0#1070

Merged
bnoordhuis merged 1 commit intodenoland:mainfrom
bnoordhuis:v8_typed_array_max_size_in_heap
Sep 18, 2022
Merged

Build with v8_typed_array_max_size_in_heap=0#1070
bnoordhuis merged 1 commit intodenoland:mainfrom
bnoordhuis:v8_typed_array_max_size_in_heap

Conversation

@bnoordhuis
Copy link
Contributor

Ensure allocation of typed arrays and arraybuffers always goes through the embedder's ArrayBufferAllocator, otherwise small buffers get moved around by the garbage collector but embedders normally want them to have fixed addresses.

Refs #1068.

Ensure allocation of typed arrays and arraybuffers always goes through
the embedder's ArrayBufferAllocator, otherwise small buffers get moved
around by the garbage collector but embedders normally want them to have
fixed addresses.
Copy link
Member

@bartlomieju bartlomieju left a comment

Choose a reason for hiding this comment

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

LGTM

@bnoordhuis bnoordhuis merged commit 5cc0a85 into denoland:main Sep 18, 2022
@bnoordhuis bnoordhuis deleted the v8_typed_array_max_size_in_heap branch September 18, 2022 22:35
@ry
Copy link
Member

ry commented Sep 19, 2022

This should be in .gn
#1073

@bnoordhuis
Copy link
Contributor Author

#1075 (oh, you edited your comment...)

What reads .gn? Neither rusty_v8 nor cargo_gn seem to reference it anywhere.

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