[all-commits] [llvm/llvm-project] f88b50: [FileCollector] Fix that the file system case-sens...

Raphael Isemann via All-commits all-commits at lists.llvm.org
Tue Feb 16 11:22:07 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: f88b502d9bc7c2d7db8db340d5b98fc7f46eba9c
      https://github.com/llvm/llvm-project/commit/f88b502d9bc7c2d7db8db340d5b98fc7f46eba9c
  Author: Raphael Isemann <teemperor at gmail.com>
  Date:   2021-02-16 (Tue, 16 Feb 2021)

  Changed paths:
    A lldb/test/API/functionalities/reproducers/fs-case-sensitivity/TestReproducerFSCaseSensitivity.py
    M llvm/lib/Support/FileCollector.cpp

  Log Message:
  -----------
  [FileCollector] Fix that the file system case-sensitivity check was inverted

real_path returns an `std::error_code` which evaluates to `true` in case an
error happens and `false` if not. This code was checking the inverse, so
case-insensitive file systems ended up being detected as case sensitive.

Tested using an LLDB reproducer test as we anyway need a real file system and
also some matching logic to detect whether the respective file system is
case-sensitive (which the test is doing via some Python checks that we can't
really emulate with the usual FileCheck logic).

Fixes rdar://67003004

Reviewed By: JDevlieghere

Differential Revision: https://reviews.llvm.org/D96795




More information about the All-commits mailing list