Remove SIMD.js from V8.

Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124, v8:5948
R=bradnelson@chromium.org,bmeurer@chromium.org,jochen@chromium.org,hpayer@chromium.org,danno@chromium.org

Review-Url: https://codereview.chromium.org/2684313003
Cr-Original-Original-Commit-Position: refs/heads/master@{#43162}
Committed: https://chromium.googlesource.com/v8/v8/+/d170c57ab996d00c4665a9d865bd5754a1806c6c
Review-Url: https://codereview.chromium.org/2684313003
Cr-Original-Commit-Position: refs/heads/master@{#43169}
Committed: https://chromium.googlesource.com/v8/v8/+/a9b59a11f1bfe069afabe5567f919727456f1f12
Review-Url: https://codereview.chromium.org/2684313003
Cr-Commit-Position: refs/heads/master@{#43176}
Committed: https://chromium.googlesource.com/v8/v8/+/deabb19abc3328149ed76990cdd3a1d5e4e5b3ed

[bbudge, 2017-02-10 17:19:52]

The CQ bit was checked by bbudge@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-10 17:20:02]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-10 17:21:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-10 17:21:53]

Dry run: Try jobs failed on following builders:
  v8_linux64_avx2_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_avx2_rel_ng/buil...)
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)
  v8_linux64_verify_csa_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_verify_csa_rel_n...)
  v8_linux_nodcheck_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel_ng/bu...)
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/17245)
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/34496)
  v8_win_compile_dbg on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_compile_dbg/builds/3...)

[bbudge, 2017-02-10 17:30:34]

The CQ bit was checked by bbudge@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-10 17:30:43]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-10 17:41:36]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-10 17:41:37]

Dry run: Try jobs failed on following builders:
  v8_win_compile_dbg on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_compile_dbg/builds/3...)
  v8_win_nosnap_shared_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_win_nosnap_shared_rel_ng...)

[bbudge, 2017-02-10 18:58:39]

The CQ bit was checked by bbudge@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-10 18:58:43]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-10 19:02:50]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-10 19:02:51]

Dry run: Try jobs failed on following builders:
  v8_linux64_gyp_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux64_gyp_rel_ng/build...)

[bbudge, 2017-02-10 19:45:56]

The CQ bit was checked by bbudge@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-10 19:45:59]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-10 19:50:52]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-10 19:50:53]

Dry run: Try jobs failed on following builders:
  v8_linux_arm64_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm64_rel_ng/build...)
  v8_linux_nodcheck_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel_ng/bu...)
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/17252)

[bbudge, 2017-02-10 21:28:47]

The CQ bit was checked by bbudge@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-10 21:28:57]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[bbudge, 2017-02-10 21:30:18]

bbudge@chromium.org changed reviewers:
+ bradnelson@chromium.org

[bbudge, 2017-02-10 21:30:19]

isolate tests fail. Not sure how to disable those. Otherwise, it looks like
everything else is ready.

[commit-bot: I haz the power, 2017-02-10 21:33:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-10 21:33:28]

Dry run: Try jobs failed on following builders:
  v8_linux_arm_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_arm_rel_ng/builds/...)
  v8_linux_nodcheck_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_nodcheck_rel_ng/bu...)
  v8_linux_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_linux_rel_ng/builds/20744)
  v8_mac_rel_ng on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_mac_rel_ng/builds/17260)

[bradnelson, 2017-02-12 05:27:10]

lgtm

Wow. Shocking amount of effort that went into that. This highlights to me the
importance of making sure we have clarity before we invest.
Bill thanks for your continued efforts.

I believe the isolate tests are failing due to the buildbot build step doing a
find on the directory, and the trybot patching logic not handling removed
directories.

I'm going to use NOTRY to attempt to force land and watch the bots.

[bradnelson, 2017-02-12 05:27:29]

Description was changed from

==========
Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124,5948
==========

to

==========
Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124,5948
NOTRY=true
==========

[bradnelson, 2017-02-12 05:27:38]

The CQ bit was checked by bradnelson@chromium.org

[commit-bot: I haz the power, 2017-02-12 05:27:44]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-12 05:31:12]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-12 05:31:14]

Try jobs failed on following builders:
  v8_presubmit on master.tryserver.v8 (JOB_FAILED,
http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/34540)

[bradnelson, 2017-02-12 05:36:09]

Description was changed from

==========
Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124,5948
NOTRY=true
==========

to

