Skip to content

Remove duplicate logic from memoryloader#235

Merged
TheDeepestSpace merged 1 commit into
mainfrom
clean-memloader
May 24, 2026
Merged

Remove duplicate logic from memoryloader#235
TheDeepestSpace merged 1 commit into
mainfrom
clean-memloader

Conversation

@tandr3w
Copy link
Copy Markdown
Contributor

@tandr3w tandr3w commented May 24, 2026

@tandr3w tandr3w self-assigned this May 24, 2026
@tandr3w tandr3w requested a review from TheDeepestSpace May 24, 2026 01:45
@github-actions
Copy link
Copy Markdown

🔧 DE1-SoC Synthesis Report Summary Diff

  • RV32I

    1. Fitter Summary

      @@ -1,4 +1,4 @@
      -Fitter Status : Successful - Sat May 23 03:33:28 2026
      +Fitter Status : Successful - Sun May 24 01:48:13 2026
       Quartus Prime Version : 25.1std.0 Build 1129 10/21/2025 SC Lite Edition
       Revision Name : utoss-risc-v
       Top-level Entity Name : top
    2. Fitter by entity
      No changes detected

    3. Timing
      No changes detected

  • RV32IB

    1. Fitter Summary

      @@ -1,4 +1,4 @@
      -Fitter Status : Successful - Sat May 23 03:33:23 2026
      +Fitter Status : Successful - Sun May 24 01:48:33 2026
       Quartus Prime Version : 25.1std.0 Build 1129 10/21/2025 SC Lite Edition
       Revision Name : utoss-risc-v
       Top-level Entity Name : top
    2. Fitter by entity
      No changes detected

    3. Timing
      No changes detected


Comparing synthesis results from main branch vs. this PR

Copy link
Copy Markdown
Member

@TheDeepestSpace TheDeepestSpace left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thanks!

@TheDeepestSpace TheDeepestSpace linked an issue May 24, 2026 that may be closed by this pull request
@TheDeepestSpace TheDeepestSpace merged commit 93f3724 into main May 24, 2026
13 checks passed
@tandr3w tandr3w deleted the clean-memloader branch May 24, 2026 18:41
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.

Revisit memory loader

2 participants