[all-commits] [llvm/llvm-project] e32609: [clangd][modules] Provide correct context to Modul...

[Aleksandr Platonov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e3260943a9bf6f14b9f92cb30de40ac6cc4ae406
      https://github.com/llvm/llvm-project/commit/e3260943a9bf6f14b9f92cb30de40ac6cc4ae406
  Author: Aleksandr Platonov <platonov.aleksandr at huawei.com>
  Date:   2026-06-05 (Fri, 05 Jun 2026)

  Changed paths:
    M clang-tools-extra/clangd/ClangdServer.cpp
    M clang-tools-extra/clangd/ClangdServer.h
    A clang-tools-extra/clangd/test/modules_config_remove_flags.test

  Log Message:
  -----------
  [clangd][modules] Provide correct context to ModulesBuilder::hasRequiredModules() call (#201419)

To make command mangler to use compile command edits from the
configuration, we need to provide the correct context to it.
Without this patch compile command edits declared in .clangd file are
not used during required modules check, which can lead to compile errors
appearing and false negative `ModulesBuilder::hadRequiredModules()`
return result.

This PR addresses problem described here
https://github.com/llvm/llvm-project/pull/200001#issuecomment-4590514342



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications