Skip to content

Some clean ups in RR graph generation and custom switch block parsing #3166

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 17 commits into from
Jun 26, 2025
Merged

Some clean ups in RR graph generation and custom switch block parsing #3166

merged 17 commits into from
Jun 26, 2025

Conversation

soheilshahrouz
Copy link
Contributor

@soheilshahrouz soheilshahrouz commented Jun 26, 2025 •
edited
Loading

  • Added doxygen comments for some function.
  • Moved custom switch block parsing to a new file to make the code more organized
  • Passed some arguments by refernece
  • Tried to make the code more consistent with the coding style guide

@github-actions github-actions bot added VPR VPR FPGA Placement & Routing Tool libarchfpga Library for handling FPGA Architecture descriptions lang-cpp C/C++ code libvtrutil labels Jun 26, 2025
@soheilshahrouz soheilshahrouz changed the title [WIP] CHANZ RR type Some clean ups in RR graph generation and custom switch block parsing Jun 26, 2025
@soheilshahrouz soheilshahrouz requested a review from amin1377 June 26, 2025 18:00
amin1377
amin1377 approved these changes Jun 26, 2025
View reviewed changes
Copy link
Contributor

@amin1377 amin1377 left a comment

Choose a reason for hiding this comment

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

Thanks, Soheil!

@amin1377 amin1377 merged commit 88cae23 into master Jun 26, 2025
33 checks passed
@amin1377 amin1377 deleted the temp_chanz_rr_type branch June 26, 2025 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amin1377 amin1377 amin1377 approved these changes

Assignees
No one assigned
Labels
lang-cpp C/C++ code libarchfpga Library for handling FPGA Architecture descriptions libvtrutil VPR VPR FPGA Placement & Routing Tool
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@soheilshahrouz @amin1377