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

Side by Side Diff: src/hydrogen-instructions.h

Issue 494063002: Rename IsDontDelete to IsConfigurable (and invert conditions) (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 years, 4 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 | Annotate | Revision Log
« no previous file with comments | « src/hydrogen.h ('k') | src/hydrogen-instructions.cc » ('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_INSTRUCTIONS_H_ 5 #ifndef V8_HYDROGEN_INSTRUCTIONS_H_
6 #define V8_HYDROGEN_INSTRUCTIONS_H_ 6 #define V8_HYDROGEN_INSTRUCTIONS_H_
7 7
8 #include "src/v8.h" 8 #include "src/v8.h"
9 9
10 #include "src/allocation.h" 10 #include "src/allocation.h"
(...skipping 5775 matching lines...) Expand 10 before | Expand all | Expand 10 after
5786 return kPointersToHereMaybeInteresting; 5786 return kPointersToHereMaybeInteresting;
5787 } 5787 }
5788 5788
5789 5789
5790 class HStoreGlobalCell V8_FINAL : public HUnaryOperation { 5790 class HStoreGlobalCell V8_FINAL : public HUnaryOperation {
5791 public: 5791 public:
5792 DECLARE_INSTRUCTION_FACTORY_P3(HStoreGlobalCell, HValue*, 5792 DECLARE_INSTRUCTION_FACTORY_P3(HStoreGlobalCell, HValue*,
5793 Handle<PropertyCell>, PropertyDetails); 5793 Handle<PropertyCell>, PropertyDetails);
5794 5794
5795 Unique<PropertyCell> cell() const { return cell_; } 5795 Unique<PropertyCell> cell() const { return cell_; }
5796 bool RequiresHoleCheck() { 5796 bool RequiresHoleCheck() { return details_.IsConfigurable(); }
Toon Verwaest 2014/08/21 08:53:23 I removed IsReadOnly because we don't use the hole
5797 return !details_.IsDontDelete() || details_.IsReadOnly();
5798 }
5799 bool NeedsWriteBarrier() { 5797 bool NeedsWriteBarrier() {
5800 return StoringValueNeedsWriteBarrier(value()); 5798 return StoringValueNeedsWriteBarrier(value());
5801 } 5799 }
5802 5800
5803 virtual void FinalizeUniqueness() V8_OVERRIDE { 5801 virtual void FinalizeUniqueness() V8_OVERRIDE {
5804 cell_ = Unique<PropertyCell>(cell_.handle()); 5802 cell_ = Unique<PropertyCell>(cell_.handle());
5805 } 5803 }
5806 5804
5807 virtual Representation RequiredInputRepresentation(int index) V8_OVERRIDE { 5805 virtual Representation RequiredInputRepresentation(int index) V8_OVERRIDE {
5808 return Representation::Tagged(); 5806 return Representation::Tagged();
(...skipping 2051 matching lines...) Expand 10 before | Expand all | Expand 10 after
7860 }; 7858 };
7861 7859
7862 7860
7863 7861
7864 #undef DECLARE_INSTRUCTION 7862 #undef DECLARE_INSTRUCTION
7865 #undef DECLARE_CONCRETE_INSTRUCTION 7863 #undef DECLARE_CONCRETE_INSTRUCTION
7866 7864
7867 } } // namespace v8::internal 7865 } } // namespace v8::internal
7868 7866
7869 #endif // V8_HYDROGEN_INSTRUCTIONS_H_ 7867 #endif // V8_HYDROGEN_INSTRUCTIONS_H_
OLDNEW
« no previous file with comments | « src/hydrogen.h ('k') | src/hydrogen-instructions.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698