-
Notifications
You must be signed in to change notification settings - Fork 12.9k
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
Compiler error while compiling Winrt #66402
Comments
Can you please provide a backtrace, and if possible, a minimal reproducer? Thanks. cc @wesleywiser |
also, I think this is the winrt repo: https://github.com/contextfree/winrt-rust |
triage: P-high for now. Removing nomination. |
I was able to reproduce it,
|
Yeah, sorry, haven't been on this machine for a few days. Areredify's backtrace looks right compared to what I saw before. Heres the full backtrace just in case.
|
Using |
closing as fixed by PR #66378 |
@wesleywiser points out this may need a regression test. Reopening, re-prioritizing as P-medium and marking E-needstest. |
Triage: I agree it would be good to have a regression test for the ICE. But now, more than three years later, I think we have to accept that no one will write a regression test. Had there been a minimal reproducer, I would have written a regression test myself. But by now it simply is too much effort to figure out how to reproduce and figure out how to minimize the reproducer. There are 3595 other C-bugs open in this repo (of which 584 are I-ICE and 113 are E-needs-test) that deserves more attention than this issue, and there will always be issues that deserves more attention than this issue. Let's close this. |
Trying to compile winrt with latest nightly msvc got a internal compiler error.
Saw similar error over with this one: #64908 so figured I would report here and not in the winrt github. Project compiles fine with latest stable on
x86-64-pc-windows-msvc
.The text was updated successfully, but these errors were encountered: