Open shakthimaan opened 2 years ago
Thanks for the logs @shakthimaan! I see three separate issues here --
Lockfree: I presume lockfree doesn't building/install on 4.14 branch. This should be do-able given we have (mock) Atomic
module 4.12 onward. I'll test this and update here.
Domains: The Domain
module is not present in 4.14, hence some of the failures. We could have a mock-Domain similar to the mock-Atomics I mentioned above. Please let me know if it'd be useful to have.
C-API changes: Some C functions have changed in the 5.x branch. We could turn off the global-roots benchmark for 4.14, or replace it with the corresponding 4.14 version. These are micro-benchmarks not run in the nightly runs, I believe, so it might be okay to just turn them off.
Yes, we can have mock Atomic
and Domain
modules with a README note on the same for 4.14. Sure, we can disable the global-roots benchmarks for 4.14. Please share a PR with the necessary changes, and I can test the same. Thanks!
The
domainslib4.x
branch withdune.2.9.0
is used to build the parallel benchmarks in Sandmark to support 4.14 using the following commands in the CI:At present it fails with the following error messages, and the benchmarks need to be updated to support both 4.x and 5.x branches:
Reference: Build support for 4.14. https://github.com/ocaml-bench/sandmark/pull/387