[all-commits] [llvm/llvm-project] c40f0f: Revert "Reland "[clang] Lower modf builtin using `...

[Hans Wennborg via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c40f0fe4348bb9304b95bd317665bf1cb2bdcc85
      https://github.com/llvm/llvm-project/commit/c40f0fe4348bb9304b95bd317665bf1cb2bdcc85
  Author: Hans Wennborg <hans at chromium.org>
  Date:   2025-03-10 (Mon, 10 Mar 2025)

  Changed paths:
    M clang/lib/CodeGen/CGBuiltin.cpp
    M clang/test/CodeGen/X86/math-builtins.c
    M clang/test/CodeGen/aix-builtin-mapping.c
    M clang/test/CodeGen/builtin-attributes.c
    M clang/test/CodeGen/math-builtins-long.c
    M clang/test/CodeGen/math-libcalls.c

  Log Message:
  -----------
  Revert "Reland "[clang] Lower modf builtin using `llvm.modf` intrinsic" (#129885)"

This broke modff calls on 32-bit x86 Windows. See comment on the PR.

> This updates the existing modf[f|l] builtin to be lowered via the
> llvm.modf.* intrinsic (rather than directly to a library call).
>
> The legalization issues exposed by the original PR (#126750) should have
> been fixed in #128055 and #129264.

This reverts commit cd1d9a8fab05524a27ffdb251f6def37786b5cc1.



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