- 02 Mar, 2012 6 commits
-
-
svenpanne@chromium.org authored
Main change from the original CL: Call::ComputeTarget does not use IsProperty anymore, because this would potentially need a holder, which we don't have here. Using Map::LookupInDescriptors with a NULL holder is a bit fishy in general, because one has to be *extremely* careful when using its LookupResult. The original CL made Chrome's NetInternalsTest.netInternalsTourTabs browser test fail, but it's a mystery how this could happen: We should never reach Call::ComputeTarget via Call::RecordTypeFeedback with a CALLBACKS property, because we never consider calls to them monomorphic, which is in turn because of the stub cache leaving them in the pre-monomorphic state. Therefore, I don't have a clue how to write a regression test for this... As an additional tiny bonus, the --trace-opt output for deoptimizations has been improved. Review URL: https://chromiumcodereview.appspot.com/9584003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10906 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
Allow Crankshaft to inline ordered relational comparisons (<, >, <=, >=) that have undefined arguments in addition to double value arguments (rather than calling the generic Compare stub). R=fschneider@chromium.org TEST=test/mjsunit/comparison-ops-and-undefined.js Review URL: https://chromiumcodereview.appspot.com/9584006 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10905 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/9580005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10904 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
There are failures on Kraken. R=fschneider@chromium.org Review URL: https://chromiumcodereview.appspot.com/9585011 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10903 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yurys@chromium.org authored
Cache result of ScriptNameOrSourceURL function. The function is quite slow on large(several MBs) scripts which causes significant slowdown when capturing stack trace for such scripts. Review URL: https://chromiumcodereview.appspot.com/9564012 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10902 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
R=fschneider@chromium.org Review URL: https://chromiumcodereview.appspot.com/9571001 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10901 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 01 Mar, 2012 10 commits
-
-
mstarzinger@chromium.org authored
This is the cleanup of an old todo from r10254 (93483b6eb). BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/9570010 Patch from Daniel Kalmar <kalmard@homejinni.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10896 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/9570016 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10893 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
These changes were not included in r10863 (226afed4) because of patch submission ordering. BUG= TEST= Patch from Daniel Kalmar <kalmard@homejinni.com>. Review URL: https://chromiumcodereview.appspot.com/9566003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10892 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
The old code used a separate HToInt32 instruction which had a wrong register constraint for the input register which caused wrong result when the stored value is used after a typed array store. (UseRegister instead of UseTempRegister) when no SSE3 is available. This change fixes it by replacing HToInt32 with the corresponding HChange instruction which has correct register contraints. TEST=mjsunit/compiler/regress-toint32.js Review URL: https://chromiumcodereview.appspot.com/9565007 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10891 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
Port r10881 (0d25c61e). Original commit message: Generates inlined code for object allocation specific to the initial map of the given constructor function. Also forces completion of inobject slack tracking while crankshafting to finalize instance size of these objects. BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/9569008 Patch from Daniel Kalmar <kalmard@homejinni.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10890 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
MIPS: Fix secondary stub cache and add a test for the stub cache lookups. This is a commit of https://chromiumcodereview.appspot.com/9566004 for Daniel Kalmar git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10885 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
This improves the generated hydrogen graph by also removing the obsolete HCheckFunction instruction if we backout of inlining constructors. R=danno@chromium.org Review URL: https://chromiumcodereview.appspot.com/9537004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10884 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/9540010 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10883 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
By default --trace-hydrogen only generates the final IR (before code generation). The other phases can be printed by specifying them by their first letter. Currently H=HIR, L=LIR or Z=final IR (default) are recognized. Review URL: https://chromiumcodereview.appspot.com/9535009 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10882 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
Generates inlined code for object allocation specific to the initial map of the given constructor function. Also forces completion of inobject slack tracking while crankshafting to finalize instance size of these objects. R=vegorov@chromium.org TEST=mjsunit/compiler/alloc-object Review URL: https://chromiumcodereview.appspot.com/9370019 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10881 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 29 Feb, 2012 13 commits
-
-
ulan@chromium.org authored
R=jkummerow@chromium.org BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/9535010 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10874 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/9535006 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10873 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
Review URL: https://chromiumcodereview.appspot.com/9539010 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10872 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/9536011 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10871 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/9286002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10870 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/9533007 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10869 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
This reverts r10847. Review URL: https://chromiumcodereview.appspot.com/9536010 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10868 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=jkummerow@chromium.org Review URL: https://chromiumcodereview.appspot.com/9536005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10867 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
rossberg@chromium.org authored
R=jkummerow@chromium.org BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/9496003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10866 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
Port r10794 (654fe910). Original commit message: Only JSObject enumerables with enum cache (fast case properties, no interceptors, no enumerable properties on the prototype) are supported. HLoadKeyedGeneric with keys produced by for-in enumeration are recognized and rewritten into direct property load by index. For this enum-cache was extended to store property indices in a separate array (see handles.cc). New hydrogen instructions: - HForInPrepareMap: checks for-in fast case preconditions and returns map that contains enum-cache; - HForInCacheArray: extracts enum-cache array from the map; - HCheckMapValue: map check with HValue map instead of immediate; - HLoadFieldByIndex: load fast property by it's index, positive indexes denote in-object properties, negative - out of object properties; Changed hydrogen instructions: - HLoadKeyedFastElement: added hole check suppression for loads from internal FixedArrays that are knows to have no holes inside. BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/9453009 Patch from Daniel Kalmar <kalmard@homejinni.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10865 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
Review URL: https://chromiumcodereview.appspot.com/9496010 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10864 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
Port r10857 (87e52e7). Original commit message: This is a very mechanical change to reduce the number of calls to Isolate::Current that come with every plain new-allocation. BUG= TEST= Patch from Daniel Kalmar <kalmard@homejinni.com>. Review URL: https://chromiumcodereview.appspot.com/9511003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10863 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
Port r10849 (b0fe79c). Also included: Fixed a bug in GenerateRecordCallTarget. This bug prevented certain functions from being registered as Monomorphic and thus prevented them from being inlined using the new system (b0fe79c). BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/9511002 Patch from Daniel Kalmar <kalmard@homejinni.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10862 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 28 Feb, 2012 11 commits
-
-
ulan@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/9464054 Patch from Iliyan Malchev <malchev@google.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10861 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
This CL is a step towards removing ZoneObject's new operator without a Zone parameter, which uses Isolate::Current. For e.g. the bulletben benchmark, this CL reduces the number of calls to this new operator by roughly 120k, but we are still left with 780k calls from other sites... Review URL: https://chromiumcodereview.appspot.com/9487010 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10860 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
This removes roughly 5k invocations of Isolate::Current from the string-tagcloud benchmark. Review URL: https://chromiumcodereview.appspot.com/9490009 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10859 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
BUG=v8:1900, 115073 Review URL: https://chromiumcodereview.appspot.com/9495005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10858 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
This is a very mechanical change to reduce the number of calls to Isolate::Current that come with every plain new-allocation. BUG=v8:1802 Review URL: https://chromiumcodereview.appspot.com/9491004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10857 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
This removes approx. 12k calls of Isolate::Current() in string-tagcloud. Review URL: https://chromiumcodereview.appspot.com/9490004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10856 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=yangguo@chromium.org BUG=v8:849 Review URL: https://chromiumcodereview.appspot.com/9491005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10855 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
rossberg@chromium.org authored
Define modules as module declarations. Separate function declarations from var declarations. R=jkummerow@chromium.org BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/9460064 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10854 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ulan@chromium.org authored
R=yangguo@chromium.org BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/9495004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10851 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=ulan@chromium.org Review URL: https://chromiumcodereview.appspot.com/9495003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10850 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=vegorov@chromium.org,kmillikin@chromium.org Review URL: https://chromiumcodereview.appspot.com/9304001 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10849 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-