[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