This is an archive of the discontinued LLVM Phabricator instance.

[flang] Harmonize standalone gtest support with other projects
ClosedPublic

Authored by mgorny on Nov 23 2022, 7:10 AM.

Details

Summary

Use the CMake rules of unittest libraries via add_subdirectory()
when using standalone builds, similarly to how e.g. Clang does that.
Besides making things more consistent, I hope this also unblocks D137035
and effectively we'll be able to simplify things further and have LLVM
semi-automatically include gtest support for us in the future.

Diff Detail

Event Timeline

mgorny created this revision.Nov 23 2022, 7:10 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 23 2022, 7:10 AM
mgorny requested review of this revision.Nov 23 2022, 7:10 AM
phosek accepted this revision.Nov 23 2022, 10:16 AM

LGTM

This revision is now accepted and ready to land.Nov 23 2022, 10:16 AM
awarzynski accepted this revision.Nov 23 2022, 12:55 PM

Thanks for all the clean-up! LGTM

Herald added a project: Restricted Project. · View Herald TranscriptNov 23 2022, 6:29 PM