[all-commits] [llvm/llvm-project] 79921f: [libc++][CI] Reenables clang-tidy. (#90077)

Mark de Wever via All-commits all-commits at lists.llvm.org
Wed May 8 11:18:20 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 79921fbd5c6223ff7e6c75ed75974b4d16cad529
      https://github.com/llvm/llvm-project/commit/79921fbd5c6223ff7e6c75ed75974b4d16cad529
  Author: Mark de Wever <koraq at xs4all.nl>
  Date:   2024-05-08 (Wed, 08 May 2024)

  Changed paths:
    M libcxx/modules/std/chrono.inc
    M libcxx/modules/std/ranges.inc
    M libcxx/test/CMakeLists.txt
    M libcxx/test/libcxx/clang_tidy.gen.py
    M libcxx/test/tools/clang_tidy_checks/CMakeLists.txt

  Log Message:
  -----------
  [libc++][CI] Reenables clang-tidy. (#90077)

The patch does several things:
- fixes module exports
- disables clang-tidy with Clang-17 due to known issues
- disabled clang-tidy on older libstdc++ versions since it lacks C++20
features used
- fixes the CMake dependency

The issue why clang-tidy was not used in the CI was the last issue; the
plugin was not a
dependency of the tests. Without a plugin the tests disable clang-tidy.

This was noticed while investigating
https://github.com/llvm/llvm-project/issues/89898



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list