This is an archive of the discontinued LLVM Phabricator instance.

Remove legacy CMake targets for libcxx and libcxxabi
ClosedPublic

Authored by ldionne on Mar 12 2020, 3:24 PM.

Details

Summary

We've been meaning to remove those targets for a while, and the fix is
simple enough cause they're all just aliases to other targets.

There's no doubt this commit will break some CI systems, however the
fix is trivial.

Diff Detail

Event Timeline

ldionne created this revision.Mar 12 2020, 3:24 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald Transcript
Herald added a reviewer: Restricted Project. · View Herald Transcript
Herald added a reviewer: Restricted Project. · View Herald Transcript
phosek accepted this revision.Mar 12 2020, 3:39 PM
phosek added a subscriber: phosek.

LGTM

Just to clear things out since the review groups are new, I'm still waiting for a thumbs up from someone from the libc++ group (so @EricWF or @mclow.lists).

EricWF accepted this revision.Mar 21 2020, 4:59 PM
This revision is now accepted and ready to land.Mar 21 2020, 4:59 PM
This revision was automatically updated to reflect the committed changes.