Unconditionally uses SSE2 instructions on x86

RESOLVED FIXED in Firefox -esr60

Status

()

defect
P1
normal
RESOLVED FIXED
10 months ago
10 months ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

unspecified
mozilla64
Points:
---

Firefox Tracking Flags

(firefox-esr60 fixed, firefox64 fixed)

Details

Attachments

(1 attachment)

While Mozilla.org builds of Firefox dropped support for x86 CPUs without SSE2, downstreams may support it. This is generally handled gracefully by having runtime detection, and many things do it in Gecko. One that doesn't, though, is the Swizzle code.
The same is done for NEON, and the check folds to nothing when building
when the build target supports SSE2 and runtime detection is not
necessary.
Comment on attachment 9009873 [details]
Bug 1492065 - Use Swizzle fallback when SSE2 is not supported

Lee Salzman [:lsalzman] has approved the revision.
Attachment #9009873 - Flags: review+
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/210b9e777cec
Use Swizzle fallback when SSE2 is not supported r=lsalzman
Priority: -- → P1
Comment on attachment 9009873 [details]
Bug 1492065 - Use Swizzle fallback when SSE2 is not supported

[Approval Request Comment]
If this is not a sec:{high,crit} bug, please state case for ESR consideration: Necessary for some downstreams.
User impact if declined: Downstreams such as Debian provide OSes that work on older x86 machines, and as such, try to keep Firefox working on those machines. For users with such machines (and there are surprisingly lots of them), Firefox crashes without this patch.
Fix Landed on Version: 64
Risk to taking this patch (and alternatives if risky): For Firefox builds that Mozilla ships, the patch actually is a no-op: the added conditions turn into "if (true)". For downstreams like Debian, this just ensures SSE2 instructions are not used on machines without SSE2.
String or UUID changes made by this patch: N/A

See https://wiki.mozilla.org/Release_Management/ESR_Landing_Process for more info.
Attachment #9009873 - Flags: approval-mozilla-esr60?
https://hg.mozilla.org/mozilla-central/rev/210b9e777cec
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla64
Comment on attachment 9009873 [details]
Bug 1492065 - Use Swizzle fallback when SSE2 is not supported

Fix for some downstream distros. Approved for ESR 60.3.
Attachment #9009873 - Flags: approval-mozilla-esr60? → approval-mozilla-esr60+
You need to log in before you can comment on or make changes to this bug.