[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