RESOLVED FIXED 127136
ArithCos in FTL 
https://bugs.webkit.org/show_bug.cgi?id=127136
Summary ArithCos in FTL
Matthew Mirman
Reported 2014-01-16 12:51:48 PST
patch forthcoming
Attachments
ArithCos patch (6.18 KB, patch)
2014-01-16 12:57 PST, Matthew Mirman 		sam: review-
DetailsFormatted DiffDiff
ArithCos Patch (15.92 KB, patch)
2014-01-20 13:21 PST, Matthew Mirman 		no flags
DetailsFormatted DiffDiff
ArithCos patch without the deleted change log (4.28 KB, patch)
2014-01-20 13:31 PST, Matthew Mirman 		fpizlo: review+
DetailsFormatted DiffDiff
Show Obsolete (2) View All Add attachment proposed patch, testcase, etc.
Matthew Mirman
Comment 1 2014-01-16 12:57:38 PST
Created attachment 221405 [details] ArithCos patch
Filip Pizlo
Comment 2 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. ;-)
Sam Weinig
Comment 3 2014-01-16 22:00:51 PST
Comment on attachment 221405 [details] ArithCos patch Looks like you uploaded both cos and sin.
Matthew Mirman
Comment 4 2014-01-20 13:21:12 PST
Created attachment 221686 [details] ArithCos Patch
Matthew Mirman
Comment 5 2014-01-20 13:31:26 PST
Created attachment 221687 [details] ArithCos patch without the deleted change log
Filip Pizlo
Comment 6 2014-01-20 13:41:09 PST
Comment on attachment 221687 [details] ArithCos patch without the deleted change log R=me, I will land.
Filip Pizlo
Comment 7 2014-01-20 13:57:24 PST
Landed in http://trac.webkit.org/changeset/162375
Note You need to log in before you can comment on or make changes to this bug.