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

Side by Side Diff: src/compiler/access-builder.cc

Issue 2810333004: [turbofan] Set proper representation for initial arguments length. (Closed)
Patch Set: Created 3 years, 8 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
« no previous file with comments | « src/bootstrapper.cc ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 698 matching lines...) Expand 10 before | Expand all | Expand 10 after
709 Type::NonInternal(), MachineType::AnyTagged(), 709 Type::NonInternal(), MachineType::AnyTagged(),
710 kFullWriteBarrier}; 710 kFullWriteBarrier};
711 return access; 711 return access;
712 } 712 }
713 713
714 714
715 // static 715 // static
716 FieldAccess AccessBuilder::ForArgumentsLength() { 716 FieldAccess AccessBuilder::ForArgumentsLength() {
717 FieldAccess access = {kTaggedBase, JSArgumentsObject::kLengthOffset, 717 FieldAccess access = {kTaggedBase, JSArgumentsObject::kLengthOffset,
718 Handle<Name>(), MaybeHandle<Map>(), 718 Handle<Name>(), MaybeHandle<Map>(),
719 Type::NonInternal(), MachineType::AnyTagged(), 719 Type::SignedSmall(), MachineType::TaggedSigned(),
720 kFullWriteBarrier}; 720 kNoWriteBarrier};
721 return access; 721 return access;
722 } 722 }
723 723
724 724
725 // static 725 // static
726 FieldAccess AccessBuilder::ForArgumentsCallee() { 726 FieldAccess AccessBuilder::ForArgumentsCallee() {
727 FieldAccess access = { 727 FieldAccess access = {
728 kTaggedBase, JSSloppyArgumentsObject::kCalleeOffset, 728 kTaggedBase, JSSloppyArgumentsObject::kCalleeOffset,
729 Handle<Name>(), MaybeHandle<Map>(), 729 Handle<Name>(), MaybeHandle<Map>(),
730 Type::NonInternal(), MachineType::AnyTagged(), 730 Type::NonInternal(), MachineType::AnyTagged(),
(...skipping 217 matching lines...) Expand 10 before | Expand all | Expand 10 after
948 MaybeHandle<Map>(), 948 MaybeHandle<Map>(),
949 Type::SignedSmall(), 949 Type::SignedSmall(),
950 MachineType::TaggedSigned(), 950 MachineType::TaggedSigned(),
951 kNoWriteBarrier}; 951 kNoWriteBarrier};
952 return access; 952 return access;
953 } 953 }
954 954
955 } // namespace compiler 955 } // namespace compiler
956 } // namespace internal 956 } // namespace internal
957 } // namespace v8 957 } // namespace v8
OLDNEW
« no previous file with comments | « src/bootstrapper.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698