Commit aab1df6d authored by Ivica Bogosavljevic's avatar Ivica Bogosavljevic Committed by Commit Bot

[wasm] Add missing parameters to MachineOperatorBuilder.

MachineOperatorBuilder was instantiated with default parameters
and this caused unaligned memory access errors.

Bug: 
Change-Id: Id95d074b2641ba4f002171b5c2990a520879f490
Reviewed-on: https://chromium-review.googlesource.com/720812Reviewed-by: 's avatarAndreas Haas <ahaas@chromium.org>
Commit-Queue: Ivica Bogosavljevic <ivica.bogosavljevic@imgtec.com>
Cr-Commit-Position: refs/heads/master@{#48588}
parent e659f456
...@@ -4195,7 +4195,10 @@ Handle<Code> CompileJSToWasmWrapper(Isolate* isolate, wasm::WasmModule* module, ...@@ -4195,7 +4195,10 @@ Handle<Code> CompileJSToWasmWrapper(Isolate* isolate, wasm::WasmModule* module,
Zone zone(isolate->allocator(), ZONE_NAME); Zone zone(isolate->allocator(), ZONE_NAME);
Graph graph(&zone); Graph graph(&zone);
CommonOperatorBuilder common(&zone); CommonOperatorBuilder common(&zone);
MachineOperatorBuilder machine(&zone); MachineOperatorBuilder machine(
&zone, MachineType::PointerRepresentation(),
InstructionSelector::SupportedMachineOperatorFlags(),
InstructionSelector::AlignmentRequirements());
JSGraph jsgraph(isolate, &graph, &common, nullptr, nullptr, &machine); JSGraph jsgraph(isolate, &graph, &common, nullptr, nullptr, &machine);
Node* control = nullptr; Node* control = nullptr;
...@@ -4305,7 +4308,10 @@ Handle<Code> CompileWasmToJSWrapper( ...@@ -4305,7 +4308,10 @@ Handle<Code> CompileWasmToJSWrapper(
Zone zone(isolate->allocator(), ZONE_NAME); Zone zone(isolate->allocator(), ZONE_NAME);
Graph graph(&zone); Graph graph(&zone);
CommonOperatorBuilder common(&zone); CommonOperatorBuilder common(&zone);
MachineOperatorBuilder machine(&zone); MachineOperatorBuilder machine(
&zone, MachineType::PointerRepresentation(),
InstructionSelector::SupportedMachineOperatorFlags(),
InstructionSelector::AlignmentRequirements());
JSGraph jsgraph(isolate, &graph, &common, nullptr, nullptr, &machine); JSGraph jsgraph(isolate, &graph, &common, nullptr, nullptr, &machine);
Node* control = nullptr; Node* control = nullptr;
...@@ -4393,7 +4399,10 @@ Handle<Code> CompileWasmToWasmWrapper(Isolate* isolate, Handle<Code> target, ...@@ -4393,7 +4399,10 @@ Handle<Code> CompileWasmToWasmWrapper(Isolate* isolate, Handle<Code> target,
Zone zone(isolate->allocator(), ZONE_NAME); Zone zone(isolate->allocator(), ZONE_NAME);
Graph graph(&zone); Graph graph(&zone);
CommonOperatorBuilder common(&zone); CommonOperatorBuilder common(&zone);
MachineOperatorBuilder machine(&zone); MachineOperatorBuilder machine(
&zone, MachineType::PointerRepresentation(),
InstructionSelector::SupportedMachineOperatorFlags(),
InstructionSelector::AlignmentRequirements());
JSGraph jsgraph(isolate, &graph, &common, nullptr, nullptr, &machine); JSGraph jsgraph(isolate, &graph, &common, nullptr, nullptr, &machine);
Node* control = nullptr; Node* control = nullptr;
...@@ -4524,7 +4533,10 @@ Handle<Code> CompileCWasmEntry(Isolate* isolate, wasm::FunctionSig* sig, ...@@ -4524,7 +4533,10 @@ Handle<Code> CompileCWasmEntry(Isolate* isolate, wasm::FunctionSig* sig,
Zone zone(isolate->allocator(), ZONE_NAME); Zone zone(isolate->allocator(), ZONE_NAME);
Graph graph(&zone); Graph graph(&zone);
CommonOperatorBuilder common(&zone); CommonOperatorBuilder common(&zone);
MachineOperatorBuilder machine(&zone); MachineOperatorBuilder machine(
&zone, MachineType::PointerRepresentation(),
InstructionSelector::SupportedMachineOperatorFlags(),
InstructionSelector::AlignmentRequirements());
JSGraph jsgraph(isolate, &graph, &common, nullptr, nullptr, &machine); JSGraph jsgraph(isolate, &graph, &common, nullptr, nullptr, &machine);
Node* control = nullptr; Node* control = nullptr;
......
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