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

Side by Side Diff: src/crankshaft/x87/lithium-x87.h

Issue 1614943002: [crankshaft] Remove the useless HMapEnumLength instruction. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Put NoObservableSideEffectsScope there Created 4 years, 11 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/crankshaft/x87/lithium-codegen-x87.cc ('k') | src/crankshaft/x87/lithium-x87.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_CRANKSHAFT_X87_LITHIUM_X87_H_ 5 #ifndef V8_CRANKSHAFT_X87_LITHIUM_X87_H_
6 #define V8_CRANKSHAFT_X87_LITHIUM_X87_H_ 6 #define V8_CRANKSHAFT_X87_LITHIUM_X87_H_
7 7
8 #include "src/crankshaft/hydrogen.h" 8 #include "src/crankshaft/hydrogen.h"
9 #include "src/crankshaft/lithium.h" 9 #include "src/crankshaft/lithium.h"
10 #include "src/crankshaft/lithium-allocator.h" 10 #include "src/crankshaft/lithium-allocator.h"
(...skipping 88 matching lines...) Expand 10 before | Expand all | Expand 10 after
99 V(LazyBailout) \ 99 V(LazyBailout) \
100 V(LoadContextSlot) \ 100 V(LoadContextSlot) \
101 V(LoadFieldByIndex) \ 101 V(LoadFieldByIndex) \
102 V(LoadFunctionPrototype) \ 102 V(LoadFunctionPrototype) \
103 V(LoadGlobalGeneric) \ 103 V(LoadGlobalGeneric) \
104 V(LoadKeyed) \ 104 V(LoadKeyed) \
105 V(LoadKeyedGeneric) \ 105 V(LoadKeyedGeneric) \
106 V(LoadNamedField) \ 106 V(LoadNamedField) \
107 V(LoadNamedGeneric) \ 107 V(LoadNamedGeneric) \
108 V(LoadRoot) \ 108 V(LoadRoot) \
109 V(MapEnumLength) \
110 V(MathAbs) \ 109 V(MathAbs) \
111 V(MathClz32) \ 110 V(MathClz32) \
112 V(MathExp) \ 111 V(MathExp) \
113 V(MathFloor) \ 112 V(MathFloor) \
114 V(MathFround) \ 113 V(MathFround) \
115 V(MathLog) \ 114 V(MathLog) \
116 V(MathMinMax) \ 115 V(MathMinMax) \
117 V(MathPowHalf) \ 116 V(MathPowHalf) \
118 V(MathRound) \ 117 V(MathRound) \
119 V(MathSqrt) \ 118 V(MathSqrt) \
(...skipping 1217 matching lines...) Expand 10 before | Expand all | Expand 10 after
1337 1336
1338 LOperand* value() { return inputs_[0]; } 1337 LOperand* value() { return inputs_[0]; }
1339 1338
1340 DECLARE_CONCRETE_INSTRUCTION(CmpMapAndBranch, "cmp-map-and-branch") 1339 DECLARE_CONCRETE_INSTRUCTION(CmpMapAndBranch, "cmp-map-and-branch")
1341 DECLARE_HYDROGEN_ACCESSOR(CompareMap) 1340 DECLARE_HYDROGEN_ACCESSOR(CompareMap)
1342 1341
1343 Handle<Map> map() const { return hydrogen()->map().handle(); } 1342 Handle<Map> map() const { return hydrogen()->map().handle(); }
1344 }; 1343 };
1345 1344
1346 1345
1347 class LMapEnumLength final : public LTemplateInstruction<1, 1, 0> {
1348 public:
1349 explicit LMapEnumLength(LOperand* value) {
1350 inputs_[0] = value;
1351 }
1352
1353 LOperand* value() { return inputs_[0]; }
1354
1355 DECLARE_CONCRETE_INSTRUCTION(MapEnumLength, "map-enum-length")
1356 };
1357
1358
1359 class LSeqStringGetChar final : public LTemplateInstruction<1, 2, 0> { 1346 class LSeqStringGetChar final : public LTemplateInstruction<1, 2, 0> {
1360 public: 1347 public:
1361 LSeqStringGetChar(LOperand* string, LOperand* index) { 1348 LSeqStringGetChar(LOperand* string, LOperand* index) {
1362 inputs_[0] = string; 1349 inputs_[0] = string;
1363 inputs_[1] = index; 1350 inputs_[1] = index;
1364 } 1351 }
1365 1352
1366 LOperand* string() const { return inputs_[0]; } 1353 LOperand* string() const { return inputs_[0]; }
1367 LOperand* index() const { return inputs_[1]; } 1354 LOperand* index() const { return inputs_[1]; }
1368 1355
(...skipping 1420 matching lines...) Expand 10 before | Expand all | Expand 10 after
2789 DISALLOW_COPY_AND_ASSIGN(LChunkBuilder); 2776 DISALLOW_COPY_AND_ASSIGN(LChunkBuilder);
2790 }; 2777 };
2791 2778
2792 #undef DECLARE_HYDROGEN_ACCESSOR 2779 #undef DECLARE_HYDROGEN_ACCESSOR
2793 #undef DECLARE_CONCRETE_INSTRUCTION 2780 #undef DECLARE_CONCRETE_INSTRUCTION
2794 2781
2795 } // namespace internal 2782 } // namespace internal
2796 } // namespace v8 2783 } // namespace v8
2797 2784
2798 #endif // V8_CRANKSHAFT_X87_LITHIUM_X87_H_ 2785 #endif // V8_CRANKSHAFT_X87_LITHIUM_X87_H_
OLDNEW
« no previous file with comments | « src/crankshaft/x87/lithium-codegen-x87.cc ('k') | src/crankshaft/x87/lithium-x87.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698