Bug 70626

Summary: Flush denormals in Biquad, ZeroPole, and DynamicsCompressor
Product: WebKit Reporter: Raymond Toy <rtoy>
Component: New BugsAssignee: Raymond Toy <rtoy>
Status: RESOLVED FIXED    
Severity: Normal CC: crogers, kbr, rtoy, webkit.review.bot
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Raymond Toy
Reported 2011-10-21 11:35:30 PDT
Flush denormals in Biquad, ZeroPole, and DynamicsCompressor
Attachments
Patch (4.27 KB, patch)
2011-10-21 11:38 PDT, Raymond Toy 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Raymond Toy
Comment 1 2011-10-21 11:38:13 PDT
Created attachment 111996 [details] Patch
Raymond Toy
Comment 2 2011-10-21 11:44:53 PDT
This is an extension of the previous patch for flushing denormals. Chris suggested that the Biquad, ZeroPole, and DynamicsCompressor have the state variables checked for denormals and flushed so that they don't propagate further within the class.
Chris Rogers
Comment 3 2011-10-21 11:57:20 PDT
This looks fine to me.
Kenneth Russell
Comment 4 2011-10-21 12:53:00 PDT
Comment on attachment 111996 [details] Patch Looks fine.
WebKit Review Bot
Comment 5 2011-10-21 14:03:49 PDT
Comment on attachment 111996 [details] Patch Clearing flags on attachment: 111996 Committed r98144: <http://trac.webkit.org/changeset/98144>
WebKit Review Bot
Comment 6 2011-10-21 14:03:56 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.