Skip to content
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.

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

Fix a bug in VPR when reading tileable rr_graph (--read_rr_graph) #340

Closed
wants to merge 2 commits into from

Conversation

rs-mahdi
Copy link
Contributor

@rs-mahdi rs-mahdi commented Aug 8, 2023

Motivate of the pull request

  • To address an existing issue. If so, please provide a link to the issue:
  • Breaking new feature. If so, please describe details in the description part.

Describe the technical details

What is currently done? (Provide issue link if applicable)

I modified VPR source code to correctly read rr_graph when it is tileable. Before this change, VPR had not been checking if the input rr_graph is tileable or not. When the rr_graph is tileable, ptc# for wires changes through different coordinates. However, VPR was assigning the same ptc# to all coordinates that a wire can cover.

What does this pull request change?

How the ptc# is assigned to wires longer than 1. This assignment is done run-time by VPR to create its internal data structure.

Which part of the code base require a change

  • Build compatibility
  • Documentation
  • Regression tests
  • Continous Integration (CI) scripts

Impact of the pull request

  • Require a change on Quality of Results (QoR)
  • Break back-compatibility. If so, please list who may be influenced.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant