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

Side by Side Diff: src/ia32/lithium-ia32.h

Issue 12208013: Separated smi check from HBoundsCheck. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Created 7 years, 10 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
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Redistribution and use in source and binary forms, with or without 2 // Redistribution and use in source and binary forms, with or without
3 // modification, are permitted provided that the following conditions are 3 // modification, are permitted provided that the following conditions are
4 // met: 4 // met:
5 // 5 //
6 // * Redistributions of source code must retain the above copyright 6 // * Redistributions of source code must retain the above copyright
7 // notice, this list of conditions and the following disclaimer. 7 // notice, this list of conditions and the following disclaimer.
8 // * Redistributions in binary form must reproduce the above 8 // * Redistributions in binary form must reproduce the above
9 // copyright notice, this list of conditions and the following 9 // copyright notice, this list of conditions and the following
10 // disclaimer in the documentation and/or other materials provided 10 // disclaimer in the documentation and/or other materials provided
(...skipping 68 matching lines...) Expand 10 before | Expand all | Expand 10 after
79 V(CmpMapAndBranch) \ 79 V(CmpMapAndBranch) \
80 V(CmpT) \ 80 V(CmpT) \
81 V(CmpConstantEqAndBranch) \ 81 V(CmpConstantEqAndBranch) \
82 V(ConstantD) \ 82 V(ConstantD) \
83 V(ConstantI) \ 83 V(ConstantI) \
84 V(ConstantT) \ 84 V(ConstantT) \
85 V(Context) \ 85 V(Context) \
86 V(DeclareGlobals) \ 86 V(DeclareGlobals) \
87 V(DeleteProperty) \ 87 V(DeleteProperty) \
88 V(Deoptimize) \ 88 V(Deoptimize) \
89 V(DeoptimizeIfTaggedIsNotSmi) \
89 V(DivI) \ 90 V(DivI) \
90 V(DoubleToI) \ 91 V(DoubleToI) \
91 V(DummyUse) \ 92 V(DummyUse) \
92 V(ElementsKind) \ 93 V(ElementsKind) \
93 V(FastLiteral) \ 94 V(FastLiteral) \
94 V(FixedArrayBaseLength) \ 95 V(FixedArrayBaseLength) \
95 V(FunctionLiteral) \ 96 V(FunctionLiteral) \
96 V(GetCachedArrayIndex) \ 97 V(GetCachedArrayIndex) \
97 V(GlobalObject) \ 98 V(GlobalObject) \
98 V(GlobalReceiver) \ 99 V(GlobalReceiver) \
(...skipping 2169 matching lines...) Expand 10 before | Expand all | Expand 10 after
2268 explicit LCheckSmi(LOperand* value) { 2269 explicit LCheckSmi(LOperand* value) {
2269 inputs_[0] = value; 2270 inputs_[0] = value;
2270 } 2271 }
2271 2272
2272 LOperand* value() { return inputs_[0]; } 2273 LOperand* value() { return inputs_[0]; }
2273 2274
2274 DECLARE_CONCRETE_INSTRUCTION(CheckSmi, "check-smi") 2275 DECLARE_CONCRETE_INSTRUCTION(CheckSmi, "check-smi")
2275 }; 2276 };
2276 2277
2277 2278
2279 class LDeoptimizeIfTaggedIsNotSmi: public LTemplateInstruction<0, 1, 0> {
Jakob Kummerow 2013/02/06 13:43:59 I think we don't need a new LInstruction. Instead,
Massi 2013/02/06 19:12:31 Done.
2280 public:
2281 explicit LDeoptimizeIfTaggedIsNotSmi(LOperand* value) {
2282 inputs_[0] = value;
2283 }
2284
2285 LOperand* value() { return inputs_[0]; }
2286
2287 DECLARE_CONCRETE_INSTRUCTION(DeoptimizeIfTaggedIsNotSmi,
2288 "deoptimize-if-tagged-is-not-smi")
2289 };
2290
2291
2278 class LClampDToUint8: public LTemplateInstruction<1, 1, 0> { 2292 class LClampDToUint8: public LTemplateInstruction<1, 1, 0> {
2279 public: 2293 public:
2280 explicit LClampDToUint8(LOperand* value) { 2294 explicit LClampDToUint8(LOperand* value) {
2281 inputs_[0] = value; 2295 inputs_[0] = value;
2282 } 2296 }
2283 2297
2284 LOperand* unclamped() { return inputs_[0]; } 2298 LOperand* unclamped() { return inputs_[0]; }
2285 2299
2286 DECLARE_CONCRETE_INSTRUCTION(ClampDToUint8, "clamp-d-to-uint8") 2300 DECLARE_CONCRETE_INSTRUCTION(ClampDToUint8, "clamp-d-to-uint8")
2287 }; 2301 };
(...skipping 487 matching lines...) Expand 10 before | Expand all | Expand 10 after
2775 2789
2776 DISALLOW_COPY_AND_ASSIGN(LChunkBuilder); 2790 DISALLOW_COPY_AND_ASSIGN(LChunkBuilder);
2777 }; 2791 };
2778 2792
2779 #undef DECLARE_HYDROGEN_ACCESSOR 2793 #undef DECLARE_HYDROGEN_ACCESSOR
2780 #undef DECLARE_CONCRETE_INSTRUCTION 2794 #undef DECLARE_CONCRETE_INSTRUCTION
2781 2795
2782 } } // namespace v8::internal 2796 } } // namespace v8::internal
2783 2797
2784 #endif // V8_IA32_LITHIUM_IA32_H_ 2798 #endif // V8_IA32_LITHIUM_IA32_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698