Closed
Bug 1516605
Opened 5 years ago
Closed 5 years ago
Android/aarch64 should use NEON on libpng
Categories
(Core :: Graphics: ImageLib, enhancement)
Tracking
()
RESOLVED
FIXED
mozilla66
| Tracking | Status | |
|---|---|---|
| firefox66 | --- | fixed |
People
(Reporter: m_kato, Assigned: m_kato)
Details
Attachments
(1 file)
LibPNG has NEON support using intrinsics, not assembler, so we should use it. (MSVC support is already in upstream, but it isn't in stable version)
|
Assignee | |
Updated•5 years ago
|
Summary: Androidd/aarch64 should use NEON on libpng → Android/aarch64 should use NEON on libpng
|
|
Assignee | |
Comment 1•5 years ago
|
||
libpng has NEON support using intrinsics, not assembler, so we should use it even if aarch64. MSVC build still turns off NEON since libpng doesn't support it on stable version although upstream has it.
Pushed by m_kato@ga2.so-net.ne.jp: https://hg.mozilla.org/integration/autoland/rev/129f8c60783c Turn on NEON on aarch64 when using gcc or clang. r=aosmond
|
||
Comment 3•5 years ago
|
||
Oh, I actually already did this in the libpng 1.6.36 update bug (bug 1513236)...
|
||
Comment 4•5 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/129f8c60783c
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla66
You need to log in
before you can comment on or make changes to this bug.
Description
•