[all-commits] [llvm/llvm-project] 93b0bf: [clangd] [Modules] Fix to correctly handle module ...

[fleeting-xx via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 93b0bf635a287aac55f6da39f38c1cf257efa824
      https://github.com/llvm/llvm-project/commit/93b0bf635a287aac55f6da39f38c1cf257efa824
  Author: fleeting-xx <bakerdt at gmail.com>
  Date:   2025-06-06 (Fri, 06 Jun 2025)

  Changed paths:
    M clang-tools-extra/clangd/ModulesBuilder.cpp
    A clang-tools-extra/clangd/test/module_dependencies.test

  Log Message:
  -----------
  [clangd] [Modules] Fix to correctly handle module dependencies (#142828)

This is a re-application of llvm/llvm-project#142090 without the unit
test changes. A subsequent PR will follow that adds a unit test for
module dependencies.

### Changes
- Fix dangling string references in the return value of
getAllRequiredModules()
- Change a couple of calls in getOrBuildModuleFile() to use the loop
variable instead of the ModuleName parameter.

@ChuanqiXu9 for review



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