Commits · 8c0c573d9694bb2525270c87a18a3962c8e19885 · Linshizhi / V8

18 Sep, 2013 9 commits

Set the proper instance-type on HAllocate in BuildFastLiteral · 8c0c573d

verwaest@chromium.org authored Sep 18, 2013

BUG=chromium:284577
R=hpayer@chromium.org

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

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

8c0c573d

Revert "Turn on i18n support by default", "Update test262 expectations, now... · eca42c42

machenbach@chromium.org authored Sep 18, 2013

Revert "Turn on i18n support by default", "Update test262 expectations, now that 15.5.4.9_CE passes", "Make testing i18n the default.", "Mark two failing mozilla tests as failing.", "Mark mozilla tests as flaky instead of failing" and "Fix test expectations for i18n."

This reverts commit 16783-16788 due to several crashers on nosnap builder.

TBR=jochen@chromium.org

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

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

eca42c42

Fix test expectations for i18n. · ef62d723

machenbach@chromium.org authored Sep 18, 2013

BUG=v8:2893

TBR=jochen@chromium.org

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

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

ef62d723

Mark mozilla tests as flaky instead of failing · a5eda99f

jochen@chromium.org authored Sep 18, 2013

Apparently, they work on windows

BUG=v8:2893
TBR=mstarzinger@chromium.org

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

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

a5eda99f

Mark two failing mozilla tests as failing. · e805b836

jochen@chromium.org authored Sep 18, 2013

They started to fail after turning on i18n support. Need to investigate
why

BUG=v8:2893
TBR=mstarzinger@chromium.org

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

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

e805b836

Make testing i18n the default. · 7e0cc857

machenbach@chromium.org authored Sep 18, 2013

If i18n support is on when building with make, the intl test suite is automatically run with "check".

On the bots, we'll turn it off explicitly, where required.

R=jochen@chromium.org

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

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

7e0cc857

Update test262 expectations, now that 15.5.4.9_CE passes · f2966242

jochen@chromium.org authored Sep 18, 2013

BUG=v8:2413
TBR=mstarzinger@chromium.org

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

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

f2966242

Turn on i18n support by default · f41f6c2b

jochen@chromium.org authored Sep 18, 2013

BUG=v8:2745
R=danno@chromium.org

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

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

f41f6c2b

Prepare push to trunk. Now working on version 3.21.18. · ae942958

hpayer@chromium.org authored Sep 18, 2013

R=machenbach@chromium.org
BUG=

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

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

ae942958

17 Sep, 2013 29 commits

MIPS: Tweak StoreKeyed. · bcaea997

plind44@gmail.com authored Sep 17, 2013

Port r16771 (536eb66)

Original commit message:
Avoid corrupting its input in some cases.

BUG=none
TEST=test/mjsunit/lithium/StoreKeyed*.js
R=plind44@gmail.com

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

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

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

bcaea997

MIPS: reland 16744: add context save for GenerateFastApiCall. · d047afb4

plind44@gmail.com authored Sep 17, 2013

Port r16763 (968acfa)

BUG=
R=plind44@gmail.com

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

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

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

d047afb4

Support stepin for combination of apply and bound function · 2974f8e3

prybin@chromium.org authored Sep 17, 2013

R=yangguo@chromium.org

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

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

2974f8e3

Implement local load/store elimination on basic blocks. · 05797e77

titzer@chromium.org authored Sep 17, 2013

BUG=
R=mstarzinger@chromium.org

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

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

05797e77

Make v8 compile on VS2013. · 9f5edd1b

dslomov@chromium.org authored Sep 17, 2013

VS2013 contains a number of improvements, most notably the addition of all C99 math functions.

I'm a little bit concerned about the change I had to make in cpu-profiler.cc, but I spent quite a bit of time looking at it and was unable to figure out any rational explanation for the warning. It's possible it's spurious. Since it seems like a useful warning in general though, I chose not to disable globally at the gyp level.

I do think someone with expertise here should probably try to determine if this is a legitimate warning.

BUG=288948
R=dslomov@chromium.org

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

Patch from Zach Turner <zturner@chromium.org>.

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

9f5edd1b

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

svenpanne@chromium.org authored Sep 17, 2013

Enables MAYBE_BOOL flags for when you want to only do something if the flag
was explicitly set to true or false. Also cleans up JSArguments struct.

BUG=None
R=svenpanne@chromium.org

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

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

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

cc17926d

Add a mechanism to override the detected cpu features. · a5e0f768

olivf@chromium.org authored Sep 17, 2013

BUG=
R=svenpanne@chromium.org

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

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

a5e0f768

ARM: Tweak StoreKeyed. · 87d0659e

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

Avoid corrupting its input in some cases.

BUG=none
TEST=test/mjsunit/lithium/StoreKeyed*.js
R=ulan@chromium.org

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

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

87d0659e

new gc callbacks with isolate parameters · f3434003

