Skip to content

Skip 32 bit relocation addend partitioning in MCCAS. #8444

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

Conversation

rastogishubham
Copy link

Relocations addednds do not deduplicate, to improve deduplication in MCCAS, we store those addends in an AddendsRef block. This is much more complicated to do in 32 bit architectures because of various corner cases and complexities with the MachO 32-bit relocation format Therefore we skip relocation partitioning for 32-bit architectures in MCCAS.

(cherry picked from commit ff2d0ee)

Relocations addednds do not deduplicate, to improve deduplication in
MCCAS, we store those addends in an AddendsRef block. This is much more
complicated to do in 32 bit architectures because of various corner
cases and complexities with the MachO 32-bit relocation format
Therefore we skip relocation partitioning for 32-bit architectures in
MCCAS.

(cherry picked from commit ff2d0ee)
@rastogishubham
Copy link
Author

@swift-ci please test

@rastogishubham rastogishubham merged commit c079606 into swiftlang:stable/20230725 Mar 21, 2024
@rastogishubham rastogishubham deleted the MCCAS32BitRelocStable branch March 21, 2024 21:18
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.

1 participant