Closed Bug 1253456 Opened 8 years ago Closed 8 years ago

Disable rust on win32

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(firefox47 fixed)

RESOLVED FIXED
mozilla47
Tracking Status
firefox47 --- fixed

People

(Reporter: rillian, Assigned: rillian)

References

Details

Attachments

(1 file)

Temporarily remove --enable-rust from win32 mozconfigs until we can address bug 1253202.
I'm not going to get a custom library build ready for testing in time for the next nightly, so work around the top-crash in the meantime by disabling the code.

This leaves rust enabled for debug builds for testing convenience.
Attachment #8726471 - Flags: review?(mshal)
Given that bug 1253202 was probably caused by the same thing bug 1252041 was caused by (using SSE2 instructions), then it's probably worth pointing out that the standard library has SSE2 enabled by default on all platforms I believe (e.g. including Linux/OSX). It looks like the only crash reports came from Windows, though? If any Linux/OSX user uses one of those old CPUs though they're likely to see the same crash.

Just FYI!
Comment on attachment 8726471 [details] [diff] [review]
Disable rust on win32

Looks fine. We can always disable it in other platforms if they exhibit the crash as well. It wasn't seen it all in win64?
Attachment #8726471 - Flags: review?(mshal) → review+
I believe x86_64 by definition has SSE2, it's just x86 that may or may not have SSE2. (I may be remembering wrong though)
OS X for x86 requires at least Core 2, so SSE2 is never an issue. I heard we have enabled SSE2 for Linux 32bit since long ago?
https://hg.mozilla.org/mozilla-central/rev/530047c97d83
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: