[Bug 612377]
Swsnyder
612377 at bugs.launchpad.net
Mon Mar 28 15:14:14 UTC 2011
(In reply to comment #148)
> Comment on attachment 471713 [details]
> Patch v1.3.1
[snip]
> >diff --git a/jpeg/jcdctmgr.c b/jpeg/jcdctmgr.c
> >--- a/jpeg/jcdctmgr.c
> >+++ b/jpeg/jcdctmgr.c
> > /*
> >+ * Find the highest bit in an integer through binary search.
> >+ */
> >+LOCAL(int)
> >+flss (UINT16 val)
> >+{
>
> It would be good to use compiler intrinsics like BitScanReverse and
> __builtin_clz here when they're available.
[snip]
https://sourceforge.net/tracker/?func=detail&aid=3249446&group_id=303195&atid=1278160
It's an improvement in both speed and size, but probably not enough to
make a compelling case for inclusion of the patch. We've seen better
results in JS & NSPR because they both examine 32 bits. The libjpeg-
turbo code only looks at 16 bits so there are less savings to be had
from the BSR instruction.
--
You received this bug notification because you are a member of Mozilla
Bugs, which is subscribed to Mozilla.
https://bugs.launchpad.net/bugs/612377
Title:
[needs-packaging] libjpeg-turbo
More information about the Ubuntu-mozillateam-bugs
mailing list