This is an archive of the discontinued LLVM Phabricator instance.

[libc] Do not add unittests LLVM_INCLUDE_TESTS is OFF.
ClosedPublic

Authored by sivachandra on Oct 9 2019, 1:37 PM.

Diff Detail

Event Timeline

sivachandra created this revision.Oct 9 2019, 1:37 PM
Herald added a project: Restricted Project. · View Herald TranscriptOct 9 2019, 1:37 PM

Hopefully this should fix the problem you reported here: https://reviews.llvm.org/D67867

Yes, it does! I can approve this if you need it but I am not an expect in cmake so I cannot comment on how proper this is.

nathanchance accepted this revision.Oct 13 2019, 6:59 PM
This revision is now accepted and ready to land.Oct 13 2019, 6:59 PM
This revision was automatically updated to reflect the committed changes.