Skip to content

[6.2][CAS] Allow uncached job from CAS based dependency scanning #82069

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

Open
wants to merge 1 commit into
base: release/6.2
Choose a base branch
from

Conversation

cachemeifyoucan
Copy link
Contributor

  • Explanation: Add a new option to allow swift compiler to load modules from CAS but skip caching and load source file from disk. This allows uncached build to quickly iterate on the current module without full dependency scanning every run.
  • Scope: The new option is added for preview support when swift caching is enabled so preview can share the same build dependency as the rest of the build.
  • Issues: rdar://152441866
  • Original PRs: [CAS] Allow uncached job from CAS based dependency scanning #81990
  • Risk: Low. A new option added to support preview when swift caching build is enabled.
  • Testing: Unit test
  • Reviewers: @benlangmuir

Create a path that swift-frontend can execute an uncached job from
modules built with CAS based explicit module build. The new flag
-import-module-from-cas will allow an uncached build to load module
from CAS, and combined with source file from real file system to build
the current module. This allows quick iterations that bypasses CAS,
without full dependency scanning every time in between.

rdar://152441866
@cachemeifyoucan cachemeifyoucan requested a review from a team as a code owner June 6, 2025 18:30
@cachemeifyoucan
Copy link
Contributor Author

@swift-ci please test

@cachemeifyoucan cachemeifyoucan changed the title [CAS] Allow uncached job from CAS based dependency scanning [6.2][CAS] Allow uncached job from CAS based dependency scanning Jun 6, 2025
Copy link
Contributor

@nkcsgexi nkcsgexi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding a new option when CAS is enabled should be safe.

@cachemeifyoucan
Copy link
Contributor Author

@swift-ci please test macOS platform

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.

3 participants