node icon indicating copy to clipboard operation
node copied to clipboard

src: migrate from deprecated SnapshotCreator constructor

Open joyeecheung opened this issue 4 months ago • 5 comments

Previously we have been using the variant of SnapshotCreator that only passes the external references instead of v8::Isolate::CreateParams and it's about to be deprecated. Switch to using the new constructor that takes a fully CreateParams instead.

This also makes sure that the snapshot building script is using the Node.js array buffer allocator instead of a separate default one that was previously used by the old constructor. The zero fill toggle in the Node.js array buffer allocator would still be ignored during snapshot building, however, until we fixes the array buffer allocator and let V8 own the toggle backing store instead, because otherwise the snapshot would contain the external toggle address and become unreproducible.

joyeecheung avatar Oct 09 '24 16:10 joyeecheung