src/wasm/module-decoder.cc - Issue 1830663002: [wasm] Binary 11: AST changes.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: src/wasm/module-decoder.cc

Issue 1830663002: [wasm] Binary 11: AST changes. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master

Patch Set: Renumber opcodes and remove tests added elsewhere Created 4 years, 9 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:

Index: src/wasm/module-decoder.cc

diff --git a/src/wasm/module-decoder.cc b/src/wasm/module-decoder.cc

index 558d17c5a6e1117e8cf4f9c37f52faaef43e965d..ce3f50a87285fe4b838e34da52f90020ac305d3a 100644

--- a/src/wasm/module-decoder.cc

+++ b/src/wasm/module-decoder.cc

@@ -566,11 +566,10 @@ class ModuleDecoder : public Decoder {

// Verifies the body (code) of a given function.

void VerifyFunctionBody(uint32_t func_num, ModuleEnv* menv,

WasmFunction* function) {

- if (FLAG_trace_wasm_decode_time) {

+ if (FLAG_trace_wasm_decoder || FLAG_trace_wasm_decode_time) {

OFStream os(stdout);

os << "Verifying WASM function " << WasmFunctionName(function, menv)

<< std::endl;

- os << std::endl;

}

FunctionBody body = {menv, function->sig, start_,

start_ + function->code_start_offset,