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

Side by Side Diff: src/hydrogen-check-elimination.h

Issue 310593004: Revert "Handle HCheckInstanceType and HIsStringAndBranch in check elimination." (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 6 years, 6 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 | « no previous file | src/hydrogen-check-elimination.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 2013 the V8 project authors. All rights reserved. 1 // Copyright 2013 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_CHECK_ELIMINATION_H_ 5 #ifndef V8_HYDROGEN_CHECK_ELIMINATION_H_
6 #define V8_HYDROGEN_CHECK_ELIMINATION_H_ 6 #define V8_HYDROGEN_CHECK_ELIMINATION_H_
7 7
8 #include "hydrogen.h" 8 #include "hydrogen.h"
9 #include "hydrogen-alias-analysis.h" 9 #include "hydrogen-alias-analysis.h"
10 10
11 namespace v8 { 11 namespace v8 {
12 namespace internal { 12 namespace internal {
13 13
14 14
15 // Remove CheckMaps instructions through flow- and branch-sensitive analysis. 15 // Remove CheckMaps instructions through flow- and branch-sensitive analysis.
16 class HCheckEliminationPhase : public HPhase { 16 class HCheckEliminationPhase : public HPhase {
17 public: 17 public:
18 explicit HCheckEliminationPhase(HGraph* graph) 18 explicit HCheckEliminationPhase(HGraph* graph)
19 : HPhase("H_Check Elimination", graph), aliasing_(), 19 : HPhase("H_Check Elimination", graph), aliasing_() {
20 string_maps_(kStringMapsSize, zone()) {
21 // Compute the set of string maps.
22 #define ADD_STRING_MAP(type, size, name, Name) \
23 string_maps_.Add(Unique<Map>::CreateImmovable( \
24 graph->isolate()->factory()->name##_map()), zone());
25 STRING_TYPE_LIST(ADD_STRING_MAP)
26 #undef ADD_STRING_MAP
27 ASSERT_EQ(kStringMapsSize, string_maps_.size());
28 #ifdef DEBUG 20 #ifdef DEBUG
29 redundant_ = 0; 21 redundant_ = 0;
30 removed_ = 0; 22 removed_ = 0;
31 removed_cho_ = 0; 23 removed_cho_ = 0;
32 removed_cit_ = 0;
33 narrowed_ = 0; 24 narrowed_ = 0;
34 loads_ = 0; 25 loads_ = 0;
35 empty_ = 0; 26 empty_ = 0;
36 compares_true_ = 0; 27 compares_true_ = 0;
37 compares_false_ = 0; 28 compares_false_ = 0;
38 transitions_ = 0; 29 transitions_ = 0;
39 #endif 30 #endif
40 } 31 }
41 32
42 void Run(); 33 void Run();
43 34
44 friend class HCheckTable; 35 friend class HCheckTable;
45 36
46 private: 37 private:
47 const UniqueSet<Map>* string_maps() const { return &string_maps_; }
48
49 void PrintStats(); 38 void PrintStats();
50 39
51 HAliasAnalyzer* aliasing_; 40 HAliasAnalyzer* aliasing_;
52 #define COUNT(type, size, name, Name) + 1
53 static const int kStringMapsSize = 0 STRING_TYPE_LIST(COUNT);
54 #undef COUNT
55 UniqueSet<Map> string_maps_;
56 #ifdef DEBUG 41 #ifdef DEBUG
57 int redundant_; 42 int redundant_;
58 int removed_; 43 int removed_;
59 int removed_cho_; 44 int removed_cho_;
60 int removed_cit_;
61 int narrowed_; 45 int narrowed_;
62 int loads_; 46 int loads_;
63 int empty_; 47 int empty_;
64 int compares_true_; 48 int compares_true_;
65 int compares_false_; 49 int compares_false_;
66 int transitions_; 50 int transitions_;
67 #endif 51 #endif
68 }; 52 };
69 53
70 54
71 } } // namespace v8::internal 55 } } // namespace v8::internal
72 56
73 #endif // V8_HYDROGEN_CHECK_ELIMINATION_H_ 57 #endif // V8_HYDROGEN_CHECK_ELIMINATION_H_
OLDNEW
« no previous file with comments | « no previous file | src/hydrogen-check-elimination.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698