Skip to content

Optimize offset_from_unsigned result bounds - #161063

Open
SomeFlyingThing wants to merge 4 commits into
rust-lang:mainfrom
SomeFlyingThing:agent/offset-from-unsigned-bound
Open

Optimize offset_from_unsigned result bounds#161063
SomeFlyingThing wants to merge 4 commits into
rust-lang:mainfrom
SomeFlyingThing:agent/offset-from-unsigned-bound

Conversation

@SomeFlyingThing

Copy link
Copy Markdown
Contributor

Hint that result cannot be larger than isize max

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Aug 13, 2026
@rustbot

rustbot commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

r? @jieyouxu

rustbot has assigned @jieyouxu.
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

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: compiler
  • compiler expanded to 75 candidates
  • Random selection from 18 candidates

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@jieyouxu

Copy link
Copy Markdown
Member

r? libs

@rustbot rustbot assigned clarfonthey and unassigned jieyouxu Aug 14, 2026
@clarfonthey

Copy link
Copy Markdown
Contributor

Seems reasonable. For lack of any niche-type APIs that represent this, it should be okay to add this assumption.

@bors r+

@rust-bors

rust-bors Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 7dbde06 has been approved by clarfonthey

It is now in the queue for this repository.

@rust-bors rust-bors Bot 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 Aug 14, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 14, 2026
…unsigned-bound, r=clarfonthey

Optimize offset_from_unsigned result bounds

Hint that result cannot be larger than isize max
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Aug 14, 2026
…unsigned-bound, r=clarfonthey

Optimize offset_from_unsigned result bounds

Hint that result cannot be larger than isize max
@JonathanBrouwer

Copy link
Copy Markdown
Member

💔 I suspect this PR failed tests as part of a rollup
@bors r-

After fixing the problem, consider running a try job for the failed job before re-approving.

Link to failure: #161089 (comment)

@bors try jobs=test-various

@rust-bors rust-bors Bot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 14, 2026
@rust-bors

rust-bors Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

This pull request was unapproved.

This PR was contained in a rollup (#161092), which was unapproved.

View changes since this unapproval

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Aug 14, 2026
…und, r=<try>

Optimize offset_from_unsigned result bounds


try-job: test-various
@rust-bors

rust-bors Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

💔 Test for f4b1164 failed: CI. Failed job:

@rust-log-analyzer

This comment has been minimized.

@clarfonthey

Copy link
Copy Markdown
Contributor

Not sure if spurious, will have to take a closer look later.

@clarfonthey

Copy link
Copy Markdown
Contributor

@rustbot review

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Aug 21, 2026
@clarfonthey

Copy link
Copy Markdown
Contributor

Finally getting back to this, confused why it passed CI in the PR but not on the rollup, but this is indeed a real issue. Running locally I also have a failure for the test that failed, tests/codegen-llvm/lib-optimizations/append-elements.rs.

So, @rustbot author

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 25, 2026
@rustbot

rustbot commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

Reminder, once the PR becomes ready for a review, use @rustbot ready.

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

Labels

S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. 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.

6 participants