[all-commits] [llvm/llvm-project] a4c8ef: [compiler-rt] Fix detecting _Float16 support for s...

[Alexander Richardson via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a4c8ef0f401d86040594cc6f01bcdad9392e8ee2
      https://github.com/llvm/llvm-project/commit/a4c8ef0f401d86040594cc6f01bcdad9392e8ee2
  Author: Alexander Richardson <alexrichardson at google.com>
  Date:   2024-11-28 (Thu, 28 Nov 2024)

  Changed paths:
    M compiler-rt/lib/builtins/CMakeLists.txt

  Log Message:
  -----------
  [compiler-rt] Fix detecting _Float16 support for secondary targets (#117813)

It turns out we were not passing -m32 to the check_c_source_compiles()
invocation since CMAKE_REQUIRE_FLAGS needs to be string separated list
and
we were passing a ;-separated CMake list which appears to be parsed by
CMake as 'ignore all arguments beyond the first'.
Fix this by transforming the list to a command line first.

With this change, Clang 17 no longer claims to support _Float16 for
i386.



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