Skip to content

riscv: rename sbadaddr to stval#345

Merged
badochov merged 1 commit intomasterfrom
badochov/riscv-stval
Sep 11, 2024
Merged

riscv: rename sbadaddr to stval#345
badochov merged 1 commit intomasterfrom
badochov/riscv-stval

Conversation

@badochov
Copy link

@badochov badochov commented Sep 10, 2024

RISC-V Privileged Architectures V1.10 renamed and generalized sbadaddr to stval. binutils >= 2.43 dropped support for sbadaddr alias.

JIRA: RTOS-913

Description

Motivation and Context

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Chore (refactoring, style fixes, git/CI config, submodule management, no code logic changes)

How Has This Been Tested?

  • Already covered by automatic testing.
  • New test added: (add PR link here).
  • Tested by hand on: (list targets here).

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing linter checks and tests passed.
  • My changes generate no new compilation warnings for any of the targets.

Special treatment

  • This PR needs additional PRs to work (list the PRs, preferably in merge-order).
  • I will merge this PR by myself when appropriate.

RISC-V Privileged Architectures V1.10 renamed and generalized sbadaddr
to stval. binutils >= 2.43 dropped support for sbadaddr alias.

JIRA: RTOS-913
@badochov badochov requested a review from lukileczo September 10, 2024 15:15
@github-actions
Copy link

Unit Test Results

7 700 tests  ±0   6 985 ✅ ±0   39m 12s ⏱️ + 1m 46s
  436 suites ±0     715 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit d54fa74. ± Comparison against base commit 9db2e1c.

@badochov badochov merged commit 9787100 into master Sep 11, 2024
@badochov badochov deleted the badochov/riscv-stval branch September 11, 2024 08:50
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.

2 participants