[webkit-reviews] review requested: [Bug 75522] Enable IPP for FFTFrame : [Attachment 121794] Patch

bugzilla-daemon at webkit.org bugzilla-daemon at webkit.org
Mon Jan 9 22:21:27 PST 2012


xingnan.wang at intel.com has asked  for review:
Bug 75522: Enable IPP for FFTFrame
https://bugs.webkit.org/show_bug.cgi?id=75522

Attachment 121794: Patch
https://bugs.webkit.org/attachment.cgi?id=121794&action=review

------- Additional Comments from xingnan.wang at intel.com
Update the patch as the convolution layout test updated, new result:
Tests ConvolverNode processing a mono channel with mono impulse response.

On success, you will see a series of "PASS" messages, followed by "TEST
COMPLETE".

PASS Initial latency of convolver is silent.
PASS Triangular portion of convolution is correct.
FAIL First part of tail of convolution is not sufficiently small: 0.00244140625
dB
FAIL Test signal was not correctly convolved.
PASS successfullyParsed is true

TEST COMPLETE

Also the FFTRrameIPP::multiply() is updated to use zvmul() and zsmul() as bug
74842 fixed.


More information about the webkit-reviews mailing list