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.

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

fix tail call checks wrt #[track_caller] #135973

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

WaffleLapkin
Copy link
Member

Only check the caller + disallow caller having the attribute.

fixes #134336

r? @compiler-errors

apparently there were no tests for #[track_caller] before... ooops

@rustbot rustbot 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 Jan 24, 2025
@WaffleLapkin WaffleLapkin added the F-explicit_tail_calls `#![feature(explicit_tail_calls)]` label Jan 24, 2025
@rust-log-analyzer

This comment has been minimized.

@WaffleLapkin WaffleLapkin force-pushed the tail-track-caller-fix branch from 032f4e7 to d746614 Compare January 24, 2025 19:14
@rust-log-analyzer

This comment has been minimized.

only check the caller + disallow caller having the attr.
@WaffleLapkin WaffleLapkin force-pushed the tail-track-caller-fix branch from d746614 to e4229ac Compare January 24, 2025 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
F-explicit_tail_calls `#![feature(explicit_tail_calls)]` 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ICE: failed to resolve instance for <Self as EnvFuture>::g
4 participants