dcarney@chromium.org [Fri, 7 Jun 2013 07:34:25 +0000 (07:34 +0000)]
ReturnValue::Set(uint32_t) is wrong
R=svenpanne@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
16365008
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14990
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
adamk@chromium.org [Thu, 6 Jun 2013 19:15:40 +0000 (19:15 +0000)]
Remove redudant deleted_count argument from EnqueueSpliceRecord
R=rossberg@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
16438010
Patch from Rafael Weinstein <rafaelw@chromium.org>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14989
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
palfia@homejinni.com [Thu, 6 Jun 2013 18:59:11 +0000 (18:59 +0000)]
MIPS: Assign less environments.
Port r14983 (960d6de)
BUG=
Review URL: https://codereview.chromium.org/
16567002
Patch from Balazs Kilvady <kilvadyb@homejinni.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14988
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
rossberg@chromium.org [Thu, 6 Jun 2013 15:40:28 +0000 (15:40 +0000)]
Allow smis for singleton types
To that end, introduce a generic Box struct.
R=danno@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
16562003
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14987
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yangguo@chromium.org [Thu, 6 Jun 2013 15:32:39 +0000 (15:32 +0000)]
Do not GC prior to every parallel recompile.
R=jkummerow@chromium.org
BUG=
Review URL: https://chromiumcodereview.appspot.com/
16410005
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14986
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yangguo@chromium.org [Thu, 6 Jun 2013 15:30:46 +0000 (15:30 +0000)]
Fix bogus assertions for tracing and verify.
R=mstarzinger@chromium.org
BUG=
Review URL: https://chromiumcodereview.appspot.com/
15778012
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14985
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
wingo@igalia.com [Thu, 6 Jun 2013 14:38:26 +0000 (14:38 +0000)]
Add initial parser support for harmony iteration
This commit adds initial parser support for harmony iteration.
Specifically, it will parse:
for (x of y) {}
for (let x of y) {}
for (var x of y) {}
The semantics are still unimplemented.
TEST=mjsunit/harmony/for-of-syntax
BUG=v8:2214
R=rossberg@chromium.org
Review URL: https://codereview.chromium.org/
15300018
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14984
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Thu, 6 Jun 2013 14:31:44 +0000 (14:31 +0000)]
Assign less environments
R=danno@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
16556002
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14983
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Thu, 6 Jun 2013 14:21:35 +0000 (14:21 +0000)]
Initialized representations of computed values to None.
R=danno@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
14721009
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14982
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
ulan@chromium.org [Thu, 6 Jun 2013 14:13:44 +0000 (14:13 +0000)]
Prepare push to trunk. Now working on version 3.19.11.
R=jkummerow@chromium.org
BUG=
Review URL: https://chromiumcodereview.appspot.com/
16473005
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14979
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
rossberg@chromium.org [Thu, 6 Jun 2013 13:28:22 +0000 (13:28 +0000)]
Add type field to AST expression nodes
More importantly, do a bunch of renamings of incidental existing "types" to avoid actual and potential name clashes (and also to improve consistency).
R=svenpanne@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
16549002
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14978
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mvstanton@chromium.org [Thu, 6 Jun 2013 13:22:42 +0000 (13:22 +0000)]
Move runtime array constructor functions from builtins.cc to runtime.cc.
Not only is runtime.cc a better location, but situations arise soon where we'll
want to make runtime calls to these functions.
BUG=
R=mstarzinger@chromium.org
Review URL: https://codereview.chromium.org/
16399007
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14977
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
dcarney@chromium.org [Thu, 6 Jun 2013 13:16:52 +0000 (13:16 +0000)]
add function to test whether string contents are definitely one byte
R=yangguo@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
16530003
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14976
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
dcarney@chromium.org [Thu, 6 Jun 2013 12:32:32 +0000 (12:32 +0000)]
build fix for 14973
R=jkummerow@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
15692020
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14975
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
ulan@chromium.org [Thu, 6 Jun 2013 12:24:05 +0000 (12:24 +0000)]
Revert "Make more GCs in idle notification handler." because of performance
regression.
R=jkummerow@chromium.org
BUG=246141
Review URL: https://chromiumcodereview.appspot.com/
15864010
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14974
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
dcarney@chromium.org [Thu, 6 Jun 2013 12:23:37 +0000 (12:23 +0000)]
update test to test new style property handlers
R=svenpanne@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
15769014
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14973
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mstarzinger@chromium.org [Thu, 6 Jun 2013 12:05:22 +0000 (12:05 +0000)]
Fix TypedArray object printer output.
R=dslomov@chromium.org
Review URL: https://codereview.chromium.org/
16545002
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14970
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mvstanton@chromium.org [Thu, 6 Jun 2013 11:58:02 +0000 (11:58 +0000)]
Fix Android build issue 247303
ARM v7 support isn't available in the android build snapshot.
BUG=247303
R=ulan@chromium.org
Review URL: https://codereview.chromium.org/
16364007
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14969
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mstarzinger@chromium.org [Thu, 6 Jun 2013 11:37:40 +0000 (11:37 +0000)]
Fix TypedArray error for parameterless constructor.
R=dslomov@chromium.org
Review URL: https://codereview.chromium.org/
16542002
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14968
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
rodolph.perfetta@gmail.com [Thu, 6 Jun 2013 10:17:07 +0000 (10:17 +0000)]
RegExp macro assembler clean up.
Removes never called methods and fix a bug on ARM.
BUG=none
TEST=none
Review URL: https://chromiumcodereview.appspot.com/
16280005
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14967
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mvstanton@chromium.org [Thu, 6 Jun 2013 09:26:30 +0000 (09:26 +0000)]
Allocation type info advice consumed in bailout path leads to assert failure.
If the runtime is taken for a constructor like "new Array(100000)", where
allocation site info already led to an elements kind of DOUBLE, then the runtime
would fail to transition the array to dictionary mode. Better to recognize this
case and avoid wasting time by following the advice. Furthermore, it offers a
way to recognize that the array should be in dictionary mode (though a future
checkin will capitalize on that).
BUG=
R=danno@chromium.org
Review URL: https://codereview.chromium.org/
15993012
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14966
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yurys@chromium.org [Thu, 6 Jun 2013 07:00:57 +0000 (07:00 +0000)]
Print accessors loop iterations count in test-cpu-profiler/NativeAccessorNameInProfile1
The test is failing on Win64 bot but passes locally I need this debug print to better understand what's different on the bot.
BUG=None
TBR=loislo@chromium.org
Review URL: https://codereview.chromium.org/
16527006
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14965
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
danno@chromium.org [Wed, 5 Jun 2013 21:02:29 +0000 (21:02 +0000)]
Fix EntryHookStub on ia32 and x64.
These stubs were computing the return address location incorrectly.
Add testing for same.
R=danno@chromium.org
Review URL: https://codereview.chromium.org/
15769017
Patch from Sigurður Ásgeirsson <siggi@chromium.org>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14962
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
palfia@homejinni.com [Wed, 5 Jun 2013 17:52:44 +0000 (17:52 +0000)]
MIPS: Add LoadIC_Slow builtin and use it for loading properties from primitive values.
Port r14950 (12748bb)
Original commit message:
This fixes recent performance regressions on Dromaeo.
BUG=chromium:242512
Review URL: https://codereview.chromium.org/
16452011
Patch from Balazs Kilvady <kilvadyb@homejinni.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14961
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Wed, 5 Jun 2013 17:24:14 +0000 (17:24 +0000)]
Don't force representations in the loop builder.
Additionally, infer None-typed CompareIDAndBranch to Smi+actual input.
R=jkummerow@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
16365004
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14960
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
palfia@homejinni.com [Wed, 5 Jun 2013 16:45:19 +0000 (16:45 +0000)]
MIPS: Fix LStoreKeyed error when tracing instruction.
Port r14951 (01dbfe2)
Original commit message:
LStoreKeyed can have a NULL value(), when we recognize we are storing a constant
hole value.
BUG=
Review URL: https://codereview.chromium.org/
16171019
Patch from Balazs Kilvady <kilvadyb@homejinni.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14959
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
palfia@homejinni.com [Wed, 5 Jun 2013 16:06:38 +0000 (16:06 +0000)]
MIPS: Adapt hydrogen-based Array constructor to also support InternalArray and function call versions.
Port r14948 (1c2f28c)
BUG=
Review URL: https://codereview.chromium.org/
16407004
Patch from Balazs Kilvady <kilvadyb@homejinni.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14958
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
rossberg@chromium.org [Wed, 5 Jun 2013 15:43:53 +0000 (15:43 +0000)]
New unified type representation
Not used yet, only unit tests.
R=jkummerow@chromium.org, svenpanne@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
16154027
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14957
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yangguo@chromium.org [Wed, 5 Jun 2013 15:35:14 +0000 (15:35 +0000)]
Cosmetic change to the handle dereference check.
R=jkummerow@chromium.org
BUG=
Review URL: https://chromiumcodereview.appspot.com/
16171017
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14956
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Wed, 5 Jun 2013 13:38:28 +0000 (13:38 +0000)]
Prepare push to trunk. Now working on version 3.19.10.
R=jkummerow@chromium.org
BUG=
Review URL: https://chromiumcodereview.appspot.com/
16379010
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14953
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
dcarney@chromium.org [Wed, 5 Jun 2013 12:36:33 +0000 (12:36 +0000)]
Cutover v8 to use new style callbacks internally
R=svenpanne@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
15793007
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14952
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mvstanton@chromium.org [Wed, 5 Jun 2013 12:09:02 +0000 (12:09 +0000)]
Fix LStoreKeyed error when tracing instruction.
LStoreKeyed can have a NULL value(), when we recognize we are storing a constant
hole value.
BUG=
R=verwaest@chromium.org
Review URL: https://codereview.chromium.org/
15793006
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14951
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
jkummerow@chromium.org [Wed, 5 Jun 2013 11:12:49 +0000 (11:12 +0000)]
Add LoadIC_Slow builtin and use it for loading properties from primitive values.
This fixes recent performance regressions on Dromaeo.
BUG=chromium:242512
R=verwaest@chromium.org
Review URL: https://codereview.chromium.org/
16226024
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14950
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
jkummerow@chromium.org [Wed, 5 Jun 2013 11:05:45 +0000 (11:05 +0000)]
Use HSoftDeoptimize instead of HDeoptimize in VisitSwitchStatement, and visit the entire graph
R=svenpanne@chromium.org
Review URL: https://codereview.chromium.org/
15929010
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14949
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mvstanton@chromium.org [Wed, 5 Jun 2013 10:43:18 +0000 (10:43 +0000)]
Adapt hydrogen-based Array constructor to also support InternalArray and function call
versions.
BUG=
R=mstarzinger@chromium.org
Review URL: https://codereview.chromium.org/
14576005
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14948
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yangguo@chromium.org [Wed, 5 Jun 2013 09:41:24 +0000 (09:41 +0000)]
Free PerThreadAssertData when possible to avoid memory leak.
R=svenpanne@chromium.org
BUG=246567
Review URL: https://chromiumcodereview.appspot.com/
16093024
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14947
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Wed, 5 Jun 2013 08:43:25 +0000 (08:43 +0000)]
Remove the optimized construct stub.
R=mstarzinger@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
15993016
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14946
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yurys@chromium.org [Wed, 5 Jun 2013 06:15:41 +0000 (06:15 +0000)]
Print accessors execution time in test-cpu-profiler/NativeAccessorNameInProfile1
The test is failing on Win64 bot but passes locally I need this debug print to better understand what's different on the bot.
BUG=None
TBR=loislo@chromium.org
Review URL: https://codereview.chromium.org/
16359016
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14945
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
adamk@chromium.org [Tue, 4 Jun 2013 23:58:49 +0000 (23:58 +0000)]
Array.observe emit splices for array length change and update index >= length
R=adamk@chromium.org, rossberg@chromium.org
Review URL: https://codereview.chromium.org/
15504002
Patch from Rafael Weinstein <rafaelw@chromium.org>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14944
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
palfia@homejinni.com [Tue, 4 Jun 2013 20:17:22 +0000 (20:17 +0000)]
MIPS: Turn off allocation site info for crankshafted array constructor calls.
Port r14934 (dbddd7e)
Original commit message:
Once we crankshaft a method, we should turn off allocation site info for
constructed arrays. Additionally, the semantics for doing this were
awkward because the constructed array code stubs get an
AllocationSiteMode as a minor key, but it's used as a permission to
determine the final mode locally based on ElementsKind. I refactored
this to a simpler boolean for override or local control.
BUG=
Review URL: https://codereview.chromium.org/
16226020
Patch from Balazs Kilvady <kilvadyb@homejinni.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14943
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
palfia@homejinni.com [Tue, 4 Jun 2013 20:14:37 +0000 (20:14 +0000)]
MIPS: Deprecate FACTORY helper macro.
Port r14931 (a914442)
Original commit message:
This removes the FACTORY helper macro to avoid accidental TLS access
when using the factory. Most internal code has access to the Isolate by
now whereas tests which are not performance critical still heavily use
TLS access through explicit Isolate::Current() calls.
BUG=
Review URL: https://codereview.chromium.org/
15994011
Patch from Balazs Kilvady <kilvadyb@homejinni.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14942
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
palfia@homejinni.com [Tue, 4 Jun 2013 20:11:16 +0000 (20:11 +0000)]
MIPS : Collect type feedback for power-of-2 right operands in BinaryOps.
Port r14928 (a2e0e18)
Original commit message:
Improved --trace-ic output for unary/binary ops a bit on the way. Moved int32_t/uint32_t conversion helpers around.
BUG=
Review URL: https://codereview.chromium.org/
16361004
Patch from Balazs Kilvady <kilvadyb@homejinni.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14941
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mstarzinger@chromium.org [Tue, 4 Jun 2013 17:20:06 +0000 (17:20 +0000)]
Prevent excessive processing of weak maps while marking.
This makes sure processing of Harmony weak maps is only triggerd twice
during a mark-compact collection. Note that the processing itself still
is a fix-point iteration with quadratic worst-case time.
R=hpayer@chromium.org
BUG=v8:2699
Review URL: https://codereview.chromium.org/
16286018
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14940
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Tue, 4 Jun 2013 17:15:32 +0000 (17:15 +0000)]
Fix win build
BUG=
R=hpayer@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
15934019
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14939
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
jkummerow@chromium.org [Tue, 4 Jun 2013 16:41:24 +0000 (16:41 +0000)]
Liveness analysis for environment slots in Hydrogen
R=titzer@chromium.org
Review URL: https://codereview.chromium.org/
15533004
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14938
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Tue, 4 Jun 2013 15:39:56 +0000 (15:39 +0000)]
Replace log2 with MostSignificantBit
R=ulan@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
15994015
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14937
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
dcarney@chromium.org [Tue, 4 Jun 2013 13:36:18 +0000 (13:36 +0000)]
Remove V8_USE_OLD_STYLE_PERSISTENT_HANDLE_VISITORS.
Blink has migrated to use the new style visitors.
BUG=
R=dcarney@chromium.org, svenpanne@chromium.org
Review URL: https://codereview.chromium.org/
16360005
Patch from Marja Hölttä <marja@chromium.org>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14936
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
hpayer@chromium.org [Tue, 4 Jun 2013 13:20:13 +0000 (13:20 +0000)]
Added pretenuring support for call new.
BUG=
R=mvstanton@chromium.org
Review URL: https://codereview.chromium.org/
16226012
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14935
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mvstanton@chromium.org [Tue, 4 Jun 2013 12:48:51 +0000 (12:48 +0000)]
Turn off allocation site info for crankshafted array constructor calls.
Once we crankshaft a method, we should turn off allocation site info for
constructed arrays. Additionally, the semantics for doing this were
awkward because the constructed array code stubs get an
AllocationSiteMode as a minor key, but it's used as a permission to
determine the final mode locally based on ElementsKind. I refactored
this to a simpler boolean for override or local control.
BUG=
R=hpayer@chromium.org
Review URL: https://codereview.chromium.org/
16206007
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14934
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Tue, 4 Jun 2013 12:26:39 +0000 (12:26 +0000)]
Infer the range of XOR as limited by the highest possible changed bit.
R=ulan@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
16361003
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14933
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yurys@chromium.org [Tue, 4 Jun 2013 10:57:32 +0000 (10:57 +0000)]
CPU profiler should support names of accessors set via v8::Object::SetAccessor
This change fixes the case when the accessors are invoked from JSObject::{Get,Set}PropertyWithCallback.
It already works for inlined calls generated by StoreStubCompiler::CompileStoreCallback. The same still needs to be fixed for getter invocations generated by BaseLoadStubCompiler::CompileLoadCallback, corresponding case is commented out in the new test.
This is a slightly modified version of r14915 which was rolled back due to test timeout on Windows. Compared to r14915 the new tests use OS::TimeCurrentMillis instead of OS::Ticks as OS::Ticks has ms precision on Windows and trying to wait 10 ticks (us) will result in at least 1 ms pause.
BUG=244580
R=jkummerow@chromium.org, loislo@chromium.org
Review URL: https://codereview.chromium.org/
15995017
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14932
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mstarzinger@chromium.org [Tue, 4 Jun 2013 10:30:05 +0000 (10:30 +0000)]
Deprecate FACTORY helper macro.
This removes the FACTORY helper macro to avoid accidental TLS access
when using the factory. Most internal code has access to the Isolate by
now whereas tests which are not performance critical still heavily use
TLS access through explicit Isolate::Current() calls.
R=svenpanne@chromium.org
Review URL: https://codereview.chromium.org/
16337005
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14931
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mstarzinger@chromium.org [Tue, 4 Jun 2013 08:28:33 +0000 (08:28 +0000)]
Deprecate HAllocateObject in favor of HAllocate.
This models the instantiation of an implicit receiver for CallNew nodes
in hydrogen using HAllocate together with generic stores instead of one
specialized HAllocateObject instruction, hence creating a single choking
point for inlined allocation in optimized code.
R=hpayer@chromium.org
Review URL: https://codereview.chromium.org/
15714005
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14930
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yangguo@chromium.org [Tue, 4 Jun 2013 08:06:53 +0000 (08:06 +0000)]
Revert accidental change to SealHandleScope.
R=dslomov@chromium.org
BUG=
Review URL: https://chromiumcodereview.appspot.com/
16226007
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14929
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
svenpanne@chromium.org [Tue, 4 Jun 2013 07:49:45 +0000 (07:49 +0000)]
Collect type feedback for power-of-2 right operands in BinaryOps.
Improved --trace-ic output for unary/binary ops a bit on the way. Moved int32_t/uint32_t conversion helpers around.
R=jkummerow@chromium.org
Review URL: https://codereview.chromium.org/
15735005
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14928
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yurys@chromium.org [Tue, 4 Jun 2013 06:17:53 +0000 (06:17 +0000)]
Mark test-cpu-profiler/SampleWhenFrameIsNotSetup as flaky on MIPS simulator
Relanding r14916 that was rolled back in r14918
BUG=v8:2628
TBR=danno
Review URL: https://codereview.chromium.org/
16355004
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14927
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Mon, 3 Jun 2013 16:57:36 +0000 (16:57 +0000)]
Improve smi support in crankshaft
- Recover smi in phis if inputs are smi-typed
- Don't record smi-typed values as pointers
BUG=
R=jkummerow@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
16240003
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14926
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yangguo@chromium.org [Mon, 3 Jun 2013 16:01:51 +0000 (16:01 +0000)]
Fix windows build.
R=rossberg@chromium.org
BUG=
Review URL: https://chromiumcodereview.appspot.com/
15927012
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14923
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
rossberg@chromium.org [Mon, 3 Jun 2013 15:43:11 +0000 (15:43 +0000)]
Prepare push to trunk. Now working on version 3.19.9.
R=danno@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
16140010
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14920
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yangguo@chromium.org [Mon, 3 Jun 2013 15:32:22 +0000 (15:32 +0000)]
Make assertion scopes thread safe.
R=svenpanne@chromium.org
BUG=
Review URL: https://chromiumcodereview.appspot.com/
15691017
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14919
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
danno@chromium.org [Mon, 3 Jun 2013 15:25:28 +0000 (15:25 +0000)]
Revert 14916: Mark test-cpu-profiler/SampleWhenFrameIsNotSetup as flaky on MIPS simulator
Revert 14915: CPU profiler should support names of accessors set via v8::Object::SetAccessor
Due to Windows failures
R=rossberg@chromium.org
Review URL: https://codereview.chromium.org/
16335005
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14918
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mvstanton@chromium.org [Mon, 3 Jun 2013 14:46:23 +0000 (14:46 +0000)]
Special Array constructor type feedback erroneously recorded when Array
was called as a function. Issue was found with optimize_constructed_array
turned on. This patch makes the fix, and turns the flag back on.
BUG=244461
R=danno@chromium.org
Review URL: https://codereview.chromium.org/
16057005
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14917
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yurys@chromium.org [Mon, 3 Jun 2013 12:58:25 +0000 (12:58 +0000)]
Mark test-cpu-profiler/SampleWhenFrameIsNotSetup as flaky on MIPS simulator
The test started failing again, disabling it for now.
BUG=v8:2628
TBR=jkummerow
Review URL: https://codereview.chromium.org/
16329004
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14916
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yurys@chromium.org [Mon, 3 Jun 2013 12:50:03 +0000 (12:50 +0000)]
CPU profiler should support names of accessors set via v8::Object::SetAccessor
This change fixes the case when the accessors are invoked from JSObject::{Get,Set}PropertyWithCallback.
It already works for inlined calls generated by StoreStubCompiler::CompileStoreCallback. The same still needs to be fixed for getter invocations generated by BaseLoadStubCompiler::CompileLoadCallback, corresponding case is commented out in the new test.
BUG=244580
R=jkummerow@chromium.org, loislo@chromium.org
Review URL: https://codereview.chromium.org/
16004007
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14915
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
dcarney@chromium.org [Mon, 3 Jun 2013 10:02:02 +0000 (10:02 +0000)]
remove V8_ALLOW_ACCESS_TO_PERSISTENT_IMPLICIT and V8_ALLOW_ACCESS_TO_PERSISTENT_ARROW
R=svenpanne@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
15979014
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14914
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yurys@chromium.org [Mon, 3 Jun 2013 09:46:32 +0000 (09:46 +0000)]
Remove unnecessary check from Logger::LogCodeObject
Logger::LogCodeObjects is the only caller of the method. And all its
call sites check for at least one of the coditions before calling
LogCodeObjects. So there is no need to check it again.
BUG=None
R=jkummerow@chromium.org, loislo@chromium.org
Review URL: https://codereview.chromium.org/
15980007
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14913
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Mon, 3 Jun 2013 09:08:00 +0000 (09:08 +0000)]
Remove unnecessary code from the json-parser.
R=yangguo@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
16306005
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14912
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
dcarney@chromium.org [Mon, 3 Jun 2013 08:17:04 +0000 (08:17 +0000)]
remove old MakeWeak
R=svenpanne@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
16160010
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14911
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yurys@chromium.org [Mon, 3 Jun 2013 07:52:08 +0000 (07:52 +0000)]
Simulator specific code that is the same on all platforms as well as TickSample instance retrieval extracted into SampleHelper class.
Drive-by: extracted platform-independent pieces of Sampler::PlatformData into PlatformDataCommon.
BUG=None
R=loislo@chromium.org, svenpanne@chromium.org
Review URL: https://codereview.chromium.org/
15957005
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14910
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Fri, 31 May 2013 19:11:09 +0000 (19:11 +0000)]
Move field index into property details, freeing up the value slot of fields.
BUG=
R=jkummerow@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
15941016
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14909
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
jkummerow@chromium.org [Fri, 31 May 2013 15:50:19 +0000 (15:50 +0000)]
Fast literals: fixed initialization of non-copied in-object property fields
BUG=chromium:245424
R=mstarzinger@chromium.org
Review URL: https://codereview.chromium.org/
16190008
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14906
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yurys@chromium.org [Fri, 31 May 2013 12:52:28 +0000 (12:52 +0000)]
Fix function name inferring inside closures
BUG=224884
R=loislo@chromium.org, yangguo@chromium.org
Review URL: https://codereview.chromium.org/
16125007
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14903
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mstarzinger@chromium.org [Fri, 31 May 2013 11:58:39 +0000 (11:58 +0000)]
Fix non-idempotent modification in JSObject::AddFastProperty.
R=verwaest@chromium.org
BUG=mozilla/ecma/Array/15.4.5.2-2
Review URL: https://codereview.chromium.org/
15955010
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14902
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
rossberg@chromium.org [Fri, 31 May 2013 11:38:36 +0000 (11:38 +0000)]
Prepare push to trunk. Now working on version 3.19.8.
R=jkummerow@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
15959021
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14899
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
dcarney@chromium.org [Fri, 31 May 2013 11:06:50 +0000 (11:06 +0000)]
ReturnValue::Set needs to check for empty handles
R=jkummerow@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
16073010
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14898
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Fri, 31 May 2013 10:48:37 +0000 (10:48 +0000)]
Remove duplicated initialization of stack_parameter_count_ of hydrogen stub descriptor.
R=verwaest@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
16097004
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14897
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Fri, 31 May 2013 08:37:34 +0000 (08:37 +0000)]
Fix DeferredTaggedToINoSSE2 to not unconditionally untag undefined to 0.
R=danno@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
16228002
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14896
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
palfia@homejinni.com [Thu, 30 May 2013 22:30:00 +0000 (22:30 +0000)]
MIPS: Replace DeoptimizeOnUndefined with whitelisting AllowUndefinedAsNan
Port r14894 (
0ca991f0)
BUG=
Review URL: https://codereview.chromium.org/
15713017
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14895
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Thu, 30 May 2013 09:11:06 +0000 (09:11 +0000)]
Replace DeoptimizeOnUndefined with whitelisting AllowUndefinedAsNan
R=danno@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
15952007
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14894
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
palfia@homejinni.com [Thu, 30 May 2013 01:24:45 +0000 (01:24 +0000)]
MIPS: Convert ToBooleanStub to a HydrogenStub. Currently just using the existing HBranch instruction, which is still fully implemented in Lithium. Will refactor HBranch in a next CL.
Port r14886 (
068e9135)
BUG=
Review URL: https://codereview.chromium.org/
15664008
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14893
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
palfia@homejinni.com [Thu, 30 May 2013 01:23:54 +0000 (01:23 +0000)]
MIPS: Fix AccessArgumentsAt on ARM. Use register for index.
Port r14885 (63dd99b)
BUG=
Review URL: https://codereview.chromium.org/
16215002
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14892
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
palfia@homejinni.com [Thu, 30 May 2013 01:23:04 +0000 (01:23 +0000)]
MIPS: Update ASSERT on arm to support SmiOrTagged in ClampToUint8
Port r14881 (
0b784159)
BUG=
Review URL: https://codereview.chromium.org/
15769005
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14891
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
palfia@homejinni.com [Thu, 30 May 2013 01:22:04 +0000 (01:22 +0000)]
MIPS: Don't explicitly pass requested representations to constants; implement ConstantS
Port r14874 (
e93cc94e)
BUG=
Review URL: https://codereview.chromium.org/
15731008
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14890
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
plind44@gmail.com [Wed, 29 May 2013 19:45:19 +0000 (19:45 +0000)]
Prevent segfault if file cannot be opened when --ll_prof is used.
TEST=
BUG=
R=jkummerow@chromium.org
Review URL: https://codereview.chromium.org/
16107002
Patch from Dusan Milosavljevic <Dusan.Milosavljevic@rt-rk.com>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14889
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
adamk@chromium.org [Wed, 29 May 2013 17:26:05 +0000 (17:26 +0000)]
Re-land: Implement ObservedArrayPop, ObservedArrayShift, ObservedArrayUnshift & ObservedArraySplice
Original issue: https://codereview.chromium.org/
15331002/
Broke mozilla/js1_5/Array/regress-451483.js, which ends up attempting to call %IsObserved() on a non-object.
IsObserved now checks for JSReceiver and returns false rather than crashing.
R=adamk@chromium.org, rossberg@chromium.org
Review URL: https://codereview.chromium.org/
15777007
Patch from Rafael Weinstein <rafaelw@chromium.org>.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14888
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mstarzinger@chromium.org [Wed, 29 May 2013 15:38:09 +0000 (15:38 +0000)]
Handlify map allocation in Genesis::InstallInternalArray.
R=dcarney@chromium.org
TEST=mozilla/js1_5/Regress/regress-367561-01
Review URL: https://codereview.chromium.org/
15690018
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14887
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
olivf@chromium.org [Wed, 29 May 2013 14:49:28 +0000 (14:49 +0000)]
Convert ToBooleanStub to a HydrogenStub. Currently just using the existing HBranch instruction, which is still fully implemented in Lithium. Will refactor HBranch in a next CL.
[
CL breaks mips, since the deoptimizer in Deoptimizer::DoTranslateCommand(...) does not convert smis passed to the stub via a0.
]
BUG=
R=verwaest@chromium.org
Review URL: https://codereview.chromium.org/
15302004
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14886
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Wed, 29 May 2013 12:48:06 +0000 (12:48 +0000)]
Fix AccessArgumentsAt on ARM. Use register for index.
R=jkummerow@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
15690009
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14885
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
svenpanne@chromium.org [Wed, 29 May 2013 12:42:04 +0000 (12:42 +0000)]
Improve range analysis for modulo operations.
R=mstarzinger@chromium.org
Review URL: https://codereview.chromium.org/
15952004
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14884
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yurys@chromium.org [Wed, 29 May 2013 12:40:21 +0000 (12:40 +0000)]
Add support for //# sourceURL similar to deprecated //@ sourceURL one.
BUG=v8:2702
R=yangguo@chromium.org, yurys@chromium.org
Review URL: https://codereview.chromium.org/
15859010
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14883
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mstarzinger@chromium.org [Wed, 29 May 2013 12:36:41 +0000 (12:36 +0000)]
Make sure IfBuilder::Return clears the current block.
This is a follow-up to an earlier bugfix. It doesn't yet trigger any
known bugs but is a safety precaution since IfBuilder usage increases.
R=jkummerow@chromium.org
BUG=chromium:243868
Review URL: https://codereview.chromium.org/
15832005
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14882
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Wed, 29 May 2013 12:20:32 +0000 (12:20 +0000)]
Update ASSERT on arm to support SmiOrTagged in ClampToUint8
R=jkummerow@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
15832004
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14881
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
yurys@chromium.org [Wed, 29 May 2013 12:09:03 +0000 (12:09 +0000)]
Fix two CPU profiler tests on ARM and MIPS simulators
Use Simulator::get_pc instead of Simulator::get_register(pc) as the latter
will return value adjusted by Instruction::kPCReadOffset.
BUG=v8:2628
R=loislo@chromium.org, svenpanne@chromium.org
Review URL: https://codereview.chromium.org/
16154004
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14880
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
ulan@chromium.org [Wed, 29 May 2013 11:44:54 +0000 (11:44 +0000)]
Make more GCs in idle notification handler.
When counting GCs in idle round, count only GC that were directly triggered by the idle notification handler.
R=hpayer@chromium.org
BUG=241815
Review URL: https://chromiumcodereview.appspot.com/
15737007
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14879
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mstarzinger@chromium.org [Wed, 29 May 2013 11:32:45 +0000 (11:32 +0000)]
Ignore test262 data files downloaded by test harness.
R=jkummerow@chromium.org
Review by mail to v8-dev@googlegroups.com list.
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14878
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
ulan@chromium.org [Wed, 29 May 2013 11:13:59 +0000 (11:13 +0000)]
Fix counting of scanned bytes in incremental marking step for large object.
R=mstarzinger@chromium.org
BUG=241815
Review URL: https://chromiumcodereview.appspot.com/
15745004
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14877
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
mstarzinger@chromium.org [Wed, 29 May 2013 11:09:01 +0000 (11:09 +0000)]
Fix handle zapping interaction with NoHandleAllocation.
This makes sure that closed handle scopes are properly zapped even if an
enclosing NoHandleAllocation shrunk the limit. It also unifies the code
that performs scope closing for internal and external handle scopes.
R=svenpanne@chromium.org
TEST=cctest/test-api/NestedLockersNoTryCatch
Review URL: https://codereview.chromium.org/
16004006
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14876
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
dcarney@chromium.org [Wed, 29 May 2013 11:04:10 +0000 (11:04 +0000)]
remove remaining V8_ALLOW_ACCESS_TO* defines from test classes
R=svenpanne@chromium.org
BUG=
Review URL: https://codereview.chromium.org/
16032010
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14875
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Wed, 29 May 2013 10:47:55 +0000 (10:47 +0000)]
Don't explicitly pass requested representations to constants; implement ConstantS
R=jkummerow@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
15932011
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14874
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
verwaest@chromium.org [Wed, 29 May 2013 09:55:50 +0000 (09:55 +0000)]
Don't explicitly pass the Smi-representation to IfCompare.
BUG=
R=jkummerow@chromium.org
Review URL: https://chromiumcodereview.appspot.com/
15891006
git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@14873
ce2b1a6d-e550-0410-aec6-
3dcde31c8c00
projects / platform / upstream / v8.git / log