==========
Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124,5948
R=bradnelson@chromium.org,bmeurer@chromium.org,jochen@chromium.org,hpayer@chr...
(notry since trybots can't patch directory deletes)
NOTRY=true
==========

[bradnelson, 2017-02-12 05:36:09]

bradnelson@chromium.org changed reviewers:
+ bmeurer@chromium.org, danno@chromium.org, hpayer@chromium.org,
jochen@chromium.org

[bradnelson, 2017-02-12 05:38:29]

Could we get OWNERS from:
bmeurer, jochen, hpayer
As this touches lots of places.
NOTE: The red on the trybot results is due to a limitation of how the trybots
patch directory deletes and should land green (affects isolate tests).
Thanks

[Benedikt Meurer, 2017-02-12 07:22:10]

On 2017/02/12 05:38:29, bradnelson wrote:
> Could we get OWNERS from:
> bmeurer, jochen, hpayer
> As this touches lots of places.
> NOTE: The red on the trybot results is due to a limitation of how the trybots
> patch directory deletes and should land green (affects isolate tests).
> Thanks

Awesome! LGTM!

[bradn, 2017-02-12 07:26:22]

bradnelson@google.com changed reviewers:
+ bradnelson@google.com

[bradn, 2017-02-12 07:26:23]

Now need jochen or hpayer for the remaining bits in the GC.

[Michael Lippautz, 2017-02-12 12:24:58]

On 2017/02/12 07:26:23, bradn wrote:
> Now need jochen or hpayer for the remaining bits in the GC.

heap lgtm

[bradn, 2017-02-12 16:20:27]

The CQ bit was checked by bradnelson@google.com

[commit-bot: I haz the power, 2017-02-12 16:20:32]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-12 16:22:43]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-12 16:22:45]

Failed to apply patch for DEPS:
While running git apply --index -p1;
  error: patch failed: DEPS:33
  error: DEPS: patch does not apply

Patch:       DEPS
Index: DEPS
diff --git a/DEPS b/DEPS
index
5c1983002eded1caba3231f037412149aa239958..c95741585e2e305123969ffeea8588b58c270f1c
100644
--- a/DEPS
+++ b/DEPS
@@ -33,7 +33,6 @@ deps = {
     Var("chromium_url") + "/v8/deps/third_party/benchmarks.git" + "@" +
"05d7188267b4560491ff9155c5ee13e207ecd65f",
   "v8/test/mozilla/data":
     Var("chromium_url") + "/v8/deps/third_party/mozilla-tests.git" + "@" +
"f6c578a10ea707b1a8ab0b88943fe5115ce2b9be",
-  "v8/test/simdjs/data": Var("chromium_url") +
"/external/github.com/tc39/ecmascript_simd.git" + "@" +
"baf493985cb9ea7cdbd0d68704860a8156de9556",
   "v8/test/test262/data":
     Var("chromium_url") + "/external/github.com/tc39/test262.git" + "@" +
"6a0f1189eb00d38ef9760cb65cbc41c066876cde",
   "v8/test/test262/harness":

[bbudge, 2017-02-12 19:57:38]

On 2017/02/12 05:27:10, bradnelson wrote:
> lgtm
> 
> Wow. Shocking amount of effort that went into that. This highlights to me the
> importance of making sure we have clarity before we invest.
> Bill thanks for your continued efforts.
> 
> I believe the isolate tests are failing due to the buildbot build step doing a
> find on the directory, and the trybot patching logic not handling removed
> directories.
> 
> I'm going to use NOTRY to attempt to force land and watch the bots.

Yes, in hindsight this was a bad idea. Taking it out should help code size, and
perhaps even performance. I tried rebasing to see if the patch failure in the
DEPS file could be fixed, but the merge finished without conflict. Feel free to
land if you can. Otherwise, I'll try Monday.

[bbudge, 2017-02-13 01:28:32]

The CQ bit was checked by bbudge@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-13 01:28:37]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-13 01:30:26]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-13 01:30:28]

Dry run: This issue passed the CQ dry run.

[Hannes Payer (out of office), 2017-02-13 08:25:19]

Please also remove kSimd128Unaligned from AllocationAlignment in globals.h 

lgtm

[ivica.bogosavljevic, 2017-02-13 14:32:17]

On 2017/02/12 05:27:10, bradnelson wrote:
> lgtm
> 
> Wow. Shocking amount of effort that went into that. This highlights to me the
> importance of making sure we have clarity before we invest.
> Bill thanks for your continued efforts.
> 
> I believe the isolate tests are failing due to the buildbot build step doing a
> find on the directory, and the trybot patching logic not handling removed
> directories.
> 
> I'm going to use NOTRY to attempt to force land and watch the bots.

