Skip to content

lower spans of error const args and test binder bounds - #163750

Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
paradoxicalguy:ice-span-wo-parent
Oct 5, 2026
Merged

rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
paradoxicalguy:ice-span-wo-parent

Conversation

@paradoxicalguy

@paradoxicalguy paradoxicalguy commented Oct 4, 2026 •

Copy link
Copy Markdown
Contributor

fixes #163592

lower spans when constructing hir nodes for error const args and test binder bound type constraints, these paths were using raw ast spans instead of routing them through lower_span
(reproduce with rust.debug-assertions = true and -Cincremental; the tests listed in the issue now pass.)

r? @bjorn3
cc @matthiaskrgr

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 4, 2026
@rustbot rustbot added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Oct 4, 2026
@bjorn3

bjorn3 commented Oct 4, 2026

Copy link
Copy Markdown
Member

Not familiar with this code.

r? compiler

@rustbot rustbot assigned oli-obk and unassigned bjorn3 Oct 4, 2026
@oli-obk

oli-obk commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@rust-bors

rust-bors Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

📌 Commit 63c07bd has been approved by oli-obk

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

rust-bors Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

⌛ Testing commit 63c07bd with merge 67e8503...

Workflow: https://github.com/rust-lang/rust/actions/runs/37280098399

rust-bors Bot pushed a commit that referenced this pull request Oct 5, 2026
lower spans of error const args and test binder bounds



fixes #163592

lower spans when constructing hir nodes for error const args and test binder bound type constraints, these paths were using raw ast spans instead of routing them through `lower_span`
(reproduce with `rust.debug-assertions = true` and `-Cincremental`; the tests listed in the issue now pass.)

r? @bjorn3 
cc @matthiaskrgr
@JonathanBrouwer

Copy link
Copy Markdown
Member

@bors yield
Yielding to rollup=never pr, this will go in the next rollup

@rust-bors

rust-bors Bot commented Oct 5, 2026

Copy link
Copy Markdown
Contributor

Auto build was cancelled. Cancelled workflows:

The next pull request likely to be tested is #156674.

rust-bors Bot pushed a commit that referenced this pull request Oct 5, 2026
…uwer

Rollup of 4 pull requests

Successful merges:

 - #163790 (miri subtree update)
 - #163610 (optimize empty `NestedGoals::extend_from_child`)
 - #163697 (cg_llvm: Use `LLVMGlobalAddMetadata` from LLVM-C)
 - #163750 (lower spans of error const args and test binder bounds)
@rust-bors
rust-bors Bot merged commit b9eb49a into rust-lang:main Oct 5, 2026
14 of 15 checks passed
@rustbot rustbot added this to the 1.101.0 milestone Oct 5, 2026
rust-bors Bot pushed a commit that referenced this pull request Oct 5, 2026
Rollup merge of #163750 - paradoxicalguy:ice-span-wo-parent, r=oli-obk

lower spans of error const args and test binder bounds

fixes #163592

lower spans when constructing hir nodes for error const args and test binder bound type constraints, these paths were using raw ast spans instead of routing them through `lower_span`
(reproduce with `rust.debug-assertions = true` and `-Cincremental`; the tests listed in the issue now pass.)

r? @bjorn3
cc @matthiaskrgr
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-compiler Relevant 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.

error: internal compiler error: compiler/rustc_ast_lowering/src/index.rs:103:17: span without a parent

5 participants