[libc-commits] [PATCH] D141112: [libc] Add a separate install target for the libc static archives.

Siva Chandra via Phabricator via libc-commits libc-commits at lists.llvm.org
Fri Jan 6 15:31:38 PST 2023


sivachandra added inline comments.


================
Comment at: libc/CMakeLists.txt:166
   set(LIBC_COMPONENT libc)
-  set(LIBC_INSTALL_DEPENDS "libc;install-libc-headers;libc-startup")
+  set(libc_sub_install_targets "install-libc-static-archives;install-libc-headers")
+  if(NOT LIBC_TARGET_OS_IS_BAREMETAL)
----------------
michaelrj wrote:
> Nit: `libc_sub_install_targets` seems like it should be capitalized to match the other variables next to it
I have removed that intermediate variable.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D141112/new/

https://reviews.llvm.org/D141112



More information about the libc-commits mailing list