stylo: Check what the status of LTO for our release builds is.

NEW
Unassigned

Status

()

P4
normal
2 years ago
a year ago

People

(Reporter: emilio, Unassigned)

Tracking

(Blocks: 1 bug)

unspecified
Points:
---

Firefox Tracking Flags

(firefox57 wontfix)

Details

(Reporter)

Description

2 years ago
We should take a look, if we are able to get our FFI calls inlined, that'd be a win in runtime performance, as well as possibly in code size (though a lighter one arguably).
Priority: -- → P3
Priority: P3 → P4
Emilio, do we need to do anything to enable LTO for Stylo code? Or is this bug just about verifying the current output of LTO code?
No longer blocks: 1243581
Flags: needinfo?(emilio)
(Reporter)

Comment 2

a year ago
Well, this was about being able to inline FFI calls, which AIUI is just not possible with MSVC, and would be a significant effort building FF with clang.

So there's a bunch of work to do here if we want to make this doable, but we figured out other solutions at the potential perf issues that we were facing a year ago.

Still would be nice, of course.
Flags: needinfo?(emilio)
status-firefox57=wontfix unless someone thinks this bug should block 57
status-firefox57: --- → wontfix
You need to log in before you can comment on or make changes to this bug.