454797 - WinCE GFX Cairo No MMX Support in ARM

Status: RESOLVED FIXED

(Core :: Graphics, defect)

Description

[John Wolfe (:wolfe)]

Attachment: Disables MMX support for WinCE builds [Checkin: Comment 5]

WinCE ARM devices have no MMX support - this patch disables MMX support for WinCE builds.

Comment 1

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Oh wow, I had completely forgotten we were using MSVC-on-ARM there.  So that patch is fine, but I think we'll also want the pixman-arm bits (especially there, where our rendering will be software).  But those bits are using inline gcc assembly -- could you work with Jeff to get those converted into MSVC inline assembly, behind #ifdefs?

Comment 2

[Jeff Muizelaar [:jrmuizel]]

As far as I can tell MSVC on ARM does not support inline assembly this means we'll have to target 'armasm' which will be harder.

Comment 3

[Vladimir Vukicevic [:vlad] [:vladv] (needinfo me, slow to respond)]

Harder? You mean EXCITING!

Comment 4

[Jeff Muizelaar [:jrmuizel]]

You're right, I do mean that.

Comment 5

[Serge Gautherie (:sgautherie)]

Comment on attachment 338099 [details] [diff] [review]
Disables MMX support for WinCE builds
[Checkin: Comment 5]

http://hg.mozilla.org/mozilla-central/rev/e3f57277e577

***

Note that patch should have Unix EOL.