-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Add replay support for MCCAS in Swift. #74202
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
156d06a
to
f894b02
Compare
@swift-ci please test |
f894b02
to
b4bc1c9
Compare
@swift-ci please test |
b4bc1c9
to
465f588
Compare
@swift-ci please test |
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.
The layering is wrong that you put MCCAS object writing into CachedDiagnostics. You need to find a better place to put it.
465f588
to
b01c2be
Compare
b01c2be
to
2b9d5d4
Compare
2b9d5d4
to
eae62dc
Compare
eae62dc
to
2572496
Compare
@swift-ci please test |
@cachemeifyoucan sorry for getting back on this a little late, but I think I addressed everything |
When MCCAS is used, filter the object file from the CASOutputBackend because we do not want to write the object file in the CAS, we want to associate the MCCAS top level CAS Object to the main output of the CASOutputBackend.
This patch stores the MCCAS ObjectRef as the main output of the CASOutputBackend when MCCAS is enabled.
2572496
to
93f514e
Compare
@swift-ci please test |
e359604
to
1d4d211
Compare
@swift-ci please test |
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.
LGTM after addressing the one last comment.
This patch adds support for MCCAS when a cache hit is encountered when trying to replay a compilation, and uses the MCCAS serialization code to materialize the object file that is the main output of the compilation.
1d4d211
to
8dabf58
Compare
@swift-ci please test |
@swift-ci please smoke test |
This patch adds support to be able to replay a cached object file from MCCAS in the swift compiler.