Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(14)

Side by Side Diff: test/fuzzer/wasm-code.cc

Issue 2336233006: [wasm] C++ style: ErrorThrower& -> ErrorThrower* (Closed)
Patch Set: more Created 4 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « test/common/wasm/wasm-module-runner.cc ('k') | test/fuzzer/wasm-section-fuzzers.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2016 the V8 project authors. All rights reserved. 1 // Copyright 2016 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include <stddef.h> 5 #include <stddef.h>
6 #include <stdint.h> 6 #include <stdint.h>
7 7
8 #include "include/v8.h" 8 #include "include/v8.h"
9 #include "src/isolate.h" 9 #include "src/isolate.h"
10 #include "src/wasm/encoder.h" 10 #include "src/wasm/encoder.h"
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 49
50 ZoneBuffer buffer(&zone); 50 ZoneBuffer buffer(&zone);
51 builder.WriteTo(buffer); 51 builder.WriteTo(buffer);
52 52
53 v8::internal::WasmJs::SetupIsolateForWasm(i_isolate); 53 v8::internal::WasmJs::SetupIsolateForWasm(i_isolate);
54 54
55 v8::internal::HandleScope scope(i_isolate); 55 v8::internal::HandleScope scope(i_isolate);
56 56
57 ErrorThrower interpreter_thrower(i_isolate, "Interpreter"); 57 ErrorThrower interpreter_thrower(i_isolate, "Interpreter");
58 std::unique_ptr<const WasmModule> module(testing::DecodeWasmModuleForTesting( 58 std::unique_ptr<const WasmModule> module(testing::DecodeWasmModuleForTesting(
59 i_isolate, &zone, interpreter_thrower, buffer.begin(), buffer.end(), 59 i_isolate, &zone, &interpreter_thrower, buffer.begin(), buffer.end(),
60 v8::internal::wasm::ModuleOrigin::kWasmOrigin)); 60 v8::internal::wasm::ModuleOrigin::kWasmOrigin));
61 61
62 if (module == nullptr) { 62 if (module == nullptr) {
63 return 0; 63 return 0;
64 } 64 }
65 int32_t result_interpreted; 65 int32_t result_interpreted;
66 { 66 {
67 WasmVal args[] = {WasmVal(1), WasmVal(2), WasmVal(3)}; 67 WasmVal args[] = {WasmVal(1), WasmVal(2), WasmVal(3)};
68 result_interpreted = testing::InterpretWasmModule( 68 result_interpreted = testing::InterpretWasmModule(
69 i_isolate, interpreter_thrower, module.get(), 0, args); 69 i_isolate, &interpreter_thrower, module.get(), 0, args);
70 } 70 }
71 71
72 ErrorThrower compiler_thrower(i_isolate, "Compiler"); 72 ErrorThrower compiler_thrower(i_isolate, "Compiler");
73 v8::internal::Handle<v8::internal::JSObject> instance = 73 v8::internal::Handle<v8::internal::JSObject> instance =
74 testing::InstantiateModuleForTesting(i_isolate, compiler_thrower, 74 testing::InstantiateModuleForTesting(i_isolate, &compiler_thrower,
75 module.get()); 75 module.get());
76 76
77 if (!interpreter_thrower.error()) { 77 if (!interpreter_thrower.error()) {
78 CHECK(!instance.is_null()); 78 CHECK(!instance.is_null());
79 } else { 79 } else {
80 return 0; 80 return 0;
81 } 81 }
82 int32_t result_compiled; 82 int32_t result_compiled;
83 { 83 {
84 v8::internal::Handle<v8::internal::Object> arguments[] = { 84 v8::internal::Handle<v8::internal::Object> arguments[] = {
85 v8::internal::handle(v8::internal::Smi::FromInt(1), i_isolate), 85 v8::internal::handle(v8::internal::Smi::FromInt(1), i_isolate),
86 v8::internal::handle(v8::internal::Smi::FromInt(2), i_isolate), 86 v8::internal::handle(v8::internal::Smi::FromInt(2), i_isolate),
87 v8::internal::handle(v8::internal::Smi::FromInt(3), i_isolate)}; 87 v8::internal::handle(v8::internal::Smi::FromInt(3), i_isolate)};
88 result_compiled = testing::CallWasmFunctionForTesting( 88 result_compiled = testing::CallWasmFunctionForTesting(
89 i_isolate, instance, compiler_thrower, "main", arraysize(arguments), 89 i_isolate, instance, &compiler_thrower, "main", arraysize(arguments),
90 arguments, v8::internal::wasm::ModuleOrigin::kWasmOrigin); 90 arguments, v8::internal::wasm::ModuleOrigin::kWasmOrigin);
91 } 91 }
92 if (result_interpreted == 0xdeadbeef) { 92 if (result_interpreted == 0xdeadbeef) {
93 CHECK(i_isolate->has_pending_exception()); 93 CHECK(i_isolate->has_pending_exception());
94 i_isolate->clear_pending_exception(); 94 i_isolate->clear_pending_exception();
95 } else { 95 } else {
96 CHECK_EQ(result_interpreted, result_compiled); 96 CHECK_EQ(result_interpreted, result_compiled);
97 } 97 }
98 return 0; 98 return 0;
99 } 99 }
OLDNEW
« no previous file with comments | « test/common/wasm/wasm-module-runner.cc ('k') | test/fuzzer/wasm-section-fuzzers.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698