Update clang 6 pre to clang 6 final

RESOLVED FIXED

Status

enhancement
RESOLVED FIXED
Last year
8 months ago

People

(Reporter: glandium, Assigned: glandium)

Tracking

3 Branch
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

We have a bunch of build types using a trunk version of clang back when version 6 was still being worked on. Since then, clang 6 was released (a couple months ago), it's time to update those to use the final version.
Attachment #8976739 - Flags: review?(core-build-config-reviews) → review?(gps)
Comment on attachment 8976739 [details]
Bug 1462498 - Update clang 6 pre to clang 6 final on windows.

https://reviewboard.mozilla.org/r/244828/#review250888
Attachment #8976739 - Flags: review?(gps) → review+
Attachment #8976739 - Flags: review?(core-build-config-reviews)
Yeah, I know, I updated a lot... but all I did was remove 4 patches that were applied upstream between the point at which we were taking trunk and the actual clang 6 release.
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/autoland/rev/4523372c4945
Update clang 6 pre to clang 6 final. r=gps
Depends on: 1427808
Flags: needinfo?(mh+mozilla)
See Also: → 1466427
Keywords: leave-open
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/mozilla-inbound/rev/556f097431e3
Update clang 6 pre to clang 6 final on linux and mac. r=gps
Landed the part that doesn't depend on bug 1427808, and updated mozreview to only contain the part left for after bug 1427808 is fixed.
Depends on: 1466242
Hitting bug 1466242 this time. Re-refreshed the patch to the full extent of what it was ; we now have to wait for both 1466242 and bug 1427808.
Flags: needinfo?(mh+mozilla)
Keywords: leave-open
Pushed by mh@glandium.org:
https://hg.mozilla.org/integration/mozilla-inbound/rev/295900f1ae69
Update clang 6 pre to clang 6 final on linux and mac. r=gps
Retrying.
Keywords: leave-open
Looks like the linux/mac part is going to stick, so updating the patch for the remainder again.
Blocks: 1467658
Keywords: leave-open
Let's mark this as fixed: Linux/mac builds were upgraded, and Windows is going to upgrade to clang 7 after bug 
1427808.
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Version: Version 3 → 3 Branch
You need to log in before you can comment on or make changes to this bug.