[all-commits] [llvm/llvm-project] 87729b: [clangd] Migrate away from std::nullopt (NFC) (#14...

Kazu Hirata via All-commits all-commits at lists.llvm.org
Thu Jun 26 08:41:18 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 87729bcbb817123b50eb12f168d359c4607af16d
      https://github.com/llvm/llvm-project/commit/87729bcbb817123b50eb12f168d359c4607af16d
  Author: Kazu Hirata <kazu at google.com>
  Date:   2025-06-26 (Thu, 26 Jun 2025)

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

  Log Message:
  -----------
  [clangd] Migrate away from std::nullopt (NFC) (#145841)

ArrayRef has a constructor that accepts std::nullopt.  This
constructor dates back to the days when we still had llvm::Optional.

Since the use of std::nullopt outside the context of std::optional is
kind of abuse and not intuitive to new comers, I would like to move
away from the constructor and eventually remove it.

This patch replaces std::nullopt with {}.



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