-
Notifications
You must be signed in to change notification settings - Fork 12.7k
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
Implement unwinding for Windows #908
Comments
Reassigning the "Application has requested the Runtime to terminate it in an abnormal way" FIXME in rust_kernel::fail to this bug, 'cause it's related to failure on windows, so I can close #2671. |
Not critical for 0.6; de-milestoning |
Nominating for Maturity 3. |
accepted for feature-complete milestone |
Some LLVM patches exist for simple Windows 64 Exception Handling support in LLVM. See LLVMdev message http://lists.cs.uiuc.edu/pipermail/llvmdev/2013-April/060838.html |
This resolves issue #908. Notable changes: - On Windows, LLVM integrated assembler emits bad stack unwind tables when segmented stacks are enabled. However, unwind info directives in the assembly output are correct, so we generate assembly first and then run it through an external assembler, just like it is already done for Android builds. - Linker is invoked via "g++" command instead of "gcc": g++ passes the appropriate magic parameters to the linker, which ensure correct registration of stack unwind tables in dynamic libraries.
This can be closed now. |
hurrah! |
Update 0840-no-panic-in-c-string.md
Bumps [proc-macro2](https://github.com/alexcrichton/proc-macro2) from 1.0.8 to 1.0.9. - [Release notes](https://github.com/alexcrichton/proc-macro2/releases) - [Commits](dtolnay/proc-macro2@1.0.8...1.0.9) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
* Custom zero * Enable on differentials * Cleanup
Windows can't unwind on task failure because LLVM exception handling doesn't work there.
The text was updated successfully, but these errors were encountered: