[all-commits] [llvm/llvm-project] 48884d: [clang][Modules] Respect case sensitivity when has...

[Cyndy Ishida via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 48884d2ddba0718960f68eb01a5b8c42c089f27c
      https://github.com/llvm/llvm-project/commit/48884d2ddba0718960f68eb01a5b8c42c089f27c
  Author: Cyndy Ishida <cyndy_ishida at apple.com>
  Date:   2026-09-25 (Fri, 25 Sep 2026)

  Changed paths:
    M clang/lib/Lex/HeaderSearch.cpp
    A clang/test/ClangScanDeps/modules-context-hash-case-sensitive-vfs-paths.c
    M clang/test/ClangScanDeps/modules-context-hash-module-map-path.c
    A clang/test/Modules/implicit-module-map-path-case-sensitive.c

  Log Message:
  -----------
  [clang][Modules] Respect case sensitivity when hashing implicit module map path (#226339)

Before this patch, the implicit module hash normalized the module map
path, but explicit modules always respected the case in paths. This
worked by depending on FileEntry caching and/or relocation checking to
resolve correctly. This behavior was both subtle and difficult to
understand. Instead, respect the casing in paths for hashing to match
explicitly built modules so they can never diverge.

This patch also updates `modules-context-hash-module-map-path.c` to
match a downstream version that already tested for the case where vfs
overlay entries differ by case but point to the same file.
`modules-context-hash-case-sensitive-vfs-paths.c` is the same but points
to different underlying files.



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