You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The domainslib4.x branch with dune.2.9.0 is used to build the parallel benchmarks in Sandmark to support 4.14 using the following commands in the CI:
TAG='"run_in_ci"' make multicore_parallel_run_config_filtered.json
TAG='"macro_bench"' make multicore_parallel_run_config_filtered_filtered.json
OPT_WAIT=0 USE_SYS_DUNE_HACK=1 BUILD_BENCH_TARGET=multibench_parallel RUN_CONFIG_JSON=multicore_parallel_run_config_filtered_filtered_2domains.json make ocaml-versions/4.14.0.bench
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:
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. #387
The text was updated successfully, but these errors were encountered: