- 08 Aug, 2013 6 commits
-
-
palfia@homejinni.com authored
Port r16114 (2d1bf399) Original commit message: This patch is to enhance the source code line information for profiler. For the Hydrogen compilation, most of the source code line information is not copied from the HInstruction the to corresponding LInstruction. This patch defines one PositionBits field for LInstruction and copies the sorce code position value from the HInstruction. When Generating the native code, we use RecordPosition(..) function to write LInstruction's position value to position recorder. BUG= Review URL: https://codereview.chromium.org/22502006 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16118 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yurys@chromium.org authored
This reverts commit r16116 due to WebKit compilation breakage. Will reland it once Blink r155755 is rolled into Chromium. TBR=svenpanne@chromium.org BUG=None Review URL: https://codereview.chromium.org/22388003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16117 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yurys@chromium.org authored
All of these values are derived from the self samples count and there is no need to evaluate them in v8 when clients can do that when needed on their side. Also added unsigned GetHitCount() which should be used instead of double GetSelfSamplesCount(). I'm going to deprecate the latter one once Blink has switched to GetHitCount. BUG=267595 R=loislo@chromium.org, svenpanne@chromium.org Review URL: https://codereview.chromium.org/22347003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16116 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
plind44@gmail.com authored
This simple change moves the Deoptimize call into the Deopt table where it belongs, and shortens the inline map-check. Gives a surprisingly large performance gain, around 4% on Octane. BUG= R=gergely@homejinni.com Review URL: https://codereview.chromium.org/22391008 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16115 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
haitao.feng@intel.com authored
This patch is to enhance the source code line information for profiler. For the Hydrogen compilation, most of the source code line information is not copied from the HInstruction the to corresponding LInstruction. This patch defines one PositionBits field for LInstruction and copies the sorce code position value from the HInstruction. When Generating the native code, we use RecordPosition(..) function to write LInstruction's position value to position recorder. For the MIPS platform, I did not touch because I have no devices to verify the modification on it. R=danno@chromium.org Review URL: https://codereview.chromium.org/21042003 Patch from Chunyang Dai <chunyang.dai@intel.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16114 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
haitao.feng@intel.com authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/22285003 Patch from Weiliang Lin <weiliang.lin@intel.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16113 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 07 Aug, 2013 21 commits
-
-
plind44@gmail.com authored
Additionally, fix object size register clobbering in Allocate routine. Based on 13284 (d42bf80). BUG= R=plind44@gmail.com Review URL: https://codereview.chromium.org/22582002 Patch from Dusan Milosavljevic <Dusan.Milosavljevic@rt-rk.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16112 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
BUG=2774,2779 R=adamk@chromium.org Review URL: https://chromiumcodereview.appspot.com/22562004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16111 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
Due to buildbot failures. BUG= R=danno@chromium.org Review URL: https://chromiumcodereview.appspot.com/22474007 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16110 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yurys@chromium.org authored
This change provides an API for the embedder to tell CPU profiler if it is idle or busy with some task. This way we can discriminate between idle time and some native code execution. BUG=268947 R=alph@chromium.org, yangguo@chromium.org Review URL: https://codereview.chromium.org/22412003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16109 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
olivf@chromium.org authored
BUG= R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/21495005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16108 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
palfia@homejinni.com authored
Port r16098 (328ff5f7) Original commit message: This change implements a simple data-flow analysis pass over captured objects to the existing escape analysis. It tracks the state of values in the Hydrogen graph through CapturedObject marker instructions that are used to construct an appropriate translation for the deoptimizer to be able to materialize these objects again. This can be considered a combination of scalar replacement of loads and stores on captured objects and sinking of unused allocations. TEST=mjsunit/compiler/escape-analysis BUG= Review URL: https://codereview.chromium.org/22580003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16107 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
R=yangguo@chromium.org Review URL: https://codereview.chromium.org/22420004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16106 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
R=palfia@homejinni.com Review URL: https://codereview.chromium.org/22571003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16102 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
This was an omission in https://code.google.com/p/v8/source/detail?r=14051. R=rossberg@chromium.org Review URL: https://codereview.chromium.org/22578002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16101 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
prybin@chromium.org authored
R=yangguo@chromium.org Review URL: https://codereview.chromium.org/22198002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16100 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jochen@chromium.org authored
BUG=v8:2745 R=dcarney@chromium.org, mstarzinger@chromium.org Review URL: https://codereview.chromium.org/22266009 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16099 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
This change implements a simple data-flow analysis pass over captured objects to the existing escape analysis. It tracks the state of values in the Hydrogen graph through CapturedObject marker instructions that are used to construct an appropriate translation for the deoptimizer to be able to materialize these objects again. This can be considered a combination of scalar replacement of loads and stores on captured objects and sinking of unused allocations. R=titzer@chromium.org TEST=mjsunit/compiler/escape-analysis Review URL: https://codereview.chromium.org/21055011 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16098 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
BUG= R=hpayer@chromium.org, mstarzinger@chromium.org Review URL: https://codereview.chromium.org/22379002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16095 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
It should be able to return Smi, etc. Not only JSObject. BUG=v8:2821 TEST=cctest/test-api/JSONParseNumber R=yangguo@chromium.org Review URL: https://codereview.chromium.org/22416003 Patch from Takeshi Yoshino <tyoshino@chromium.org>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16092 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
dslomov@chromium.org authored
R=danno@chromium.org Review URL: https://codereview.chromium.org/22468004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16091 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=yangguo@chromium.org Review URL: https://codereview.chromium.org/22556002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16090 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
dcarney@chromium.org authored
R=svenpanne@chromium.org BUG= Review URL: https://codereview.chromium.org/22384003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16089 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/22554002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16088 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
bmeurer@chromium.org authored
R=danno@chromium.org Review URL: https://codereview.chromium.org/22274017 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16087 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jochen@chromium.org authored
BUG=v8:2745 R=dcarney@chromium.org, mstarzinger@chromium.org Review URL: https://codereview.chromium.org/22411003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16086 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
palfia@homejinni.com authored
* This makes sure that DoModI calculates correct result even when the left register is the same as the result register. * Microoptmization: it generates smaller code when left and result are different registers. TEST=benchmarks/sunspider/crypto-aes.js BUG= Review URL: https://codereview.chromium.org/22491002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16085 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 06 Aug, 2013 13 commits
-
-
dslomov@chromium.org authored
These factory methods used pointers for constructor fucntions, therefore those pointers could corrupt if allocation triggered gc. R=mstarzinger@chromium.org Review URL: https://codereview.chromium.org/22426003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16084 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
dslomov@chromium.org authored
TBR=yangguo@chromium.org Review URL: https://codereview.chromium.org/22473002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16083 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ulan@chromium.org authored
BUG=264203 TEST=test/mjsunit/regress/regress-264203 R=danno@chromium.org Review URL: https://chromiumcodereview.appspot.com/22314012 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16082 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
R=bmeurer@chromium.org Review URL: https://codereview.chromium.org/22314010 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16081 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
machenbach@chromium.org authored
Test expectations can now include outcome: FLAKY. The test runner can now run a class of tests (flaky|non-flaky|all). All tests are in the non-flaky class that are not marked as FLAKY. The slash correction for windows is now pulled into the test name method. Currently the progress output on windows contains a mixture of / and \. R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/22381003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16080 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
R=titzer@chromium.org BUG= Review URL: https://codereview.chromium.org/22424002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16079 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yurys@chromium.org authored
V8EXPORT is defined in each header of V8 public API and the definitions already have some slight discrepancies. This CL makes all headers use the same definition in v8.h BUG=None R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/22363003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16078 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
rossberg@chromium.org authored
TBR=mstarzinger@chromium.org BUG=265369 Review URL: https://codereview.chromium.org/22428002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16077 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
rossberg@chromium.org authored
R=mstarzinger@chromium.org BUG=265369 Review URL: https://codereview.chromium.org/22284009 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16076 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
If an developer attempts to "subclass" Error by running `MyError.prototype = new Error();`, then the internal v8::Message object that's produced and handed off to `window.onerror` handlers is poorly stringified as "[object Object]". This patch adjusts the stringification process for these objects to include not only native Error objects, but also objects that have Error in their prototype chain, and haven't overwritten Error.toString with some custom variant. BUG=2822 R=mstarzinger@chromium.org, yangguo@chromium.org Review URL: https://codereview.chromium.org/21761002 Patch from Mike West <mkwst@chromium.org>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16075 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
rossberg@chromium.org authored
https://codereview.chromium.org/19541010/ The main problem is that if you called Object.getNotifier(obj) on an object, %SetObserved(object) would never get called on it, and thus it would be unobservable (new test added for this). Additionally, Runtime::SetObserved was asserting obj->IsJSObject() which would fail if called on a proxy. It just happens that our existing test always called getNotifier() before Object.observe on proxies, and thus we never previously attempted to transition the map of a proxy. Both issues are now fixed and properly tested. R=rossberg@chromium.org Review URL: https://codereview.chromium.org/21891008 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16074 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
R=mstarzinger@chromium.org, verwaest@chromium.org Review URL: https://codereview.chromium.org/22184004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16073 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
dslomov@chromium.org authored
At the last face-to-face meeting it was decided that we should use numeric indexes for the Array Iterator values. https://github.com/rwldrn/tc39-notes/blob/master/es6/2013-07/july-24.md#514-keys-entries-return-numbers-for-array-index-properties BUG=v8:2818 R=dslomov@chromium.org Review URL: https://codereview.chromium.org/21180008 Patch from Erik Arvidsson <arv@chromium.org>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@16072 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-