[all-commits] [llvm/llvm-project] e5f100: [clangd] [C++20] [Modules] Add modules suffix for ...

Chuanqi Xu via All-commits all-commits at lists.llvm.org
Mon Mar 24 18:43:35 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e5f100676ed478ec099d0c820648bfe1dd0b52d2
      https://github.com/llvm/llvm-project/commit/e5f100676ed478ec099d0c820648bfe1dd0b52d2
  Author: Chuanqi Xu <yedeng.yd at linux.alibaba.com>
  Date:   2025-03-25 (Tue, 25 Mar 2025)

  Changed paths:
    M clang-tools-extra/clangd/HeaderSourceSwitch.cpp
    M clang-tools-extra/clangd/unittests/HeaderSourceSwitchTests.cpp

  Log Message:
  -----------
  [clangd] [C++20] [Modules] Add modules suffix for 'Header' Source Switch (#131591)

Support the trivial "header"/source switch for module interfaces.

I initially thought the naming are bad and we should rename it. But
later I feel it is better to split patches as much as possible.

>From the codes it looks like there are problems. e.g., `isHeaderFile`.
But let's try to fix them in different patches.



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