Index: src/compiler/wasm-compiler.cc |
diff --git a/src/compiler/wasm-compiler.cc b/src/compiler/wasm-compiler.cc |
index b0e76e6b2daf6d2efa4d67ea5680e88edd7d404e..35d94ba04bee1d5d4b96fdbf61f931b477a15a7b 100644 |
--- a/src/compiler/wasm-compiler.cc |
+++ b/src/compiler/wasm-compiler.cc |
@@ -168,21 +168,60 @@ class WasmTrapHelper : public ZoneObject { |
return TrapIfEq64(reason, node, 0, position); |
} |
+ int32_t GetFunctionIdForTrap(wasm::TrapReason reason) { |
+ int32_t trap_id; |
+ if (builder_->module_ && !builder_->module_->instance->context.is_null()) { |
+ trap_id = wasm::WasmOpcodes::TrapReasonToFunctionId(reason); |
+ } else { |
+ // We use Runtime::kNumFunctions as a marker to tell the code generator |
+ // to generate a call to a testing c-function instead of a runtime |
+ // function. This code should only be called from a cctest. |
+ trap_id = Runtime::kNumFunctions; |
+ } |
+ return trap_id; |
+ } |
+ |
+#if V8_TARGET_ARCH_X64 |
+#define WASM_TRAP_IF_SUPPORTED |
+#endif |
+ |
// Add a trap if {cond} is true. |
void AddTrapIfTrue(wasm::TrapReason reason, Node* cond, |
wasm::WasmCodePosition position) { |
- AddTrapIf(reason, cond, true, position); |
+#ifdef WASM_TRAP_IF_SUPPORTED |
+ if (FLAG_wasm_trap_if) { |
+ int32_t trap_id = GetFunctionIdForTrap(reason); |
+ Node* node = graph()->NewNode(common()->TrapIf(trap_id), cond, |
+ builder_->Effect(), builder_->Control()); |
+ *builder_->control_ = node; |
+ builder_->SetSourcePosition(node, position); |
+ return; |
+ } |
+#endif // V8_TARGET_ARCH_X64 |
+ BuildTrapIf(reason, cond, true, position); |
} |
// Add a trap if {cond} is false. |
void AddTrapIfFalse(wasm::TrapReason reason, Node* cond, |
wasm::WasmCodePosition position) { |
- AddTrapIf(reason, cond, false, position); |
+#ifdef WASM_TRAP_IF_SUPPORTED |
+ if (FLAG_wasm_trap_if) { |
+ int32_t trap_id = GetFunctionIdForTrap(reason); |
+ |
+ Node* node = graph()->NewNode(common()->TrapUnless(trap_id), cond, |
+ builder_->Effect(), builder_->Control()); |
+ *builder_->control_ = node; |
+ builder_->SetSourcePosition(node, position); |
+ return; |
+ } |
+#endif // V8_TARGET_ARCH_X64 |
+ |
+ BuildTrapIf(reason, cond, false, position); |
} |
// Add a trap if {cond} is true or false according to {iftrue}. |
- void AddTrapIf(wasm::TrapReason reason, Node* cond, bool iftrue, |
- wasm::WasmCodePosition position) { |
+ void BuildTrapIf(wasm::TrapReason reason, Node* cond, bool iftrue, |
+ wasm::WasmCodePosition position) { |
Node** effect_ptr = builder_->effect_; |
Node** control_ptr = builder_->control_; |
Node* before = *effect_ptr; |
@@ -2091,6 +2130,8 @@ Node* WasmGraphBuilder::BuildI64RemS(Node* left, Node* right, |
graph()->NewNode(jsgraph()->machine()->Word64Equal(), right, |
jsgraph()->Int64Constant(-1))); |
+ d.Chain(*control_); |
+ |
Node* rem = graph()->NewNode(jsgraph()->machine()->Int64Mod(), left, right, |
d.if_false); |