[all-commits] [llvm/llvm-project] 05502d: [libc] Clean up errno header usage in a few math/s...

[Alexey Samsonov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 05502de64ccf1793a49e2f7c80ca1afe5d1e0f43
      https://github.com/llvm/llvm-project/commit/05502de64ccf1793a49e2f7c80ca1afe5d1e0f43
  Author: Alexey Samsonov <vonosmas at gmail.com>
  Date:   2025-09-08 (Mon, 08 Sep 2025)

  Changed paths:
    M libc/test/src/math/smoke/CMakeLists.txt
    M libc/test/src/math/smoke/acos_test.cpp
    M libc/test/src/math/smoke/acosf16_test.cpp
    M libc/test/src/math/smoke/acosf_test.cpp
    M libc/test/src/math/smoke/acoshf16_test.cpp
    M libc/test/src/math/smoke/acoshf_test.cpp
    M libc/test/src/math/smoke/acospif16_test.cpp
    M libc/test/src/math/smoke/asinf16_test.cpp
    M libc/test/src/math/smoke/asinf_test.cpp
    M libc/test/src/math/smoke/asinhf16_test.cpp
    M libc/test/src/math/smoke/asinhf_test.cpp
    M libc/test/src/math/smoke/asinpif16_test.cpp

  Log Message:
  -----------
  [libc] Clean up errno header usage in a few math/smoke tests. (#157517)

Most of the unit tests don't (or don't need to) read/write libc_errno
code directly - it's cleared by the ErrnoCheckingTest harness, and is
verified by framework-provided scripts such as EXPECT_MATH_ERRNO.

Use the following rule of thumb for header inclusion:
* if you use libc_errno in code literally, include
src/__support/libc_errno.h
* if you only rely on errno constants, include hdr/errno_macros.h

This PR only updates a few tests for acos/asin variants, as a
proof-of-concept. If it goes in, a follow-up PR would update the rest.



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