Commit 42ece78c authored by Clemens Hammacher's avatar Clemens Hammacher Committed by Commit Bot

[wasm] Remove unused CompilationEnv for wrappers

R=titzer@chromium.org

Bug: v8:8238
Change-Id: I458656fcd04b7d27054717842910d563d81c5392
Reviewed-on: https://chromium-review.googlesource.com/c/1332301Reviewed-by: 's avatarBen Titzer <titzer@chromium.org>
Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#57575}
parent 61029a5c
...@@ -4130,11 +4130,10 @@ void RecordFunctionCompilation(CodeEventListener::LogEventsAndTags tag, ...@@ -4130,11 +4130,10 @@ void RecordFunctionCompilation(CodeEventListener::LogEventsAndTags tag,
class WasmWrapperGraphBuilder : public WasmGraphBuilder { class WasmWrapperGraphBuilder : public WasmGraphBuilder {
public: public:
WasmWrapperGraphBuilder(Zone* zone, wasm::CompilationEnv* env, WasmWrapperGraphBuilder(Zone* zone, JSGraph* jsgraph, wasm::FunctionSig* sig,
JSGraph* jsgraph, wasm::FunctionSig* sig,
compiler::SourcePositionTable* spt, compiler::SourcePositionTable* spt,
StubCallMode stub_mode) StubCallMode stub_mode)
: WasmGraphBuilder(env, zone, jsgraph, sig, spt), : WasmGraphBuilder(nullptr, zone, jsgraph, sig, spt),
isolate_(jsgraph->isolate()), isolate_(jsgraph->isolate()),
jsgraph_(jsgraph), jsgraph_(jsgraph),
stub_mode_(stub_mode) {} stub_mode_(stub_mode) {}
...@@ -4947,9 +4946,7 @@ MaybeHandle<Code> CompileJSToWasmWrapper(Isolate* isolate, ...@@ -4947,9 +4946,7 @@ MaybeHandle<Code> CompileJSToWasmWrapper(Isolate* isolate,
Node* control = nullptr; Node* control = nullptr;
Node* effect = nullptr; Node* effect = nullptr;
wasm::CompilationEnv env(nullptr, wasm::kNoTrapHandler, WasmWrapperGraphBuilder builder(&zone, &jsgraph, sig, nullptr,
wasm::kRuntimeExceptionSupport);
WasmWrapperGraphBuilder builder(&zone, &env, &jsgraph, sig, nullptr,
StubCallMode::kCallOnHeapBuiltin); StubCallMode::kCallOnHeapBuiltin);
builder.set_control_ptr(&control); builder.set_control_ptr(&control);
builder.set_effect_ptr(&effect); builder.set_effect_ptr(&effect);
...@@ -5058,11 +5055,7 @@ wasm::WasmCode* CompileWasmImportCallWrapper(Isolate* isolate, ...@@ -5058,11 +5055,7 @@ wasm::WasmCode* CompileWasmImportCallWrapper(Isolate* isolate,
SourcePositionTable* source_position_table = SourcePositionTable* source_position_table =
source_positions ? new (&zone) SourcePositionTable(&graph) : nullptr; source_positions ? new (&zone) SourcePositionTable(&graph) : nullptr;
wasm::CompilationEnv env(nullptr, wasm::kNoTrapHandler, WasmWrapperGraphBuilder builder(&zone, &jsgraph, sig, source_position_table,
wasm::kRuntimeExceptionSupport);
WasmWrapperGraphBuilder builder(&zone, &env, &jsgraph, sig,
source_position_table,
StubCallMode::kCallWasmRuntimeStub); StubCallMode::kCallWasmRuntimeStub);
builder.set_control_ptr(&control); builder.set_control_ptr(&control);
builder.set_effect_ptr(&effect); builder.set_effect_ptr(&effect);
...@@ -5105,7 +5098,7 @@ wasm::WasmCode* CompileWasmInterpreterEntry(Isolate* isolate, ...@@ -5105,7 +5098,7 @@ wasm::WasmCode* CompileWasmInterpreterEntry(Isolate* isolate,
Node* control = nullptr; Node* control = nullptr;
Node* effect = nullptr; Node* effect = nullptr;
WasmWrapperGraphBuilder builder(&zone, nullptr, &jsgraph, sig, nullptr, WasmWrapperGraphBuilder builder(&zone, &jsgraph, sig, nullptr,
StubCallMode::kCallWasmRuntimeStub); StubCallMode::kCallWasmRuntimeStub);
builder.set_control_ptr(&control); builder.set_control_ptr(&control);
builder.set_effect_ptr(&effect); builder.set_effect_ptr(&effect);
...@@ -5143,7 +5136,7 @@ MaybeHandle<Code> CompileCWasmEntry(Isolate* isolate, wasm::FunctionSig* sig) { ...@@ -5143,7 +5136,7 @@ MaybeHandle<Code> CompileCWasmEntry(Isolate* isolate, wasm::FunctionSig* sig) {
Node* control = nullptr; Node* control = nullptr;
Node* effect = nullptr; Node* effect = nullptr;
WasmWrapperGraphBuilder builder(&zone, nullptr, &jsgraph, sig, nullptr, WasmWrapperGraphBuilder builder(&zone, &jsgraph, sig, nullptr,
StubCallMode::kCallOnHeapBuiltin); StubCallMode::kCallOnHeapBuiltin);
builder.set_control_ptr(&control); builder.set_control_ptr(&control);
builder.set_effect_ptr(&effect); builder.set_effect_ptr(&effect);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment