-
fpizlo@apple.com authored
https://bugs.webkit.org/show_bug.cgi?id=123574 Source/JavaScriptCore: Reviewed by Mark Hahnenberg. This is performance-neutral because I also make Math.cos/sin intrinsic. This means that we gain the "overhead" of actually computing sin and cos but we lose the overhead of going through the native call thunks. Caching transcendental functions is a really ugly idea. It works for SunSpider because that benchmark makes very predictable calls into Math.sin. But I don't believe that this is representative of any kind of reality, and so for sensible uses of Math.sin/cos all that this was doing was adding more call overhead and some hashing overhead. * JavaScriptCore.xcodeproj/project.pbxproj: * dfg/DFGAbstractInterpreterInlines.h: (JSC::DFG::::executeEffects): * dfg/DFGBackwardsPropagationPhase.cpp: (JSC::DFG::BackwardsPropagationPhase::propagate): * dfg/DFGByteCodeParser.cpp: (JSC::DFG::ByteCodeParser::handleI...
b3336c7b