EDIT: Comment claims its AI with no proof -> I prove its not AI -> I get downvoted. Classic reddit.

More Proof its not AI:

Check out the repo at this commit, at this point the structure of my code was quite bad, as you'd expect from a beginner.

• The fetch/decode/execute loop was not inside the CPU class but inside main.cpp, furthermore the fetch function was also in main when it should be inside the CPU class.
  • You can see how fetch, decode, execute and Step took arguments whereas it would be unnecessary if they were a part of the CPU class so they had access to the internal state
  • I was using main.cpp to write temporary tests before I added an actual testing framework (gtest).
• In execute.cpp, the execute function took the CPU class and Memory itself as arguments when it should just be a part of the CPU class so it can access the internal state (as is in the latest version of the repo).
  • The switch statement, although at this point only had 3 instructions, was going to be wildly repetitive, I was reading/writing operands from the register file in each case instead of once in the beginning and end.
• In decode.cpp, the extract_fields function took boolean values to determine which fields to extract but in the latest version there are separate functions for each instruction type.
  • Confusing variable names like instruction.instruction.

Hi all,

I built my first C++ project (second ever project) and would appreciate honest feedback. Here's the GitHub.

CPU simulator

• An instruction set simulator that simulates a RISC-V architecture CPU
• Object-oriented design
• Implemented all base RV32I instructions
• Implemented Multiply/Divide instructions
• GoogleTest suite

I'm new to programming in general and only starting learning a few months ago (CS50P, CS50x). So I'd love any feedback on how I can improve my code and become a better (c++) programmer.

I'd also love to hear suggestions on what I should add, more extensions? Test against official RISC-V tests?

Thank you!