Skip to content

[Cherry-pick into next] Register the symbol context's module's AST files first. #8628

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Apr 23, 2024
Merged

[Cherry-pick into next] Register the symbol context's module's AST files first. #8628

merged 1 commit into from
Apr 23, 2024

Conversation

adrian-prantl
Copy link 

commit 7dbd6b79448f6d7abf672025cf08022cf3e7d249
Author: Adrian Prantl <[email protected]>
Date:   Wed Apr 17 16:36:46 2024 -0700

    Register the symbol context's module's AST files first.
    
    This makes it more likely that compatible AST blobs are found first,
    since then the local AST blobs overwrite any ones with the same import
    path from another dylib. The AST blobs are registered in a DFS
    following the dynamic library dependencies in the binary.

@adrian-prantl
Register the symbol context's module's AST files first.
09752d7 
This makes it more likely that compatible AST blobs are found first,
since then the local AST blobs overwrite any ones with the same import
path from another dylib. The AST blobs are registered in a DFS
following the dynamic library dependencies in the binary.

(cherry picked from commit 7dbd6b7)
@adrian-prantl adrian-prantl merged commit 4d929a1 into swiftlang:next Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JDevlieghere JDevlieghere Awaiting requested review from JDevlieghere JDevlieghere is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@adrian-prantl