This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Use generator expression to simplify the CMake code
ClosedPublic

Authored by ldionne on Feb 23 2021, 3:27 PM.

Details

Summary

A comment was left for when we would require CMake >= 3, which we do now.
I expect this should be a NFC.

Diff Detail

Event Timeline

ldionne created this revision.Feb 23 2021, 3:27 PM
ldionne requested review of this revision.Feb 23 2021, 3:27 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 23 2021, 3:27 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante added inline comments.
libcxx/src/CMakeLists.txt
382

Buildkite fails with " Target "cxx" is not an executable or library.", so seem the target is no longer valid.

ldionne updated this revision to Diff 326457.Feb 25 2021, 11:43 AM
ldionne marked an inline comment as done.

Fix incorrect target name

Mordante accepted this revision.Mar 3 2021, 9:55 AM

Since we require CMake > 3 and the build passes, LGTM.

ldionne accepted this revision as: Restricted Project.Mar 3 2021, 9:59 AM
This revision is now accepted and ready to land.Mar 3 2021, 9:59 AM