Skip to content

Commit

Permalink
Build+test bn_mp_set_double.c on more platforms
Browse files Browse the repository at this point in the history
Not all platforms/environments/architectures that support enough of
IEEE 754 for the purposes of mp_set_double() actually support enough
to legitimately define __STDC_IEC_559__, so only relying on that is
too strict. Fixes #159
  • Loading branch information
MasterDuke17 authored and minad committed Feb 17, 2020
1 parent ffd8066 commit 4e90f31
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion demo/test.c
Expand Up @@ -522,7 +522,7 @@ static int test_mp_invmod(void)

}

#if defined(__STDC_IEC_559__) || defined(__GCC_IEC_559)
#if defined(__STDC_IEC_559__) || defined(__GCC_IEC_559) || defined(__x86_64__) || defined(_M_X64) || defined(_M_AMD64) || defined(__i386__) || defined(_M_X86) || defined(__aarch64__) || defined(__arm__)
static int test_mp_set_double(void)
{
int i;
Expand Down
2 changes: 1 addition & 1 deletion mp_set_double.c
Expand Up @@ -3,7 +3,7 @@
/* LibTomMath, multiple-precision integer library -- Tom St Denis */
/* SPDX-License-Identifier: Unlicense */

#if defined(__STDC_IEC_559__) || defined(__GCC_IEC_559)
#if defined(__STDC_IEC_559__) || defined(__GCC_IEC_559) || defined(__x86_64__) || defined(_M_X64) || defined(_M_AMD64) || defined(__i386__) || defined(_M_X86) || defined(__aarch64__) || defined(__arm__)
mp_err mp_set_double(mp_int *a, double b)
{
uint64_t frac;
Expand Down

0 comments on commit 4e90f31

Please sign in to comment.