This is an archive of the discontinued LLVM Phabricator instance.

CMake: Consolidate gtest detection code
AbandonedPublic

Authored by tstellar on Aug 31 2018, 10:46 PM.

Details

Reviewers
chandlerc
beanz
Summary

This makes it possible to build the unittests with only the gtest sources
and without a full LLVM source tree.

Diff Detail

Event Timeline

tstellar created this revision.Aug 31 2018, 10:46 PM
chandlerc accepted this revision.Sep 4 2018, 3:17 AM

I mean, sure.

I really don't know that supporting this ever expanding diversity of build strategies is worth its cost, but I don't see a specific reason to not take this patch....

This revision is now accepted and ready to land.Sep 4 2018, 3:17 AM

I mean, sure.

I really don't know that supporting this ever expanding diversity of build strategies is worth its cost, but I don't see a specific reason to not take this patch....

I actually agree with you that we don't want to be supporting every possible strategy in trunk. I usually try to keep Fedora specific stuff like this out of trunk, but in this case it looked like a useful code simplification that might be generally helpful.

tstellar abandoned this revision.Nov 14 2019, 3:29 PM

Dropping this since it is more Fedora specific.

Herald added a project: Restricted Project. · View Herald TranscriptNov 14 2019, 3:29 PM