We were about to start port of SIMD.js to MIPS. I am curious why is SIMD.js
being removed
and will there be any plans to add it again?

[bradn, 2017-02-13 15:56:35]

We're focusing forward going SIMD effort on WebAssembly SIMD.
SIMD.js is being used as a starting point to guide that effort.
A prototype is being developed in tree in src/compiler + src/wasm
Bringing up MIPS support for those opcodes would be very helpful.
I'll reach out to you to discuss further.

[bradn, 2017-02-13 15:56:51]

The CQ bit was checked by bradnelson@google.com

[bradn, 2017-02-13 15:56:51]

The patchset sent to the CQ was uploaded after l-g-t-m from
mlippautz@chromium.org, bradnelson@chromium.org, bmeurer@chromium.org
Link to the patchset: https://codereview.chromium.org/2684313003/#ps120001
(title: "Rebase.")

[commit-bot: I haz the power, 2017-02-13 15:57:01]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-13 15:59:15]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1487001411280120,
"parent_rev": "5183933362c3ae77cdfa490a4f0652d642103a71", "commit_rev":
"0e6ea115ec4f5a148c678df06ee36158934159a8"}

[commit-bot: I haz the power, 2017-02-13 15:59:23]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1487001411280120,
"parent_rev": "eecbdc42a6190457f760bdce6c6554da39c4835c", "commit_rev":
"d170c57ab996d00c4665a9d865bd5754a1806c6c"}

[commit-bot: I haz the power, 2017-02-13 15:59:34]

Description was changed from

