[all-commits] [llvm/llvm-project] 061eb6: [Clang] [NFC] Remove default argument in ASTUnit.h...

Sirraide via All-commits all-commits at lists.llvm.org
Fri Jan 19 05:29:35 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 061eb62a9051ed06935135ffa3ea03eb250fd55e
      https://github.com/llvm/llvm-project/commit/061eb62a9051ed06935135ffa3ea03eb250fd55e
  Author: Sirraide <74590115+Sirraide at users.noreply.github.com>
  Date:   2024-01-19 (Fri, 19 Jan 2024)

  Changed paths:
    M clang/include/clang/Frontend/ASTUnit.h
    M clang/tools/libclang/CIndexCodeCompletion.cpp

  Log Message:
  -----------
  [Clang] [NFC] Remove default argument in ASTUnit.h (#78566)

This removes a default argument that is currently broken in C++23 mode
due to `std::default_delete` now being `constexpr`. This is a known
problem (see #74963, #59966, #69996, and a couple more), fixing which
will probably take some time, so this at least makes it possible to
compile `ASTUnit.h` in C++23 mode.

Note that we can’t simply include the header that provides the
definition of the class causing the problem either, as that would create
a circular dependency.




More information about the All-commits mailing list