[all-commits] [llvm/llvm-project] 60969a: [Support][FileSystem] Prefer `status` with `file_t...

[Steven Wu via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 60969adcbc2fca5779580ca93ba00a7b06cc0e40
      https://github.com/llvm/llvm-project/commit/60969adcbc2fca5779580ca93ba00a7b06cc0e40
  Author: Steven Wu <stevenwu at apple.com>
  Date:   2026-09-09 (Wed, 09 Sep 2026)

  Changed paths:
    M llvm/include/llvm/Support/FileSystem.h

  Log Message:
  -----------
  [Support][FileSystem] Prefer `status` with `file_t` parameter. NFC. (#157581)

When searching the documentation for LLVM for `llvm::sys::fs::status`,
only the version with file descriptor can be found because the
documentation is built on the UNIX host and the file handle version is
defined out.

In reality, file handle version is more portable and should be displayed
instead. On UNIX, these two functions have the same underlying type,
thus it shouldn't matter which one is declared, and both versions are
still available on Windows for compatibility.



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