Skip to content

[6.2][IRGen] Set generic context before getting call emission in visitFull… #83074

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

Open
wants to merge 1 commit into
base: release/6.2
Choose a base branch
from

Conversation

drexin
Copy link
Contributor

@drexin drexin commented Jul 15, 2025

…ApplySite

  • Explanation: Without the generic context, the result type can't be mapped into the current context, causing the compiler to crash.
  • Scope: Code gen for typed throws in generic context
  • Issues: rdar://149007227
  • Original PRs: [IRGen] Set generic context before getting call emission in visitFull… #80746
  • Risk: The only change is that the generic context gets set earlier, so it's available in the call emission. This should not affect anything but the bug this fixes.
  • Testing: Added regression tests.
  • Reviewers: @aschwaighofer

…ApplySite

rdar://149007227

Without the generic context, the result type can't be mapped into the current context, causing the compiler to crash.
@drexin drexin requested a review from a team as a code owner July 15, 2025 20:41
@drexin
Copy link
Contributor Author

drexin commented Jul 15, 2025

@swift-ci test

@drexin drexin added 🍒 release cherry pick Flag: Release branch cherry picks swift 6.2 labels Jul 15, 2025
@drexin
Copy link
Contributor Author

drexin commented Jul 15, 2025

cc: @tbkka

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 6.2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants