search

armdevvel / mxe-STATIC

MXE (M cross environment) (For ARM32 Windows development)
https://mxe.cc
Other
4 stars 2 forks source link

Disable the OpenMP test on ARM targets as OpenMP build needs UASM. #4

Closed treeswift closed 2 years ago

treeswift commented 2 years ago

As far as I understand, UASM, the OpenMP prerequisite, only supports Intel. OMP builds are disabled for ARM*. Therefore the unit test that depends on OpenMP needs to be conditionally turned off. This fixes building pthreads. Feel free to reject if this temporary fix is not enough and the root case issue (OMP on ARM) should be addressed instead.

pahaze commented 2 years ago

Huh. I didn't know pthreads was broken, it was building fine on my system. I'll merge and if needbe revert and put another fix in later (if there is one for OMP on ARM).