(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 Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/612377 Title: [needs-packaging] libjpeg-turbo -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs