7

Promote loongarch64-unknown-none* to Tier 2 by heiher · Pull Request #115368 · r...

 11 months ago
source link: https://github.com/rust-lang/rust/pull/115368
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
neoserver,ios ssh client

Promote loongarch64-unknown-none* to Tier 2 #115368

Conversation

Contributor

sairumea reacted with rocket emoji

Collaborator

r? @JohnTitor

(rustbot has picked a reviewer for you, use r? to override)

rustbot

added A-testsuite Area: The testsuite used to check the correctness of rustc S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.

labels

Aug 30, 2023

Contributor

Author

Could you please trigger a try build?

@bors try

heiher reacted with heart emoji

Contributor

⌛ Trying commit afb9e961c4d0852daca63b67c72ca13283a2ae1a with merge 33a6d20727ff80d19d8e4fb41f083039d4555a55...

Contributor

☀️ Try build successful - checks-actions
Build commit: 33a6d20727ff80d19d8e4fb41f083039d4555a55 (33a6d20727ff80d19d8e4fb41f083039d4555a55)

Member

Potential blockers:
#115509
#115609

EDIT: I didn't realize that we already have a tier 2 loongarch target. I guess adding more of them doesn't make this any worse. Ideally this would have had a more careful ABI audit before becoming tier 2.

Member

r? compiler

Contributor

r? @wesleywiser as the MCP seconder

wesleywiser and heiher reacted with thumbs up emoji

Contributor

Author

@wesleywiser gentle ping

Contributor

Author

Now, #115509 has been resolved, and #115609 is still in progress.

Quote @cuviper's comment:

the -linux-gnu target is already tier 2, so I don't see why those issues would block -none

I request that #115609 is not considered a blocker.

Contributor

Author

r? @Mark-Simulacrum "Promotion" may have more to do with infrastructure.

I see that only a small amount of library code needs to be compiled for the bare metal target, which puts very little pressure on the build. The 1.74 version will be branched off from the master on Sep 29th. Could you please review this before then? 😃

wesleywiser reacted with thumbs up emoji

Member

@wesleywiser wesleywiser

left a comment

The compiler team has approved the MCP for promotion to Tier 2 and these changes look correct to me but the Infra team needs to sign off on the addition to CI.

heiher reacted with thumbs up emoji

Member

CI overhead should be negligible as this doesn't add a new standalone job.

@bors r+

heiher reacted with heart emoji

Contributor

📌 Commit d185e0d has been approved by pietroalbini

It is now in the queue for this repository.

bors

added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.

and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.

labels

Sep 29, 2023

Contributor

⌛ Testing commit d185e0d with merge b8b376a...

Contributor

☀️ Test successful - checks-actions
Approved by: pietroalbini
Pushing b8b376a to master...

wesleywiser reacted with hooray emoji

bors

added the merged-by-bors This PR was explicitly merged by bors label

Sep 30, 2023

bors

merged commit b8b376a into

rust-lang:master

Sep 30, 2023

12 checks passed

Contributor

Author

@rustbot label: +relnotes

rustbot

added the relnotes Marks issues that should be documented in the release notes of the next release. label

Sep 30, 2023

Collaborator

Finished benchmarking commit (b8b376a): comparison URL.

Overall result: no relevant changes - no action needed

@rustbot label: -perf-regression

Instruction count

This benchmark run did not return any relevant results for this metric.

Max RSS (memory usage)

Results

Cycles

Results

Binary size

This benchmark run did not return any relevant results for this metric.

Bootstrap: 632.26s -> 631.081s (-0.19%)
Artifact size: 317.36 MiB -> 317.36 MiB (0.00%)

Contributor

Looks like this PR narrowly missed the beta promotion? Do we still want to stuff it into beta?

Member

It did not miss -- it merged before the version bump in #116302, so it was still auto-added to milestone 1.74.0.

But generally, no, we would not do a beta backport for a feature like this, only for high-impact bugs. The point of having frequent releases is that it should not be a big deal to wait for the next one.

compiler-errors reacted with thumbs up emoji

Contributor

It did not miss -- it merged before the version bump in #116302, so it was still auto-added to milestone 1.74.0.

But generally, no, we would not do a beta backport for a feature like this, only for high-impact bugs. The point of having frequent releases is that it should not be a big deal to wait for the next one.

Thanks for the explanation! I checked the beta branch previously but didn't see this commit, hence the question. (I'd agree with waiting for the next release if the change actually missed 1.74.0, though.)

bors-ferrocene bot

added a commit to ferrocene/ferrocene that referenced this pull request

Oct 5, 2023

bors-ferrocene bot

added a commit to ferrocene/ferrocene that referenced this pull request

Oct 6, 2023

bors-ferrocene bot

added a commit to ferrocene/ferrocene that referenced this pull request

Oct 9, 2023

bors-ferrocene bot

added a commit to ferrocene/ferrocene that referenced this pull request

Oct 9, 2023

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

wesleywiser

wesleywiser approved these changes
Labels
A-testsuite Area: The testsuite used to check the correctness of rustc merged-by-bors This PR was explicitly merged by bors relnotes Marks issues that should be documented in the release notes of the next release. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.
Projects

None yet

Milestone

1.74.0

Development

Successfully merging this pull request may close these issues.

None yet

13 participants

About Joyk


Aggregate valuable and interesting links.
Joyk means Joy of geeK