This is an archive of the discontinued LLVM Phabricator instance.

[CMake] Use cxx-headers as a depedency for C++ headers
ClosedPublic

Authored by phosek on Jul 14 2018, 8:24 PM.

Details

Summary

We no longer pass CLANG_DEFAULT_CXX_STDLIB to the runtimes build
as it was causing issues so we can no longer use this variable. We
instead use cxx-headers as a dependency whenever this is available
since both XRay and libFuzzer are built as static libraries so this
is sufficient.

Diff Detail

Event Timeline

phosek created this revision.Jul 14 2018, 8:24 PM
Herald added subscribers: Restricted Project, llvm-commits, mgorny. · View Herald TranscriptJul 14 2018, 8:24 PM
dberris accepted this revision.Jul 15 2018, 5:23 PM

LGTM on the XRay side.

This revision is now accepted and ready to land.Jul 15 2018, 5:23 PM
morehouse accepted this revision.Jul 16 2018, 9:31 AM
This revision was automatically updated to reflect the committed changes.