Closed
Bug 1449834
Opened 7 years ago
Closed 6 years ago
Do not use Skia Assembly on x64 MinGW Build
Categories
(Core :: Graphics, enhancement, P5)
Core
Graphics
Tracking
()
RESOLVED
FIXED
People
(Reporter: tjr, Assigned: tjr)
References
Details
(Whiteboard: [tor][gfx-noted])
Attachments
(1 file)
In Bug 1364560 we fixed the Skia Build for MinGW which included not using the jumper assembly. It turns out that didn't quite work - there seems not to have been any jumper assembly for x86, so now that we're doing x64 we tried to use the assembly (and failed.) We need to actually be sure not to use that assembly. Lots of errors like this > [task 2018-03-28T16:33:03.657Z] 16:33:03 INFO - /builds/worker/workspace/build/src/gfx/skia/skia/src/jumper/SkJumper.cpp:250: undefined reference to `_sk_hsl_to_rgb_avx'
Comment hidden (mozreview-request) |
Comment 2•7 years ago
|
||
mozreview-review |
Comment on attachment 8963457 [details] Bug 1449834 Define SK_JUMPER_USE_ASSEMBLY as 0, not False https://reviewboard.mozilla.org/r/232382/#review237792 This is already fixed in beta and nightly as part of the Skia m66 update. The only build that should even hypothetically be affected by this anymore is release? If that is the case, backporting this is okay, I suppose.
Attachment #8963457 -
Flags: review?(lsalzman) → review+
Assignee | ||
Comment 3•7 years ago
|
||
Thanks! I am indeed behind tree because of the --disable-stylo removal. I am considering trying to get this backported to esr60 though.
Updated•6 years ago
|
Priority: -- → P5
Whiteboard: [tor] → [tor][gfx-noted]
Assignee | ||
Updated•6 years ago
|
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•