This is an archive of the discontinued LLVM Phabricator instance.

Use a sufficiently current MSVCRT when using MinGW
ClosedPublic

Authored by jsroemer on Jan 28 2015, 1:52 AM.

Details

Reviewers
Bigcheese
Summary

The patch finds all installed MSVCRT versions from 11.0 (highest currently supported by MinGW) down to 9.0 and sets the appropriate compiler and linker flags for the newest version found. An installed MSVCRT is required due to the call to llvm-tblgen at build time.

Diff Detail

Repository
rL LLVM

Event Timeline

jsroemer updated this revision to Diff 18879.Jan 28 2015, 1:52 AM
jsroemer retitled this revision from to Use a sufficiently current MSVCRT when using MinGW.
jsroemer updated this object.
jsroemer edited the test plan for this revision. (Show Details)
jsroemer set the repository for this revision to rL LLVM.
jsroemer added subscribers: Unknown Object (MLST), rnk, Bigcheese.
Bigcheese accepted this revision.Jan 28 2015, 4:12 AM
Bigcheese added a reviewer: Bigcheese.

lgtm

This revision is now accepted and ready to land.Jan 28 2015, 4:12 AM

Thanks for the review. As I do not have commit access someone else will have to land the patch.

Bigcheese closed this revision.Jan 30 2015, 7:34 AM

I've committed a different fix that removes the need for this. Thanks for the patch though.

I assume you are referring to http://reviews.llvm.org/D7230?

Is it really desirable to disable the functionality for MinGW? Any recent MinGW is perfectly capable of compiling it with the right flags. My issue was only that running the generated executables requires the corresponding runtimes to be installed (also true for MSVC, but if that is installed the runtime is automatically installed as well)