This is an archive of the discontinued LLVM Phabricator instance.

[CMake] Change target 'check' from 'check-llvm' to 'check-all'
ClosedPublic

Authored by MaskRay on May 29 2020, 11:18 AM.

Diff Detail

Event Timeline

MaskRay created this revision.May 29 2020, 11:18 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 29 2020, 11:18 AM
mehdi_amini accepted this revision.May 29 2020, 11:45 AM
mehdi_amini added a subscriber: mehdi_amini.

LGTM: I assume this is no change when only LLVM is configured, but it provides a better default otherwise right?

This revision is now accepted and ready to land.May 29 2020, 11:45 AM
echristo accepted this revision.May 29 2020, 12:16 PM

That's my perspective too and seems to make sense. People that need just check-llvm will have to be explicit now.

This revision was automatically updated to reflect the committed changes.