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
[DNM] Use normalized paths when emitting a module trace. #29967
base: main
Are you sure you want to change the base?
[DNM] Use normalized paths when emitting a module trace. #29967
Conversation
@swift-ci please smoke test Windows platform |
Actually, this is not 100% right, because when we search, we should normalize the paths as well. |
@swift-ci please smoke test |
But I think this should work as a temporary fix since the paths that are problematic are the ones in the map. I wrote some code to normalize the paths that are being looked up, but that crashed for some odd reason inside DenseMap's find, probably a lifetime problem 😕. |
@swift-ci please test Windows platform |
@swift-ci please test Windows platform |
Marking DNM temporarily, I'll get back to this on the weekend. |
@swift-ci please test Windows platform |
No description provided.