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

Unified Diff: src/builtins/builtins-wasm.cc

Issue 2691993004: [wasm] Introduce WasmStackGuard builtin (Closed)
Patch Set: Add builtins-wasm.cc ;) Created 3 years, 10 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 | « src/builtins/builtins.h ('k') | src/compiler/wasm-compiler.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/builtins/builtins-wasm.cc
diff --git a/src/builtins/builtins-wasm.cc b/src/builtins/builtins-wasm.cc
new file mode 100644
index 0000000000000000000000000000000000000000..2a9438ca8822f0732c3fe334906e17fcd2a813c4
--- /dev/null
+++ b/src/builtins/builtins-wasm.cc
@@ -0,0 +1,24 @@
+// Copyright 2017 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/builtins/builtins-utils.h"
+#include "src/builtins/builtins.h"
+#include "src/code-stub-assembler.h"
+#include "src/wasm/wasm-module.h"
ahaas 2017/02/14 14:45:38 Do we need all these includes here?
Clemens Hammacher 2017/02/14 17:23:14 Thanks for the hint. Removed three of them.
+#include "src/wasm/wasm-objects.h"
+
+namespace v8 {
+namespace internal {
+
+typedef compiler::Node Node;
+typedef CodeStubAssembler::ParameterMode ParameterMode;
Michael Starzinger 2017/02/14 14:43:22 nit: The "ParameterMode" typedef seems to be unuse
Clemens Hammacher 2017/02/14 17:23:14 Done, also CodeAssemblerState.
+typedef compiler::CodeAssemblerState CodeAssemblerState;
+
+TF_BUILTIN(WasmStackGuard, CodeStubAssembler) {
+ Node* context = SmiConstant(0);
ahaas 2017/02/14 14:45:39 I wonder if SmiConstant(Smi::kZero) would be bette
Clemens Hammacher 2017/02/14 17:23:14 Done.
+ TailCallRuntime(Runtime::kWasmStackGuard, context);
+}
+
+} // namespace internal
+} // namespace v8
« no previous file with comments | « src/builtins/builtins.h ('k') | src/compiler/wasm-compiler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698