Skip to content

Commit 4b6abf3

Browse files
committed
Add V8 issue reference
1 parent 5cec516 commit 4b6abf3

2 files changed

Lines changed: 11 additions & 3 deletions

File tree

docs/native_math_bugs.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
* [Trigonometric functions in V8][@chromium:320097]
77
* [V8 not IEEE 754-2008 compliant][@v8:3089]
88
* [Mozilla discussion on sine and cosine in V8][@bugzilla:967709]
9-
* [V8 replaced a lookup table by computing `Math.tan` as `Math.sin/Math.cos`][5]
9+
* [V8 replaced a lookup table by computing `Math.tan` as `Math.sin/Math.cos`][@chromium:78263005]
1010
* [Browser math accuracy issues][6]
1111
* [ES6 accuracy of special functions][7]
1212
* [Accuracy of `Math.exp` in V8][8]
@@ -35,7 +35,7 @@
3535
[@chromium:320097]: https://bugs.chromium.org/p/chromium/issues/detail?id=320097
3636
[@v8:3089]: https://bugs.chromium.org/p/v8/issues/detail?id=3089
3737
[@bugzilla:967709]: https://bugzilla.mozilla.org/show_bug.cgi?id=967709
38-
[5]: https://github.com/v8/v8/commit/33b5db090258c2a2dc825659c3ad109bd02110c1
38+
[@chromium:78263005]: https://github.com/v8/v8/commit/33b5db090258c2a2dc825659c3ad109bd02110c1
3939
[6]: https://github.com/kangax/compat-table/issues/392
4040
[7]: https://esdiscuss.org/topic/es6-accuracy-of-special-functions
4141
[8]: https://bugs.chromium.org/p/v8/issues/detail?id=3468

docs/references/bib.bib

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -743,10 +743,18 @@ @misc{v8:3089
743743
year = {2014}
744744
}
745745

746-
@misc{mozilla:967709,
746+
@misc{bugzilla:967709,
747747
abstract = {On the attached microbenchmark, which just pounds on sin with non-repeating values, V8 is about 2.8x faster on my Linux machine. (Our sin/cos just call in the C stdlib's sin/cos, so this is highly dependent on OS and stdlib version. I'd appreciate seeing what numbers other people get.) Profiling the box2d benchmark on awfy shows about 50% of its time is just calling sin/cos and this gives V8 better overall throughput on my machine. It looks like V8 rolls their own sin/cos (https://code.google.com/p/v8/source/detail?r=17594) which gives them more predictable performance. They self-host sin/cos which also avoids the call out from JIT code and all the overhead that that incurs. Since the sin/code code isn't all that complex, it seems like we could do even better with MSin/MCos MIR/LIR ops.},
748748
keywords = {mozilla, firefox, ff, math, sine, sin, cosine, cos, trig, bug, performance, perf},
749749
title = {{V8 is 2.8x faster at sin/cos}},
750750
url = {https://bugzilla.mozilla.org/show_bug.cgi?id=967709#c33},
751751
year = {2014}
752752
}
753+
754+
@misc{chromium:78263005,
755+
abstract = {Embed trigonometric lookup table.},
756+
keywords = {math, v8, chrome, sine, sin, cosine, cos, tan, tangent, trig, trigonometry, algorithm, algo},
757+
title = {{Reland: Embed trigonometric lookup table.}},
758+
url = {https://github.com/v8/v8/commit/33b5db090258c2a2dc825659c3ad109bd02110c1},
759+
year = {2013}
760+
}

0 commit comments

Comments
 (0)