Skip to content
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

[sram-exec] Avoid using compressed instructions when loading sp. #26065

Merged
merged 1 commit into from
Jan 30, 2025

Conversation

moidx
Copy link
Contributor

@moidx moidx commented Jan 30, 2025

This changes solves an issue that caused larger programs to return the following linker error:

relocation truncated to fit: R_RISCV_GPREL_I against symbol `_stack_end' defined in .sram_start

Using the norelax option avoids the use of compressed instructions which was causing the _stack_end address calculation error.

Patch submitted by [email protected].

This changes solves an issue that caused larger programs to return the
following linker error:

```
relocation truncated to fit: R_RISCV_GPREL_I against symbol `_stack_end' defined in .sram_start
```

Using the `norelax` option avoids the use of compressed instructions
which was causing the `_stack_end` address calculation error.

Patch submitted by [email protected].

Signed-off-by: Miguel Osorio <[email protected]>
@moidx moidx merged commit 5b072fc into lowRISC:earlgrey_1.0.0 Jan 30, 2025
32 checks passed
@moidx moidx deleted the sram-sp-calculation branch January 30, 2025 05:38
@moidx moidx added the CherryPick:master This PR should be cherry-picked to master label Jan 30, 2025
Copy link

Copy link

Git push to origin failed for master with exitcode 1

@github-actions github-actions bot added the Manually CherryPick This PR should be manually cherry picked. label Jan 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CherryPick:master This PR should be cherry-picked to master Manually CherryPick This PR should be manually cherry picked.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants