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

Repository
rL LLVM

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.