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

Side by Side Diff: src/crankshaft/arm/lithium-arm.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 | « no previous file | src/crankshaft/arm/lithium-arm.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_ARM_LITHIUM_ARM_H_ 5 #ifndef V8_CRANKSHAFT_ARM_LITHIUM_ARM_H_
6 #define V8_CRANKSHAFT_ARM_LITHIUM_ARM_H_ 6 #define V8_CRANKSHAFT_ARM_LITHIUM_ARM_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 83 matching lines...) Expand 10 before | Expand all | Expand 10 after
94 V(LazyBailout) \ 94 V(LazyBailout) \
95 V(LoadContextSlot) \ 95 V(LoadContextSlot) \
96 V(LoadRoot) \ 96 V(LoadRoot) \
97 V(LoadFieldByIndex) \ 97 V(LoadFieldByIndex) \
98 V(LoadFunctionPrototype) \ 98 V(LoadFunctionPrototype) \
99 V(LoadGlobalGeneric) \ 99 V(LoadGlobalGeneric) \
100 V(LoadKeyed) \ 100 V(LoadKeyed) \
101 V(LoadKeyedGeneric) \ 101 V(LoadKeyedGeneric) \
102 V(LoadNamedField) \ 102 V(LoadNamedField) \
103 V(LoadNamedGeneric) \ 103 V(LoadNamedGeneric) \
104 V(MapEnumLength) \
105 V(MathAbs) \ 104 V(MathAbs) \
106 V(MathClz32) \ 105 V(MathClz32) \
107 V(MathExp) \ 106 V(MathExp) \
108 V(MathFloor) \ 107 V(MathFloor) \
109 V(MathFround) \ 108 V(MathFround) \
110 V(MathLog) \ 109 V(MathLog) \
111 V(MathMinMax) \ 110 V(MathMinMax) \
112 V(MathPowHalf) \ 111 V(MathPowHalf) \
113 V(MathRound) \ 112 V(MathRound) \
114 V(MathSqrt) \ 113 V(MathSqrt) \
(...skipping 1239 matching lines...) Expand 10 before | Expand all | Expand 10 after
1354 LOperand* value() { return inputs_[0]; } 1353 LOperand* value() { return inputs_[0]; }
1355 LOperand* temp() { return temps_[0]; } 1354 LOperand* temp() { return temps_[0]; }
1356 1355
1357 DECLARE_CONCRETE_INSTRUCTION(CmpMapAndBranch, "cmp-map-and-branch") 1356 DECLARE_CONCRETE_INSTRUCTION(CmpMapAndBranch, "cmp-map-and-branch")
1358 DECLARE_HYDROGEN_ACCESSOR(CompareMap) 1357 DECLARE_HYDROGEN_ACCESSOR(CompareMap)
1359 1358
1360 Handle<Map> map() const { return hydrogen()->map().handle(); } 1359 Handle<Map> map() const { return hydrogen()->map().handle(); }
1361 }; 1360 };
1362 1361
1363 1362
1364 class LMapEnumLength final : public LTemplateInstruction<1, 1, 0> {
1365 public:
1366 explicit LMapEnumLength(LOperand* value) {
1367 inputs_[0] = value;
1368 }
1369
1370 LOperand* value() { return inputs_[0]; }
1371
1372 DECLARE_CONCRETE_INSTRUCTION(MapEnumLength, "map-enum-length")
1373 };
1374
1375
1376 class LSeqStringGetChar final : public LTemplateInstruction<1, 2, 0> { 1363 class LSeqStringGetChar final : public LTemplateInstruction<1, 2, 0> {
1377 public: 1364 public:
1378 LSeqStringGetChar(LOperand* string, LOperand* index) { 1365 LSeqStringGetChar(LOperand* string, LOperand* index) {
1379 inputs_[0] = string; 1366 inputs_[0] = string;
1380 inputs_[1] = index; 1367 inputs_[1] = index;
1381 } 1368 }
1382 1369
1383 LOperand* string() const { return inputs_[0]; } 1370 LOperand* string() const { return inputs_[0]; }
1384 LOperand* index() const { return inputs_[1]; } 1371 LOperand* index() const { return inputs_[1]; }
1385 1372
(...skipping 1393 matching lines...) Expand 10 before | Expand all | Expand 10 after
2779 DISALLOW_COPY_AND_ASSIGN(LChunkBuilder); 2766 DISALLOW_COPY_AND_ASSIGN(LChunkBuilder);
2780 }; 2767 };
2781 2768
2782 #undef DECLARE_HYDROGEN_ACCESSOR 2769 #undef DECLARE_HYDROGEN_ACCESSOR
2783 #undef DECLARE_CONCRETE_INSTRUCTION 2770 #undef DECLARE_CONCRETE_INSTRUCTION
2784 2771
2785 } // namespace internal 2772 } // namespace internal
2786 } // namespace v8 2773 } // namespace v8
2787 2774
2788 #endif // V8_CRANKSHAFT_ARM_LITHIUM_ARM_H_ 2775 #endif // V8_CRANKSHAFT_ARM_LITHIUM_ARM_H_
OLDNEW
« no previous file with comments | « no previous file | src/crankshaft/arm/lithium-arm.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698