[wasm] Introduce the TrapIf and TrapUnless operators to generate trap code.

src/compiler/wasm-compiler.cc

 // Copyright 2015 the V8 project authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "src/compiler/wasm-compiler.h"
 
 #include <memory>
 
 #include "src/isolate-inl.h"
 
@@ skipping 150 matching lines @@
                   position);
     return builder_->Control();
   }
 
   // Add a check that traps if {node} is zero.
   Node* ZeroCheck64(wasm::TrapReason reason, Node* node,
                     wasm::WasmCodePosition position) {
     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;
+  }
+
   // Add a trap if {cond} is true.
   void AddTrapIfTrue(wasm::TrapReason reason, Node* cond,
                      wasm::WasmCodePosition position) {
+#if V8_TARGET_ARCH_X64

[titzer, 2016/12/12 14:21:33]

Can you introduce another preprocessor definition, like WASM_TRAP_IF_SUPPORTED,
so that we can just change that macro as each new platform gets added (and
eventually remove)?

[ahaas, 2016/12/13 12:38:59]

Done.

+    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
     AddTrapIf(reason, cond, true, position);

[titzer, 2016/12/12 14:21:33]

Can we rename this internal function to BuildTrapIf() since they build an actual
subgraph as opposed to using the special TrapIf operator?

[ahaas, 2016/12/13 12:38:59]

Done.

   }
 
   // Add a trap if {cond} is false.
   void AddTrapIfFalse(wasm::TrapReason reason, Node* cond,
                       wasm::WasmCodePosition position) {
+#if V8_TARGET_ARCH_X64
+    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
+
     AddTrapIf(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) {
     Node** effect_ptr = builder_->effect_;
     Node** control_ptr = builder_->control_;
     Node* before = *effect_ptr;
     BranchHint hint = iftrue ? BranchHint::kFalse : BranchHint::kTrue;
@@ skipping 3421 matching lines @@
                       Smi::FromInt(instruction.instr_offset));
     fn_protected->set(Code::kTrapDataSize * i + Code::kTrapLandingOffset,
                       Smi::FromInt(instruction.landing_offset));
   }
   return fn_protected;
 }
 
 }  // namespace compiler
 }  // namespace internal
 }  // namespace v8