[wasm] Make sure the interpreter only executes preprocessed code.

src/wasm/wasm-interpreter.cc

 // Copyright 2016 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/wasm/wasm-interpreter.h"
 
 #include "src/utils.h"
 #include "src/wasm/ast-decoder.h"
 #include "src/wasm/decoder.h"
 #include "src/wasm/wasm-external-refs.h"
@@ skipping 1018 matching lines @@
     TRACE("  => PushFrame(#%u @%zu)\n", code->function->func_index,
           frames_.back().ret_pc);
   }
 
   virtual WasmInterpreter::State Run() {
     do {
       TRACE("  => Run()\n");
       if (state_ == WasmInterpreter::STOPPED ||
           state_ == WasmInterpreter::PAUSED) {
         state_ = WasmInterpreter::RUNNING;
-        Execute(frames_.back().code, frames_.back().ret_pc, kRunSteps);
+        Execute(codemap()->Preprocess(frames_.back().code),

[titzer, 2016/09/22 14:18:24]

Maybe PushFrame should do this?

[ahaas, 2016/09/22 14:40:37]

I moved Preprocess to FindCode in the CodeMap. I think that's an even better
place than PushFrame, and PushFrame uses FindCode.

+                frames_.back().ret_pc, kRunSteps);
       }
     } while (state_ == WasmInterpreter::STOPPED);
     return state_;
   }
 
   virtual WasmInterpreter::State Step() {
     TRACE("  => Step()\n");
     if (state_ == WasmInterpreter::STOPPED ||
         state_ == WasmInterpreter::PAUSED) {
       state_ = WasmInterpreter::RUNNING;
@@ skipping 800 matching lines @@
 
 ControlTransferMap WasmInterpreter::ComputeControlTransfersForTesting(
     Zone* zone, const byte* start, const byte* end) {
   ControlTransfers targets(zone, 0, start, end);
   return targets.map_;
 }
 
 }  // namespace wasm
 }  // namespace internal
 }  // namespace v8