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

Remove some duplicates from emitted compilation traces for Julia 1.10 #53776

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

kpamnany
Copy link
Contributor

When multiple threads concurrently attempt to compile the same method, --trace-compile could emit duplicate precompile statements. This small tweak eliminates one source of these duplicates.

Backported to 1.10 from #53774.

When multiple threads concurrently attempt to compile the same method,
`--trace-compile` could emit duplicate `precompile` statements. This
small tweak eliminates one source of these duplicates.
@kpamnany kpamnany requested a review from vtjnash March 18, 2024 17:01
@kpamnany kpamnany changed the title Remove some duplicates from emitted compilation traces Remove some duplicates from emitted compilation traces for Julia 1.10 Mar 18, 2024
@kpamnany
Copy link
Contributor Author

@KristofferC: this is a backported version of #53774. Could you take a look please?

I haven't opened a backport for 1.11; maybe this can be cherry-picked directly?

@KristofferC KristofferC merged commit f6a3c5f into backports-release-1.10 Mar 20, 2024
4 of 7 checks passed
@KristofferC KristofferC deleted the kp/rm-dup-precompiles-110 branch March 20, 2024 13:28
KristofferC pushed a commit that referenced this pull request Mar 20, 2024
…#53776)

When multiple threads concurrently attempt to compile the same method,
`--trace-compile` could emit duplicate `precompile` statements. This
small tweak eliminates one source of these duplicates.

Backported to 1.10 from #53774.

(cherry picked from commit f6a3c5f)
kpamnany added a commit to RelationalAI/julia that referenced this pull request Mar 20, 2024
…JuliaLang#53776)

When multiple threads concurrently attempt to compile the same method,
`--trace-compile` could emit duplicate `precompile` statements. This
small tweak eliminates one source of these duplicates.
kpamnany added a commit to RelationalAI/julia that referenced this pull request Mar 20, 2024
…JuliaLang#53776)

When multiple threads concurrently attempt to compile the same method,
`--trace-compile` could emit duplicate `precompile` statements. This
small tweak eliminates one source of these duplicates.
Drvi pushed a commit to RelationalAI/julia that referenced this pull request Jun 7, 2024
…JuliaLang#53776)

When multiple threads concurrently attempt to compile the same method,
`--trace-compile` could emit duplicate `precompile` statements. This
small tweak eliminates one source of these duplicates.

Backported to 1.10 from JuliaLang#53774.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants