Commits · 4270a0f2a84066480fb7d0878b58b1880d8f5bd4 · Linshizhi / V8

21 Nov, 2013 7 commits

Prepare push to trunk. Now working on version 3.23.10. · 4270a0f2

machenbach@chromium.org authored Nov 21, 2013

R=jkummerow@chromium.org, danno@chromium.org

Review URL: https://codereview.chromium.org/77243006

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17952 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

4270a0f2

Let ChangeLog get auto-generated in push-to-trunk script. · 121ef80c

machenbach@chromium.org authored Nov 21, 2013

BUG=
R=jkummerow@chromium.org

Review URL: https://codereview.chromium.org/78683002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17951 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

121ef80c

Do deferred migration of maps after deoptimizing once. · df35c0e4

hpayer@chromium.org authored Nov 21, 2013

BUG=
R=verwaest@chromium.org

Review URL: https://codereview.chromium.org/50213003

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17949 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

df35c0e4

Fix Windows build for real · 96c9095f

jkummerow@chromium.org authored Nov 21, 2013

R=svenpanne@chromium.org

Review URL: https://codereview.chromium.org/80513002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17948 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

96c9095f

Fix Windows build · 293b8f73

jkummerow@chromium.org authored Nov 21, 2013

R=svenpanne@chromium.org

Review URL: https://codereview.chromium.org/79963003

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17946 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

293b8f73

API: Change AdjustAmountOfExternalAllocatedMemory calls to use int64_t instead · 9318e1cc

jkummerow@chromium.org authored Nov 21, 2013

of intptr_t

This prevents an overflow in FreeArrayBuffer, which in turn caused needless GCs
as well as crashes on isolate teardown.

LOG=Y
R=ulan@chromium.org

Review URL: https://codereview.chromium.org/70233010

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17944 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

9318e1cc

Removed dead stack printing code. · 617c2dd7

svenpanne@chromium.org authored Nov 21, 2013

R=verwaest@chromium.org

Review URL: https://codereview.chromium.org/77723007

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17943 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

617c2dd7

20 Nov, 2013 32 commits

MIPS: Improvement of “MIPS: Fixed crashes exposed though fuzzing.” (r17886). · c41ca55f

plind44@gmail.com authored Nov 20, 2013

BUG=
R=plind44@gmail.com

Review URL: https://codereview.chromium.org/78423002

Patch from Balazs Kilvady <kilvadyb@homejinni.com>.

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17942 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

c41ca55f

MIPS: Use SmiTst and TrySmiTag MacroAssembler instructions. · 391f05ae

plind44@gmail.com authored Nov 20, 2013

BUG=
R=plind44@gmail.com

Review URL: https://codereview.chromium.org/78403002

Patch from Balazs Kilvady <kilvadyb@homejinni.com>.

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17941 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

391f05ae

MIPS: Replace hard-coded stack frame size literals with... · c33cbd9f

plind44@gmail.com authored Nov 20, 2013

MIPS: Replace hard-coded stack frame size literals with StandardFrameConstants::kFixedFrameSizeFromFp.

Port r17925 (0b95731)

BUG=
R=plind44@gmail.com

Review URL: https://codereview.chromium.org/77763006

Patch from Balazs Kilvady <kilvadyb@homejinni.com>.

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17940 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

c33cbd9f

Mark flaky debug test as failing. · e5f18799

yangguo@chromium.org authored Nov 20, 2013

The issues are known. For the time being, we mark it as failing.

R=machenbach@chromium.org
BUG=v8:2921, v8:3005
LOG=N

Review URL: https://codereview.chromium.org/77723008

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17938 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

e5f18799

MIPS: Remove the first_right_arg hack for HMod. · 29e47d1f

plind44@gmail.com authored Nov 20, 2013

Port r17919 (23e9644)

Original commit message:
Generate a proper subgraph in BuildBinaryOperation instead.

BUG=
R=plind44@gmail.com

Review URL: https://codereview.chromium.org/78663002

Patch from Balazs Kilvady <kilvadyb@homejinni.com>.

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17937 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

29e47d1f

Reland r17907 - Make it possible to add more than one piece of embedder data to isolates" · 840bc42d

jochen@chromium.org authored Nov 20, 2013

This will allow for using gin and blink bindings in the same
process.

Over r17907, I changed the order of fields in Isolate to be stable across different platforms, since the ABI defined packing is not the same on
all targets, and I initialize the embedder data field in Isolate.

BUG=317398
R=svenpanne@chromium.org, dcarney@chromium.org
LOG=n

Review URL: https://codereview.chromium.org/78453002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17935 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

840bc42d

