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

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

Issue 2256603002: [wasm] Add stack checks at the beginning of each function. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Do not trap but use the runtime stack guard call 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
Index: src/wasm/ast-decoder.cc
diff --git a/src/wasm/ast-decoder.cc b/src/wasm/ast-decoder.cc
index 740199c29817a5a6ab4cf4ffb6a37f1bd16ec3e1..d869e04ef38538438a445a1f529aed426c6ab1ef 100644
--- a/src/wasm/ast-decoder.cc
+++ b/src/wasm/ast-decoder.cc
@@ -559,6 +559,9 @@ class WasmFullDecoder : public WasmDecoder {
ssa_env->control = start;
ssa_env->effect = start;
SetEnv("initial", ssa_env);
+ if (builder_) {
titzer 2016/08/17 11:17:23 Will the BUILD() macro work here?
ahaas 2016/08/17 16:12:57 Actually not, because InitStackCheck does not retu
+ builder_->InitStackCheck(position());
+ }
}
TFNode* DefaultValue(LocalType type) {

Powered by Google App Engine
This is Rietveld 408576698