Use GCC builtins in MathAlgorithms.h on mingw.

RESOLVED FIXED in mozilla37

Status

()

defect
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: jacek, Assigned: jacek)

Tracking

Trunk
mozilla37
x86_64
Windows 7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Posted patch patch.diffSplinter Review
MSVC-specific code has been enabled for mingw as a side effect of bug 1111255. It works fine except of unknown pragmas warnings. However, GCC branch seems nicer, so I chose that over #ifdefing pragmas.
Attachment #8539828 - Flags: review?(nfroyd)
Comment on attachment 8539828 [details] [diff] [review]
patch.diff

One of these days, I'll actually remember what gets enabled on mingw, and what doesn't, and we can stop breaking things like this.  Thanks for the patch!
Attachment #8539828 - Flags: review?(nfroyd) → review+
https://hg.mozilla.org/mozilla-central/rev/c0c165a61e9d
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla37
You need to log in before you can comment on or make changes to this bug.