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

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

Issue 300423003: Handle HCheckInstanceType and HIsStringAndBranch in check elimination. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Addressed comments 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());
20 #ifdef DEBUG 28 #ifdef DEBUG
21 redundant_ = 0; 29 redundant_ = 0;
22 removed_ = 0; 30 removed_ = 0;
23 removed_cho_ = 0; 31 removed_cho_ = 0;
32 removed_cit_ = 0;
24 narrowed_ = 0; 33 narrowed_ = 0;
25 loads_ = 0; 34 loads_ = 0;
26 empty_ = 0; 35 empty_ = 0;
27 compares_true_ = 0; 36 compares_true_ = 0;
28 compares_false_ = 0; 37 compares_false_ = 0;
29 transitions_ = 0; 38 transitions_ = 0;
30 #endif 39 #endif
31 } 40 }
32 41
33 void Run(); 42 void Run();
34 43
35 friend class HCheckTable; 44 friend class HCheckTable;
36 45
37 private: 46 private:
47 const UniqueSet<Map>* string_maps() const { return &string_maps_; }
48
38 void PrintStats(); 49 void PrintStats();
39 50
40 HAliasAnalyzer* aliasing_; 51 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_;
41 #ifdef DEBUG 56 #ifdef DEBUG
42 int redundant_; 57 int redundant_;
43 int removed_; 58 int removed_;
44 int removed_cho_; 59 int removed_cho_;
60 int removed_cit_;
45 int narrowed_; 61 int narrowed_;
46 int loads_; 62 int loads_;
47 int empty_; 63 int empty_;
48 int compares_true_; 64 int compares_true_;
49 int compares_false_; 65 int compares_false_;
50 int transitions_; 66 int transitions_;
51 #endif 67 #endif
52 }; 68 };
53 69
54 70
55 } } // namespace v8::internal 71 } } // namespace v8::internal
56 72
57 #endif // V8_HYDROGEN_CHECK_ELIMINATION_H_ 73 #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