Bug 127136 - ArithCos in FTL
Summary: ArithCos in FTL
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: 528+ (Nightly build)
Hardware: All All
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-01-16 12:51 PST by Matthew Mirman
Modified: 2014-01-20 13:57 PST (History)
1 user (show)

See Also:


Attachments
ArithCos patch (6.18 KB, patch)
2014-01-16 12:57 PST, Matthew Mirman
sam: review-
Details | Formatted Diff | Diff
ArithCos Patch (15.92 KB, patch)
2014-01-20 13:21 PST, Matthew Mirman
no flags Details | Formatted Diff | Diff
ArithCos patch without the deleted change log (4.28 KB, patch)
2014-01-20 13:31 PST, Matthew Mirman
fpizlo: review+
Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Matthew Mirman 2014-01-16 12:51:48 PST
patch forthcoming
Comment 1 Matthew Mirman 2014-01-16 12:57:38 PST
Created attachment 221405 [details]
ArithCos patch
Comment 2 Filip Pizlo 2014-01-16 13:35:06 PST
Comment on attachment 221405 [details]
ArithCos patch

View in context: https://bugs.webkit.org/attachment.cgi?id=221405&action=review

> Source/JavaScriptCore/ChangeLog:37
> +2014-01-16  Matthew Mirman  <mmirman@apple.com>
> +
> +        ArithSin in FTL
> +        https://bugs.webkit.org/show_bug.cgi?id=127129
> +
> +        Reviewed by NOBODY (OOPS!).
> +
> +        * ftl/FTLCapabilities.cpp:
> +        (JSC::FTL::canCompile):
> +        * ftl/FTLIntrinsicRepository.h:
> +        * ftl/FTLLowerDFGToLLVM.cpp:
> +        (JSC::FTL::LowerDFGToLLVM::compileNode):
> +        (JSC::FTL::LowerDFGToLLVM::compileArithSin):
> +        * ftl/FTLOutput.h:
> +        (JSC::FTL::Output::doubleSin):
> +        * tests/stress/ftl-arithsin.js: Added.
> +

You should rebase. ;-)
Comment 3 Sam Weinig 2014-01-16 22:00:51 PST
Comment on attachment 221405 [details]
ArithCos patch

Looks like you uploaded both cos and sin.
Comment 4 Matthew Mirman 2014-01-20 13:21:12 PST
Created attachment 221686 [details]
ArithCos Patch
Comment 5 Matthew Mirman 2014-01-20 13:31:26 PST
Created attachment 221687 [details]
ArithCos patch without the deleted change log
Comment 6 Filip Pizlo 2014-01-20 13:41:09 PST
Comment on attachment 221687 [details]
ArithCos patch without the deleted change log 

R=me, I will land.
Comment 7 Filip Pizlo 2014-01-20 13:57:24 PST
Landed in http://trac.webkit.org/changeset/162375