[all-commits] [llvm/llvm-project] 152f29: [clang][deps] Fix race condition

Jan Svoboda via All-commits all-commits at lists.llvm.org
Tue Feb 7 09:27:58 PST 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 152f2914607ffe632202ee660208d6dfba6c8870
      https://github.com/llvm/llvm-project/commit/152f2914607ffe632202ee660208d6dfba6c8870
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2023-02-07 (Tue, 07 Feb 2023)

  Changed paths:
    M clang/tools/clang-scan-deps/ClangScanDeps.cpp

  Log Message:
  -----------
  [clang][deps] Fix race condition

D140176 introduced new `FullDeps` API that's not thread-safe, breaking the class invariant. This was causing race condition when `clang-scan-deps` was run with multiple threads.

Reviewed By: steven_wu, akyrtzi

Differential Revision: https://reviews.llvm.org/D143428


  Commit: 5e14a48463b4c3eb3f36eeef30dee7312f9127da
      https://github.com/llvm/llvm-project/commit/5e14a48463b4c3eb3f36eeef30dee7312f9127da
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2023-02-07 (Tue, 07 Feb 2023)

  Changed paths:
    M clang/include/clang/Basic/FileEntry.h
    M clang/lib/Basic/FileManager.cpp

  Log Message:
  -----------
  [clang] NFC: Remove GCC 5.1 workaround in FileEntry

We no longer support GCC 5.1 (D122976), so let's remove the workaround in FileEntry.

Reviewed By: benlangmuir

Differential Revision: https://reviews.llvm.org/D143427


Compare: https://github.com/llvm/llvm-project/compare/140bc411b245...5e14a48463b4


More information about the All-commits mailing list