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

Side by Side 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 unified diff | Download patch
OLDNEW
1 // Copyright 2014 the V8 project authors. All rights reserved. 1 // Copyright 2014 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/compiler/access-builder.h" 5 #include "src/compiler/access-builder.h"
6 6
7 #include "src/compiler/type-cache.h" 7 #include "src/compiler/type-cache.h"
8 #include "src/contexts.h" 8 #include "src/contexts.h"
9 #include "src/frames.h" 9 #include "src/frames.h"
10 #include "src/handles-inl.h" 10 #include "src/handles-inl.h"
(...skipping 725 matching lines...) Expand 10 before | Expand all | Expand 10 after
736 MachineType::Float64(), kNoWriteBarrier}; 736 MachineType::Float64(), kNoWriteBarrier};
737 return access; 737 return access;
738 } 738 }
739 } 739 }
740 UNREACHABLE(); 740 UNREACHABLE();
741 ElementAccess access = {kUntaggedBase, 0, Type::None(), MachineType::None(), 741 ElementAccess access = {kUntaggedBase, 0, Type::None(), MachineType::None(),
742 kNoWriteBarrier}; 742 kNoWriteBarrier};
743 return access; 743 return access;
744 } 744 }
745 745
746 // static
747 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
748 FieldAccess access = {
749 kTaggedBase, ContextExtension::kScopeInfoOffset, Handle<Name>(),
750 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
751 return access;
752 }
753
754 // static
755 FieldAccess AccessBuilder::ForContextExtensionExtension() {
756 FieldAccess access = {
757 kTaggedBase, ContextExtension::kExtensionOffset, Handle<Name>(),
758 Type::Any(), MachineType::AnyTagged(), kFullWriteBarrier};
759 return access;
760 }
761
746 } // namespace compiler 762 } // namespace compiler
747 } // namespace internal 763 } // namespace internal
748 } // namespace v8 764 } // namespace v8
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698