==========
Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124,5948
R=bradnelson@chromium.org,bmeurer@chromium.org,jochen@chromium.org,hpayer@chr...
(notry since trybots can't patch directory deletes)
NOTRY=true
==========

to

==========
Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124,5948
R=bradnelson@chromium.org,bmeurer@chromium.org,jochen@chromium.org,hpayer@chr...
(notry since trybots can't patch directory deletes)
NOTRY=true

Review-Url: https://codereview.chromium.org/2684313003
Cr-Commit-Position: refs/heads/master@{#43162}
Committed:
https://chromium.googlesource.com/v8/v8/+/d170c57ab996d00c4665a9d865bd5754a18...
==========

[commit-bot: I haz the power, 2017-02-13 15:59:36]

Committed patchset #7 (id:120001) as
https://chromium.googlesource.com/v8/v8/+/d170c57ab996d00c4665a9d865bd5754a18...

[Michael Achenbach, 2017-02-13 17:09:48]

Why did this large CL land with notry? It broke the main waterfall and the
trybots in patch 6 are broken in the same way.

[bradn, 2017-02-13 17:45:04]

A revert of this CL (patchset #7 id:120001) has been created in
https://codereview.chromium.org/2691903003/ by bradnelson@google.com.

The reason for reverting is: Much red..

[bradnelson, 2017-02-13 18:11:32]

A revert of this CL (patchset #7 id:120001) has been created in
https://codereview.chromium.org/2692933002/ by bradnelson@chromium.org.

The reason for reverting is: red.

[bbudge, 2017-02-13 18:43:20]

On 2017/02/13 08:25:19, Hannes Payer wrote:
> Please also remove kSimd128Unaligned from AllocationAlignment in globals.h 
> 
> lgtm

I was planning on removing that in a separate CL to reduce the risk on this one.

[Michael Achenbach, 2017-02-13 19:16:15]

Before you can rip out the .isolate stuff, you need to tell the infrastructure
to not run the tests anymore:
https://cs.chromium.org/search/?q=simdjs+file:%5Ebuild/+package:%5Echromium$&...

Lemme see if I can hack something up...

[Michael Achenbach, 2017-02-13 19:18:24]

On 2017/02/13 19:16:15, Michael Achenbach wrote:
> Before you can rip out the .isolate stuff, you need to tell the infrastructure
> to not run the tests anymore:
>
https://cs.chromium.org/search/?q=simdjs+file:%5Ebuild/+package:%5Echromium$&...
> 
> Lemme see if I can hack something up...

Is this also removed from the internal perf infra?

[Michael Achenbach, 2017-02-13 19:26:46]

On 2017/02/13 19:18:24, Michael Achenbach wrote:
> On 2017/02/13 19:16:15, Michael Achenbach wrote:
> > Before you can rip out the .isolate stuff, you need to tell the
infrastructure
> > to not run the tests anymore:
> >
>
https://cs.chromium.org/search/?q=simdjs+file:%5Ebuild/+package:%5Echromium$&...
> > 
> > Lemme see if I can hack something up...
> 
> Is this also removed from the internal perf infra?

Feel free to +1 or CQ this: https://chromium-review.googlesource.com/c/441826/

[Michael Achenbach, 2017-02-13 20:06:40]

Description was changed from

==========
Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124,5948
R=bradnelson@chromium.org,bmeurer@chromium.org,jochen@chromium.org,hpayer@chr...
(notry since trybots can't patch directory deletes)
NOTRY=true

Review-Url: https://codereview.chromium.org/2684313003
Cr-Commit-Position: refs/heads/master@{#43162}
Committed:
https://chromium.googlesource.com/v8/v8/+/d170c57ab996d00c4665a9d865bd5754a18...
==========

to

==========
Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124,v8:5948
R=bradnelson@chromium.org,bmeurer@chromium.org,jochen@chromium.org,hpayer@chr...

Review-Url: https://codereview.chromium.org/2684313003
Cr-Commit-Position: refs/heads/master@{#43162}
Committed:
https://chromium.googlesource.com/v8/v8/+/d170c57ab996d00c4665a9d865bd5754a18...
==========

[Michael Achenbach, 2017-02-13 20:07:04]

On 2017/02/13 19:26:46, Michael Achenbach wrote:
> On 2017/02/13 19:18:24, Michael Achenbach wrote:
> > On 2017/02/13 19:16:15, Michael Achenbach wrote:
> > > Before you can rip out the .isolate stuff, you need to tell the
> infrastructure
> > > to not run the tests anymore:
> > >
> >
>
https://cs.chromium.org/search/?q=simdjs+file:%5Ebuild/+package:%5Echromium$&...
> > > 
> > > Lemme see if I can hack something up...
> > 
> > Is this also removed from the internal perf infra?
> 
> Feel free to +1 or CQ this: https://chromium-review.googlesource.com/c/441826/

Infra changes landed. Reopened this for dry-running.

[Michael Achenbach, 2017-02-13 20:07:17]

The CQ bit was checked by machenbach@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-13 20:07:23]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[bbudge, 2017-02-13 20:18:20]

The CQ bit was checked by bbudge@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-02-13 20:18:23]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-13 20:35:35]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-02-13 20:35:37]

Dry run: This issue passed the CQ dry run.

[bbudge, 2017-02-13 20:40:49]

The CQ bit was checked by bbudge@chromium.org

[commit-bot: I haz the power, 2017-02-13 20:40:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-13 20:43:11]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1487018449471910,
"parent_rev": "11f88ef53f8e7c5aecfccd52e2a076a73efd413a", "commit_rev":
"a9b59a11f1bfe069afabe5567f919727456f1f12"}

[commit-bot: I haz the power, 2017-02-13 20:43:19]

Description was changed from

==========
Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124,v8:5948
R=bradnelson@chromium.org,bmeurer@chromium.org,jochen@chromium.org,hpayer@chr...

Review-Url: https://codereview.chromium.org/2684313003
Cr-Commit-Position: refs/heads/master@{#43162}
Committed:
https://chromium.googlesource.com/v8/v8/+/d170c57ab996d00c4665a9d865bd5754a18...
==========

to

==========
Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124,v8:5948
R=bradnelson@chromium.org,bmeurer@chromium.org,jochen@chromium.org,hpayer@chr...

Review-Url: https://codereview.chromium.org/2684313003
Cr-Original-Commit-Position: refs/heads/master@{#43162}
Committed:
https://chromium.googlesource.com/v8/v8/+/d170c57ab996d00c4665a9d865bd5754a18...
Review-Url: https://codereview.chromium.org/2684313003
Cr-Commit-Position: refs/heads/master@{#43169}
Committed:
https://chromium.googlesource.com/v8/v8/+/a9b59a11f1bfe069afabe5567f919727456...
==========

[commit-bot: I haz the power, 2017-02-13 20:43:21]

Committed patchset #7 (id:120001) as
https://chromium.googlesource.com/v8/v8/+/a9b59a11f1bfe069afabe5567f919727456...

[Franzi, 2017-02-13 21:33:07]

A revert of this CL (patchset #7 id:120001) has been created in
https://codereview.chromium.org/2695653005/ by franzih@chromium.org.

The reason for reverting is: Breaks Node integration build..

[Franzi, 2017-02-14 06:54:46]

Description was changed from

==========
Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124,v8:5948
R=bradnelson@chromium.org,bmeurer@chromium.org,jochen@chromium.org,hpayer@chr...

Review-Url: https://codereview.chromium.org/2684313003
Cr-Original-Commit-Position: refs/heads/master@{#43162}
Committed:
https://chromium.googlesource.com/v8/v8/+/d170c57ab996d00c4665a9d865bd5754a18...
Review-Url: https://codereview.chromium.org/2684313003
Cr-Commit-Position: refs/heads/master@{#43169}
Committed:
https://chromium.googlesource.com/v8/v8/+/a9b59a11f1bfe069afabe5567f919727456...
==========

to

==========
Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124,v8:5948
R=bradnelson@chromium.org,bmeurer@chromium.org,jochen@chromium.org,hpayer@chr...

Review-Url: https://codereview.chromium.org/2684313003
Cr-Original-Commit-Position: refs/heads/master@{#43162}
Committed:
https://chromium.googlesource.com/v8/v8/+/d170c57ab996d00c4665a9d865bd5754a18...
Review-Url: https://codereview.chromium.org/2684313003
Cr-Commit-Position: refs/heads/master@{#43169}
Committed:
https://chromium.googlesource.com/v8/v8/+/a9b59a11f1bfe069afabe5567f919727456...
==========

[Franzi, 2017-02-14 06:55:02]

The CQ bit was checked by franzih@chromium.org

[commit-bot: I haz the power, 2017-02-14 06:55:11]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-02-14 06:57:28]

CQ is committing da patch.

Bot data: {"patchset_id": 120001, "attempt_start_ts": 1487055302673180,
"parent_rev": "033d3b577b47a985d8720e48b725686249829c50", "commit_rev":
"deabb19abc3328149ed76990cdd3a1d5e4e5b3ed"}

[commit-bot: I haz the power, 2017-02-14 06:57:35]

Description was changed from

==========
Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124,v8:5948
R=bradnelson@chromium.org,bmeurer@chromium.org,jochen@chromium.org,hpayer@chr...

Review-Url: https://codereview.chromium.org/2684313003
Cr-Original-Commit-Position: refs/heads/master@{#43162}
Committed:
https://chromium.googlesource.com/v8/v8/+/d170c57ab996d00c4665a9d865bd5754a18...
Review-Url: https://codereview.chromium.org/2684313003
Cr-Commit-Position: refs/heads/master@{#43169}
Committed:
https://chromium.googlesource.com/v8/v8/+/a9b59a11f1bfe069afabe5567f919727456...
==========

to

==========
Remove SIMD.js from V8.

LOG=Y
BUG=v8:4124,v8:5948
R=bradnelson@chromium.org,bmeurer@chromium.org,jochen@chromium.org,hpayer@chr...

Review-Url: https://codereview.chromium.org/2684313003
Cr-Original-Original-Commit-Position: refs/heads/master@{#43162}
Committed:
https://chromium.googlesource.com/v8/v8/+/d170c57ab996d00c4665a9d865bd5754a18...
Review-Url: https://codereview.chromium.org/2684313003
Cr-Original-Commit-Position: refs/heads/master@{#43169}
Committed:
https://chromium.googlesource.com/v8/v8/+/a9b59a11f1bfe069afabe5567f919727456...
Review-Url: https://codereview.chromium.org/2684313003
Cr-Commit-Position: refs/heads/master@{#43176}
Committed:
https://chromium.googlesource.com/v8/v8/+/deabb19abc3328149ed76990cdd3a1d5e4e...
==========

[commit-bot: I haz the power, 2017-02-14 06:57:37]

Committed patchset #7 (id:120001) as
https://chromium.googlesource.com/v8/v8/+/deabb19abc3328149ed76990cdd3a1d5e4e...

[agrieve, 2017-02-14 20:40:58]

On 2017/02/14 06:57:37, commit-bot: I haz the power wrote:
> Committed patchset #7 (id:120001) as
>
https://chromium.googlesource.com/v8/v8/+/deabb19abc3328149ed76990cdd3a1d5e4e...

FYI - this shrank Chrome.apk by 468kb!

[bbudge, 2017-02-14 21:52:50]

On 2017/02/14 20:40:58, agrieve wrote:
> On 2017/02/14 06:57:37, commit-bot: I haz the power wrote:
> > Committed patchset #7 (id:120001) as
> >
>
https://chromium.googlesource.com/v8/v8/+/deabb19abc3328149ed76990cdd3a1d5e4e...
> 
> FYI - this shrank Chrome.apk by 468kb!

Wow! I didn't expect that much. There should be at least a small performance
gain as well.