Skip to content

Commit

Permalink
Parser: no jnf for Apple
Browse files Browse the repository at this point in the history
At least on AppleClang and also seen in flang source-code, we
just call the double variant of `jn` instead of using a specialized
call to `jnf`.

This might be less efficient than needed with Intel on Apple, but
I don't have a system to test this on at the moment and this is thus
a save bet for compile stability.
  • Loading branch information
ax3l committed Apr 1, 2021
1 parent e7f1543 commit 8da0900
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion Source/Parser/wp_parser_y.h
Original file line number Diff line number Diff line change
Expand Up @@ -247,7 +247,7 @@ wp_call_f2 (enum wp_f2_t type, T a, T b)
amrex::Abort();
return 0.0;
#else
# ifdef AMREX_USE_FLOAT
# ifdef AMREX_USE_FLOAT && !defined(__APPLE__)
return jnf(a, b);
# else
return jn(a, b);
Expand Down

0 comments on commit 8da0900

Please sign in to comment.