[libcxx-commits] [PATCH] D117268: [libc++] Install clang-tidy in docker containers and create command db symlink

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Fri Jan 14 08:52:31 PST 2022


ldionne added a comment.

I'm wondering whether we should perhaps do this from the libc++ build itself, which might help resolve https://github.com/llvm/llvm-project/issues/45348 at the same time. Any thoughts?



================
Comment at: libcxx/utils/ci/run-buildbot:200
                    -DLIBUNWIND_TEST_CONFIG="llvm-libunwind-shared.cfg.in"
+    create-db-symlink
     check-runtimes
----------------
Mordante wrote:
> I wonder whether we need this in all configurations.
> Secondly let's make sure we export the compile commands in the configuration where we use this.
I think we need to, because the `clang-tidy` check is run in all configurations.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117268



More information about the libcxx-commits mailing list