[all-commits] [llvm/llvm-project] 0c545a: Revert "clang: Use new frexp intrinsic for builtin...

[Hans via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 0c545a441285a73e00b859dd52f1a85cb9eeeefc
      https://github.com/llvm/llvm-project/commit/0c545a441285a73e00b859dd52f1a85cb9eeeefc
  Author: Hans Wennborg <hans at chromium.org>
  Date:   2023-06-30 (Fri, 30 Jun 2023)

  Changed paths:
    M clang/include/clang/Basic/Builtins.def
    M clang/lib/CodeGen/CGBuiltin.cpp
    M clang/lib/Headers/__clang_hip_math.h
    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-builtins.c
    M clang/test/CodeGenOpenCL/builtins-generic-amdgcn.cl
    M clang/test/Headers/__clang_hip_math.hip

  Log Message:
  -----------
  Revert "clang: Use new frexp intrinsic for builtins and add f16 version"

This caused asserts in some Android and Windows builds:

SelectionDAGNodes.h:1138: llvm::SDValue::SDValue(SDNode *, unsigned int):
Assertion `(!Node || !ResNo || ResNo < Node->getNumValues()) && "Invalid result number for the given node!"' failed.

See comment on https://github.com/llvm/llvm-project/commit/85bdea023f5116f789095b606554739403042a21

Also revert "HIP: Use frexp builtins in math headers"
which seems to depend on this change.

This reverts commit 85bdea023f5116f789095b606554739403042a21.
This reverts commit bf8e92c0e792cbe3c9cc50607a1e33c6912ffd0e.