Skip to content

Add a test for a past ICE when calling a const fn of an unresolved type with the wrong number of args - #154671

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
jakubadamw:issue-127423
Apr 2, 2026
Merged

Add a test for a past ICE when calling a const fn of an unresolved type with the wrong number of args#154671
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
jakubadamw:issue-127423

Conversation

@jakubadamw

@jakubadamwjakubadamw commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

The ICE is fixed, but there needs to be a regression test for it.

Closes#127423.

@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 Apr 1, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

r? @adwinwhite

rustbot has assigned @adwinwhite.
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 69 candidates
  • Random selection from 13 candidates

Comment threadtests/ui/consts/ice-extra-args-fn-abi-issue-127423.rs Outdated
@rustbotrustbot 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 Apr 1, 2026
@rustbot

Copy link
Copy Markdown
Collaborator

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

@jakubadamw

Copy link
Copy Markdown
ContributorAuthor

@rustbot ready

@rustbotrustbot 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 Apr 1, 2026
@Kivooeo

Copy link
Copy Markdown
Member

r? me @bors r+ rollup

@rust-bors

rust-borsBot commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

📌 Commit d1f11b2 has been approved by Kivooeo

It is now in the queue for this repository.

@rust-borsrust-borsBot added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Apr 1, 2026
@rust-borsrust-borsBot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 1, 2026
jhpratt added a commit to jhpratt/rust that referenced this pull request Apr 2, 2026
Add a test for a past ICE when calling a const fn of an unresolved type with the wrong number of args
The ICE is fixed, but there needs to be a regression test for it.
Closesrust-lang#127423.
@jhprattjhpratt mentioned this pull request Apr 2, 2026
rust-borsBot pushed a commit that referenced this pull request Apr 2, 2026
Rollup of 11 pull requests
Successful merges:
- #153105 (Compute the result of a projection type with region errors)
- #153286 (various fixes for scalable vectors)
- #153960 (Make `layout_of` cycles fatal errors)
- #154527 (Emit pre-expansion feature gate warnings for negative impls and specialization)
- #154666 (Remove `StableHashContext` impls)
- #154669 (Introduce #[diagnostic::on_move] on `Arc`)
- #154442 (Export `derive` at the crate root: `core::derive` and `std::derive`)
- #154644 (rustdoc: seperate methods and associated functions in sidebar)
- #154660 (Avoid creating async return opaques for foreign async fns)
- #154671 (Add a test for a past ICE when calling a const fn of an unresolved type with the wrong number of args)
- #154680 ([rustdoc] Replace `DocContext` with `TyCtxt` wherever possible)
jhpratt added a commit to jhpratt/rust that referenced this pull request Apr 2, 2026
Add a test for a past ICE when calling a const fn of an unresolved type with the wrong number of args
The ICE is fixed, but there needs to be a regression test for it.
Closesrust-lang#127423.
@jhprattjhpratt mentioned this pull request Apr 2, 2026
rust-borsBot pushed a commit that referenced this pull request Apr 2, 2026
Rollup of 11 pull requests
Successful merges:
- #153105 (Compute the result of a projection type with region errors)
- #153960 (Make `layout_of` cycles fatal errors)
- #154527 (Emit pre-expansion feature gate warnings for negative impls and specialization)
- #154666 (Remove `StableHashContext` impls)
- #154669 (Introduce #[diagnostic::on_move] on `Arc`)
- #154213 (tidy-alphabetical: fix line number in error message)
- #154442 (Export `derive` at the crate root: `core::derive` and `std::derive`)
- #154644 (rustdoc: seperate methods and associated functions in sidebar)
- #154660 (Avoid creating async return opaques for foreign async fns)
- #154671 (Add a test for a past ICE when calling a const fn of an unresolved type with the wrong number of args)
- #154680 ([rustdoc] Replace `DocContext` with `TyCtxt` wherever possible)
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Apr 2, 2026
Add a test for a past ICE when calling a const fn of an unresolved type with the wrong number of args
The ICE is fixed, but there needs to be a regression test for it.
Closesrust-lang#127423.
rust-borsBot pushed a commit that referenced this pull request Apr 2, 2026
…uwer
Rollup of 21 pull requests
Successful merges:
- #153105 (Compute the result of a projection type with region errors)
- #153286 (various fixes for scalable vectors)
- #153532 (Attributes containing rustc)
- #153960 (Make `layout_of` cycles fatal errors)
- #154527 (Emit pre-expansion feature gate warnings for negative impls and specialization)
- #154666 (Remove `StableHashContext` impls)
- #154669 (Introduce #[diagnostic::on_move] on `Arc`)
- #154710 (opaque_generic_const_args -> generic_const_args)
- #154712 (Revert "`-Znext-solver` Remove the forced ambiguity hack from search graph")
- #154713 (Stop compiling when we get resolving crate failure)
- #154213 (tidy-alphabetical: fix line number in error message)
- #154425 (Migrate transmute tests)
- #154442 (Export `derive` at the crate root: `core::derive` and `std::derive`)
- #154469 (mGCA: Lower spans for literal const args)
- #154578 (Rename `probe_ty_var` to `try_resolve_ty_var`)
- #154615 (Moving issues)
- #154644 (rustdoc: seperate methods and associated functions in sidebar)
- #154660 (Avoid creating async return opaques for foreign async fns)
- #154671 (Add a test for a past ICE when calling a const fn of an unresolved type with the wrong number of args)
- #154680 ([rustdoc] Replace `DocContext` with `TyCtxt` wherever possible)
- #154709 (Revert `Ty` type alias in `rustc_type_ir`)
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request Apr 2, 2026
Add a test for a past ICE when calling a const fn of an unresolved type with the wrong number of args
The ICE is fixed, but there needs to be a regression test for it.
Closesrust-lang#127423.
rust-borsBot pushed a commit that referenced this pull request Apr 2, 2026
…uwer
Rollup of 21 pull requests
Successful merges:
- #153105 (Compute the result of a projection type with region errors)
- #153286 (various fixes for scalable vectors)
- #153532 (Attributes containing rustc)
- #153960 (Make `layout_of` cycles fatal errors)
- #154527 (Emit pre-expansion feature gate warnings for negative impls and specialization)
- #154666 (Remove `StableHashContext` impls)
- #154669 (Introduce #[diagnostic::on_move] on `Arc`)
- #154710 (opaque_generic_const_args -> generic_const_args)
- #154712 (Revert "`-Znext-solver` Remove the forced ambiguity hack from search graph")
- #153614 (`FindParamInClause` handle edge-cases)
- #154213 (tidy-alphabetical: fix line number in error message)
- #154425 (Migrate transmute tests)
- #154442 (Export `derive` at the crate root: `core::derive` and `std::derive`)
- #154469 (mGCA: Lower spans for literal const args)
- #154578 (Rename `probe_ty_var` to `try_resolve_ty_var`)
- #154615 (Moving issues)
- #154644 (rustdoc: seperate methods and associated functions in sidebar)
- #154660 (Avoid creating async return opaques for foreign async fns)
- #154671 (Add a test for a past ICE when calling a const fn of an unresolved type with the wrong number of args)
- #154680 ([rustdoc] Replace `DocContext` with `TyCtxt` wherever possible)
- #154709 (Revert `Ty` type alias in `rustc_type_ir`)
rust-borsBot pushed a commit that referenced this pull request Apr 2, 2026
…uwer
Rollup of 20 pull requests
Successful merges:
- #153105 (Compute the result of a projection type with region errors)
- #153532 (Attributes containing rustc)
- #153960 (Make `layout_of` cycles fatal errors)
- #154527 (Emit pre-expansion feature gate warnings for negative impls and specialization)
- #154666 (Remove `StableHashContext` impls)
- #154669 (Introduce #[diagnostic::on_move] on `Arc`)
- #154710 (opaque_generic_const_args -> generic_const_args)
- #154712 (Revert "`-Znext-solver` Remove the forced ambiguity hack from search graph")
- #153614 (`FindParamInClause` handle edge-cases)
- #154213 (tidy-alphabetical: fix line number in error message)
- #154425 (Migrate transmute tests)
- #154442 (Export `derive` at the crate root: `core::derive` and `std::derive`)
- #154469 (mGCA: Lower spans for literal const args)
- #154578 (Rename `probe_ty_var` to `try_resolve_ty_var`)
- #154615 (Moving issues)
- #154644 (rustdoc: seperate methods and associated functions in sidebar)
- #154660 (Avoid creating async return opaques for foreign async fns)
- #154671 (Add a test for a past ICE when calling a const fn of an unresolved type with the wrong number of args)
- #154680 ([rustdoc] Replace `DocContext` with `TyCtxt` wherever possible)
- #154709 (Revert `Ty` type alias in `rustc_type_ir`)
@rust-bors
rust-borsBot merged commit 39ba2cd into rust-lang:mainApr 2, 2026
11 checks passed
@rustbotrustbot added this to the 1.96.0 milestone Apr 2, 2026
rust-timer added a commit that referenced this pull request Apr 2, 2026
Rollup merge of #154671 - jakubadamw:issue-127423, r=Kivooeo
Add a test for a past ICE when calling a const fn of an unresolved type with the wrong number of args
The ICE is fixed, but there needs to be a regression test for it.
Closes#127423.
github-actionsBot pushed a commit to rust-lang/miri that referenced this pull request Apr 3, 2026
…uwer
Rollup of 20 pull requests
Successful merges:
- rust-lang/rust#153105 (Compute the result of a projection type with region errors)
- rust-lang/rust#153532 (Attributes containing rustc)
- rust-lang/rust#153960 (Make `layout_of` cycles fatal errors)
- rust-lang/rust#154527 (Emit pre-expansion feature gate warnings for negative impls and specialization)
- rust-lang/rust#154666 (Remove `StableHashContext` impls)
- rust-lang/rust#154669 (Introduce #[diagnostic::on_move] on `Arc`)
- rust-lang/rust#154710 (opaque_generic_const_args -> generic_const_args)
- rust-lang/rust#154712 (Revert "`-Znext-solver` Remove the forced ambiguity hack from search graph")
- rust-lang/rust#153614 (`FindParamInClause` handle edge-cases)
- rust-lang/rust#154213 (tidy-alphabetical: fix line number in error message)
- rust-lang/rust#154425 (Migrate transmute tests)
- rust-lang/rust#154442 (Export `derive` at the crate root: `core::derive` and `std::derive`)
- rust-lang/rust#154469 (mGCA: Lower spans for literal const args)
- rust-lang/rust#154578 (Rename `probe_ty_var` to `try_resolve_ty_var`)
- rust-lang/rust#154615 (Moving issues)
- rust-lang/rust#154644 (rustdoc: seperate methods and associated functions in sidebar)
- rust-lang/rust#154660 (Avoid creating async return opaques for foreign async fns)
- rust-lang/rust#154671 (Add a test for a past ICE when calling a const fn of an unresolved type with the wrong number of args)
- rust-lang/rust#154680 ([rustdoc] Replace `DocContext` with `TyCtxt` wherever possible)
- rust-lang/rust#154709 (Revert `Ty` type alias in `rustc_type_ir`)
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.

ICE: sig.c_variadic || extra_args.is_empty()

4 participants

@jakubadamw@rustbot@Kivooeo@adwinwhite