search

JuliaMath / openlibm

High quality system independent, portable, open source libm implementation
https://openlibm.org
Other
515 stars 140 forks source link

Fixes for Windows ARM64 #251

Closed jeremyd2019 closed 3 years ago

jeremyd2019 commented 3 years ago

With these changes, it builds and tests succeed on Windows ARM64. Fixes #250

jeremyd2019 commented 3 years ago

Oops, this worked locally, but when applied to MINGW-packages it fails with:

  In file included from src/s_fmax.c:32:
  In file included from C:/_/M/mingw-w64-openlibm/src/build-aarch64/src/fpmath.h:33:
  C:/_/M/mingw-w64-openlibm/src/build-aarch64/src/aarch64_fpmath.h:33:3: error: unknown type name 'uint64_t'
                  uint64_t        manl    :64;
                  ^
  C:/_/M/mingw-w64-openlibm/src/build-aarch64/src/aarch64_fpmath.h:34:3: error: unknown type name 'uint64_t'
                  uint64_t        manh    :48;
                  ^
  C:/_/M/mingw-w64-openlibm/src/build-aarch64/src/aarch64_fpmath.h:40:3: error: unknown type name 'uint64_t'
                  uint64_t        manl    :64;
                  ^
  C:/_/M/mingw-w64-openlibm/src/build-aarch64/src/aarch64_fpmath.h:41:3: error: unknown type name 'uint64_t'
                  uint64_t        manh    :48;
                  ^
  4 errors generated.
  make: *** [Make.inc:160: src/s_fmax.c.o] Error 1

Maybe there are more changes in master from 0.7.5 that allowed it to work?

jeremyd2019 commented 3 years ago

Happens on master too. I swear I tested this, but maybe only incrementally as I was changing things.