Commit 3f65598
src: remove kEagerCompile for CompileFunction
It wasn't doing anything, and actually enabling it would cause some
tests to fail.
Refs: #48576
PR-URL: #48671
Refs: v8/v8@cb00db4
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>1 parent 3b8ec34 commit 3f65598
2 files changed
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -297,7 +297,7 @@ MaybeLocal<Function> BuiltinLoader::LookupAndCompileInternal( | |||
| 297 | 297 | const bool has_cache = cached_data != nullptr; | |
| 298 | 298 | ScriptCompiler::CompileOptions options = | |
| 299 | 299 | has_cache ? ScriptCompiler::kConsumeCodeCache | |
| 300 | - : ScriptCompiler::kEagerCompile; | ||
| 300 | + : ScriptCompiler::kNoCompileOptions; | ||
| 301 | 301 | ScriptCompiler::Source script_source(source, origin, cached_data); | |
| 302 | 302 | ||
| 303 | 303 | per_process::Debug(DebugCategory::CODE_CACHE, | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -1484,7 +1484,7 @@ void CompileSerializeMain(const FunctionCallbackInfo<Value>& args) { | |||
| 1484 | 1484 | parameters.data(), | |
| 1485 | 1485 | 0, | |
| 1486 | 1486 | nullptr, | |
| 1487 | - ScriptCompiler::kEagerCompile) | ||
| 1487 | + ScriptCompiler::kNoCompileOptions) | ||
| 1488 | 1488 | .ToLocal(&fn)) { | |
| 1489 | 1489 | args.GetReturnValue().Set(fn); | |
| 1490 | 1490 | } | |
0 commit comments