Skip to content

Conversation

@rwardd
Copy link
Contributor

@rwardd rwardd commented Nov 20, 2024

Description

Whilst reading through the RISC-V portASM.S, I noticed the documentation on lines 35 and 49 referenced an old directory (RISC-V-RV32/). After grepping through the other files in the directory, I noticed some discrepancies (some referencing RISC-V/, and some referencing RISC-V-RV32/). I've gone through and made the necessary changes.

Checklist:

  • I have tested my changes. No regression in existing tests.
  • I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@rwardd rwardd requested a review from a team as a code owner November 20, 2024 06:07
@sonarqubecloud
Copy link

@aggarg aggarg merged commit 874fa7b into FreeRTOS:main Nov 20, 2024
16 checks passed
@aggarg
Copy link
Member

aggarg commented Nov 20, 2024

Thank you for your contribution!

@tomsalpietro
Copy link

Great work!

moninom1 pushed a commit to moninom1/FreeRTOS-Kernel that referenced this pull request Sep 30, 2025
* Update result section in README under CBMC folder

* Code review suggestions

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>

* Update proofs result path

---------

Signed-off-by: Gaurav Aggarwal <aggarg@amazon.com>
Co-authored-by: Gaurav Aggarwal <aggarg@amazon.com>
Co-authored-by: Gaurav-Aggarwal-AWS <33462878+aggarg@users.noreply.github.com>
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.

4 participants