[Webkit-unassigned] [Bug 79739] Use IPP malloc for AudioArray

Tue Feb 28 18:37:51 PST 2012

https://bugs.webkit.org/show_bug.cgi?id=79739

--- Comment #5 from Chris Rogers <crogers at google.com>  2012-02-28 18:37:51 PST ---
(In reply to comment #4)
> (In reply to comment #3)
> > Will 32-byte alignment get us any advantage over the 16-byte alignment?
> 
> I think 32-byte and 16-byte alignment are not too much different here, I have to use it because the IPP only provides 32-byte aligned ippmalloc function. The advantage we can get is the aligned memory is more easily allocated than original way.

I'm not sure what you mean by "more easily allocated".  I understand that the run-time code path in the IPP case would be slightly simpler in AudioArray.h, but is it actually a measurable performance win?
The disadvantage of having this special-casing is that the code is somewhat harder to read with this patch.  And this particular file is already somewhat tweaky even as it is right now, so it would be great to avoid adding additional complexity (harder to read through the code) unless there's a pretty good win.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.