[all-commits] [llvm/llvm-project] 229e70: [libc] Avoid system fenv.h in FEnvImpl.h when LIBC...

[Marcos Ramirez Joos via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 229e7000493f118d49d23073d3bfa613b3798d85
      https://github.com/llvm/llvm-project/commit/229e7000493f118d49d23073d3bfa613b3798d85
  Author: Marcos Ramirez Joos <mramirezjoos.oss at proton.me>
  Date:   2026-09-25 (Fri, 25 Sep 2026)

  Changed paths:
    M libc/src/__support/FPUtil/FEnvImpl.h

  Log Message:
  -----------
  [libc] Avoid system fenv.h in FEnvImpl.h when LIBC_MATH_HAS_NO_EXCEPT is set (#224384)

The warning being added under #199009 triggers errors in subprojects
(e.g., flang) using libc headers that gate the use of `#pragma STDC
FENV_ACCESS ON` on the internal libc
`LIBC_COMPILER_HAS_STDC_FENV_ACCESS` flag, when building under
`-Wfenv-access -Werror`. As the idea is for these subprojects to not
access the libc FP environment (see
https://github.com/llvm/llvm-project/pull/199009#issuecomment-5299119396),
gating the use of system fenv.h functions on the
`LIBC_MATH_HAS_NO_EXCEPT` flag avoids having to specify the
`LIBC_COMPILER_HAS_STDC_FENV_ACCESS` flag when building them.

The three line removals are a clean-up related to
https://github.com/llvm/llvm-project/pull/200268. These uses are
redundant as the called functions already use the pragma when
appropriate.



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