[all-commits] [llvm/llvm-project] eea232: [clang][tools] Remove path separator assumption

Jan Svoboda via All-commits all-commits at lists.llvm.org
Sat Sep 9 11:38:48 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: eea232da827abae1a4fd71a0fb1f4d239d7b2f6a
      https://github.com/llvm/llvm-project/commit/eea232da827abae1a4fd71a0fb1f4d239d7b2f6a
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2023-09-09 (Sat, 09 Sep 2023)

  Changed paths:
    M clang-tools-extra/include-cleaner/lib/FindHeaders.cpp

  Log Message:
  -----------
  [clang][tools] Remove path separator assumption

After 98e6deb6 the 'HeadersForSymbolTest.IWYUTransitiveExportWithPrivate' test in 'ClangIncludeCleanerTest' started failing. This is most likely because `FileEntryRef::getName()` now starts with ".\" on Windows, whereas `FileEntry::getName()` did not. This commit fixes assumption of forward slash separators.




More information about the All-commits mailing list