Skip to content

internal: Run analysis-stats on CI, with opt-level = 1 #13401

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

Merged
merged 3 commits into from
Oct 14, 2022

Conversation

lnicola
Copy link
Member

@lnicola lnicola commented Oct 12, 2022

We might want to run analysis-stats on PRs, and this makes it less unbearable.

@lnicola
Copy link
Member Author

lnicola commented Oct 12, 2022

This looks reasonable:

image

vs. a previous run:

image

@lnicola lnicola marked this pull request as draft October 12, 2022 17:47
@lnicola lnicola marked this pull request as ready for review October 13, 2022 09:49
@lnicola lnicola changed the title Set opt-level = 1 on dev profile Run analysis-stats on CI, with opt-level = 1 Oct 13, 2022
@lnicola
Copy link
Member Author

lnicola commented Oct 13, 2022

With analysis-stats:

image

image

@lnicola lnicola mentioned this pull request Oct 13, 2022
@lnicola
Copy link
Member Author

lnicola commented Oct 14, 2022

@bors r+

@bors
Copy link
Contributor

bors commented Oct 14, 2022

📌 Commit 4adf09b has been approved by lnicola

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Oct 14, 2022

⌛ Testing commit 4adf09b with merge 36a70b7...

@lnicola lnicola changed the title Run analysis-stats on CI, with opt-level = 1 internal: Run analysis-stats on CI, with opt-level = 1 Oct 14, 2022
@bors
Copy link
Contributor

bors commented Oct 14, 2022

☀️ Test successful - checks-actions
Approved by: lnicola
Pushing 36a70b7 to master...

@bors bors merged commit 36a70b7 into rust-lang:master Oct 14, 2022
@lnicola lnicola deleted the opt-tests branch October 14, 2022 07:18
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