Skip to content

[Caching] Use swift-frontend as executable name in replays #75555

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

Merged
merged 2 commits into from
Jul 30, 2024

Conversation

cachemeifyoucan
Copy link
Contributor

Don't try to figure out the executable names during replay from
libSwiftScan.dylib. The actual executable path for the process actually
doesn't matter in this case to reconstruct the invocation and might
actually be misleading.

Just use swift-frontend as a placeheader executable name for
in-process cache replay.

rdar://132758308

@cachemeifyoucan cachemeifyoucan changed the title [Caching] Use swift-frontend as exectuable name in replays [Caching] Use swift-frontend as executable name in replays Jul 29, 2024
@cachemeifyoucan
Copy link
Contributor Author

@swift-ci please smoke test

Fix an assertion when replayDiagnostics failed in libSwiftScan.
Don't try to figure out the executable names during replay from
libSwiftScan.dylib. The actual executable path for the process actually
doesn't matter in this case to reconstruct the invocation and might
actually be misleading.

Just use `swift-frontend` as a placeheader executable name for
in-process cache replay.

rdar://132758308
@cachemeifyoucan
Copy link
Contributor Author

@swift-ci please smoke test

@cachemeifyoucan
Copy link
Contributor Author

@swift-ci please smoke test linux platform

@cachemeifyoucan cachemeifyoucan merged commit d736106 into swiftlang:main Jul 30, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants