-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Generate CodeView by deafult for Windows #22625
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
Conversation
After #22624 goes in this would be the most sane default. |
Please add a driver level test to ensure that we do this. |
CC: @dcci |
llvm, lldb and swift combine to offer better support for PDBs on Windows than it does DWARF. Default to generating PDBs on Windows.
@compnerd This fails to build on Windows due to the pdb gen bug you hit. WIP |
Let's see what happens @swift-ci please smoke test |
@swift-ci please test Windows platform |
welp, thats going to destroy the test suite on Windows; I think that we should ensure that those work before merging this |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree. Also it feels like a bug to me that Windows failing doesn't prevent the PR from being merged.
@lanza I think we can merge this if you XFAIL or fix these tests:
|
I wouldn't really XFAIL these tests. It means that there are potentially big holes in CodeView Windows support and merging this patch would lead to false expectations of what works and what doesn't (or maybe tests just need to be updated and everything works fine, in which case, well, we should do it sooner rather than later). |
@lanza I'm going to close this out since it seems like this needs to be handled incrementally to avoid disrupting the Windows build. |
llvm, lldb and swift combine to offer better support for PDBs on
Windows than it does DWARF. Default to generating PDBs on Windows.