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

Side by Side Diff: test/cctest/test-compiler.cc

Issue 1148973005: Revert of Remove obsolete JSFunction::IsOptimizable predicate. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@local_cleanup-compiler-15
Patch Set: Created 5 years, 7 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/runtime/runtime-test.cc ('k') | no next file » | 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 // 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 377 matching lines...) Expand 10 before | Expand all | Expand 10 after
388 "var closure0 = MakeClosure();" 388 "var closure0 = MakeClosure();"
389 "%DebugPrint(closure0());" 389 "%DebugPrint(closure0());"
390 "%OptimizeFunctionOnNextCall(closure0);" 390 "%OptimizeFunctionOnNextCall(closure0);"
391 "%DebugPrint(closure0());" 391 "%DebugPrint(closure0());"
392 "var closure1 = MakeClosure();" 392 "var closure1 = MakeClosure();"
393 "var closure2 = MakeClosure();"); 393 "var closure2 = MakeClosure();");
394 Handle<JSFunction> fun1 = v8::Utils::OpenHandle( 394 Handle<JSFunction> fun1 = v8::Utils::OpenHandle(
395 *v8::Local<v8::Function>::Cast(env->Global()->Get(v8_str("closure1")))); 395 *v8::Local<v8::Function>::Cast(env->Global()->Get(v8_str("closure1"))));
396 Handle<JSFunction> fun2 = v8::Utils::OpenHandle( 396 Handle<JSFunction> fun2 = v8::Utils::OpenHandle(
397 *v8::Local<v8::Function>::Cast(env->Global()->Get(v8_str("closure2")))); 397 *v8::Local<v8::Function>::Cast(env->Global()->Get(v8_str("closure2"))));
398 CHECK(fun1->IsOptimized() || !CcTest::i_isolate()->use_crankshaft()); 398 CHECK(fun1->IsOptimized()
399 CHECK(fun2->IsOptimized() || !CcTest::i_isolate()->use_crankshaft()); 399 || !CcTest::i_isolate()->use_crankshaft() || !fun1->IsOptimizable());
400 CHECK(fun2->IsOptimized()
401 || !CcTest::i_isolate()->use_crankshaft() || !fun2->IsOptimizable());
400 CHECK_EQ(fun1->code(), fun2->code()); 402 CHECK_EQ(fun1->code(), fun2->code());
401 } 403 }
402 } 404 }
403 405
404 406
405 TEST(CompileFunctionInContext) { 407 TEST(CompileFunctionInContext) {
406 CcTest::InitializeVM(); 408 CcTest::InitializeVM();
407 v8::HandleScope scope(CcTest::isolate()); 409 v8::HandleScope scope(CcTest::isolate());
408 LocalContext env; 410 LocalContext env;
409 CompileRun("var r = 10;"); 411 CompileRun("var r = 10;");
(...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
557 CompileRun("function f() { a = 12345678 }; f();"); 559 CompileRun("function f() { a = 12345678 }; f();");
558 CheckCodeForUnsafeLiteral(GetJSFunction(context->Global(), "f")); 560 CheckCodeForUnsafeLiteral(GetJSFunction(context->Global(), "f"));
559 CompileRun("function f(x) { a = 12345678 + x}; f(1);"); 561 CompileRun("function f(x) { a = 12345678 + x}; f(1);");
560 CheckCodeForUnsafeLiteral(GetJSFunction(context->Global(), "f")); 562 CheckCodeForUnsafeLiteral(GetJSFunction(context->Global(), "f"));
561 CompileRun("function f(x) { var arguments = 1; x += 12345678}; f(1);"); 563 CompileRun("function f(x) { var arguments = 1; x += 12345678}; f(1);");
562 CheckCodeForUnsafeLiteral(GetJSFunction(context->Global(), "f")); 564 CheckCodeForUnsafeLiteral(GetJSFunction(context->Global(), "f"));
563 CompileRun("function f(x) { var arguments = 1; x = 12345678}; f(1);"); 565 CompileRun("function f(x) { var arguments = 1; x = 12345678}; f(1);");
564 CheckCodeForUnsafeLiteral(GetJSFunction(context->Global(), "f")); 566 CheckCodeForUnsafeLiteral(GetJSFunction(context->Global(), "f"));
565 } 567 }
566 #endif 568 #endif
OLDNEW
« no previous file with comments | « src/runtime/runtime-test.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698