This is an archive of the discontinued LLVM Phabricator instance.

[compiler-rt] [test] [fuzzer] Don't pass msvc specific flags to mingw tools
ClosedPublic

Authored by mstorsjo on Apr 5 2023, 11:49 AM.

Details

Summary

This fixes building the tests so that the tests can start
executing (even if there still are lots of failures).

Diff Detail

Event Timeline

mstorsjo created this revision.Apr 5 2023, 11:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 5 2023, 11:49 AM
Herald added subscribers: Enna1, dberris. · View Herald Transcript
mstorsjo requested review of this revision.Apr 5 2023, 11:49 AM
Herald added a project: Restricted Project. · View Herald TranscriptApr 5 2023, 11:49 AM
Herald added a subscriber: Restricted Project. · View Herald Transcript
vitalybuka accepted this revision.Apr 5 2023, 11:55 AM
This revision is now accepted and ready to land.Apr 5 2023, 11:55 AM
vitalybuka requested changes to this revision.Apr 5 2023, 11:56 AM

What is about clang-cl?

This revision now requires changes to proceed.Apr 5 2023, 11:56 AM

What is about clang-cl?

When building with clang-cl, the cmake variable MSVC is set, since it’s considered roughly the same. (It’s considered the same “environment “.) Distinguishing between clang-cl and actual MSVC requires checking more specific variables.

vitalybuka accepted this revision.Apr 5 2023, 12:10 PM

What is about clang-cl?

When building with clang-cl, the cmake variable MSVC is set, since it’s considered roughly the same. (It’s considered the same “environment “.) Distinguishing between clang-cl and actual MSVC requires checking more specific variables.

Thanks, we are good if it's covered.

This revision is now accepted and ready to land.Apr 5 2023, 12:10 PM
phosek accepted this revision.Apr 5 2023, 7:23 PM

LGTM