[all-commits] [llvm/llvm-project] 51689c: [libc][NFC] clean internal fd handling (#143991)

[Michael Jones via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 51689c9df2fbb81aab1ff802f3efb86cac926853
      https://github.com/llvm/llvm-project/commit/51689c9df2fbb81aab1ff802f3efb86cac926853
  Author: Michael Jones <michaelrj at google.com>
  Date:   2025-06-13 (Fri, 13 Jun 2025)

  Changed paths:
    M libc/src/__support/File/linux/file.cpp
    M libc/src/__support/OSUtil/fcntl.h
    M libc/src/__support/OSUtil/linux/CMakeLists.txt
    M libc/src/__support/OSUtil/linux/fcntl.cpp
    M libc/src/fcntl/linux/CMakeLists.txt
    M libc/src/fcntl/linux/fcntl.cpp
    M libc/src/fcntl/linux/open.cpp
    M libc/src/sys/auxv/linux/getauxval.cpp
    M libc/src/sys/mman/linux/shm_common.h
    M libc/src/sys/mman/linux/shm_open.cpp
    M libc/src/sys/mman/linux/shm_unlink.cpp
    M libc/src/unistd/linux/close.cpp
    M utils/bazel/llvm-project-overlay/libc/BUILD.bazel

  Log Message:
  -----------
  [libc][NFC] clean internal fd handling (#143991)

The previous internal fcntl implementation modified errno directly, this
patch fixes that. This patch also moves open and close into OSUtil since
they are used in multiple places. There are more places that need
similar cleanup but only got comments in this patch to keep it
relatively reviewable.

Related to: https://github.com/llvm/llvm-project/issues/143937



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