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

Unified Diff: src/compiler/access-builder.cc

Issue 2302013002: Store the scope info in catch contexts (Closed)
Patch Set: updates Created 4 years, 3 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/compiler/access-builder.cc
diff --git a/src/compiler/access-builder.cc b/src/compiler/access-builder.cc
index f7dbd54b5096dbf57a63efcbfa5e566c53494cab..60a24e730112bf91fc9a4e38f05b4ff5cb17cb7d 100644
--- a/src/compiler/access-builder.cc
+++ b/src/compiler/access-builder.cc
@@ -743,6 +743,22 @@ ElementAccess AccessBuilder::ForTypedArrayElement(ExternalArrayType type,
return access;
}
+// static
+FieldAccess AccessBuilder::ForContextExtensionScopeInfo() {
Michael Starzinger 2016/09/05 11:35:29 nit: Likewise, lets move it up to before "elements
jochen (gone - plz use gerrit) 2016/09/05 11:42:44 done
+ FieldAccess access = {
+ kTaggedBase, ContextExtension::kScopeInfoOffset, Handle<Name>(),
+ Type::Any(), MachineType::AnyTagged(), kFullWriteBarrier};
Michael Starzinger 2016/09/05 11:35:29 nit: Lets use Type::OtherInternal instead.
jochen (gone - plz use gerrit) 2016/09/05 11:42:44 done
+ return access;
+}
+
+// static
+FieldAccess AccessBuilder::ForContextExtensionExtension() {
+ FieldAccess access = {
+ kTaggedBase, ContextExtension::kExtensionOffset, Handle<Name>(),
+ Type::Any(), MachineType::AnyTagged(), kFullWriteBarrier};
+ return access;
+}
+
} // namespace compiler
} // namespace internal
} // namespace v8

Powered by Google App Engine
This is Rietveld 408576698