awarzynski added a comment.
Hi @xgupta , thank you for working on this!
It's quite easy to miss dependencies when building with `BUILD_SHARED_LIBS=Off`
(i.e. with the default setting - static libs). IIUC, that's what happened here.
This example clearly requires the following:
- `ASTConsumer` implemented in `clangAST`
- `FullSourceLoc` implemented in `clangBasic`
- `ASTFrontendAction` implemented in `clangFrontend`
- `PCHContainerOperations` (required for the invocation of `runToolOnCode`)
implemented in `clangSerialization`
All the above were missing and your patch fixes that, thank you a ton! I think
that this is also missing (implements `llvm::outs()`):
set(LLVM_LINK_COMPONENTS
Support
)
I tested on Darwin with `BUILD_SHARED_LIBS=On` and I'm getting build errors
without it:
Undefined symbols for architecture x86_64:
"llvm::outs()", referenced from:
FindNamedClassVisitor::VisitCXXRecordDecl(clang::CXXRecordDecl*) in
FindClassDecls.cpp.o
PS Apologies for taking so much time to review this!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D95737/new/
https://reviews.llvm.org/D95737
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
