From: mstarzinger@chromium.org Date: Thu, 24 Jan 2013 17:54:30 +0000 (+0000) Subject: Allow monomorphic loads when static type is known. X-Git-Tag: upstream/4.7.83~15206 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=44ffa0dcd3443bed8b8aad68f342e0735e7e8fc7;p=platform%2Fupstream%2Fv8.git Allow monomorphic loads when static type is known. This allows Crankshaft to generate monomorphic loads when the receiver type is statically known even though the load site has polymorphic type feedback. This applies to inlined constructor calls and literals. R=jkummerow@chromium.org TEST=mjsunit/compiler/property-static Review URL: https://codereview.chromium.org/12051058 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13500 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- diff --git a/src/hydrogen-instructions.cc b/src/hydrogen-instructions.cc index 0bf208b..38f3dac 100644 --- a/src/hydrogen-instructions.cc +++ b/src/hydrogen-instructions.cc @@ -571,6 +571,11 @@ void HValue::PrintNameTo(StringStream* stream) { } +bool HValue::HasMonomorphicJSObjectType() { + return !GetMonomorphicJSObjectMap().is_null(); +} + + bool HValue::UpdateInferredType() { HType type = CalculateInferredType(); bool result = (!type.Equals(type_)); diff --git a/src/hydrogen-instructions.h b/src/hydrogen-instructions.h index 367d2b3..aa513e7 100644 --- a/src/hydrogen-instructions.h +++ b/src/hydrogen-instructions.h @@ -770,6 +770,14 @@ class HValue: public ZoneObject { const char* Mnemonic() const; + // Type information helpers. + bool HasMonomorphicJSObjectType(); + + // TODO(mstarzinger): For now instructions can override this function to + // specify statically known types, once HType can convey more information + // it should be based on the HType. + virtual Handle GetMonomorphicJSObjectMap() { return Handle(); } + // Updated the inferred type of this instruction and returns true if // it has changed. bool UpdateInferredType(); @@ -5019,6 +5027,10 @@ class HAllocateObject: public HTemplateInstruction<1> { virtual Representation RequiredInputRepresentation(int index) { return Representation::Tagged(); } + virtual Handle GetMonomorphicJSObjectMap() { + ASSERT(constructor()->has_initial_map()); + return Handle(constructor()->initial_map()); + } virtual HType CalculateInferredType(); DECLARE_CONCRETE_INSTRUCTION(AllocateObject) @@ -5086,6 +5098,9 @@ class HFastLiteral: public HMaterializedLiteral<1> { virtual Representation RequiredInputRepresentation(int index) { return Representation::Tagged(); } + virtual Handle GetMonomorphicJSObjectMap() { + return Handle(boilerplate()->map()); + } virtual HType CalculateInferredType(); DECLARE_CONCRETE_INSTRUCTION(FastLiteral) diff --git a/src/hydrogen.cc b/src/hydrogen.cc index 8de8e52..019f39a 100644 --- a/src/hydrogen.cc +++ b/src/hydrogen.cc @@ -6796,12 +6796,16 @@ void HOptimizedGraphBuilder::VisitProperty(Property* expr) { } else if (expr->key()->IsPropertyName()) { Handle name = expr->key()->AsLiteral()->AsPropertyName(); SmallMapList* types = expr->GetReceiverTypes(); + HValue* object = Top(); - bool monomorphic = expr->IsMonomorphic(); Handle map; + bool monomorphic = false; if (expr->IsMonomorphic()) { map = types->first(); - if (map->is_dictionary_map()) monomorphic = false; + monomorphic = !map->is_dictionary_map(); + } else if (object->HasMonomorphicJSObjectType()) { + map = object->GetMonomorphicJSObjectMap(); + monomorphic = !map->is_dictionary_map(); } if (monomorphic) { Handle getter; diff --git a/test/mjsunit/compiler/property-static.js b/test/mjsunit/compiler/property-static.js new file mode 100644 index 0000000..0702134 --- /dev/null +++ b/test/mjsunit/compiler/property-static.js @@ -0,0 +1,69 @@ +// 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: --allow-natives-syntax + +// Test usage of static type information for loads that would otherwise +// turn into polymorphic or generic loads. + +// Prepare a highly polymorphic load to be used by all tests. +Object.prototype.load = function() { return this.property; }; +Object.prototype.load.call({ A:0, property:10 }); +Object.prototype.load.call({ A:0, B:0, property:11 }); +Object.prototype.load.call({ A:0, B:0, C:0, property:12 }); +Object.prototype.load.call({ A:0, B:0, C:0, D:0, property:13 }); +Object.prototype.load.call({ A:0, B:0, C:0, D:0, E:0, property:14 }); +Object.prototype.load.call({ A:0, B:0, C:0, D:0, E:0, F:0, property:15 }); + +// Test for object literals. +(function() { + function f(x) { + var object = { property:x }; + return object.load(); + } + + assertSame(1, f(1)); + assertSame(2, f(2)); + %OptimizeFunctionOnNextCall(f); + assertSame(3, f(3)); +})(); + +// Test for inlined constructors. +(function() { + function c(x) { + this.property = x; + } + function f(x) { + var object = new c(x); + return object.load(); + } + + assertSame(1, f(1)); + assertSame(2, f(2)); + %OptimizeFunctionOnNextCall(f); + assertSame(3, f(3)); +})();