This is an archive of the discontinued LLVM Phabricator instance.

[libcxx][CMake] Move the warning to HandleOutOfTreeLLVM
ClosedPublic

Authored by phosek on Dec 9 2016, 4:59 PM.

Details

Summary

This currently gives a warning when building libcxx under runtimes. When building as part of runtimes, the value of CMAKE_SOURCE_DIR is different from CMAKE_CURRENT_SOURCE_DIR, but runtimes build sets ${runtime}_STANDALONE_BUILD to 1 which triggers this warning. This was introduced by r286789.

Diff Detail

Repository
rL LLVM

Event Timeline

phosek updated this revision to Diff 80980.Dec 9 2016, 4:59 PM
phosek retitled this revision from to [libcxx][CMake] Move the warning to HandleOutOfTreeLLVM.
phosek updated this object.
phosek added reviewers: EricWF, beanz.
phosek set the repository for this revision to rL LLVM.
phosek added a subscriber: llvm-commits.
phosek added a comment.Dec 9 2016, 5:01 PM

On a related note, there seems to be difference in handling this part of configuration between libcxxabi and libunwind, and libcxx. For example, when libcxxabi or libunwind cannot find llvm-config they throw FATAL_ERROR while libcxx only prints WARNING and carries on. Would it make sense to unify this logic, and if so which is correct?

EricWF accepted this revision.Dec 10 2016, 11:49 PM
EricWF edited edge metadata.
This revision is now accepted and ready to land.Dec 10 2016, 11:49 PM
This revision was automatically updated to reflect the committed changes.