Revision: 15270
Author: [email protected]
Date: Fri Jun 21 08:34:12 2013
Log: Merged r15193, r15263, r15267 into trunk branch.
Turn on parallel recompilation.
Short-circuit embedded cons strings.
HBoundsCheck shouldn't ignore Tagged input representations unconditionally.
BUG=v8:2740
[email protected]
Review URL: https://codereview.chromium.org/17176021
http://code.google.com/p/v8/source/detail?r=15270
Added:
/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
=======================================
--- /dev/null
+++ /trunk/test/mjsunit/regress/regress-embedded-cons-string.js Fri Jun 21
08:34:12 2013
@@ -0,0 +1,66 @@
+// 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 Wed Jun 12 15:31:22 2013
+++ /trunk/src/arm/assembler-arm-inl.h Fri Jun 21 08:34:12 2013
@@ -149,6 +149,7 @@
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 Tue Jun 18 04:54:54 2013
+++ /trunk/src/factory.cc Fri Jun 21 08:34:12 2013
@@ -257,6 +257,32 @@
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 Tue Jun 18 04:54:54 2013
+++ /trunk/src/factory.h Fri Jun 21 08:34:12 2013
@@ -152,6 +152,10 @@
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 Wed Jun 19 11:17:45 2013
+++ /trunk/src/flag-definitions.h Fri Jun 21 08:34:12 2013
@@ -290,10 +290,10 @@
"allow uint32 values on optimize frames if they are used only
in "
"safe operations")
-DEFINE_bool(parallel_recompilation, false,
+DEFINE_bool(parallel_recompilation, true,
"optimizing hot functions asynchronously on a separate thread")
DEFINE_bool(trace_parallel_recompilation, false, "track parallel
recompilation")
-DEFINE_int(parallel_recompilation_queue_length, 3,
+DEFINE_int(parallel_recompilation_queue_length, 8,
"the length of the parallel compilation queue")
DEFINE_int(parallel_recompilation_delay, 0,
"artificial compilation delay in ms")
=======================================
--- /trunk/src/hydrogen-instructions.cc Tue Jun 18 04:54:54 2013
+++ /trunk/src/hydrogen-instructions.cc Fri Jun 21 08:34:12 2013
@@ -1155,8 +1155,12 @@
HValue* actual_length = length()->ActualValue();
Representation index_rep = actual_index->representation();
Representation length_rep = actual_length->representation();
- if (index_rep.IsTagged()) index_rep = Representation::Smi();
- if (length_rep.IsTagged()) length_rep = Representation::Smi();
+ if (index_rep.IsTagged() && actual_index->type().IsSmi()) {
+ index_rep = Representation::Smi();
+ }
+ if (length_rep.IsTagged() && actual_length->type().IsSmi()) {
+ length_rep = Representation::Smi();
+ }
Representation r = index_rep.generalize(length_rep);
if (r.is_more_general_than(Representation::Integer32())) {
r = Representation::Integer32();
@@ -3323,10 +3327,9 @@
HConstant* c_right = HConstant::cast(right);
HConstant* c_left = HConstant::cast(left);
if (c_left->HasStringValue() && c_right->HasStringValue()) {
- Factory* factory = Isolate::Current()->factory();
- return new(zone)
HConstant(factory->NewConsString(c_left->StringValue(),
-
c_right->StringValue()),
- Representation::Tagged());
+ Handle<String> concat =
zone->isolate()->factory()->NewFlatConcatString(
+ c_left->StringValue(), c_right->StringValue());
+ return new(zone) HConstant(concat, Representation::Tagged());
}
}
return new(zone) HStringAdd(context, left, right);
=======================================
--- /trunk/src/ia32/assembler-ia32-inl.h Wed Jun 12 15:31:22 2013
+++ /trunk/src/ia32/assembler-ia32-inl.h Fri Jun 21 08:34:12 2013
@@ -132,6 +132,7 @@
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 Thu Jun 13 07:40:17 2013
+++ /trunk/src/mips/assembler-mips-inl.h Fri Jun 21 08:34:12 2013
@@ -202,6 +202,7 @@
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 Thu Jun 20 09:56:08 2013
+++ /trunk/src/version.cc Fri Jun 21 08:34:12 2013
@@ -35,7 +35,7 @@
#define MAJOR_VERSION 3
#define MINOR_VERSION 19
#define BUILD_NUMBER 18
-#define PATCH_LEVEL 4
+#define PATCH_LEVEL 5
// 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 Wed Jun 12 15:31:22 2013
+++ /trunk/src/x64/assembler-x64-inl.h Fri Jun 21 08:34:12 2013
@@ -308,6 +308,7 @@
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.
