Closed Bug 1423307 Opened 7 years ago Closed 7 years ago

Update Windows clang-cl builds to a newer upstream LLVM version to fix bustage with VS2017 15.5

Categories

(Firefox Build System :: General, enhancement)

enhancement
Not set
normal

Tracking

(firefox59 fixed)

RESOLVED FIXED
mozilla59
Tracking Status
firefox59 --- fixed

People

(Reporter: RyanVM, Assigned: RyanVM)

References

(Depends on 1 open bug)

Details

Attachments

(1 file)

With the current 5.0.0svn version we're bootstrapping, we hit a build failure with the new VC++ v14.12 toolset shipping in VS2017 15.5: C:\PROGRA~2\MIB055~1\2017\COMMUN~1\VC\Tools\MSVC\1412~1.258\include\type_traits:898:47: error: '_Ty' does not refer to a value Which in turns leads to a Rust panic and build failure. I've confirmed locally that updating to SVN tip (r319785) fixes this failure and allows me to build again. I'll work on cleaning up the patch a bit and more thoroughly test it with all our different build variants so we can make this the new default and unblock Visual Studio upgrades.
As an update, I tried a build off SVN rev 317840 (same as what we're using for Linux ASAN builds), and while it mostly seems to work OK on Try (and successfully builds m-c locally), I'm seeing Win32 static analysis build failures: https://treeherder.mozilla.org/logviewer.html#?job_id=150042241&repo=try Win64 builds are fine. Newer SVN revisions hit other static analysis bustage in clang-plugin.dll, but dmajor has a tentative fix that I'll try pushing on top of tomorrow.
This is the same revision we're currently using for Linux ASAN builds. Confirmed locally that this fixes the build issues with VS2017 15.5. https://treeherder.mozilla.org/#/jobs?repo=try&revision=bfe9d8401b8e6922238b2e3e65b12e4225a23074&group_state=expanded
Attachment #8935054 - Flags: review?(core-build-config-reviews)
Depends on: 1423667
Attachment #8935054 - Flags: review?(core-build-config-reviews) → review+
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/9072ae5dd4f8 Update clang-cl builds to LLVM revision 317840. r=ted
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Depends on: 1425983
Depends on: 1425984
Depends on: 1440983
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: