Skip to content

cmake: arm: linker: remove usage of ld-specific ADDR() function #88599

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 1 commit into from
Apr 24, 2025

Conversation

danieldegrasse
Copy link
Collaborator

Remove usage of ld-specific ADDR function within the cmake linker generator scripting. Since the linker generator scripting doesn't support MPUs, we can simply set the __ramfunc_region_start symbol to be equal to __ramfunc_start

Fixes #87200

Remove usage of ld-specific ADDR function within the cmake linker
generator scripting. Since the linker generator scripting doesn't
support MPUs, we can simply set the __ramfunc_region_start symbol to be
equal to __ramfunc_start

Fixes zephyrproject-rtos#87200

Signed-off-by: Daniel DeGrasse <[email protected]>
@danieldegrasse danieldegrasse requested a review from tejlmand April 14, 2025 15:22
@danieldegrasse
Copy link
Collaborator Author

@tejlmand can you verify this fixes the issue on Arm Compiler 6? I don't have a license to verify

@github-actions github-actions bot added area: Build System size: XS A PR changing only a single line of code labels Apr 14, 2025
@carlescufi
Copy link
Member

@tejlmand can you verify this fixes the issue on Arm Compiler 6? I don't have a license to verify

Not that it matters much, the proprietary toolchain is being retired in favor of just vanilla LLVM/Clang:
#88584 (comment)

@tejlmand
Copy link
Collaborator

@tejlmand can you verify this fixes the issue on Arm Compiler 6? I don't have a license to verify

This PR fixes the issue, though while testing I have hit another unrelated issue.

@kartben kartben merged commit b84076a into zephyrproject-rtos:main Apr 24, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Build System size: XS A PR changing only a single line of code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Use of ld specic function (ADDR) break non-ld linkers
4 participants