Issue 2083473002: [wasm] Use the new TF operators for F64Cos, F64Sin, F64Tan, and F64Exp

Issue 2083473002: [wasm] Use the new TF operators for F64Cos, F64Sin, F64Tan, and F64Exp (Closed)

Created:
4 years, 6 months ago by ahaas

Modified:
4 years, 6 months ago

Reviewers:
Benedikt Meurer, bradnelson, bradn

CC:
v8-reviews_googlegroups.com

Base URL:
https://chromium.googlesource.com/v8/v8.git@master

Target Ref:
refs/pending/heads/master

Project:
v8

Visibility:
Public.

[wasm] Use the new TF operators for F64Cos, F64Sin, F64Tan, and F64Exp R=bmeurer@chromium.org, bradnelson@chromium.org BUG=v8:5086 Committed: https://crrev.com/cdf4d10d131532e273a3750b92a45f259450e2d7 Cr-Commit-Position: refs/heads/master@{#37094}

Created: 4 years, 6 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+8 lines, -84 lines)			Patch
M	src/assembler.h	View	1 chunk	+0 lines, -4 lines	0 comments	Download
M	src/assembler.cc	View	1 chunk	+0 lines, -36 lines	0 comments	Download
M	src/compiler/wasm-compiler.h	View	1 chunk	+0 lines, -4 lines	0 comments	Download
M	src/compiler/wasm-compiler.cc	View	2 chunks	+8 lines, -32 lines	0 comments	Download
M	src/external-reference-table.cc	View	1 chunk	+0 lines, -8 lines	0 comments	Download

Total messages: 9 (3 generated)

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2083473002/1

4 years, 6 months ago (2016-06-20 09:00:15 UTC) #4

commit-bot: I haz the power

Patchset 1 (id:??) landed as https://crrev.com/cdf4d10d131532e273a3750b92a45f259450e2d7 Cr-Commit-Position: refs/heads/master@{#37094}

4 years, 6 months ago (2016-06-20 09:29:22 UTC) #7

bradn

4 years, 6 months ago (2016-06-20 16:50:05 UTC) #9

Message was sent while issue was closed.

Nice indeed.
Is the plan that others like Atan will become operators as well eventually?