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

Unified Diff: src/ast/ast.cc

Issue 2441543005: [full-codegen] Eliminate unnecessary hole checks for stores (Closed)
Patch Set: Merge and fix modules Created 4 years, 2 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/ast/ast.h ('k') | src/full-codegen/arm/full-codegen-arm.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/ast/ast.cc
diff --git a/src/ast/ast.cc b/src/ast/ast.cc
index 906bd4a9c62d32f3247cbfa30197b07bb99277da..9db91e2b9cd774d7abd5f3f0c6bb073f3ac5b9a6 100644
--- a/src/ast/ast.cc
+++ b/src/ast/ast.cc
@@ -165,9 +165,10 @@ VariableProxy::VariableProxy(Variable* var, int start_position,
end_position_(end_position),
raw_name_(var->raw_name()),
next_unresolved_(nullptr) {
- bit_field_ |=
- IsThisField::encode(var->is_this()) | IsAssignedField::encode(false) |
- IsResolvedField::encode(false) | NeedsHoleCheckField::encode(false);
+ bit_field_ |= IsThisField::encode(var->is_this()) |
+ IsAssignedField::encode(false) |
+ IsResolvedField::encode(false) |
+ HoleCheckModeField::encode(HoleCheckMode::kElided);
BindTo(var);
}
@@ -181,7 +182,7 @@ VariableProxy::VariableProxy(const AstRawString* name,
bit_field_ |= IsThisField::encode(variable_kind == THIS_VARIABLE) |
IsAssignedField::encode(false) |
IsResolvedField::encode(false) |
- NeedsHoleCheckField::encode(false);
+ HoleCheckModeField::encode(HoleCheckMode::kElided);
}
VariableProxy::VariableProxy(const VariableProxy* copy_from)
« no previous file with comments | « src/ast/ast.h ('k') | src/full-codegen/arm/full-codegen-arm.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698