remove check for LLVM PR 8927

RESOLVED FIXED in Firefox 43

Status

defect
RESOLVED FIXED
4 years ago
Last year

People

(Reporter: froydnj, Assigned: froydnj)

Tracking

unspecified
mozilla43

Firefox Tracking Flags

(firefox43 fixed)

Details

Attachments

(1 attachment)

The PR was fixed in early 2011.  clang 3.3, the oldest version of clang
that we support, was released in mid-2013.  It's safe to say that all
versions of clang now have this fix, and we can delete the check.
Ask and ye shall receive! :)
Attachment #8644993 - Flags: review?(mh+mozilla)
Comment on attachment 8644993 [details] [diff] [review]
remove check for LLVM PR 8927

Review of attachment 8644993 [details] [diff] [review]:
-----------------------------------------------------------------

s/that we support/that we build with/

AFAIK we don't have an explicit supported version of clang (although we should).
Attachment #8644993 - Flags: review?(mh+mozilla) → review+
https://hg.mozilla.org/mozilla-central/rev/683f439ef93c
https://hg.mozilla.org/mozilla-central/rev/7c2524bbcb40
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla43

Updated

Last year
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.