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

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

Issue 1782893002: [Interpreter] Fixes cctest/test-serialize/SerializeInternalReference for ignition. (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Rebased the patch Created 4 years, 9 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 | « test/cctest/cctest.status ('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 2007-2010 the V8 project authors. All rights reserved. 1 // Copyright 2007-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 1550 matching lines...) Expand 10 before | Expand all | Expand 10 after
1561 CHECK(result_str->Equals(isolate2->GetCurrentContext(), v8_str("XY")) 1561 CHECK(result_str->Equals(isolate2->GetCurrentContext(), v8_str("XY"))
1562 .FromJust()); 1562 .FromJust());
1563 } 1563 }
1564 isolate2->Dispose(); 1564 isolate2->Dispose();
1565 } 1565 }
1566 1566
1567 TEST(CodeSerializerInternalReference) { 1567 TEST(CodeSerializerInternalReference) {
1568 #if V8_TARGET_ARCH_ARM || V8_TARGET_ARCH_ARM64 1568 #if V8_TARGET_ARCH_ARM || V8_TARGET_ARCH_ARM64
1569 return; 1569 return;
1570 #endif 1570 #endif
1571 // In ignition there are only relative jumps, so the following code
1572 // would not have any internal references. This test is not relevant
1573 // for ignition.
1574 if (FLAG_ignition) {
1575 return;
1576 }
1571 // Disable experimental natives that are loaded after deserialization. 1577 // Disable experimental natives that are loaded after deserialization.
1572 FLAG_function_context_specialization = false; 1578 FLAG_function_context_specialization = false;
1573 FLAG_always_opt = true; 1579 FLAG_always_opt = true;
1574 const char* flag = "--turbo-filter=foo"; 1580 const char* flag = "--turbo-filter=foo";
1575 FlagList::SetFlagsFromString(flag, StrLength(flag)); 1581 FlagList::SetFlagsFromString(flag, StrLength(flag));
1576 1582
1577 const char* source = 1583 const char* source =
1578 "var foo = (function(stdlib, foreign, heap) {" 1584 "var foo = (function(stdlib, foreign, heap) {"
1579 " function foo(i) {" 1585 " function foo(i) {"
1580 " i = i|0;" 1586 " i = i|0;"
(...skipping 91 matching lines...) Expand 10 before | Expand all | Expand 10 after
1672 delete script_data; 1678 delete script_data;
1673 } 1679 }
1674 1680
1675 1681
1676 TEST(SerializationMemoryStats) { 1682 TEST(SerializationMemoryStats) {
1677 FLAG_profile_deserialization = true; 1683 FLAG_profile_deserialization = true;
1678 FLAG_always_opt = false; 1684 FLAG_always_opt = false;
1679 v8::StartupData blob = v8::V8::CreateSnapshotDataBlob(); 1685 v8::StartupData blob = v8::V8::CreateSnapshotDataBlob();
1680 delete[] blob.data; 1686 delete[] blob.data;
1681 } 1687 }
OLDNEW
« 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