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

Unified Diff: test/cctest/compiler/test-run-inlining.cc

Issue 551413002: Switch inlining to use simplified instead of machine loads. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Addressed comments by Sigurd. Created 6 years, 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « test/cctest/cctest.status ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: test/cctest/compiler/test-run-inlining.cc
diff --git a/test/cctest/compiler/test-run-inlining.cc b/test/cctest/compiler/test-run-inlining.cc
index 8759ac5bf26b781b7d18e90e99c6f5fac88151e3..1e52b3506da7f6426890dd2d6999d09c240f0c04 100644
--- a/test/cctest/compiler/test-run-inlining.cc
+++ b/test/cctest/compiler/test-run-inlining.cc
@@ -42,7 +42,8 @@ TEST(SimpleInlining) {
"function bar(s, t) { return foo(s); };"
"return bar;})();",
CompilationInfo::kInliningEnabled |
- CompilationInfo::kContextSpecializing);
+ CompilationInfo::kContextSpecializing |
+ CompilationInfo::kTypingEnabled);
InstallAssertStackDepthHelper(CcTest::isolate());
T.CheckCall(T.Val(1), T.Val(1), T.Val(2));
@@ -57,7 +58,8 @@ TEST(SimpleInliningContext) {
"return bar;"
"})();",
CompilationInfo::kInliningEnabled |
- CompilationInfo::kContextSpecializing);
+ CompilationInfo::kContextSpecializing |
+ CompilationInfo::kTypingEnabled);
InstallAssertStackDepthHelper(CcTest::isolate());
T.CheckCall(T.Val(13), T.Val(1), T.Val(2));
@@ -73,7 +75,8 @@ TEST(CaptureContext) {
"})();"
"(function (s) { return f(s)})",
CompilationInfo::kInliningEnabled |
- CompilationInfo::kContextSpecializing);
+ CompilationInfo::kContextSpecializing |
+ CompilationInfo::kTypingEnabled);
InstallAssertStackDepthHelper(CcTest::isolate());
T.CheckCall(T.Val(42 + 12), T.Val(12), T.undefined());
@@ -90,7 +93,8 @@ TEST(DontInlineEval) {
"return bar;"
"})();",
CompilationInfo::kInliningEnabled |
- CompilationInfo::kContextSpecializing);
+ CompilationInfo::kContextSpecializing |
+ CompilationInfo::kTypingEnabled);
InstallAssertStackDepthHelper(CcTest::isolate());
T.CheckCall(T.Val(42), T.Val("x"), T.undefined());
@@ -105,7 +109,8 @@ TEST(InlineOmitArguments) {
"return (function (s,t) { return bar(s); });"
"})();",
CompilationInfo::kInliningEnabled |
- CompilationInfo::kContextSpecializing);
+ CompilationInfo::kContextSpecializing |
+ CompilationInfo::kTypingEnabled);
InstallAssertStackDepthHelper(CcTest::isolate());
T.CheckCall(T.Val(42 + 12), T.Val(12), T.undefined());
@@ -121,7 +126,8 @@ TEST(InlineSurplusArguments) {
"return bar;"
"})();",
CompilationInfo::kInliningEnabled |
- CompilationInfo::kContextSpecializing);
+ CompilationInfo::kContextSpecializing |
+ CompilationInfo::kTypingEnabled);
InstallAssertStackDepthHelper(CcTest::isolate());
T.CheckCall(T.Val(42 + 12), T.Val(12), T.undefined());
@@ -136,7 +142,8 @@ TEST(InlineTwice) {
"return (function (s,t) { return bar(s) + bar(t); });"
"})();",
CompilationInfo::kInliningEnabled |
- CompilationInfo::kContextSpecializing);
+ CompilationInfo::kContextSpecializing |
+ CompilationInfo::kTypingEnabled);
InstallAssertStackDepthHelper(CcTest::isolate());
T.CheckCall(T.Val(2 * 42 + 12 + 4), T.Val(12), T.Val(4));
@@ -152,7 +159,8 @@ TEST(InlineTwiceDependent) {
"return bar;"
"})();",
CompilationInfo::kInliningEnabled |
- CompilationInfo::kContextSpecializing);
+ CompilationInfo::kContextSpecializing |
+ CompilationInfo::kTypingEnabled);
InstallAssertStackDepthHelper(CcTest::isolate());
T.CheckCall(T.Val(42 + 42 + 12), T.Val(12), T.Val(4));
@@ -169,7 +177,8 @@ TEST(InlineTwiceDependentDiamond) {
"return bar;"
"})();",
CompilationInfo::kInliningEnabled |
- CompilationInfo::kContextSpecializing);
+ CompilationInfo::kContextSpecializing |
+ CompilationInfo::kTypingEnabled);
InstallAssertStackDepthHelper(CcTest::isolate());
T.CheckCall(T.Val(-11), T.Val(11), T.Val(4));
@@ -186,7 +195,8 @@ TEST(InlineTwiceDependentDiamondDifferent) {
"return bar;"
"})();",
CompilationInfo::kInliningEnabled |
- CompilationInfo::kContextSpecializing);
+ CompilationInfo::kContextSpecializing |
+ CompilationInfo::kTypingEnabled);
InstallAssertStackDepthHelper(CcTest::isolate());
T.CheckCall(T.Val(-329), T.Val(11), T.Val(4));
@@ -203,7 +213,8 @@ TEST(InlineLoop) {
"return bar;"
"})();",
CompilationInfo::kInliningEnabled |
- CompilationInfo::kContextSpecializing);
+ CompilationInfo::kContextSpecializing |
+ CompilationInfo::kTypingEnabled);
InstallAssertStackDepthHelper(CcTest::isolate());
T.CheckCall(T.Val(0.0), T.Val(11), T.Val(4));
@@ -220,7 +231,8 @@ TEST(InlineStrictIntoNonStrict) {
"return bar;"
"})();",
CompilationInfo::kInliningEnabled |
- CompilationInfo::kContextSpecializing);
+ CompilationInfo::kContextSpecializing |
+ CompilationInfo::kTypingEnabled);
InstallAssertStackDepthHelper(CcTest::isolate());
T.CheckThrows(T.undefined(), T.undefined());
@@ -236,7 +248,8 @@ TEST(InlineNonStrictIntoStrict) {
"return bar;"
"})();",
CompilationInfo::kInliningEnabled |
- CompilationInfo::kContextSpecializing);
+ CompilationInfo::kContextSpecializing |
+ CompilationInfo::kTypingEnabled);
InstallAssertStackDepthHelper(CcTest::isolate());
T.CheckCall(T.Val(42), T.undefined(), T.undefined());
« no previous file with comments | « test/cctest/cctest.status ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698