- 21 Sep, 2011 8 commits
-
-
vegorov@chromium.org authored
Otherwise we get the wrong page when start == limit. R=lrn@chromium.org Review URL: http://codereview.chromium.org/7979019 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9362 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
BUG=v8:1573 TEST=cctest test-debug/TestDebuggerAgent Review URL: http://codereview.chromium.org/7978025 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9361 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
When aborting incremental marking with compaction discard all slots collected on evacuation candidates. R=ricow@chromium.org BUG=v8:1700 TEST=inspector/debugger/debugger-step-in.html Review URL: http://codereview.chromium.org/7977004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9360 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
rossberg@chromium.org authored
R=jkummerow@chromium.org BUG=v8:1698 TEST= Review URL: http://codereview.chromium.org/7977021 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9358 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
kmillikin@chromium.org authored
Last minute cleanup lost a '*'. TBR=ricow@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/7977020 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9357 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
kmillikin@chromium.org authored
The threading test failures seem to be due to a GC-unsafe place, that mixed handles and raw pointers, in the runtime code for context lookup. R=fschneider@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/7890002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9356 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
This is not perfect, but it should fix the problem at hand. We should really clean up the memory handling responsibilities for the (macro)assemblers. BUG=v8:1706 Review URL: http://codereview.chromium.org/7978023 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9351 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
Review URL: http://codereview.chromium.org/7978022 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9350 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 20 Sep, 2011 16 commits
-
-
mstarzinger@chromium.org authored
This prevents potential misuse of SeqString::kHeaderSize as in the case of live byte counting in incremental marking stub. All stubs picked up the undefined size constant SeqString::kHeaderSize, thus the computed size of all strings was off by two pointers slots. R=lrn@chromium.org BUG=v8:1672 TEST=mjsunit/object-seal.js,... Review URL: http://codereview.chromium.org/7971009 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9349 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=vegorov@chromium.org BUG=v8:1702 TEST=cctest/test-heap/GrowAndShrinkNewSpace Review URL: http://codereview.chromium.org/7983001 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9348 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
The formerly unused variable was found by GCC 4.6.1, BTW... Review URL: http://codereview.chromium.org/7979005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9347 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=lrn@chromium.org BUG=v8:1702 TEST=cctest/test-heap/GrowAndShrinkNewSpace Review URL: http://codereview.chromium.org/7976003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9346 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
R=erik.corry@gmail.com Review URL: http://codereview.chromium.org/7980004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9345 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
Fix some incorrect comments. Review URL: http://codereview.chromium.org/7977005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9344 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
We should not use slot address to find out the page. This does not work for large pages. Also this assertion is not always true (violated during incremental marking). R=erik.corry@gmail.com TEST=mozilla/js1_5/extensions/regress-371636 Review URL: http://codereview.chromium.org/7981001 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9339 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
lrn@chromium.org authored
The "live bytes" count is *really* a "marked black" count - i.e., the count of bytes *known* to be live. Fix aggravating bug on X64 where assembler code used a value that was off by a factor of 2^31. Ensure that sweeping clears live-bytes. Added other missing increments. Added print statements to trace live-byte modifications, under a flag. Still a few cases of undercounting left. (New issue to merge from GC branch to bleeding_edge) Review URL: http://codereview.chromium.org/7970009 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9338 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
We have to check for uninitialized uses before phi-elimination. Otherwise we may miss such a use and result in using the hole value instead. This causes a NULL-dereference or assertion failure. BUG=96989 TEST=mjsunit/compiler/regress-96989.js Review URL: http://codereview.chromium.org/7974009 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9337 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
object. Rename it to be the inner pointer to code cache. Review URL: http://codereview.chromium.org/7969013 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9336 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
BUG=94873 Review URL: http://codereview.chromium.org/7929001 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9335 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
Review URL: http://codereview.chromium.org/7969014 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9334 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
Review URL: http://codereview.chromium.org/7970008 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9333 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
the new large pages it takes too long to iterate from the start of the page to find a code object. Review URL: http://codereview.chromium.org/7974007 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9332 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
kmillikin@chromium.org authored
We reset the count of outgoing arguments, used for the index in the frame's outgoing arguments, for each level of inlining. This could result in the arguments overlapping, rather than stacking on top of each other. R=whesse@chromium.org,fschneider@chromium.org BUG= TEST= Review URL: http://codereview.chromium.org/7932022 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9331 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
BUG=v8:1698 Review URL: http://codereview.chromium.org/7974006 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9330 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 19 Sep, 2011 6 commits
-
-
erik.corry@gmail.com authored
BUG=1696 Review URL: http://codereview.chromium.org/7956002 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9329 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
vegorov@chromium.org authored
Review URL: http://codereview.chromium.org/7945009 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9328 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
fschneider@chromium.org authored
We don't modify the graph in later phases. This speeds up debug tests by a lot. Review URL: http://codereview.chromium.org/7942001 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9325 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
Although this patch is not small, most parts of it are rather mechanical: * First of all, the concept of a 'nil-like' value is introduced, which can be null or undefined. They are treated symmetrically regarding comparisons, so it makes sense to handle them in a uniform manner. It is a mystery why JavaScript defines two of those beasts, when even *one* is a design wart... * Extended and renamed a few things which now handle undefined in addition to null. * Made the parts of the full code generator and the hydrogen generation which deal with comparisons a bit more similar regarding their handling of special cases. * Refactored the syntactical detection of special cases for comparisons, hopefully making them a bit more readable and less copy-n-paste-oriented. Things like this should really be a one-liner in any sane programming language... :-P * Cut down the length of the argument lists of a few functions to something less insane, making them more easily understandable locally. This involves minor code duplication, but this was a good tradeoff and can be remedied later if necessary. * Replaced some boolean arguments with more readable enums. * Fixed a TODO: Values which are definitely a Smi or unboxed can never be equal to null or undefined. Review URL: http://codereview.chromium.org/7918012 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9323 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
ported r9310 (72a5468) BUG= TEST= Review URL: http://codereview.chromium.org/7931002 Patch from Paul Lind <plind44@gmail.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9322 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
generation on ARM without VFP3. Review URL: http://codereview.chromium.org/7937004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9319 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 18 Sep, 2011 4 commits
-
-
whesse@chromium.org authored
BUG= TEST= Review URL: http://codereview.chromium.org/7931023 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9318 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erikcorry authored
An ARM PreIndex store was badly ported. Landing http://codereview.chromium.org/7934003/ for Paul Lind. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9317 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erikcorry authored
port r9258 (c8709a9) Note on mips implementation: Arm reg r4 (call type) normally maps to mips reg t0. We had already used t0 as a temp in Generate_FunctionCall() and Generate_FunctionApply(), so I replaced that existing t0 usage with t3, and now use t0 only for call type. Original commit message: Introduce new %Apply native. Extend Execution::Call to optionally handle receiver rewriting (needed for %Apply). Fix Function.prototype.bind for functions that have .apply modified. Landing http://codereview.chromium.org/7891033/ for Paul Lind. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9316 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erikcorry authored
Fix a mips-specific CallCFunction assertion, when checking for valid frames on the stack. We fake it with AllowExternalCallThatCantCauseGC(). Landing http://codereview.chromium.org/7932010/ for Paul Lind. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9315 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 17 Sep, 2011 1 commit
-
-
fschneider@chromium.org authored
This flag is used to debug problems with inlining. By default we still limit inlining to small functions and up to a maximum depth to avoid too much code size growth. Review URL: http://codereview.chromium.org/7919012 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9314 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 16 Sep, 2011 5 commits
-
-
rossberg@chromium.org authored
Fix a couple of other proxy bugs along the way. Refactor trap invocation in native code. R=kmillikin@chromium.org BUG=v8:1543 TEST= Review URL: http://codereview.chromium.org/7799026 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9312 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
Review URL: http://codereview.chromium.org/7920006 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9311 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
rossberg@chromium.org authored
R=kmillikin@chromium.org BUG=v8:1543 TEST= Review URL: http://codereview.chromium.org/7628021 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9310 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
erik.corry@gmail.com authored
Review URL: http://codereview.chromium.org/7922008 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9309 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
Highlights: - code-stubs-mips.cc -- use EmitFPUTruncate in place of inline code in several places. -- use BranchF macro rather than lower-level FP cmp and branch for readability. -- Port of Sven's r8859 (Implement type recording for ToBoolean) and r8886 (Simplify and optimize ToBoolean handling.) -- Fix bug in TranscendentalCacheStub::Generate where some regs were not saved across CFunction call. -- use updated xxxCFunction macros. -- update InstanceOfStub to support crankshaft DoDeferredLInstanceOfKnownGlobal -- Provide code-patching and I-cache flushing support for generated code, used for InstanceOfStub under crankshaft (not submitted here). This requires adding new ExternalReference to src/assember.cc,h - stub-cache-mips.cc -- port Danno's r8901 (Create a common base class for Fixed-, FixedDouble- and ExternalArrays) to mips crankshaft branch. BUG= TEST= Review URL: http://codereview.chromium.org/7890001 Patch from Paul Lind <plind44@gmail.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@9308 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-