- 11 Feb, 2013 3 commits
-
-
vegorov@chromium.org authored
Output of --trace-deopt --code-comments does not always allow to reliably match deoptimization to the lithium instruction (and it is actually never accurate on x64 due to one level of indirection). This change allows to reliably figure out which instruction deoptimized just by looking up bailout id in the hydrogen.cfg. R=danno@chromium.org BUG= Review URL: https://codereview.chromium.org/12217103 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13639 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
This is not only inherently slow, but it also forces the caller to enter an Isolate before. Both is bad, so we have to do some heap plumbing. BUG=v8:2531 Review URL: https://codereview.chromium.org/12217106 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13638 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
rossberg@chromium.org authored
VS10 generates bogus code for specific arguments to the Number::New function when invoked in specific places of an ExpectRecords array definition, passing a garbage value to the function (as printf debugging reveals). Changing the argument from 1 to 7 apparently circumvents the problem. R=jkummerow@chromium.org BUG= Review URL: https://codereview.chromium.org/12212108 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13637 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 08 Feb, 2013 4 commits
-
-
mstarzinger@chromium.org authored
This also implements --trap-on-deopt on x64 and simplifies the implementation of this flag on all architectures. R=danno@chromium.org Review URL: https://codereview.chromium.org/12223053 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13633 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/12211095 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13632 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
svenpanne@chromium.org authored
Simplified the HeapStatistics class a bit, following Uncle Bob's advice that adding accessors to DTOs only satisfies some design fundamentalists, but serves no other purpose. :-) Review URL: https://codereview.chromium.org/12207076 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13631 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/12208088 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13630 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 07 Feb, 2013 6 commits
-
-
jkummerow@chromium.org authored
Review URL: https://codereview.chromium.org/12223039 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13624 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
Review URL: https://codereview.chromium.org/12114002 Patch from Haitao Feng <haitao.feng@intel.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13623 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
Many thanks to Vyacheslav Egorov for coming up with this test! BUG=173907 Review URL: https://codereview.chromium.org/12212066 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13622 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
jkummerow@chromium.org authored
BUG=173907 Review URL: https://codereview.chromium.org/12207063 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13617 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/12218051 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13614 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=ulan@chromium.org Review URL: https://codereview.chromium.org/12220040 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13613 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 06 Feb, 2013 11 commits
-
-
adamk@chromium.org authored
Because observed arrays should always be in dictionary mode, we'll always fall back to the JS version anyway. Review URL: https://codereview.chromium.org/12221033 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13612 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mmassi@chromium.org authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/12212032 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13609 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/12218034 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13608 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
If MINGW_HAS_SECURE_API is defined, the definitions of localtime_s, fopen_s, _vsnprintf_s, strncpy_s had to be renamed using #define so they would not conflict with definitions in the standard headers. A simpler way to handle this is to undefine MINGW_HAS_SECURE_API before including the standard headers. Contributed by net147@gmail.com BUGS= TEST= Review URL: https://chromiumcodereview.appspot.com/12210033 Patch from Jonathan Liu <net147@gmail.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13607 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
R=ulan@chromium.org BUG= Review URL: https://chromiumcodereview.appspot.com/12184015 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13606 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/12177012 Patch from Zheng Z. Liu <zheng.z.liu@intel.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13605 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
Initial step towards separating IC (map check(s)), handler frontend (prototype-check) and handler backend (actual handler code). - Still need to split the map-check (IC) from rest of the prototype chain check. - Still need to turn different parts in own code objects and cache them in more optimal places. Review URL: https://chromiumcodereview.appspot.com/12207016 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13604 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
yangguo@chromium.org authored
Windows XP does not provide secure API functions in msvcrt.dll but newer versions of Windows do. Avoid using secure API functions for compatibility with msvcrt.dll on Windows XP. Contributed by net147@gmail.com BUGS= TEST= Review URL: https://chromiumcodereview.appspot.com/12189010 Patch from Jonathan Liu <net147@gmail.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13603 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/12225043 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13602 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ulan@chromium.org authored
We were doing a redundant VCVT operation in MacroAssembler::EmitECMATruncate. Also, setting the FPSCR exception flags is expensive on some CPUs, wo we should try to avoid it if we can. Thanks to Rodolph Perfetta for the input on this! Review URL: https://chromiumcodereview.appspot.com/12217014 Patch from Hans Wennborg <hans@chromium.org>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13601 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/12228004 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13600 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 05 Feb, 2013 7 commits
-
-
danno@chromium.org authored
R=mvstanton@chromium.org Review URL: https://codereview.chromium.org/12223007 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13599 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=svenpanne@chromium.org Review URL: https://codereview.chromium.org/12208011 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13598 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
mstarzinger@chromium.org authored
R=rossberg@chromium.org TEST=cctest/test-object-observe/ObservationWeakMap Review URL: https://codereview.chromium.org/12217017 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13597 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
hpayer@chromium.org authored
BUG= Review URL: https://codereview.chromium.org/12177017 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13596 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/12211011 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13595 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=verwaest@chromium.org Review URL: https://codereview.chromium.org/12221012 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13594 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
Review URL: https://codereview.chromium.org/12093089 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13593 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
- 04 Feb, 2013 9 commits
-
-
adamk@chromium.org authored
Review URL: https://codereview.chromium.org/12041084 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13592 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
adamk@chromium.org authored
object-observe.js uses weak maps to add "hidden" properties to objects. Previously, the hash tables it was using weren't actually weak. This patch changes the existing runtime functions to create instances of JSWeakMap instead of exposing ObjectHashTable directly. Review URL: https://codereview.chromium.org/12092079 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13591 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
Review URL: https://chromiumcodereview.appspot.com/12181012 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13590 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
R=jkummerow@chromium.org Review URL: https://codereview.chromium.org/12190003 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13587 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
BUG=chromium:173974 Review URL: https://chromiumcodereview.appspot.com/12178017 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13586 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
danno@chromium.org authored
This includes: * Adding support for saving callee-clobbered double registers in Crankshaft code. * Adding a new "HTrapAllocationMemento" hydrogen instruction to handle AllocationSiteInfo data in crankshafted stubs. * Adding a new "HAllocate" hydrogen instruction that can allocate raw memory from the GC in crankshafted code. * Support for manipulation of the hole in HChange instructions for Crankshafted stubs. * Utility routines to manually build loops and if statements containing hydrogen code. Review URL: https://codereview.chromium.org/11659022 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13585 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ulan@chromium.org authored
BUG=172488,172489 R=mstarzinger@chromium.org Review URL: https://chromiumcodereview.appspot.com/12094036 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13584 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
ulan@chromium.org authored
This fixes the following two warnings, so that "make all" builds cleanly with Clang: src/arm/macro-assembler-arm.h:1410:7: error: private field 'instructions_' is not used [-Werror,-Wunused-private-field] int instructions_; // Number of instructions of the expected patch size. ^ src/arm/simulator-arm.cc:402:20: error: variable 'words' is used uninitialized whenever 'if' condition is false [-Werror,-Wsometimes-uninitialized] } else if (argc == next_arg + 1) { ^~~~~~~~~~~~~~~~~~~~ ../src/arm/simulator-arm.cc:407:21: note: uninitialized use occurs here end = cur + words; ^~~~~ ../src/arm/simulator-arm.cc:402:16: note: remove the 'if' if its condition is always true } else if (argc == next_arg + 1) { ^~~~~~~~~~~~~~~~~~~~~~~~~~ Review URL: https://chromiumcodereview.appspot.com/12087131 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13583 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-
verwaest@chromium.org authored
Port r13579 (e640269) BUG= Review URL: https://chromiumcodereview.appspot.com/12186003 Patch from Paul Lind <plind44@gmail.com>. git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@13580 ce2b1a6d-e550-0410-aec6-3dcde31c8c00
-