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

[WIP] Test compiling the compiler with -Z share-generics #70560

Closed
wants to merge 1 commit into from

Conversation

Zoxc
Copy link
Contributor

@Zoxc Zoxc commented Mar 30, 2020

r? @ghost

@Zoxc
Copy link
Contributor Author

Zoxc commented Mar 30, 2020

@bors try @rust-timer queue

@rust-timer
Copy link
Collaborator

Awaiting bors try build completion

@bors
Copy link
Contributor

bors commented Mar 30, 2020

⌛ Trying commit 95bf273 with merge 02c7088d2cedc389184f692fa5318503fe12cbce...

@bors
Copy link
Contributor

bors commented Mar 30, 2020

☀️ Try build successful - checks-azure
Build commit: 02c7088d2cedc389184f692fa5318503fe12cbce (02c7088d2cedc389184f692fa5318503fe12cbce)

@rust-timer
Copy link
Collaborator

Queued 02c7088d2cedc389184f692fa5318503fe12cbce with parent 8926bb4, future comparison URL.

@rust-timer
Copy link
Collaborator

Finished benchmarking try commit 02c7088d2cedc389184f692fa5318503fe12cbce, comparison URL.

@bjorn3
Copy link
Member

bjorn3 commented Mar 30, 2020

For keccak-check clean:

Query/Function Time % Total Time Δ Full Executions Δ Loading Δ
Totals 7.710 100.08%* 71.4% (3.212) 139383 0.0% (0.000) 0.000 0.0% (0.000)
typeck_tables_of 3.759 48.76% 51.0% (1.269) 3373 0.0% (0.000) 0.000 0.0% (0.000)
mir_borrowck 2.834 36.76% 105.9% (1.458) 3373 0.0% (0.000) 0.000 0.0% (0.000)
check_mod_liveness 0.301 3.90% 141.4% (0.176) 1 0.0% (0.000) 0.000 0.0% (0.000)
mir_built 0.232 3.01% 99.4% (0.116) 3373 0.0% (0.000) 0.000 0.0% (0.000)

For script-servo-check clean almost half of all queries regressed by 50% or more.

@nikomatsakis
Copy link
Contributor

Closing this pull request as Zoxc is stepping back from compiler development; see rust-lang/team#316.

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.

5 participants