[all-commits] [llvm/llvm-project] 2f5dc5: [IncludeCleaner] Also check for spellings of physi...

[kadir çetinkaya via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 2f5dc596b5719e5ed7f6978dafbce994f425a033
      https://github.com/llvm/llvm-project/commit/2f5dc596b5719e5ed7f6978dafbce994f425a033
  Author: kadir çetinkaya <kadircet at google.com>
  Date:   2024-07-22 (Mon, 22 Jul 2024)

  Changed paths:
    M clang-tools-extra/clangd/IncludeCleaner.cpp
    M clang-tools-extra/clangd/unittests/IncludeCleanerTests.cpp
    M clang-tools-extra/include-cleaner/lib/Analysis.cpp
    M clang-tools-extra/include-cleaner/unittests/AnalysisTest.cpp

  Log Message:
  -----------
  [IncludeCleaner] Also check for spellings of physical headers (#99843)

Some physical headers can have "conflicting" spellings, when same
filename exists under two different include search paths. e.g.
<stdlib.h>
can be provided by both standard library and underlying libc headers. In
such scenarios if the usage is from the latter include-search path,
users can't spell it directly.

This patch ensures we also consider spellings of such includes, in
addition to their physical files to prevent conflicting suggestions.



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