Skip to content

coverage: Reduce and clarify the context-mismatch test case - #156604

Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
Zalathar:context-mismatch
May 16, 2026
Merged

coverage: Reduce and clarify the context-mismatch test case#156604
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
Zalathar:context-mismatch

Conversation

@Zalathar

Copy link
Copy Markdown
Member

Most of the complexity in this test case isn't needed to reproduce the condition observed in #147339.

I have checked that this reduced test case still triggers the context-mismatch in extract_refined_covspans.

@rustbotrustbot 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. labels May 15, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

r? @mu001999

rustbot has assigned @mu001999.
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 73 candidates
  • Random selection from 17 candidates

@mu001999

Copy link
Copy Markdown
Member

@bors r+ rollup

@rust-bors

rust-borsBot commented May 15, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 5d15124 has been approved by mu001999

It is now in the queue for this repository.

@rust-borsrust-borsBot 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 May 15, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 15, 2026
coverage: Reduce and clarify the context-mismatch test case
Most of the complexity in this test case isn't needed to reproduce the condition observed in rust-lang#147339.
I have checked that this reduced test case still triggers the context-mismatch in `extract_refined_covspans`.
rust-borsBot pushed a commit that referenced this pull request May 15, 2026
…uwer
Rollup of 12 pull requests
Successful merges:
- #153238 (debuginfo: slices are DW_TAG_array_type's)
- #156319 (Require EIIs to be defined when we compile a rust dylib)
- #156452 (Implement pinned drop sugar)
- #156554 (Allow user-provided `llvm_args` to override target spec arguments)
- #156571 (Disable `main_needs_argc_argv` for Wasm)
- #156600 (Make const param default test reproduce original ICE)
- #156493 (actually run the temp_dir doctest)
- #156556 (Require UTF-8 in `Utf8Pattern::StringPattern`)
- #156565 (delegation: emit error when self type is not specified and accessed)
- #156586 (Use DropCtxt::new_block and new_block_with_statements systematically.)
- #156587 (Correctly handle associated items in rustdoc macro expansion)
- #156604 (coverage: Reduce and clarify the context-mismatch test case)
rust-borsBot pushed a commit that referenced this pull request May 15, 2026
…uwer
Rollup of 12 pull requests
Successful merges:
- #148788 (Unconstrained parameter fix)
- #156319 (Require EIIs to be defined when we compile a rust dylib)
- #156452 (Implement pinned drop sugar)
- #156554 (Allow user-provided `llvm_args` to override target spec arguments)
- #156571 (Disable `main_needs_argc_argv` for Wasm)
- #156600 (Make const param default test reproduce original ICE)
- #156493 (actually run the temp_dir doctest)
- #156556 (Require UTF-8 in `Utf8Pattern::StringPattern`)
- #156565 (delegation: emit error when self type is not specified and accessed)
- #156586 (Use DropCtxt::new_block and new_block_with_statements systematically.)
- #156587 (Correctly handle associated items in rustdoc macro expansion)
- #156604 (coverage: Reduce and clarify the context-mismatch test case)
@rust-bors
rust-borsBot merged commit d1da72a into rust-lang:mainMay 16, 2026
11 checks passed
rust-timer added a commit that referenced this pull request May 16, 2026
Rollup merge of #156604 - Zalathar:context-mismatch, r=mu001999
coverage: Reduce and clarify the context-mismatch test case
Most of the complexity in this test case isn't needed to reproduce the condition observed in #147339.
I have checked that this reduced test case still triggers the context-mismatch in `extract_refined_covspans`.
@rustbotrustbot added this to the 1.97.0 milestone May 16, 2026
@Zalathar
Zalathar deleted the context-mismatch branch May 16, 2026 01:07
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@Zalathar@rustbot@mu001999