Build cairo in unified mode

RESOLVED FIXED in mozilla28

Status

()

Core
Graphics
RESOLVED FIXED
5 years ago
4 years ago

People

(Reporter: Ehsan, Assigned: BenWa)

Tracking

Trunk
mozilla28
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Benoit, are you interested in taking this?
(Assignee)

Comment 1

5 years ago
FWIW I tried to unify cairo. It's easy but it doesn't really provide a win because it's c compilication and cairo doesn't include much stuff.
(Reporter)

Comment 2

5 years ago
Do you have numbers?  I would actually expect this to be much bigger win on C code since the ratio of the actual compilation time is probably lower relative to the (near constant/linear) overhead...
(Assignee)

Comment 3

5 years ago
No I don't. But it was easy to get working.
(Reporter)

Comment 4

5 years ago
(In reply to comment #3)
> No I don't. But it was easy to get working.

In that case, it would be nice if you can resurrect the patch!  :-)
(Assignee)

Comment 5

5 years ago
Created attachment 8334541 [details] [diff] [review]
unify cairo
Assignee: nobody → bgirard
Status: NEW → ASSIGNED
Attachment #8334541 - Flags: review?(ehsan)
(Assignee)

Comment 6

5 years ago
before:
real	0m2.146s
user	0m9.979s
sys	0m1.397s
after:
real	0m1.633s
user	0m6.482s
sys	0m0.473s
(Reporter)

Updated

5 years ago
Attachment #8334541 - Flags: review?(ehsan) → review+
https://hg.mozilla.org/mozilla-central/rev/970ef8123dc3
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla28

Updated

4 years ago
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.