Closed Bug 475225 Opened 11 years ago Closed 9 years ago

Add JPEG optimization for x86_64 platform

Categories

(Core :: ImageLib, defect)

x86_64
All
defect
Not set

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: m_kato, Assigned: m_kato)

References

Details

Attachments

(1 file)

Currently, JPEG SSE2 code is only Windows x86.  I will add it for x86_64 platform.

After adding it, JPEG decoding is 15% faster.
Summary: Add JPEG optimazation ofr x86_64 platform → Add JPEG optimization for x86_64 platform
Attached patch patch v1Splinter Review
tested on Windows x64 and Ubuntu 8.10 x86_64.

After testing CentOS 5.2, I will send a review.
Attachment #359419 - Flags: review?(pavlov)
I'm working on a bug with the Intel SSE2 code right now (it also exists with the MMX code) so it might be worthwhile to hold off on this. There isn't a bugzilla entry for this yet (to my knowledge) but I'll enter one if the reporter doesn't do so soon. I'm assuming that this code was derived from the Intel code.
Attachment #359419 - Flags: review?(pavlov)
Comment on attachment 359419 [details] [diff] [review]
patch v1

Jeff, you have looked at Michael's SSE work in bug 477728 - you should look at this too. :)
Attachment #359419 - Flags: review?(jmuizelaar)
Attachment #359419 - Flags: review?(jmuizelaar) → review-
Comment on attachment 359419 [details] [diff] [review]
patch v1

I think we want to port the code in bug 477728 instead of the existing code.
Depends on: 477728
Depends on: 573948
Did the work on libjpeg-turbo make this bug obsolete?
(In reply to comment #5)
> Did the work on libjpeg-turbo make this bug obsolete?

Oh, Yes.  bug 646704 is landed, so this should be fixed.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → WORKSFORME
You need to log in before you can comment on or make changes to this bug.