Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

#[track_caller] on a macro declaration causes ICE #95151

Closed
c410-f3r opened this issue Mar 20, 2022 · 2 comments · Fixed by #96925
Closed

#[track_caller] on a macro declaration causes ICE #95151

c410-f3r opened this issue Mar 20, 2022 · 2 comments · Fixed by #96925
Assignees
@c410-f3r
Labels
E-needs-test Call for participation: An issue has been fixed and does not reproduce, but no test has been added. glacier ICE tracked in rust-lang/glacier. I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️

Comments

@c410-f3r
Copy link
Contributor

c410-f3r commented Mar 20, 2022 •
edited by rustbot
Loading

rustc 1.61.0-nightly (285fa7e 2022-03-14)

#[track_caller]
macro_rules! _foo {
    () => {};
}

fn main() {
}
error: internal compiler error: compiler/rustc_typeck/src/collect.rs:1978:13: unexpected sort of node in fn_sig()
@matthiaskrgr matthiaskrgr added the I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️ label Mar 20, 2022
@rust-lang-glacier-bot rust-lang-glacier-bot added the glacier ICE tracked in rust-lang/glacier. label Mar 22, 2022
@github-actions github-actions bot mentioned this issue May 10, 2022
Merged
@Alexendoo
Copy link
Member

No longer appears to ICE since #96473

@Alexendoo Alexendoo added the E-needs-test Call for participation: An issue has been fixed and does not reproduce, but no test has been added. label May 10, 2022
c410-f3r added a commit to c410-f3r/rust that referenced this issue May 10, 2022
@c410-f3r
Fix issue rust-lang#95151
aea3b22
@c410-f3r c410-f3r mentioned this issue May 10, 2022
Merged
@c410-f3r
Copy link
Contributor Author

@rustbot claim

@JohnTitor JohnTitor mentioned this issue May 11, 2022
Merged
bors added a commit to rust-lang-ci/rust that referenced this issue May 11, 2022
@bors
Auto merge of rust-lang#96931 - JohnTitor:rollup-3um8o4j, r=JohnTitor
ee6eaab 
Rollup of 7 pull requests

Successful merges:

 - rust-lang#96543 (Remove hacks in `make_token_stream`.)
 - rust-lang#96887 (rustdoc: correct path to type alias methods)
 - rust-lang#96896 (Add regression test for rust-lang#68408)
 - rust-lang#96900 (Fix js error)
 - rust-lang#96903 (Use lifetimes on type-alias-impl-trait used in function signatures to infer output type lifetimes)
 - rust-lang#96916 (simplify length count)
 - rust-lang#96925 (Fix issue rust-lang#95151)

Failed merges:

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors closed this as completed in 514b585 May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@c410-f3r c410-f3r

Labels
E-needs-test Call for participation: An issue has been fixed and does not reproduce, but no test has been added. glacier ICE tracked in rust-lang/glacier. I-ICE Issue: The compiler panicked, giving an Internal Compilation Error (ICE) ❄️
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix issue #95151 c410-f3r/rust
4 participants
@matthiaskrgr @Alexendoo @c410-f3r @rust-lang-glacier-bot