Reland of [wasm] Enforce that function bodies end with the \"end\" opcode.

test/cctest/wasm/test-wasm-breakpoints.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/debug/debug-interface.h"
 #include "src/wasm/wasm-macro-gen.h"
 #include "src/wasm/wasm-objects.h"
 
 #include "test/cctest/cctest.h"
 #include "test/cctest/compiler/value-helper.h"
@@ skipping 42 matching lines @@
 }  // namespace
 
 TEST(CollectPossibleBreakpoints) {
   WasmRunner<int> runner(kExecuteCompiled);
 
   BUILD(runner, WASM_NOP, WASM_I32_ADD(WASM_ZERO, WASM_ONE));
 
   Handle<WasmInstanceObject> instance = runner.module().instance_object();
   std::vector<debug::Location> locations;
   CheckLocations(instance->compiled_module(), {0, 0}, {1, 0},
-                 {{0, 1}, {0, 2}, {0, 4}, {0, 6}});
+                 {{0, 1}, {0, 2}, {0, 4}, {0, 6}, {0, 7}});
   CheckLocations(instance->compiled_module(), {0, 2}, {0, 4}, {{0, 2}});
   CheckLocations(instance->compiled_module(), {0, 2}, {0, 5}, {{0, 2}, {0, 4}});
-  CheckLocations(instance->compiled_module(), {0, 6}, {0, 7}, {{0, 6}});
-  CheckLocations(instance->compiled_module(), {0, 6}, {1, 0}, {{0, 6}});
-  CheckLocations(instance->compiled_module(), {0, 7}, {1, 0}, {});
-  CheckLocationsFail(instance->compiled_module(), {0, 8}, {1, 0});
+  CheckLocations(instance->compiled_module(), {0, 7}, {0, 8}, {{0, 7}});
+  CheckLocations(instance->compiled_module(), {0, 7}, {1, 0}, {{0, 7}});
+  CheckLocations(instance->compiled_module(), {0, 8}, {1, 0}, {});
+  CheckLocationsFail(instance->compiled_module(), {0, 9}, {1, 0});
 }