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
-linkall is not forced when not necessary. As far as I understand, dune sites (without any plugins) do not actually need to force it because there's no dynamic loading/linking happening.
Correct me if I'm wrong, but manually doing the final executable step to remove -linkall still works fine because as long as the generated dune_site__Dune_site_data.cmx is explicitly linked. -linkall is not necessary for just the data.
Actual Behavior
This code seems to be responsible for forcing -linkall when dune site is in use:
Expected Behavior
-linkall
is not forced when not necessary. As far as I understand, dune sites (without any plugins) do not actually need to force it because there's no dynamic loading/linking happening.Correct me if I'm wrong, but manually doing the final executable step to remove
-linkall
still works fine because as long as the generateddune_site__Dune_site_data.cmx
is explicitly linked.-linkall
is not necessary for just the data.Actual Behavior
This code seems to be responsible for forcing
-linkall
when dune site is in use:dune/src/dune_rules/link_time_code_gen.ml
Lines 305 to 325 in 098117d
Notably, #4348 added the forcing, but it even happens when dune plugins are not used, only sites.
Specifications
dune
(output ofdune --version
): 3.16.0ocaml
(output ofocamlc --version
): 4.14.2The text was updated successfully, but these errors were encountered: