- 03 Aug, 2012 1 commit
-
-
svenpanne@chromium.org authored
The underlying problem is that for compound/count operations we use the *load* type feedback for storing, too. For normal properties this doesn't matter, but for accessor properties we should better use the *store* type feedback, which would be available, too. This consistent feedback usage could be guaranteed if we removed the heavy copy-n-paste in the crankshaft code generation for compound/count operations and assignments/property loads. To be on the safe side, we postpone this refactoring and do a quick and easily mergeable fix. BUG=140083 TEST=mjsunit/regress/regress-crbug-140083.js Review URL: https://chromiumcodereview.appspot.com/10828146 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12252 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 02 Aug, 2012 2 commits
-
-
mstarzinger@chromium.org authored
R=ulan@chromium.org Review URL: https://chromiumcodereview.appspot.com/10843052 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12251 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
This is a forward-compatible change to avoid type/naming conflicts when the Android platform/NDK will update its <signal.h> header to properly define 'struct sigcontext', 'mcontext_t' and 'ucontext_t'. In particular: - Do not define 'struct sigcontext.h' to avoid conflicts with the C library definition (which is different, see below). - Only provide custom ucontext_t declarations if the Android <signal.h> doesn't provide it. This can be tested with a macro check (__BIONIC_HAVE_UCONTEXT_T) + Use 'gettid()' on Android since it is available (at all API levels). See http://code.google.com/p/android/issues/detail?id=34784 Review URL: https://chromiumcodereview.appspot.com/10829122 Patch from David Turner <digit@chromium.org>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12250 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 01 Aug, 2012 8 commits
-
-
ulan@chromium.org authored
R=yangguo@chromium.org Review URL: https://chromiumcodereview.appspot.com/10836049 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12249 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
When specifying a new create histogram callback, reset all histograms so they recreate themselves on next use. We only do this for histograms, and not for counters, as counters might be used directly from generated code BUG=none TEST=none Review URL: https://chromiumcodereview.appspot.com/10828113 Patch from Jochen Eisinger <jochen@chromium.org>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12248 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
case objects). We should be testing the holder for dictionary mode, not the receiver. Review URL: https://chromiumcodereview.appspot.com/10827113 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12247 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=ulan@chromium.org Review URL: https://chromiumcodereview.appspot.com/10826079 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12246 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
This extends the existing clearing of type feedback cells during GC to incremental marking in order to prevent cross-context retention that would last until the next non-incremental GC. R=erik.corry@gmail.com TEST=cctest/test-heap/IncrementalMarkingClearsTypeFeedbackCells Review URL: https://chromiumcodereview.appspot.com/10823082 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12245 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
R=yangguo@chromium.org Review URL: https://chromiumcodereview.appspot.com/10824125 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12242 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/10828112 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12241 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=erik.corry@gmail.com TEST=cctest/test-serialize (--gc-interval=100) Review URL: https://chromiumcodereview.appspot.com/10829097 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12240 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 31 Jul, 2012 8 commits
-
-
erik.corry@gmail.com authored
mksnapshot or a VM that is booted from a snapshot. --debug-code can still have an effect on stub and optimized code and it still works on the full code generator when running without snapshots. The deoptimizer generates full-code-generator code and relies on it having the same layout as last time. This means that the code the full code generator makes for the snapshot should be the same as the code it makes later. This change makes the full code generator create more consistent code between mksnapshot time and run time. This is a bug fix and a step towards making the snapshot code more robust. Review URL: https://chromiumcodereview.appspot.com/10834085 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12239 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=mstarzinger@chromium.org Review URL: https://chromiumcodereview.appspot.com/10824107 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12238 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
Review URL: https://chromiumcodereview.appspot.com/10830093 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12237 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/10825099 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12236 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=jkummerow@chromium.org Review URL: https://chromiumcodereview.appspot.com/10830088 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12235 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=mmassi@chromium.org Review URL: https://chromiumcodereview.appspot.com/10837019 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12234 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
The deoptimizer generates full-code-generator code and relies on it having the same layout as last time. This means that the code the full code generator makes for the snapshot should be the same as the code it makes later. This change makes the full code generator create more consistent code between mksnapshot time and run time. This is a bug fix and a step towards making the snapshot code more robust. Review URL: https://chromiumcodereview.appspot.com/10824084 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12233 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=jkummerow@chromium.org Review URL: https://chromiumcodereview.appspot.com/10831049 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12232 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 30 Jul, 2012 14 commits
-
-
yangguo@chromium.org authored
R=verwaest@chromium.org BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/10827084 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12231 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
BUG=v8:2211 TEST=test-heap/Regress2211 Review URL: https://chromiumcodereview.appspot.com/10827040 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12230 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=erik.corry@gmail.com TEST=cctest/test-api,cctest/test-heap (--gc-interval=100) Review URL: https://chromiumcodereview.appspot.com/10824082 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12229 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
x64 BinaryOpStub::GenerateSmiCode use wrong registers in one code path. e.g. d8 --trace_ic var dd = new Float64Array(2); dd[1] = 1; (function fn() { for (var i = 0; i < 1000; i++) dd[0] = 2 / dd[1]; })(); It keeps falling into runtime call and patching. No regression test, because I don't how to test it. Review URL: https://chromiumcodereview.appspot.com/10834064 Patch from Zheng Liu <shdwthr@gmail.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12226 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ulan@chromium.org authored
R=jkummerow@chromium.org Review URL: https://chromiumcodereview.appspot.com/10825076 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12225 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
This adds the --verify-global-context-separation flag which can be used to verify that no code object embeds pointers to more than one global context after a full GC. It uses an object visitor that just performs shallow traversal of the object graph spanned by one code object, and breaks at points where application objects are encountered. So it will not trip on cross-context leaks introduced by the application itself. R=verwaest@chromium.org Review URL: https://chromiumcodereview.appspot.com/10830049 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12224 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
Currently only simple getter calls are handled (i.e. no calls in count operations or compound assignments), and deoptimization in the getter is not handled at all. Because of the latter, we temporarily hide this feature behind a new flag --inline-accessors, which is false by default. Review URL: https://chromiumcodereview.appspot.com/10828066 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12223 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=verwaest@chromium.org Review URL: https://chromiumcodereview.appspot.com/10823054 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12222 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
R=svenpanne@chromium.org BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/10825074 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12221 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
R=jkummerow@chromium.org BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/10829075 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12220 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
R=svenpanne@chromium.org BUG=v8:2239 TEST= Review URL: https://chromiumcodereview.appspot.com/10821088 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12219 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/10833002 Patch from Haitao Feng <haitao.feng@intel.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12218 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
R=mstarzinger@chromium.org BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/10832061 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12217 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
R=svenpanne@chromium.org BUG=v8:2260 TEST=test-utils/Utils1 Review URL: https://chromiumcodereview.appspot.com/10820047 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12216 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 27 Jul, 2012 7 commits
-
-
verwaest@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/10802048 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12215 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=yangguo@chromium.org TEST=mjsunit/harmony/proxies (--gc-interval=100) Review URL: https://chromiumcodereview.appspot.com/10827072 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12214 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
This is a refactoring-only CL and the fourth one in a series for enabling inlining of accessors. Later when we try to inline accessor calls, their arguments must still be on the expression stack, so we must not remove them too early. Review URL: https://chromiumcodereview.appspot.com/10828049 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12213 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=jkummerow@chromium.org Review URL: https://chromiumcodereview.appspot.com/10833049 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12212 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
BUG=v8:2257 Review URL: https://chromiumcodereview.appspot.com/10824039 Patch from Haitao Feng <haitao.feng@intel.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12211 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
R=mstarzinger@chromium.org BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/10828048 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12210 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
This is a refactoring-only CL and the third one in a series for enabling inlining of accessors. The goal of this CL is to move the builders for accessors to the places where we might be able to inline them later, i.e. the VisitFoo and HandleBar member functions of HGraphBuilder. Extracted duplicate code into LookupAccessorPair. Review URL: https://chromiumcodereview.appspot.com/10831047 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@12209 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-