[all-commits] [llvm/llvm-project] 43bec3: Remove HAVE_STRERROR

Fangrui Song via All-commits all-commits at lists.llvm.org
Tue May 30 14:12:33 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 43bec3376c07c9940361adef29bb2e4fb1f526a7
      https://github.com/llvm/llvm-project/commit/43bec3376c07c9940361adef29bb2e4fb1f526a7
  Author: Fangrui Song <i at maskray.me>
  Date:   2023-05-30 (Tue, 30 May 2023)

  Changed paths:
    M flang/runtime/CMakeLists.txt
    M flang/runtime/io-error.cpp
    M llvm/cmake/config-ix.cmake
    M llvm/include/llvm/Config/config.h.cmake
    M llvm/lib/Support/Errno.cpp
    M llvm/utils/gn/secondary/llvm/include/llvm/Config/BUILD.gn
    M utils/bazel/llvm-project-overlay/llvm/include/llvm/Config/config.h
    M utils/bazel/llvm_configs/config.h.cmake

  Log Message:
  -----------
  Remove HAVE_STRERROR

Most systems support strerror_r. For the remaining systems (e.g. MSVC) strerror_s and
strerror can be used as fallbacks. We don't have a supported operating
system/compiler that doesn't provide `strerror`.

Close https://github.com/llvm/llvm-project/issues/62804

https://github.com/flang-compiler/f18/pull/1068 added a fallback
when strerror is unavailable, but I think the code path is dead.

Reviewed By: serge-sans-paille, vzakhari

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




More information about the All-commits mailing list