dcarney@chromium.org authored Sep 17, 2013

R=svenpanne@chromium.org
BUG=

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

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

f3434003

Open new handle scope when deep copying. · 873050f1

yangguo@chromium.org authored Sep 17, 2013

R=mstarzinger@chromium.org
BUG=292903

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

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

873050f1

remove js_accessor_ics flag · 21ed55d9

dcarney@chromium.org authored Sep 17, 2013

R=mstarzinger@chromium.org
BUG=

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

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

21ed55d9

LLoadKeyed does not clobber double anymore. · 4f3d58dd

olivf@chromium.org authored Sep 17, 2013

BUG=
R=verwaest@chromium.org

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

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

4f3d58dd

Ensure constant truncation is only done when it is safe. · 1a50eb66

olivf@chromium.org authored Sep 17, 2013

BUG=
R=mstarzinger@chromium.org

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

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

1a50eb66

Adjust Android test runner to push tools/profviz to device. · 4a90f2e1

ulan@chromium.org authored Sep 17, 2013

R=bmeurer@chromium.org

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

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

4a90f2e1

Fix i18n compilation after r16762. · 4c880ef2

mstarzinger@chromium.org authored Sep 17, 2013

R=dcarney@chromium.org

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

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

4c880ef2

reland 16744: add context save for GenerateFastApiCall · 0e90c689

dcarney@chromium.org authored Sep 17, 2013

R=mstarzinger@chromium.org
BUG=

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

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

0e90c689

Handlify JSReceiver::HasProperty and friends. · 4c85efb5

mstarzinger@chromium.org authored Sep 17, 2013

R=yangguo@chromium.org

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

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

4c85efb5

Remove unused HIsNumberAndBranch instruction. · 722c6192

bmeurer@chromium.org authored Sep 17, 2013

R=svenpanne@chromium.org

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

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

722c6192

Handlify JSReceiver::SetProperty and friends. · 16a22a96

mstarzinger@chromium.org authored Sep 17, 2013

R=verwaest@chromium.org

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

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

16a22a96

Don't run tests with the profiler when USE_SIMULATOR is defined. · f89996a1

bmeurer@chromium.org authored Sep 17, 2013

BUG=v8:2874
R=machenbach@chromium.org

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

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

f89996a1

Revert "ARM: Add tests for CopyBytes." · ae5ddcd9

bmeurer@chromium.org authored Sep 17, 2013

This reverts commit r16751 for breaking arm.debug checks.

TBR=ulan@chromium.org

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

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

ae5ddcd9

Ensure escape analysis preserves boxed HeapNumber. · 680af93c

mstarzinger@chromium.org authored Sep 17, 2013

R=hpayer@chromium.org

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

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

680af93c

Fix typo in x64 disassembler. · 03d3ef80

bmeurer@chromium.org authored Sep 17, 2013

R=svenpanne@chromium.org

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

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

03d3ef80

ARM: Add tests for CopyBytes. · a68d89d0

ulan@chromium.org authored Sep 17, 2013

TEST=cctest/test-macro-assembler-arm.cc

R=ulan@chromium.org

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

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

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

a68d89d0

Fix handling of Integer32 in HLoadNamedField and HStoreNamedField on x64. · 8e285060

bmeurer@chromium.org authored Sep 17, 2013

The current implementation of Integer32 named loads and stores
is broken on x64.

R=danno@chromium.org

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

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

8e285060

Use icu_gyp_path variable to reference ICU gyp file · b3310c3a

jochen@chromium.org authored Sep 17, 2013

That way, embedders can put the icu sources whereever they want and
aren't forced to put them under third_party/gyp

BUG=none
R=jkummerow@chromium.org

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

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

b3310c3a

revert 16744 for breaking build · c09d00d1

dcarney@chromium.org authored Sep 17, 2013

TBR=mstarzinger@chromium.org
BUG=

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

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

c09d00d1

Refactoring only: Make it clear that MOD is special. · 87ff987f

svenpanne@chromium.org authored Sep 17, 2013

LArithmeticD should really be 5 different instructions... :-/

R=bmeurer@chromium.org

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

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

87ff987f

add context save for GenerateFastApiCall · 0020146f

dcarney@chromium.org authored Sep 17, 2013

R=mstarzinger@chromium.org
BUG=

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

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

0020146f

16 Sep, 2013 2 commits

MIPS: Orthogonalize Lithium binary op instructions. · 29d0f52a

palfia@homejinni.com authored Sep 16, 2013

Port r16739 (cd8e4a21)

BUG=
R=gergely@homejinni.com

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

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

29d0f52a

Chromium 284577 needs a mitigation CL added. There is a TODO to remove · 61a3605a

mvstanton@chromium.org authored Sep 16, 2013

the mitigation when the cause of the bug is discovered.

BUG=
R=hpayer@chromium.org, mstarzinger@chromium.org

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

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

61a3605a