[flang] Fix building on aarch64 *BSD and musl libc after 9d8dc45d17088300e9e2086594ca581b119193c8

[brad0]

The fpu_control.h header appears to be GLIBC specific.

[llvmbot]

@llvm/pr-subscribers-flang-runtime

Author: Brad Smith (brad0)

Changes

---

Full diff: https://github.com/llvm/llvm-project/pull/125183.diff

1 Files Affected:

• (modified) flang/runtime/exceptions.cpp (+1-1)

diff --git a/flang/runtime/exceptions.cpp b/flang/runtime/exceptions.cpp
index 257c71b51edb3d..344e7216cfaae8 100644
--- a/flang/runtime/exceptions.cpp
+++ b/flang/runtime/exceptions.cpp
@@ -11,7 +11,7 @@
 #include "flang/Runtime/exceptions.h"
 #include "terminator.h"
 #include <cfenv>
-#if defined(__aarch64__) && !defined(_WIN32)
+#if defined(__aarch64__) && defined(__GLIBC__)
 #include <fpu_control.h>
 #elif defined(__x86_64__) && !defined(_WIN32)
 #include <xmmintrin.h>

[luporl]

fpu_control.h is also absent on macOS. This PR fixes the build on it.

[vdonaldson]

Thanks for the fix!

[vdonaldson]

[There is no "request for changes". Thanks for the fix!]

[DavidTruby]

This also fixes macOS builds. Can we cherry pick this onto the release branch? Builds for all the mentioned targets are broken there at the moment.

[DavidTruby]

You already did... never mind! I didn't know there was a different way to do it except for the cherry-pick command 👍

[brad0]

You already did... never mind! I didn't know there was a different way to do it except for the cherry-pick command 👍

That is the way of doing things. But I opened another issue under the milestone and then did a cherry-pick.