Closed Bug 1192277 Opened 5 years ago Closed 5 years ago

remove check for LLVM PR 8927

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(firefox43 fixed)

RESOLVED FIXED
mozilla43
Tracking Status
firefox43 --- fixed

People

(Reporter: froydnj, Assigned: froydnj)

Details

Attachments

(1 file)

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+
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.