configure.ac |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 5078591de9a0e65ca560a4f1913e90dfe95f66bf
Author:     Stephan Bergmann <sberg...@redhat.com>
AuthorDate: Wed Aug 3 16:55:07 2022 +0200
Commit:     Stephan Bergmann <sberg...@redhat.com>
CommitDate: Wed Aug 3 21:37:16 2022 +0200

    CLANGTOOLLIBS needs to include -lclangSupport now
    
    ...presumably since
    
<https://github.com/llvm/llvm-project/commit/7a5cb15ea6facd82756adafae76d60f36a0b60fd>
    "[RISCV] Lazily add RVV C intrinsics", otherwise causing
    
    > [GEN] compilerplugins/clang/sharedvisitor/analyzer
    > /usr/bin/ld: ~/llvm/inst/lib/libclangSema.a(SemaRISCVVectorLookup.cpp.o): 
in function `clang::CreateRISCVIntrinsicManager(clang::Sema&)':
    > 
~/github.com/llvm/llvm-project/clang/lib/Sema/SemaRISCVVectorLookup.cpp:195: 
undefined reference to 
`clang::RISCV::RVVIntrinsic::computeBuiltinTypes(llvm::ArrayRef<clang::RISCV::PrototypeDescriptor>,
 bool, bool, bool, unsigned int, bool, clang::RISCV::PolicyScheme, 
clang::RISCV::Policy)'
    [...]
    
    with recent Clang 16 trunk.
    
    It would have been nice to extend the configure check to actually catch 
that,
    but I didn't find any small code sample that would diagnose that:
    * Reverting 79ca031fb2e8802aa4383362f83306fad6895729 "make the clang plugins
      configure check faster" would cause the configure check to erroneously 
fail
      due to "no member named 'tooling' in namespace 'clang'".
    * And then replacing clang/AST/RecursiveASTVisitor.h with
      clang/Tooling/Tooling.h would make the configure check succeed, and not
      diagnose the above issue.
    
    Change-Id: I6b2eb38daaa1c514fb61652e5bd522f91aa67059
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137755
    Tested-by: Jenkins
    Reviewed-by: Stephan Bergmann <sberg...@redhat.com>

diff --git a/configure.ac b/configure.ac
index 15eaca30b2d4..e17c3a345d4e 100644
--- a/configure.ac
+++ b/configure.ac
@@ -8383,7 +8383,7 @@ if test "$COM_IS_CLANG" = "TRUE"; then
             AC_MSG_CHECKING([for clang libraries to use])
             if test -z "$CLANGTOOLLIBS"; then
                 LIBS="-lclangTooling -lclangFrontend -lclangDriver 
-lclangParse -lclangSema -lclangEdit \
- -lclangAnalysis -lclangAST -lclangLex -lclangSerialization -lclangBasic 
$COMPILER_PLUGINS_LINKFLAGS"
+ -lclangAnalysis -lclangAST -lclangLex -lclangSerialization -lclangBasic 
-lclangSupport $COMPILER_PLUGINS_LINKFLAGS"
                 AC_LINK_IFELSE([
                     AC_LANG_PROGRAM([[#include 
"clang/Basic/SourceLocation.h"]],
                         [[ clang::FullSourceLoc().dump(); ]])

Reply via email to