This is an archive of the discontinued LLVM Phabricator instance.

Make all x86 target builds on MSVC use the amd64_x86 toolchain
ClosedPublic

Authored by zturner on Jan 22 2016, 10:54 AM.

Details

Summary

x86 builds of libclang.dll are failing with out of memory errors (Example: http://lab.llvm.org:8011/builders/lldb-x86-windows-msvc/builds/14604). This patch is a global change against every MSVC build running on zorg to use the x64 -> x86 cross compilation toolchain when the target is x86, so that the linker will never run out of memory.

This amd64_x86 toolchain was introduced in VS2013 so it's guaranteed to be present on all versions of MSVC that we support.

This was supposed to go to llvm-commits in addition to lldb-commits. Updated subscriber list.

Diff Detail

Event Timeline

zturner updated this revision to Diff 45706.Jan 22 2016, 10:54 AM
zturner retitled this revision from to Make all x86 target builds on MSVC use the amd64_x86 toolchain.
zturner updated this object.
zturner added a reviewer: gkistanova.
zturner added a subscriber: lldb-commits.
zturner updated this object.Jan 22 2016, 4:34 PM
zturner added a subscriber: llvm-commits.
cray added a subscriber: cray.Jan 22 2016, 8:36 PM

The amd64_x86 toolchain is not present on the community edition of visual studio. I am not sure if any build bots use this version though.

gkistanova accepted this revision.Jan 25 2016, 2:33 PM
gkistanova edited edge metadata.

LGTM

Thanks

Galina

This revision is now accepted and ready to land.Jan 25 2016, 2:33 PM
Eugene.Zelenko added a subscriber: Eugene.Zelenko.

Committed in r258738.