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

Unified Diff: src/asmjs/asm-js.cc

Issue 2264913002: [wasm] asm.js - Remove Wasm.instantiateModuleFromAsm, use asm.js directly. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: fix Created 4 years, 4 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | src/asmjs/asm-typer.cc » ('j') | test/mjsunit/regress/regress-592352.js » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/asmjs/asm-js.cc
diff --git a/src/asmjs/asm-js.cc b/src/asmjs/asm-js.cc
index c38c0537358b1f5f38606c05af9efcc568fedb2a..cbd69644a5e49ae81b4051f6b29cdce2d74c77ba 100644
--- a/src/asmjs/asm-js.cc
+++ b/src/asmjs/asm-js.cc
@@ -67,6 +67,9 @@ bool IsStdlibMemberValid(i::Isolate* isolate, Handle<JSReceiver> stdlib,
return true;
}
case wasm::AsmTyper::StandardMember::kNaN: {
+ if (stdlib.is_null()) {
+ return false;
+ }
i::Handle<i::Name> name(isolate->factory()->InternalizeOneByteString(
STATIC_CHAR_VECTOR("NaN")));
i::MaybeHandle<i::Object> maybe_value =
@@ -212,11 +215,8 @@ MaybeHandle<Object> AsmJs::InstantiateAsmWasm(i::Isolate* isolate,
isolate, init, undefined, foreign_globals->length(), foreign_args_array);
delete[] foreign_args_array;
- if (retval.is_null()) {
- thrower.Error(
- "WASM.instantiateModuleFromAsm(): foreign init function failed");
- return MaybeHandle<Object>();
- }
+ DCHECK(!retval.is_null());
+
return maybe_module_object;
}
« no previous file with comments | « no previous file | src/asmjs/asm-typer.cc » ('j') | test/mjsunit/regress/regress-592352.js » ('J')

Powered by Google App Engine
This is Rietveld 408576698