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

Side by Side Diff: src/crankshaft/x64/lithium-x64.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/x64/lithium-codegen-x64.cc ('k') | src/crankshaft/x64/lithium-x64.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_X64_LITHIUM_X64_H_ 5 #ifndef V8_CRANKSHAFT_X64_LITHIUM_X64_H_
6 #define V8_CRANKSHAFT_X64_LITHIUM_X64_H_ 6 #define V8_CRANKSHAFT_X64_LITHIUM_X64_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 1208 matching lines...) Expand 10 before | Expand all | Expand 10 after
1323 1322
1324 LOperand* value() { return inputs_[0]; } 1323 LOperand* value() { return inputs_[0]; }
1325 1324
1326 DECLARE_CONCRETE_INSTRUCTION(CmpMapAndBranch, "cmp-map-and-branch") 1325 DECLARE_CONCRETE_INSTRUCTION(CmpMapAndBranch, "cmp-map-and-branch")
1327 DECLARE_HYDROGEN_ACCESSOR(CompareMap) 1326 DECLARE_HYDROGEN_ACCESSOR(CompareMap)
1328 1327
1329 Handle<Map> map() const { return hydrogen()->map().handle(); } 1328 Handle<Map> map() const { return hydrogen()->map().handle(); }
1330 }; 1329 };
1331 1330
1332 1331
1333 class LMapEnumLength final : public LTemplateInstruction<1, 1, 0> {
1334 public:
1335 explicit LMapEnumLength(LOperand* value) {
1336 inputs_[0] = value;
1337 }
1338
1339 LOperand* value() { return inputs_[0]; }
1340
1341 DECLARE_CONCRETE_INSTRUCTION(MapEnumLength, "map-enum-length")
1342 };
1343
1344
1345 class LSeqStringGetChar final : public LTemplateInstruction<1, 2, 0> { 1332 class LSeqStringGetChar final : public LTemplateInstruction<1, 2, 0> {
1346 public: 1333 public:
1347 LSeqStringGetChar(LOperand* string, LOperand* index) { 1334 LSeqStringGetChar(LOperand* string, LOperand* index) {
1348 inputs_[0] = string; 1335 inputs_[0] = string;
1349 inputs_[1] = index; 1336 inputs_[1] = index;
1350 } 1337 }
1351 1338
1352 LOperand* string() const { return inputs_[0]; } 1339 LOperand* string() const { return inputs_[0]; }
1353 LOperand* index() const { return inputs_[1]; } 1340 LOperand* index() const { return inputs_[1]; }
1354 1341
(...skipping 1404 matching lines...) Expand 10 before | Expand all | Expand 10 after
2759 DISALLOW_COPY_AND_ASSIGN(LChunkBuilder); 2746 DISALLOW_COPY_AND_ASSIGN(LChunkBuilder);
2760 }; 2747 };
2761 2748
2762 #undef DECLARE_HYDROGEN_ACCESSOR 2749 #undef DECLARE_HYDROGEN_ACCESSOR
2763 #undef DECLARE_CONCRETE_INSTRUCTION 2750 #undef DECLARE_CONCRETE_INSTRUCTION
2764 2751
2765 } // namespace internal 2752 } // namespace internal
2766 } // namespace v8 2753 } // namespace v8
2767 2754
2768 #endif // V8_CRANKSHAFT_X64_LITHIUM_X64_H_ 2755 #endif // V8_CRANKSHAFT_X64_LITHIUM_X64_H_
OLDNEW
« no previous file with comments | « src/crankshaft/x64/lithium-codegen-x64.cc ('k') | src/crankshaft/x64/lithium-x64.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698