sea: add support for V8 bytecode-only caching - #48191

Merged

nodejs-github-bot merged 27 commits into

nodejs:mainfrom

RaisinTen:sea-V8-code-cache

Jul 26, 2023

Commits on Jul 21, 2023

sea: add support for V8 bytecode-only caching
RaisinTen
committed
18e0a49
src: use `std::nullopt` instead of `{}`
RaisinTen
committed
184e9e6
src: pass parameters vector by a pointer
RaisinTen
committed
51cf8c1
sea: pass code cache data as an external buffer to JS
RaisinTen
committed
ae48e90
sea: fix entry point file name
RaisinTen
committed
145335b
src: use MaybeLocal<Function>() instead of {}

RaisinTen
and
jasnell
committed
f855bd5
src: use DataView instead of Buffer for code cache
RaisinTen
committed
f549743
sea: warn if code cache is rejected
RaisinTen
committed
81f896f
fixup! sea: warn if code cache is rejected
RaisinTen
committed
4210887
src: add TODO for supporting `import()`
RaisinTen
committed
e8aa501
src: return result of CompileFunction directly
RaisinTen
committed
b5b2d78
sea: make code caching optional
RaisinTen
committed
5cd337f
src: pass length to v8::String::NewFromUtf8() for std::string_views
RaisinTen
committed
27b9a6c
test: adjust line numbers in test/fixtures/errors/force_colors.snapshot
RaisinTen
committed
c62e892
src: use kNoCompileOptions instead of kEagerCompile
RaisinTen
committed
1f8f86b
Apply suggestions from code review

RaisinTen
and
joyeecheung
committed
6589352
src: pass only the context to CompileFunction
RaisinTen
committed
9fe2f4d
test: add a comment to explain how unexpected warnings are treated
RaisinTen
committed
4020a9f
src: fix conflicts
RaisinTen
committed
72ed49f
doc: run -> compile
RaisinTen
committed
86d3407
doc: add note about import() not working
RaisinTen
committed
fc9509b
sea: call getCodePath() only when isSea() is true
RaisinTen
committed
80dbcbf
lib: do not load SEA binding for non-SEA code paths
RaisinTen
committed
b1792b5
src: remove unnecessary HandleScope
RaisinTen
committed
2a2d12e
src: guard against calling sea.getCodePath() in userland
RaisinTen
committed
80cd1ff
src: remove unnecessary HandleScope
RaisinTen
committed
d44aa0d
test: fix test/parallel/test-node-output-errors.mjs error
RaisinTen
committed
91bf2df

Navigation Menu

Uh oh!

Uh oh!

sea: add support for V8 bytecode-only caching - #48191

sea: add support for V8 bytecode-only caching#48191
nodejs-github-bot merged 27 commits into
nodejs:mainfrom
RaisinTen:sea-V8-code-cache

Commits on Jul 21, 2023

sea: add support for V8 bytecode-only caching

src: use `std::nullopt` instead of `{}`

src: pass parameters vector by a pointer

sea: pass code cache data as an external buffer to JS

sea: fix entry point file name

src: use MaybeLocal<Function>() instead of {}

src: use DataView instead of Buffer for code cache

sea: warn if code cache is rejected

fixup! sea: warn if code cache is rejected

src: add TODO for supporting `import()`

src: return result of CompileFunction directly

sea: make code caching optional

src: pass length to v8::String::NewFromUtf8() for std::string_views

test: adjust line numbers in test/fixtures/errors/force_colors.snapshot

src: use kNoCompileOptions instead of kEagerCompile

Apply suggestions from code review

src: pass only the context to CompileFunction

test: add a comment to explain how unexpected warnings are treated

src: fix conflicts

doc: run -> compile

doc: add note about import() not working

sea: call getCodePath() only when isSea() is true

lib: do not load SEA binding for non-SEA code paths

src: remove unnecessary HandleScope

src: guard against calling sea.getCodePath() in userland

src: remove unnecessary HandleScope

test: fix test/parallel/test-node-output-errors.mjs error