[all-commits] [llvm/llvm-project] 38473c: [clang][headers] Need a way for math.h to share th...

[Ian Anderson via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 38473c5d351d6b2ae21b5eb1ce66c2bcc2f7c6f7
      https://github.com/llvm/llvm-project/commit/38473c5d351d6b2ae21b5eb1ce66c2bcc2f7c6f7
  Author: Ian Anderson <iana at apple.com>
  Date:   2025-10-23 (Thu, 23 Oct 2025)

  Changed paths:
    M clang-tools-extra/clang-include-fixer/find-all-symbols/STLPostfixHeaderMap.cpp
    M clang-tools-extra/clangd/index/CanonicalIncludes.cpp
    M clang/lib/Headers/CMakeLists.txt
    A clang/lib/Headers/__float_float.h
    A clang/lib/Headers/__float_header_macro.h
    A clang/lib/Headers/__float_infinity_nan.h
    M clang/lib/Headers/float.h
    M clang/lib/Headers/module.modulemap
    A clang/test/Headers/floatneeds.c
    M clang/test/Modules/relative-resource-dir.m
    M llvm/utils/gn/secondary/clang/lib/Headers/BUILD.gn

  Log Message:
  -----------
  [clang][headers] Need a way for math.h to share the definitions of INIFINITY and NAN with float.h (#164348)

In C23 mode, both float.h and math.h are required to define INIFINITY
and NAN. However, with clang modules, there needs to be a single owner
for the declarations. Let that be float.h since that's the suggested
true owner in Annex F.2.2 of ISO/IEC 9899:2024, and introduce
`__need_infinity_nan` so that math.h can pick up the compiler
definitions.



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