Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[pull] master from JuliaLang:master #1789

Merged
merged 20 commits into from
Mar 10, 2023
Merged

Conversation

pull[bot]
Copy link

@pull pull bot commented Mar 10, 2023

See Commits and Changes for more details.


Created by pull[bot]

Can you help keep this open source service alive? 💖 Please sponsor : )

stevengj and others added 20 commits February 24, 2023 09:10
This commit adds libTracyClient as an optional build dependency,
to be used for profiling the base Julia runtime. Support can be
enabled by adding `WITH_TRACY := 1` to Make.user

Here's an example instrumentation:
```c
    TracyCZoneN(ctx, "jl_subtype_env", true);
    int r = jl_subtype_env(x, y, NULL, 0);
    TracyCZoneEnd(ctx);
    return r;
```

This only adds Tracy itself. In a follow-up commit, I'll update the
JL_TIMING macros to hook into Tracy when available.
This version includes the full set of feature flags that we use to
customize Tracy to, e.g., accept on-demand connections instead of
queuing profiling data until receiving a connection.
Loading libjulia-internal and libjulia-codegen can trigger constructors
in their dependencies, including some which may spawn threads (e.g.
Tracy). It is important that these not be configured to receive any
signals that libjulia-internal is expecting to `sigwait()` on, esp. when
their default action is to terminate the process.

Technically, the masked signal set should probably be synchronized with
the `sigwait_sigs` set in `signals-unix.c`, but for now this simply masks
all signals.
This is the right strategy according to #31530, so that the patching
process doesn't hit any races between multiple builds running from the
same SRCCACHE.
Add Tracy as optional dependency for profiling
cross-reference names from varinfo docstring
Let Base handle concatenation of arrays and numbers
@pull pull bot added the ⤵️ pull label Mar 10, 2023
@pull pull bot merged commit 162b9e9 into MLH-Fellowship:master Mar 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants