Skip to content

[Relocatable] Follow-up 23: Dynlink-related tidying following ld.conf changes#185

Open
dra27 wants to merge 1 commit intorelocatable-testingfrom
dynamic-tidying
Open

[Relocatable] Follow-up 23: Dynlink-related tidying following ld.conf changes#185
dra27 wants to merge 1 commit intorelocatable-testingfrom
dynamic-tidying

Conversation

@dra27
Copy link
Copy Markdown
Owner

@dra27 dra27 commented Feb 1, 2025

No description provided.

@dra27 dra27 added the relocatable PRs related to the Relocatable Compiler project label Feb 1, 2025
@dra27 dra27 force-pushed the installation-tests branch from 8538cf3 to cf7bb12 Compare February 1, 2025 21:59
@dra27 dra27 force-pushed the installation-tests branch from cf7bb12 to a300876 Compare February 5, 2025 17:50
@dra27 dra27 force-pushed the installation-tests branch from a300876 to b840f43 Compare February 9, 2025 16:06
@dra27 dra27 force-pushed the installation-tests branch from b840f43 to 12a6760 Compare February 9, 2025 17:42
@dra27 dra27 force-pushed the installation-tests branch from 12a6760 to 9312be2 Compare February 11, 2025 16:54
@dra27 dra27 force-pushed the installation-tests branch 13 times, most recently from 85ea65d to 7d2a2d9 Compare February 14, 2025 13:30
@dra27 dra27 force-pushed the installation-tests branch from 7d2a2d9 to d1a72a0 Compare February 28, 2025 13:20
@dra27 dra27 force-pushed the installation-tests branch from d1a72a0 to b5d5bb8 Compare February 28, 2025 17:23
@dra27 dra27 force-pushed the installation-tests branch from b5d5bb8 to 8ad918d Compare March 5, 2025 18:08
@dra27 dra27 force-pushed the installation-tests branch from 1ff8aae to 3adcf49 Compare March 26, 2025 15:30
@dra27 dra27 force-pushed the dynamic-tidying branch from 561ac3f to de8519f Compare March 26, 2025 15:30
@dra27 dra27 force-pushed the installation-tests branch 4 times, most recently from 08e6e34 to ef7bedc Compare April 4, 2025 21:44
@dra27 dra27 force-pushed the installation-tests branch from ada9ed4 to 27ee9a7 Compare April 5, 2025 15:36
@dra27 dra27 force-pushed the dynamic-tidying branch from de8519f to 8364e01 Compare April 5, 2025 15:37
@dra27 dra27 force-pushed the installation-tests branch from 27ee9a7 to c602311 Compare April 5, 2025 16:43
@dra27 dra27 force-pushed the dynamic-tidying branch from 8364e01 to ddeceab Compare April 5, 2025 17:02
@dra27 dra27 force-pushed the installation-tests branch from c602311 to ec86560 Compare April 5, 2025 21:12
@dra27 dra27 force-pushed the dynamic-tidying branch from ddeceab to bdeadce Compare April 5, 2025 22:16
@dra27 dra27 force-pushed the installation-tests branch 5 times, most recently from 2488f9b to 299ae51 Compare April 8, 2025 14:55
@dra27 dra27 force-pushed the dynamic-tidying branch from bdeadce to e61c396 Compare April 8, 2025 19:43
@dra27 dra27 force-pushed the installation-tests branch from 299ae51 to 2bf8a73 Compare April 21, 2025 09:06
@dra27 dra27 force-pushed the dynamic-tidying branch from e61c396 to 7019bbd Compare April 21, 2025 09:06
@dra27 dra27 force-pushed the installation-tests branch from 2bf8a73 to 92bf763 Compare April 21, 2025 16:41
@dra27 dra27 force-pushed the dynamic-tidying branch from 7019bbd to 305889a Compare April 21, 2025 16:41
@dra27 dra27 added the CI: Full matrix Full CI test matrix label Apr 21, 2025
@dra27 dra27 force-pushed the dynamic-tidying branch 3 times, most recently from f2d5bf1 to 3c43f64 Compare April 22, 2025 10:39
@dra27 dra27 force-pushed the installation-tests branch from 7c36266 to 5d3eea2 Compare April 24, 2025 18:58
In the debug runtime, caml_prim_name_table remains for the lifetime of
the program, as it's used by instrtrace.c, but in normal operation, once
the list of primitives has been handed over to Dynlink, it's no longer
required. In the normal runtime, it's now freed after this handover.

In passing, strings themselves are no longer duplicated, as the code
path is a good deal simpler than it used to be, and the bytecode section
itself can reliably be used as the underlying buffer for
caml_prim_name_table.

caml_shared_libs_path is only kept at all to be handed over to Dynlink.
It, along with the two underlying buffers for CAML_LD_LIBRARY_PATH and
ld.conf are freed after the call.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI: Full matrix Full CI test matrix no-change-entry-needed Causes the check for a Changes entry to be skipped for PRs relocatable PRs related to the Relocatable Compiler project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant