[all-commits] [llvm/llvm-project] 6ac3fd: [clangd] Fix race in Global CDB shutdown

Sam McCall via All-commits all-commits at lists.llvm.org
Tue Feb 2 06:24:49 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6ac3fd9706047304c52a678884122a3a6bc55432
      https://github.com/llvm/llvm-project/commit/6ac3fd9706047304c52a678884122a3a6bc55432
  Author: Sam McCall <sam.mccall at gmail.com>
  Date:   2021-02-02 (Tue, 02 Feb 2021)

  Changed paths:
    M clang-tools-extra/clangd/GlobalCompilationDatabase.cpp

  Log Message:
  -----------
  [clangd] Fix race in Global CDB shutdown

I believe the atomic write can be reordered after the notify, and that
seems to be happening on mac m1: http://45.33.8.238/macm1/2654/step_8.txt
In practice maybe seq_cst is enough? But no reason not to lock here.

https://bugs.llvm.org/show_bug.cgi?id=48998




More information about the All-commits mailing list