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

Side by Side Diff: src/ia32/lithium-codegen-ia32.cc

Issue 5804003: Revert change 5989, which causes failures in some benchmarks. (Closed) Base URL: http://v8.googlecode.com/svn/branches/bleeding_edge/
Patch Set: Created 10 years 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 | « src/ia32/lithium-codegen-ia32.h ('k') | src/ia32/lithium-ia32.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 2010 the V8 project authors. All rights reserved. 1 // Copyright 2010 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 2217 matching lines...) Expand 10 before | Expand all | Expand 10 after
2228 2228
2229 // Return value is in st(0) on ia32. 2229 // Return value is in st(0) on ia32.
2230 // Store it into the (fixed) result register. 2230 // Store it into the (fixed) result register.
2231 __ sub(Operand(esp), Immediate(kDoubleSize)); 2231 __ sub(Operand(esp), Immediate(kDoubleSize));
2232 __ fstp_d(Operand(esp, 0)); 2232 __ fstp_d(Operand(esp, 0));
2233 __ movdbl(ToDoubleRegister(instr->result()), Operand(esp, 0)); 2233 __ movdbl(ToDoubleRegister(instr->result()), Operand(esp, 0));
2234 __ add(Operand(esp), Immediate(kDoubleSize)); 2234 __ add(Operand(esp), Immediate(kDoubleSize));
2235 } 2235 }
2236 2236
2237 2237
2238 void LCodeGen::DoMathLog(LUnaryMathOperation* instr) {
2239 ASSERT(ToDoubleRegister(instr->result()).is(xmm1));
2240 TranscendentalCacheSSE2Stub stub(TranscendentalCache::LOG);
2241 CallCode(stub.GetCode(), RelocInfo::CODE_TARGET, instr);
2242 }
2243
2244
2245 void LCodeGen::DoUnaryMathOperation(LUnaryMathOperation* instr) { 2238 void LCodeGen::DoUnaryMathOperation(LUnaryMathOperation* instr) {
2246 switch (instr->op()) { 2239 switch (instr->op()) {
2247 case kMathAbs: 2240 case kMathAbs:
2248 DoMathAbs(instr); 2241 DoMathAbs(instr);
2249 break; 2242 break;
2250 case kMathFloor: 2243 case kMathFloor:
2251 DoMathFloor(instr); 2244 DoMathFloor(instr);
2252 break; 2245 break;
2253 case kMathRound: 2246 case kMathRound:
2254 DoMathRound(instr); 2247 DoMathRound(instr);
2255 break; 2248 break;
2256 case kMathSqrt: 2249 case kMathSqrt:
2257 DoMathSqrt(instr); 2250 DoMathSqrt(instr);
2258 break; 2251 break;
2259 case kMathPowHalf: 2252 case kMathPowHalf:
2260 DoMathPowHalf(instr); 2253 DoMathPowHalf(instr);
2261 break; 2254 break;
2262 case kMathLog:
2263 DoMathLog(instr);
2264 break;
2265
2266 default: 2255 default:
2267 UNREACHABLE(); 2256 UNREACHABLE();
2268 } 2257 }
2269 } 2258 }
2270 2259
2271 2260
2272 void LCodeGen::DoCallKeyed(LCallKeyed* instr) { 2261 void LCodeGen::DoCallKeyed(LCallKeyed* instr) {
2273 ASSERT(ToRegister(instr->result()).is(eax)); 2262 ASSERT(ToRegister(instr->result()).is(eax));
2274 2263
2275 int arity = instr->arity(); 2264 int arity = instr->arity();
(...skipping 890 matching lines...) Expand 10 before | Expand all | Expand 10 after
3166 ASSERT(!environment->HasBeenRegistered()); 3155 ASSERT(!environment->HasBeenRegistered());
3167 RegisterEnvironmentForDeoptimization(environment); 3156 RegisterEnvironmentForDeoptimization(environment);
3168 ASSERT(osr_pc_offset_ == -1); 3157 ASSERT(osr_pc_offset_ == -1);
3169 osr_pc_offset_ = masm()->pc_offset(); 3158 osr_pc_offset_ = masm()->pc_offset();
3170 } 3159 }
3171 3160
3172 3161
3173 #undef __ 3162 #undef __
3174 3163
3175 } } // namespace v8::internal 3164 } } // namespace v8::internal
OLDNEW
« no previous file with comments | « src/ia32/lithium-codegen-ia32.h ('k') | src/ia32/lithium-ia32.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698