Commits · 1d07d3152ce6496b89d55eb4a0ee297a02cbad93 · Linshizhi / V8

16 Sep, 2013 18 commits

yangguo@chromium.org authored Sep 16, 2013

BUG=
R=mstarzinger@chromium.org

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

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

1d07d315

Add mutex when accessing concurrent recompilation output queue. · ea5ab721

yangguo@chromium.org authored Sep 16, 2013

Also small refactorings.

R=mstarzinger@chromium.org
BUG=291236

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

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

ea5ab721

Orthogonalize Lithium binary op instructions. · 42879d10

olivf@chromium.org authored Sep 16, 2013

BUG=
R=svenpanne@chromium.org

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

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

42879d10

HeapProfiler: compile fix for win x64. · 849815fa

loislo@chromium.org authored Sep 16, 2013

BUG=none
TBR=yangguo@chromium.org

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

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

849815fa

Don't lookup the cache for the result of Function::New · be8621a4

mstarzinger@chromium.org authored Sep 16, 2013

Since isFunctionCached condition is wrong, we lookup the cache even if
doNotCache is true. As a result, Function::New always returns null
except for the first time.

BUG=272579
R=dcarney@chromium.org, mstarzinger@chromium.org, yhirano@chromium.org

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

Patch from Yusuke Suzuki <yusukesuzuki@chromium.org>.

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

be8621a4

Revert "Add flags to force or prevent setting of isolate.is_memory_constrained." · bf345f02

svenpanne@chromium.org authored Sep 16, 2013

It introduces static initializers for the new "MAYBE_BOOL" kind of
flags, which is a no-no for Chrome. This has to be done differently.

TBR=danno@chromium.org

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

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

bf345f02

In the case of shift amounts with two constants and if their sum is equal 32,... · 8ed9b08e

svenpanne@chromium.org authored Sep 16, 2013

In the case of shift amounts with two constants and if their sum is equal 32, then shift can also be replaced with bit rotate.

R=svenpanne@chromium.org

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

Patch from Bangfu Tao <bangfu.tao@samsung.com>.

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

8ed9b08e

HeapSnapshot: replace O(N*ln(N)) algorithm of sorting with O(N) one. · 0daf224c

loislo@chromium.org authored Sep 16, 2013

We have HashMap for the strings. They got id sequentially. So we could use index sort.

BUG=none
R=alph@chromium.org, yangguo@chromium.org

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

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

0daf224c

Add flags to force or prevent setting of isolate.is_memory_constrained. · f30a7343

svenpanne@chromium.org authored Sep 16, 2013

Also enable MAYBE_BOOL flags for when you want to only do something if the flag was explicitly set to true or false.

BUG=None
R=hpayer@chromium.org, svenpanne@chromium.org

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

Patch from Ross McIlroy <rmcilroy@chromium.org>.

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

f30a7343

Make HValue::ActualValue() traverse all idefs. · af87218f

titzer@chromium.org authored Sep 16, 2013

BUG=
R=mstarzinger@chromium.org

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

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

af87218f

Tweak HConstant::EmitAtUses() to eliminate useless constant generation · d5d7b1d9

danno@chromium.org authored Sep 16, 2013

BUG=2881
R=danno@chromium.org

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

Patch from Weiliang Lin <weiliang.lin2@gmail.com>.

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

d5d7b1d9

Make HCompareNumericAndBranch safe for snapshots on ia32. · 73b03b89

bmeurer@chromium.org authored Sep 16, 2013

R=danno@chromium.org

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

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

73b03b89

Fixed shared_library build on Windows. · 7b41d6d4

svenpanne@chromium.org authored Sep 16, 2013

BUG=v8:2879
R=ulan@chromium.org

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

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

7b41d6d4

Remove unused context parameter from AddIncrementCounter(). · 8588b744

bmeurer@chromium.org authored Sep 16, 2013

R=mvstanton@chromium.org

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

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

8588b744

ARM: Tweak the integer division operation. · 39f0ccc2

rodolph.perfetta@gmail.com authored Sep 16, 2013

BUG=none
TEST=test/mjsunit/lithium/DivI.js
R=bmeurer@chromium.org

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

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

39f0ccc2

Rename StubType MAP_TRANSITION to TRANSITION. · 223ac69f

verwaest@chromium.org authored Sep 16, 2013

R=mvstanton@chromium.org

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

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

223ac69f

HeapProfiler: replace pointer based matching algorithm with string matching... · 2abe14b8

loislo@chromium.org authored Sep 16, 2013

HeapProfiler: replace pointer based matching algorithm with string matching algorithm for strings_ member.

pros: decreased snapshot size.
cons: increased serialization time.

I've tested the implementation on gmail 90mb heap.
I saw no speed degradation on the serialization step.
The snapshot size lost ~3% of its size. 100Mb -> 97Mb.

BUG=none
R=alph@chromium.org, yangguo@chromium.org

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

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

2abe14b8

Prepare push to trunk. Now working on version 3.21.17. · 043abcd3

mvstanton@chromium.org authored Sep 16, 2013

R=machenbach@chromium.org
BUG=

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

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

043abcd3

13 Sep, 2013 22 commits

MIPS: remove getcurrent from stubs. · 6d3b3255

plind44@gmail.com authored Sep 13, 2013

