[PATCH] D61541: [compiler-rt] Create install targets for Darwin libraries
Shoaib Meenai via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 6 15:59:12 PDT 2019
smeenai marked an inline comment as done.
smeenai added inline comments.
================
Comment at: compiler-rt/cmake/Modules/CompilerRTUtils.cmake:423
+
+ if(ARG_PARENT_TARGET AND NOT TARGET install-${ARG_PARENT_TARGET})
+ # The parent install target specifies the parent component to scrape up
----------------
phosek wrote:
> IIUC, only the first invocation of this function will create the parent target and all the following ones will reuse it (for all systems/architectures with the same parent target that is). One possible alternative would be to split this into its function to avoid all those extra checks on subsequent invocations of this function.
Hmm, so the split function would just create the install-$parent targets? How would we ensure the split function gets called for each $parent without doing some sort of existence check?
Note that I've copied this logic from similar logic in AddCompilerRT.cmake: https://github.com/llvm/llvm-project/blob/491746a584723c147f8910c1ad5051c507afd735/compiler-rt/cmake/Modules/AddCompilerRT.cmake#L237
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D61541/new/
https://reviews.llvm.org/D61541
More information about the llvm-commits
mailing list