Compilation failure of ocaml-bls-12-381-unix in Mac OS X about ocaml-bls12-381

[camlspotter]

When building https://github.com/supranational/blst with build.sh I see the following compilation:

$ ./build.sh                                                                                               
+ cc -O -fno-builtin-memcpy -fPIC -Wall -Wextra -Werror -D__ADX__ -mno-avx -c ./src/server.c                                   
+ cc -O -fno-builtin-memcpy -fPIC -Wall -Wextra -Werror -D__ADX__ -mno-avx -c ./build/assembly.S                               
+ cc -O -fno-builtin-memcpy -fPIC -Wall -Wextra -Werror -D__ADX__ -mno-avx -nostdlib -r assembly.o server.o -o blst.o          
+ objcopy --localize-symbols=/tmp/localize.blst.533339 blst.o                                                                  
+ ar rc libblst.a blst.o   

-D__ADX__ is required for Mac OS X?

Sorry, this was the result in Linux.

[dannywillems]

Which commit did you use for blst? For bls12-381-unix.1.0.0 we use the commit
095a8c53787d6c91b725152ebfbbf33acf05a931

[dannywillems]

Also, which C compiler (+ version) do you use?

[camlspotter]

I am sorry, I misused a Linux machine connected with ssh.

I used commit blst 98044ad6af47179ab4004ea0fbca5f2f5b63702f and ./build.sh has failed with the same errors in the first post.

The version of cc is:

$ cc --version
Apple LLVM version 10.0.1 (clang-1001.0.46.4)
Target: x86_64-apple-darwin18.7.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

[camlspotter]

supranational/blst#70 ?

[dannywillems]

@camlspotter I would recommend you to update Xcode to use at least clang 11. Xcode 11 comes with clang 11 and Xcode 11 is supported from macOS Mojave 10.14.4. As Majove won't be supported soon (released in 2018 and Apple supports for around 3 years each MacOS version according to some Google links), I won't add support for Xcode 10 (coming with clang 10).
I added a GitHub action workflow to test for clang 11 and clang 12, see https://gitlab.com/dannywillems/ocaml-bls12-381/-/merge_requests/108/ but couldn't get it work for clang 10.
The same MR fixes the build for clang 11, and I think it will also fix for clang 10. If you confirm, I would suggest you close this thread. A new release (bls12-381.1.0.1) will be done soon.