Revision: 15283
Author: [email protected]
Date: Mon Jun 24 04:29:10 2013
Log: Rollback of r15267, r15263, r15193 in trunk branch in preparation
for 3.19 branch
HBoundsCheck shouldn't ignore Tagged input representations unconditionally
Short-circuit embedded cons strings.
Turn on parallel recompilation.
BUG=v8:2740
[email protected]
Review URL: https://codereview.chromium.org/17585003
http://code.google.com/p/v8/source/detail?r=15283
Deleted:
/trunk/test/mjsunit/regress/regress-embedded-cons-string.js
Modified:
/trunk/src/arm/assembler-arm-inl.h
/trunk/src/factory.cc
/trunk/src/factory.h
/trunk/src/flag-definitions.h
/trunk/src/hydrogen-instructions.cc
/trunk/src/ia32/assembler-ia32-inl.h
/trunk/src/mips/assembler-mips-inl.h
/trunk/src/version.cc
/trunk/src/x64/assembler-x64-inl.h
=======================================
--- /trunk/test/mjsunit/regress/regress-embedded-cons-string.js Fri Jun 21
08:34:12 2013
+++ /dev/null
@@ -1,66 +0,0 @@
-// Copyright 2013 the V8 project authors. All rights reserved.
-// Redistribution and use in source and binary forms, with or without
-// modification, are permitted provided that the following conditions are
-// met:
-//
-// * Redistributions of source code must retain the above copyright
-// notice, this list of conditions and the following disclaimer.
-// * Redistributions in binary form must reproduce the above
-// copyright notice, this list of conditions and the following
-// disclaimer in the documentation and/or other materials provided
-// with the distribution.
-// * Neither the name of Google Inc. nor the names of its
-// contributors may be used to endorse or promote products derived
-// from this software without specific prior written permission.
-//
-// THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
-// "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
-// LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
-// A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
-// OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
-// SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
-// LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
-// DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
-// THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
-// (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
-// OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
-
-// Flags: --fold-constants --nodead-code-elimination
-// Flags: --expose-gc --allow-natives-syntax
-// Flags: --parallel-recompilation --parallel-recompilation-delay=300
-
-if (!%IsParallelRecompilationSupported()) {
- print("Parallel recompilation is disabled. Skipping this test.");
- quit();
-}
-
-function assertUnoptimized(fun) {
- assertTrue(%GetOptimizationStatus(fun) != 1);
-}
-
-function test(fun) {
- fun();
- fun();
- %OptimizeFunctionOnNextCall(fun, "parallel");
- fun(); // Trigger optimization in the background.
- gc(); // Tenure cons string.
- assertUnoptimized(fun); // Compilation not complete yet.
- %CompleteOptimization(fun); // Compilation embeds tenured cons string.
- gc(); // Visit embedded cons string during mark compact.
-}
-
-function f() {
- return "abcdefghijklmn" + "123456789";
-}
-
-function g() {
- return "abcdefghijklmn\u2603" + "123456789";
-}
-
-function h() {
- return "abcdefghijklmn\u2603" + "123456789\u2604";
-}
-
-test(f);
-test(g);
-test(h);
=======================================
--- /trunk/src/arm/assembler-arm-inl.h Fri Jun 21 08:34:12 2013
+++ /trunk/src/arm/assembler-arm-inl.h Mon Jun 24 04:29:10 2013
@@ -149,7 +149,6 @@
void RelocInfo::set_target_object(Object* target, WriteBarrierMode mode) {
ASSERT(IsCodeTarget(rmode_) || rmode_ == EMBEDDED_OBJECT);
- ASSERT(!target->IsConsString());
Assembler::set_target_pointer_at(pc_, reinterpret_cast<Address>(target));
if (mode == UPDATE_WRITE_BARRIER &&
host() != NULL &&
=======================================
--- /trunk/src/factory.cc Fri Jun 21 08:34:12 2013
+++ /trunk/src/factory.cc Mon Jun 24 04:29:10 2013
@@ -257,32 +257,6 @@
isolate()->heap()->AllocateConsString(*first,
*second),
String);
}
-
-
-template<typename SinkChar, typename StringType>
-Handle<String> ConcatStringContent(Handle<StringType> result,
- Handle<String> first,
- Handle<String> second) {
- DisallowHeapAllocation pointer_stays_valid;
- SinkChar* sink = result->GetChars();
- String::WriteToFlat(*first, sink, 0, first->length());
- String::WriteToFlat(*second, sink + first->length(), 0,
second->length());
- return result;
-}
-
-
-Handle<String> Factory::NewFlatConcatString(Handle<String> first,
- Handle<String> second) {
- int total_length = first->length() + second->length();
- if (first->IsOneByteRepresentationUnderneath() &&
- second->IsOneByteRepresentationUnderneath()) {
- return ConcatStringContent<uint8_t>(
- NewRawOneByteString(total_length), first, second);
- } else {
- return ConcatStringContent<uc16>(
- NewRawTwoByteString(total_length), first, second);
- }
-}
Handle<String> Factory::NewSubString(Handle<String> str,
=======================================
--- /trunk/src/factory.h Fri Jun 21 08:34:12 2013
+++ /trunk/src/factory.h Mon Jun 24 04:29:10 2013
@@ -152,10 +152,6 @@
Handle<String> NewConsString(Handle<String> first,
Handle<String> second);
- // Create a new sequential string containing the concatenation of the
inputs.
- Handle<String> NewFlatConcatString(Handle<String> first,
- Handle<String> second);
-
// Create a new string object which holds a substring of a string.
Handle<String> NewSubString(Handle<String> str,
int begin,
=======================================
--- /trunk/src/flag-definitions.h Fri Jun 21 08:34:12 2013
+++ /trunk/src/flag-definitions.h Mon Jun 24 04:29:10 2013
@@ -290,10 +290,10 @@
"allow uint32 values on optimize frames if they are used only
in "
"safe operations")
-DEFINE_bool(parallel_recompilation, true,
+DEFINE_bool(parallel_recompilation, false,
"optimizing hot functions asynchronously on a separate thread")
DEFINE_bool(trace_parallel_recompilation, false, "track parallel
recompilation")
-DEFINE_int(parallel_recompilation_queue_length, 8,
+DEFINE_int(parallel_recompilation_queue_length, 3,
"the length of the parallel compilation queue")
DEFINE_int(parallel_recompilation_delay, 0,
"artificial compilation delay in ms")
=======================================
--- /trunk/src/hydrogen-instructions.cc Fri Jun 21 08:34:12 2013
+++ /trunk/src/hydrogen-instructions.cc Mon Jun 24 04:29:10 2013
@@ -1155,12 +1155,8 @@
HValue* actual_length = length()->ActualValue();
Representation index_rep = actual_index->representation();
Representation length_rep = actual_length->representation();
- if (index_rep.IsTagged() && actual_index->type().IsSmi()) {
- index_rep = Representation::Smi();
- }
- if (length_rep.IsTagged() && actual_length->type().IsSmi()) {
- length_rep = Representation::Smi();
- }
+ if (index_rep.IsTagged()) index_rep = Representation::Smi();
+ if (length_rep.IsTagged()) length_rep = Representation::Smi();
Representation r = index_rep.generalize(length_rep);
if (r.is_more_general_than(Representation::Integer32())) {
r = Representation::Integer32();
@@ -3327,9 +3323,10 @@
HConstant* c_right = HConstant::cast(right);
HConstant* c_left = HConstant::cast(left);
if (c_left->HasStringValue() && c_right->HasStringValue()) {
- Handle<String> concat =
zone->isolate()->factory()->NewFlatConcatString(
- c_left->StringValue(), c_right->StringValue());
- return new(zone) HConstant(concat, Representation::Tagged());
+ Factory* factory = Isolate::Current()->factory();
+ return new(zone)
HConstant(factory->NewConsString(c_left->StringValue(),
+
c_right->StringValue()),
+ Representation::Tagged());
}
}
return new(zone) HStringAdd(context, left, right);
=======================================
--- /trunk/src/ia32/assembler-ia32-inl.h Fri Jun 21 08:34:12 2013
+++ /trunk/src/ia32/assembler-ia32-inl.h Mon Jun 24 04:29:10 2013
@@ -132,7 +132,6 @@
void RelocInfo::set_target_object(Object* target, WriteBarrierMode mode) {
ASSERT(IsCodeTarget(rmode_) || rmode_ == EMBEDDED_OBJECT);
- ASSERT(!target->IsConsString());
Memory::Object_at(pc_) = target;
CPU::FlushICache(pc_, sizeof(Address));
if (mode == UPDATE_WRITE_BARRIER &&
=======================================
--- /trunk/src/mips/assembler-mips-inl.h Fri Jun 21 08:34:12 2013
+++ /trunk/src/mips/assembler-mips-inl.h Mon Jun 24 04:29:10 2013
@@ -202,7 +202,6 @@
void RelocInfo::set_target_object(Object* target, WriteBarrierMode mode) {
ASSERT(IsCodeTarget(rmode_) || rmode_ == EMBEDDED_OBJECT);
- ASSERT(!target->IsConsString());
Assembler::set_target_address_at(pc_, reinterpret_cast<Address>(target));
if (mode == UPDATE_WRITE_BARRIER &&
host() != NULL &&
=======================================
--- /trunk/src/version.cc Fri Jun 21 08:34:12 2013
+++ /trunk/src/version.cc Mon Jun 24 04:29:10 2013
@@ -35,7 +35,7 @@
#define MAJOR_VERSION 3
#define MINOR_VERSION 19
#define BUILD_NUMBER 18
-#define PATCH_LEVEL 5
+#define PATCH_LEVEL 6
// Use 1 for candidates and 0 otherwise.
// (Boolean macro values are not supported by all preprocessors.)
#define IS_CANDIDATE_VERSION 0
=======================================
--- /trunk/src/x64/assembler-x64-inl.h Fri Jun 21 08:34:12 2013
+++ /trunk/src/x64/assembler-x64-inl.h Mon Jun 24 04:29:10 2013
@@ -308,7 +308,6 @@
void RelocInfo::set_target_object(Object* target, WriteBarrierMode mode) {
ASSERT(IsCodeTarget(rmode_) || rmode_ == EMBEDDED_OBJECT);
- ASSERT(!target->IsConsString());
Memory::Object_at(pc_) = target;
CPU::FlushICache(pc_, sizeof(Address));
if (mode == UPDATE_WRITE_BARRIER &&
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/groups/opt_out.
