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 libtom#159
  • Loading branch information
MasterDuke17 committed Feb 9, 2020
1 parent 628cf84 commit db0d387
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion bn_mp_set_double.c
Original file line number Diff line number Diff line change
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
2 changes: 1 addition & 1 deletion demo/test.c
Original file line number Diff line number Diff line change
Expand Up @@ -625,7 +625,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

1 comment on commit db0d387

@chenrui333
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This commit did not get into 1.3.0 release, any idea?

Please sign in to comment.