[PATCH] D135245: [clang][Tooling] Move STL recognizer to its own library
Nico Weber via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 6 06:43:11 PDT 2022
thakis added a comment.
Thanks!
================
Comment at: clang/lib/Tooling/Inclusions/STL/CMakeLists.txt:2
+add_clang_library(clangToolingInclusionsSTL
+ StandardLibrary.cpp
+
----------------
kadircet wrote:
> sammccall wrote:
> > This means the implementation files and the header files have a different directory structure, which may be confusing to people trying to work out which library to link against based on the headers they included.
> >
> > On the other hand, I think the cascading effect of dependencies => libraries => directory structure => header structure is pretty unfortunate leaking of llvm's sad cmake structure. Up to you
> >
> >
> right, i was also torn between moving the headers around vs not. but i finally managed to convince myself that the implementation being in a different subdirectory is actually an unfortunate detail about the way LLVM is build (I didn't want to have PARTIAL_SOURCES_INTENDED, either) and shouldn't matter for the applications that want to use it.
FWIW, I'd move the header.
FWIW, I also don't think llvm's cmake setup is particularly unfortunate here either. It makes it easy to see where library boundaries are.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D135245/new/
https://reviews.llvm.org/D135245
More information about the cfe-commits
mailing list