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

Side by Side Diff: src/hydrogen.h

Issue 1052253003: Revert of Revert of Protect the emptiness of Array prototype elements with a PropertyCell. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 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/heap/heap.cc ('k') | src/isolate.h » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 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 #ifndef V8_HYDROGEN_H_ 5 #ifndef V8_HYDROGEN_H_
6 #define V8_HYDROGEN_H_ 6 #define V8_HYDROGEN_H_
7 7
8 #include "src/v8.h" 8 #include "src/v8.h"
9 9
10 #include "src/accessors.h" 10 #include "src/accessors.h"
(...skipping 398 matching lines...) Expand 10 before | Expand all | Expand 10 after
409 409
410 void MarkRecursive() { is_recursive_ = true; } 410 void MarkRecursive() { is_recursive_ = true; }
411 bool is_recursive() const { return is_recursive_; } 411 bool is_recursive() const { return is_recursive_; }
412 412
413 void MarkThisHasUses() { this_has_uses_ = true; } 413 void MarkThisHasUses() { this_has_uses_ = true; }
414 bool this_has_uses() const { return this_has_uses_; } 414 bool this_has_uses() const { return this_has_uses_; }
415 415
416 void MarkDependsOnEmptyArrayProtoElements() { 416 void MarkDependsOnEmptyArrayProtoElements() {
417 // Add map dependency if not already added. 417 // Add map dependency if not already added.
418 if (depends_on_empty_array_proto_elements_) return; 418 if (depends_on_empty_array_proto_elements_) return;
419 info()->dependencies()->AssumeElementsCantBeAdded( 419 info()->dependencies()->AssumePropertyCell(
420 handle(isolate()->initial_object_prototype()->map())); 420 isolate()->factory()->array_protector());
421 info()->dependencies()->AssumeElementsCantBeAdded(
422 handle(isolate()->initial_array_prototype()->map()));
423 depends_on_empty_array_proto_elements_ = true; 421 depends_on_empty_array_proto_elements_ = true;
424 } 422 }
425 423
426 bool depends_on_empty_array_proto_elements() { 424 bool depends_on_empty_array_proto_elements() {
427 return depends_on_empty_array_proto_elements_; 425 return depends_on_empty_array_proto_elements_;
428 } 426 }
429 427
430 bool has_uint32_instructions() { 428 bool has_uint32_instructions() {
431 DCHECK(uint32_instructions_ == NULL || !uint32_instructions_->is_empty()); 429 DCHECK(uint32_instructions_ == NULL || !uint32_instructions_->is_empty());
432 return uint32_instructions_ != NULL; 430 return uint32_instructions_ != NULL;
(...skipping 2605 matching lines...) Expand 10 before | Expand all | Expand 10 after
3038 } 3036 }
3039 3037
3040 private: 3038 private:
3041 HGraphBuilder* builder_; 3039 HGraphBuilder* builder_;
3042 }; 3040 };
3043 3041
3044 3042
3045 } } // namespace v8::internal 3043 } } // namespace v8::internal
3046 3044
3047 #endif // V8_HYDROGEN_H_ 3045 #endif // V8_HYDROGEN_H_
OLDNEW
« no previous file with comments | « src/heap/heap.cc ('k') | src/isolate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698