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

Unified Diff: runtime/vm/intermediate_language.h

Issue 2793163002: Do not embed is_auto_setup_scope into the compilation of native calls. (Closed)
Patch Set: . Created 3 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:
View side-by-side diff with in-line comments
Download patch
Index: runtime/vm/intermediate_language.h
diff --git a/runtime/vm/intermediate_language.h b/runtime/vm/intermediate_language.h
index d496fbf15b1b9803e2dacab76c0a79dd89538569..6b00393b110b239bacacb0db99de400919cdf2aa 100644
--- a/runtime/vm/intermediate_language.h
+++ b/runtime/vm/intermediate_language.h
@@ -3398,6 +3398,7 @@ class NativeCallInstr : public TemplateDefinition<0, Throws> {
function_(function),
native_c_function_(NULL),
is_bootstrap_native_(false),
+ is_auto_scope_(true),
link_lazily_(link_lazily),
token_pos_(position) {}
@@ -3407,6 +3408,7 @@ class NativeCallInstr : public TemplateDefinition<0, Throws> {
const Function& function() const { return *function_; }
NativeFunction native_c_function() const { return native_c_function_; }
bool is_bootstrap_native() const { return is_bootstrap_native_; }
+ bool is_auto_scope() const { return is_auto_scope_; }
bool link_lazily() const { return link_lazily_; }
virtual TokenPosition token_pos() const { return token_pos_; }
@@ -3424,11 +3426,13 @@ class NativeCallInstr : public TemplateDefinition<0, Throws> {
}
void set_is_bootstrap_native(bool value) { is_bootstrap_native_ = value; }
+ void set_is_auto_scope(bool value) { is_auto_scope_ = value; }
const String* native_name_;
const Function* function_;
NativeFunction native_c_function_;
bool is_bootstrap_native_;
+ bool is_auto_scope_;
bool link_lazily_;
const TokenPosition token_pos_;

Powered by Google App Engine
This is Rietveld 408576698