This is an archive of the discontinued LLVM Phabricator instance.

Stop setting LLVM_ON_WIN32 in config.h and llvm-config.h.
ClosedPublic

Authored by thakis on Apr 30 2018, 6:27 AM.

Details

Reviewers
rnk
ruiu
Summary

See thread "Replacing LLVM_ON_WIN32 with just _WIN32" on llvm-dev and cfe-dev.

I replaced all uses of LLVM_ON_WIN32 with _WIN32 in r331127 (llvm), r331069 (clang), r329697 (lldb), r329696 (lld), r329696 (clang-tools-extra).

If your out-of-tree program used LLVM_ON_WIN32, just use _WIN32 instead, which is set at exactly the same time to exactly the same value.

Diff Detail

Event Timeline

thakis created this revision.Apr 30 2018, 6:27 AM
ruiu accepted this revision.Apr 30 2018, 9:29 AM
ruiu added a subscriber: ruiu.

LGTM

This revision is now accepted and ready to land.Apr 30 2018, 9:29 AM
thakis closed this revision.Apr 30 2018, 1:23 PM

r331224, thanks!