This is an archive of the discontinued LLVM Phabricator instance.

[libc] Add a separate install target for the libc static archives.
ClosedPublic

Authored by sivachandra on Jan 6 2023, 12:47 AM.

Details

Summary

Also, skip installing startup objects for baremetal targets for now.

Diff Detail

Event Timeline

sivachandra created this revision.Jan 6 2023, 12:47 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptJan 6 2023, 12:47 AM
sivachandra requested review of this revision.Jan 6 2023, 12:47 AM
michaelrj accepted this revision.Jan 6 2023, 1:53 PM

LGTM with one minor nit

libc/CMakeLists.txt
166–172

Nit: libc_sub_install_targets seems like it should be capitalized to match the other variables next to it

This revision is now accepted and ready to land.Jan 6 2023, 1:53 PM

Remove an intermediate variable.

libc/CMakeLists.txt
166–172

I have removed that intermediate variable.

This revision was landed with ongoing or failed builds.Jan 6 2023, 3:32 PM
This revision was automatically updated to reflect the committed changes.