Port r16710 (c5203f8)

BUG=
R=plind44@gmail.com

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

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

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

6d3b3255

MIPS: Improve SeqStringSetChar implementation. · 23fc6218

plind44@gmail.com authored Sep 13, 2013

Port r16707 (a25d669)

TEST=/test/mjsunit/lithium/SeqStringSetChar.js
BUG=
R=plind44@gmail.com

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

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

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

23fc6218

Every place where AllocationMemento is initialized with an · 516339cb

mvstanton@chromium.org authored Sep 13, 2013

AllocationSite is now checked to be sure a valid Site goes in.

This is temporary code to diagnose chromium bug 284577.

(This is a second attempt, the first attempt ran into the problem of undefined ordering of function calls in Windows and Mac optimized builds, see the fixes in code-stubs-hydrogen.cc).

BUG=
R=mstarzinger@chromium.org

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

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

516339cb

revert 16717 for breaking everything · 31105962

dcarney@chromium.org authored Sep 13, 2013

TBR=svenpanne@chromium.org
BUG=

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

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

31105962

line up crashing with chrome · bbaf775d

dcarney@chromium.org authored Sep 13, 2013

R=svenpanne@chromium.org
BUG=

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

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

bbaf775d

Add Contains(), at(), and a constructor with raw addresses to UniqueSet<T> and Unique<T>. · 25372ac1

titzer@chromium.org authored Sep 13, 2013

BUG=
R=verwaest@chromium.org

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

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

25372ac1

Add missing factory method for HDebugBreak. · fe9f9e20

bmeurer@chromium.org authored Sep 13, 2013

R=mvstanton@chromium.org

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

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

fe9f9e20

Intel: Branch reordering and port arm EmitNumberUntagD() Improvement · 81425a20

olivf@chromium.org authored Sep 13, 2013

R=bmeurer@chromium.org

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

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

81425a20

Fix heap verifier for partially constructed arrays when allocation folding is off. · a8443629

hpayer@chromium.org authored Sep 13, 2013

BUG=
R=mvstanton@chromium.org

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

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

a8443629

Don't update ICs when the receiver has a deprecated map. · 3824d596

verwaest@chromium.org authored Sep 13, 2013

R=ulan@chromium.org

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

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

3824d596

remove OS::StackWalk · cb490c99

dcarney@chromium.org authored Sep 13, 2013

R=bmeurer@chromium.org
BUG=

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

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

cb490c99

remove getcurrent from stubs · 564a9a68

dcarney@chromium.org authored Sep 13, 2013

R=svenpanne@chromium.org
BUG=

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

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

564a9a68

Use regular map-checks to guard string-length loading. · ea77a8ed

verwaest@chromium.org authored Sep 13, 2013

R=ulan@chromium.org

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

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

ea77a8ed

Handlify JSObject::DeepCopy method. · 50b05676

mstarzinger@chromium.org authored Sep 13, 2013

R=verwaest@chromium.org

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

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

50b05676

ARM: Improve SeqStringSetChar implementation. · 85401293

bmeurer@chromium.org authored Sep 13, 2013

TEST=/test/mjsunit/lithium/SeqStringSetChar.js
R=bmeurer@chromium.org

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

Patch from Rodolph Perfetta <rodolph.perfetta@gmail.com>.

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

85401293

Revert "Every place where AllocationMemento is initialized with an" · c29295e8

mvstanton@chromium.org authored Sep 13, 2013

This reverts r16705 for breaking Windows build.

R=dslomov@chromium.org

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

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

c29295e8

Every place where AllocationMemento is initialized with an · 25c4f763

mvstanton@chromium.org authored Sep 13, 2013

AllocationSite is now checked to be sure a valid Site goes in.
This is temporary code to diagnose chromium bug 284577.

BUG=
R=mstarzinger@chromium.org

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

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

25c4f763

Array "splice" changeRecords should be emitted after the performChange has completed (per spec) · 94c4c596

rossberg@chromium.org authored Sep 13, 2013

R=rossberg@chromium.org
BUG=

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

Patch from Rafael Weinstein <rafaelw@chromium.org>.

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

94c4c596

Allow implicit conversion of acceptList values to string during Object.observe · f99298bf

rossberg@chromium.org authored Sep 13, 2013

R=rossberg@chromium.org
BUG=

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

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

f99298bf

Use xorps to break the cvtsi2sd unnecessary dependence due to its partially written · e4c179e2

yangguo@chromium.org authored Sep 13, 2013

BUG=
R=yangguo@chromium.org

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

Patch from Weiliang Lin <weiliang.lin2@gmail.com>.

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

e4c179e2

Prepare push to trunk. Now working on version 3.21.16. · 8650fae5

dslomov@chromium.org authored Sep 13, 2013

R=machenbach@chromium.org

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

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

8650fae5

MIPS: Bugfix: array constructors that expect a type feedback cell that points... · 2cec3754

palfia@homejinni.com authored Sep 13, 2013

MIPS: Bugfix: array constructors that expect a type feedback cell that points to an AllocationSite were being passed the undefined object in some cases. Clearly separate the cases where we have an AllocationSite and where we don't in the general ArrayConstructorStub.

Port r16693 (0852d4ab)

BUG=
R=gergely@homejinni.com

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

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

2cec3754