Update benchmark test suite. · 42f920cf

yangguo@chromium.org authored Nov 20, 2013

R=jkummerow@chromium.org

Review URL: https://codereview.chromium.org/67763007

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17934 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

42f920cf

Increase precision when finding the remainder after division by pi/2. · 2c7ebfa7

yangguo@chromium.org authored Nov 20, 2013

R=jkummerow@chromium.org

Review URL: https://codereview.chromium.org/66703005

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17933 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

2c7ebfa7

Do not recompile existing functions when enabling allocation tracking · e7511b62

yurys@chromium.org authored Nov 20, 2013

Now that we don't have special hooks for the allocations in the generated code there is no need to recompile existing functions when enabling allocation tracking.

BUG=chromium:27798
LOG=N
R=loislo@chromium.org, mstarzinger@chromium.org

Review URL: https://codereview.chromium.org/67743006

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17932 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

e7511b62

Re-enable EnsureAllocationSiteDependentCodesProcessed. · 1e8c7c56

mvstanton@chromium.org authored Nov 20, 2013

The problem was that we don't use crankshaft on non-sse2 builds.
(on ARM non-vfp3).

R=danno@chromium.org

Review URL: https://codereview.chromium.org/77963002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17930 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

1e8c7c56

MaterializedLiteral expressions need to cache expression depth. · b90d87c3

mvstanton@chromium.org authored Nov 20, 2013

A problem arises in recursive literal expressions due to recent
changes that defer allocation of constant literal properties
from parse time. We were calculating expression depth as a
side-effect of a lazy constant property build, but subsequent
calls for the depth always returned 1. Cache the correct depth
in the MaterializedLiteral instead.

(Related-to/very-partial-revert-of
https://codereview.chromium.org/61873003)

R=ulan@chromium.org

Review URL: https://codereview.chromium.org/78493002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17929 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

b90d87c3

Force 'git cl upload' in push-to-trunk forced mode. · 718b3864

machenbach@chromium.org authored Nov 20, 2013

R=jkummerow@chromium.org

Review URL: https://codereview.chromium.org/77853003

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17926 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

718b3864

Replace hard-coded stack frame size literals with StandardFrameConstants::kFixedFrameSizeFromFp · df38e6f9

rmcilroy@chromium.org authored Nov 20, 2013

R=ulan@chromium.org

Review URL: https://codereview.chromium.org/60763006

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17925 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

df38e6f9

ARM: Optimize AccessArgumentsAt · 2b1aeec5

m.m.capewell@googlemail.com authored Nov 20, 2013

Optimize register constraints and code generated for AccessArgumentsAt
Lithium instruction.

TEST=none
BUG=
R=ulan@chromium.org

Review URL: https://codereview.chromium.org/78093004

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17923 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

2b1aeec5

Use SimpleLoadField for JSObjectFieldAccessors, rather than compiling a new stub. · 6557c85f

verwaest@chromium.org authored Nov 20, 2013

R=dslomov@chromium.org

Review URL: https://chromiumcodereview.appspot.com/76403002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17922 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

6557c85f

Activate Hydrogen-generated KeyedLoadDictionary ICs · adb39abd

danno@chromium.org authored Nov 20, 2013

R=ulan@chromium.org

Review URL: https://chromiumcodereview.appspot.com/59813011

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17921 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

adb39abd

Remove the first_right_arg hack for HMod. · d38abc3b

bmeurer@chromium.org authored Nov 20, 2013

Generate a proper subgraph in BuildBinaryOperation instead.

R=svenpanne@chromium.org

Review URL: https://codereview.chromium.org/77053003

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17919 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

d38abc3b

Removed unused --preallocate-message-memory flag. · 8f88467b

svenpanne@chromium.org authored Nov 20, 2013

It results in a lot of dead code, and Isolate::PrintStack itself
crashes most of the time when something went wrong earlier.
Furthermore, we have plans do get better information into the
minidump, anyway.

R=bmeurer@chromium.org

Review URL: https://codereview.chromium.org/78003002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17918 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

8f88467b

Convert constant numbers at compile time. · 4c4d911c

bmeurer@chromium.org authored Nov 20, 2013

TEST=mjsunit/number-tostring
R=mvstanton@chromium.org

Review URL: https://codereview.chromium.org/77553005

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17917 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

4c4d911c

Revert r17907 - Make it possible to add more than one piece of embedder data to isolates · bd099373

jochen@chromium.org authored Nov 20, 2013

> This will allow for using gin and blink bindings in the same process
>
> BUG=317398
> R=svenpanne@chromium.org, dcarney@chromium.org
> LOG=y
>
> Review URL: https://codereview.chromium.org/77913003

BUG=none
R=svenpanne@chromium.org
TBR=svenpanne@chromium.org
LOG=n

Review URL: https://codereview.chromium.org/78093005

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17915 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

bd099373

Make BuildBinaryOperation use ReturnValue instead of ReturnInstruction. · defee14d

bmeurer@chromium.org authored Nov 20, 2013

R=svenpanne@chromium.org

Review URL: https://codereview.chromium.org/78183002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17914 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

defee14d

Replace unsafe uses of Add<> with AddUncasted<>. · 6826b747

bmeurer@chromium.org authored Nov 20, 2013

R=svenpanne@chromium.org

Review URL: https://codereview.chromium.org/78213002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17913 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

6826b747

Don't generate useless string checks for string adds. · da6d5586

bmeurer@chromium.org authored Nov 20, 2013

If we know that one side of a string add is definitely a string
(i.e. if it's a string constant), then we don't need to emit a
string check for the argument.

This adds a new BuildCheckString() method to the graph builder,
which does "the right thing".

TEST=mjsunit/string-add
R=mvstanton@chromium.org

Review URL: https://codereview.chromium.org/78063002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17912 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

da6d5586

Remove deprecated v8::SetResourceConstraints without Isolate parameter · 782040d2

rmcilroy@chromium.org authored Nov 20, 2013

LOG=Y
R=bmeurer@chromium.org

Review URL: https://codereview.chromium.org/62193003

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17910 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

782040d2

Remove deprecated v8-defaults.h and defaults.cc. · 57dc3139

rmcilroy@chromium.org authored Nov 20, 2013

BUG=312241
LOG=Y
R=svenpanne@chromium.org

Review URL: https://codereview.chromium.org/59823008

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17909 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

57dc3139

Make it possible to add more than one piece of embedder data to isolates · 4515fb5c

jochen@chromium.org authored Nov 20, 2013

This will allow for using gin and blink bindings in the same process

BUG=317398
R=svenpanne@chromium.org, dcarney@chromium.org
LOG=y

Review URL: https://codereview.chromium.org/77913003

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17907 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

4515fb5c

faster stack frame generation for accessor property ics · c595853b

dcarney@chromium.org authored Nov 20, 2013

this is implemented for intel platforms only, as they are the only ones which show speed up with this change

R=svenpanne@chromium.org
BUG=

Review URL: https://codereview.chromium.org/48303002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17903 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

c595853b

Reset overwrite mode for BinaryOpStub unless we can actually make use of it. · 54dc61a5

bmeurer@chromium.org authored Nov 20, 2013

R=svenpanne@chromium.org

Review URL: https://codereview.chromium.org/77943002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17899 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

54dc61a5

Prepare push to trunk. Now working on version 3.23.9. · 15582e24

machenbach@chromium.org authored Nov 20, 2013

R=danno@chromium.org
BUG=

Review URL: https://codereview.chromium.org/77673004

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17896 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

15582e24

Quit running EnsureAllocationSiteDependentCodesProcessed test until · daf4e658

mvstanton@chromium.org authored Nov 20, 2013

it's failure is diagnosed.

R=danno@chromium.org

Review URL: https://codereview.chromium.org/77923002

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17894 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

daf4e658

Add forced mode to push-to-trunk script. · 8b382597

machenbach@chromium.org authored Nov 20, 2013

This CL depends on https://codereview.chromium.org/65933003/.

BUG=
R=jkummerow@chromium.org

Review URL: https://codereview.chromium.org/67763005

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17893 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

8b382597

Refactor and improve bug line generation for push-to-trunk. · 6fcd94cc

machenbach@chromium.org authored Nov 20, 2013

BUG=
R=jkummerow@chromium.org

Review URL: https://codereview.chromium.org/65933003

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17892 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

6fcd94cc

19 Nov, 2013 1 commit

MIPS: Fixed crashes exposed though fuzzing. · 79de22a5

plind44@gmail.com authored Nov 19, 2013

Port r17886 (e2fb3ed)

Original commit message:
The %_OneByteSeqStringSetChar intrinsic expects its arguments to be checked before being called for efficiency reasons, but the fuzzer provided no such checks. Now the intrinsic is robust to bad input if FLAG_debug_code is set.

R=plind44@gmail.com, yangguo@chromium.org
TEST=test/mjsunit/regress/regress-320948.js
BUG=chromium:320948
LOG=Y

Review URL: https://codereview.chromium.org/68793008

Patch from Balazs Kilvady <kilvadyb@homejinni.com>.

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@17891 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

79de22a5