Skip to content

Rollup of 14 pull requests - #160667

Closed
jhpratt wants to merge 87 commits into
rust-lang:mainfrom
jhpratt:rollup-JUufhLd
Closed

Rollup of 14 pull requests#160667
jhpratt wants to merge 87 commits into
rust-lang:mainfrom
jhpratt:rollup-JUufhLd

Conversation

@jhpratt

@jhpratt jhpratt commented Aug 7, 2026

Copy link
Copy Markdown
Member

Successful merges:

r? @ghost

Create a similar rollup

teor2345 and others added 30 commits July 31, 2026 19:53
Add tests for generic function pointers
Change FnPtr tests to use assert_eq!() rather than println!()
don't force intrinsic results into memory
This updates the rust-version file to 7218ebe.
Move command-result printing into a helper and keep CLI loop control at the call site.
Consume Priroda's --dap flag before handing arguments to rustc_driver::run_compiler, then dispatch the freshly-created PrirodaContext to either the existing CLI loop or a new DAP loop stub.
Add FirstUserSourceLocation ResumeMode variant that stops when the
interpreter reaches a user-relevant frame with a source location.
This gives the DAP frontend an entry-stop primitive that skips
Miri-internal and std frames.
Wire `next` and `stepIn` DAP requests to Priroda's existing source-line
step.  Both commands use the same `handle_step` handler for now; true
step-over vs step-in semantics are deferred.

Add `stopped_reason` to map `StepResult` variants to DAP
`StoppedEventReason` so the editor can distinguish a manual step from a
breakpoint hit.  Add a `handle_disconnect` handler that sends the
`terminated` event and exits the session cleanly.

Document the `SourceLocation` span-storage rationale and refine the
`SourceLine` resume-mode comment to be clearer about the "no source
location → first mapped location" semantics.
When the session receives an unsupported DAP request, return
`DispatchOutcome::Continue` instead of `Exit` so the debug adapter keeps
running after sending the error response.  Remove the `eprintln!` side
channel from `handle_unsupported_request` since the framed DAP error
response is the single authoritative error-reporting path.

Include the command name in the error message string so the DAP client
sees which request was rejected.
@rustbot rustbot added 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. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue. labels Aug 7, 2026
@jhpratt

jhpratt commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

@bors r+ p=6

@bors try jobs=dist-various-1,test-various,x86_64-gnu-aux,x86_64-gnu-llvm-21-3,x86_64-msvc-1,aarch64-apple-,x86_64-mingw-1,i686-msvc-

@rust-bors

rust-bors Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

📌 Commit a996a4d has been approved by jhpratt

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 7, 2026
@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Aug 7, 2026
Rollup of 14 pull requests


try-job: dist-various-1
try-job: test-various
try-job: x86_64-gnu-aux
try-job: x86_64-gnu-llvm-21-3
try-job: x86_64-msvc-1
try-job: aarch64-apple-*
try-job: x86_64-mingw-1
try-job: i686-msvc-*
@jhpratt

jhpratt commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

GitHub is showing that they're fully operational.

@bors treeclosed-

@rust-bors

rust-bors Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Tree is now open for merging.

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Aug 7, 2026
Rollup of 14 pull requests

Successful merges:

 - #150885 (Revive L4Re target)
 - #159643 (Add support for splatted function pointers)
 - #160433 (delegation: add support for wrapping of the return value with `From::from`)
 - #160530 (refactor handling of target features in Session)
 - #160606 (bootstrap: Store and use an explicit CheckKind in `check::Rustc`)
 - #160628 (fix ICE in `suggest_add_reference_to_arg` for non-callable items)
 - #160634 (miri subtree update)
 - #158904 (Fix FutureDropPoll shim for by-move async closures)
 - #160335 (dlopen offload)
 - #160445 (codegen: classify localized MSVC linker progress as linker_info)
 - #160504 (cleanup borrowck, improve c-variadic handling)
 - #160587 (Add regression test for associated type outlives bound at call site)
 - #160625 (platform-support/netbsd.md: No longer mention 8.x, due to EoL.)
 - #160636 (derive(Diagnostic): link to proper docs)
@rust-bors

rust-bors Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: 76a8b0e (76a8b0ea796adba486491d72a20ec44d8e2bd8fa)
Base parent: 84b36a7 (84b36a78a28a63f134171c670be1932ffa2485f8)

@rust-log-analyzer

Copy link
Copy Markdown
Collaborator

The job dist-x86_64-musl failed! Check out the build log: (web) (plain enhanced) (plain)

Click to see the possible cause of the failure (guessed by this bot)

@rust-bors rust-bors Bot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 7, 2026
@rust-bors

rust-bors Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

💔 Test for 9fdf63e failed: CI. Failed job:

@jhpratt

jhpratt commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

@bors retry

@bors try dist-x86_64-musl

@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 7, 2026
@rust-bors

rust-bors Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Unknown argument "dist-x86_64-musl". Did you mean to use @bors jobs=<jobs>|parent=<parent>? Run @bors help or go to https://bors.rust-lang.org/help to see available commands.

@jhpratt

jhpratt commented Aug 7, 2026

Copy link
Copy Markdown
Member Author

@bors try jobs=dist-x86_64-musl

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Aug 7, 2026
Rollup of 14 pull requests


try-job: dist-x86_64-musl
@rust-bors

rust-bors Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: b5fbc42 (b5fbc422942446010f6d430783e204a6f5235a41)
Base parent: 88f7399 (88f7399cb4912680976e1c342ea7661b9dc84940)

@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 7, 2026
@rust-bors

rust-bors Bot commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

This pull request was unapproved due to being closed.

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

Labels

A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. A-run-make Area: port run-make Makefiles to rmake.rs A-rustdoc-json Area: Rustdoc JSON backend O-unix Operating system: Unix-like rollup A PR which is a rollup S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) 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. T-rustdoc Relevant to the rustdoc team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.