Skip to content

Conversation

KristofferC
Copy link
Member

Reverts #55909

Fixes #56063. Caused a 3x startup regression in 1.11.0.

@KristofferC KristofferC added the backport 1.11 Change should be backported to release-1.11 label Oct 9, 2024
@giordano
Copy link
Member

giordano commented Oct 9, 2024

Fixes #56063.

That's still an issue on master for v1.12, isn't it?

@KristofferC KristofferC mentioned this pull request Oct 9, 2024
44 tasks
@KristofferC
Copy link
Member Author

That's still an issue on master for v1.12, isn't it?

I don't get it. This PR reverts the PR that caused the regression?

@nsajko
Copy link
Member

nsajko commented Oct 9, 2024

The regression doesn't seem to exist on master, though, just on v1.11:

$ hyperfine './nightly_normal/julia-ecf41b18cc/bin/julia --startup-file=no -e "1+1"'
Benchmark 1: ./nightly_normal/julia-ecf41b18cc/bin/julia --startup-file=no -e "1+1"
  Time (mean ± σ):     139.6 ms ±   0.8 ms    [User: 71.1 ms, System: 66.8 ms]
  Range (min … max):   138.2 ms … 141.6 ms    21 runs

--trace-compile=stderr also doesn't produce any output for startup on nightly.

@nsajko nsajko added the revert This reverts a previously merged PR. label Oct 9, 2024
@oscardssmith oscardssmith requested a review from vtjnash October 9, 2024 12:59
@IanButterworth
Copy link
Member

Does anyone happen to see the issue with that PR?

@vtjnash
Copy link
Member

vtjnash commented Oct 9, 2024

This should be against the backport branch, not master, since it doesn't affect master

@KristofferC
Copy link
Member Author

I just pushed the commit to the backport branch.

@giordano giordano deleted the revert-55909-jn/precompile-error branch October 10, 2024 10:18
@topolarity
Copy link
Member

Did we ever figure out why this was causing issues on 1.11? Seems pretty mysterious

@DilumAluthge
Copy link
Member

I just pushed the commit to the backport branch.

Per #56069 (comment), I am assuming that we no longer need the backport label on this PR?

@DilumAluthge DilumAluthge removed the backport 1.11 Change should be backported to release-1.11 label Sep 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

revert This reverts a previously merged PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

big startup time regression with v1.11 release

7 participants