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

Update the definition of borrowing_sub #133796

Merged
merged 1 commit into from
Dec 3, 2024
Merged

Conversation

TDecking
Copy link
Contributor

@TDecking TDecking commented Dec 3, 2024

Complementary PR to #133674, which only updated carrying_add.

@rustbot
Copy link
Collaborator

rustbot commented Dec 3, 2024

r? @tgross35

rustbot has assigned @tgross35.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Dec 3, 2024
@tgross35
Copy link
Contributor

tgross35 commented Dec 3, 2024

Could you add a test similar to what is in #133674? You can just rename that file and add it there.

Cc @scottmcm

@tgross35
Copy link
Contributor

tgross35 commented Dec 3, 2024

The test lgtm, verified that the current nightly codegen doesn't make use of sbb.

But please combine x86_64-bigint-sub.rs and x86_64-bigint-add.rs into something like x86_64-bigint-helpers.rs, we don't need to have the same headers and comments duplicated in two files.

r=me with that, green CI, and a squash

This ensures that it matches the one in `carrying_add`.
Copy link
Contributor

@tgross35 tgross35 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@tgross35
Copy link
Contributor

tgross35 commented Dec 3, 2024

@bors r+

@bors
Copy link
Contributor

bors commented Dec 3, 2024

📌 Commit 8b7d3d3 has been approved by tgross35

It is now in the queue for this repository.

@bors 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 Dec 3, 2024
bors added a commit to rust-lang-ci/rust that referenced this pull request Dec 3, 2024
…iaskrgr

Rollup of 9 pull requests

Successful merges:

 - rust-lang#132612 (Gate async fn trait bound modifier on `async_trait_bounds`)
 - rust-lang#133545 (Lint against Symbol::intern on a string literal)
 - rust-lang#133558 (Structurally resolve in `probe_adt`)
 - rust-lang#133696 (stabilize const_collections_with_hasher and build_hasher_default_const_new)
 - rust-lang#133753 (Reduce false positives on some common cases from if-let-rescope lint)
 - rust-lang#133762 (stabilize const_{size,align}_of_val)
 - rust-lang#133777 (document -Zrandomize-layout in the unstable book)
 - rust-lang#133779 (Use correct `hir_id` for array const arg infers)
 - rust-lang#133796 (Update the definition of `borrowing_sub`)

r? `@ghost`
`@rustbot` modify labels: rollup
@scottmcm
Copy link
Member

scottmcm commented Dec 3, 2024

Oops, I should have done this too 😬 Thanks for the PR and for including the assembly test!

@bors bors merged commit b78ab2f into rust-lang:master Dec 3, 2024
6 checks passed
rust-timer added a commit to rust-lang-ci/rust that referenced this pull request Dec 3, 2024
Rollup merge of rust-lang#133796 - TDecking:borrowing-sub, r=tgross35

Update the definition of `borrowing_sub`

Complementary PR to rust-lang#133674, which only updated `carrying_add`.
@rustbot rustbot added this to the 1.85.0 milestone Dec 3, 2024
@TDecking TDecking deleted the borrowing-sub branch December 3, 2024 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants