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

[Test] Add a new test case to validate the fix_pins option #620

Merged
merged 3 commits into from
Apr 13, 2022

Conversation

tangxifan
Copy link
Collaborator

@tangxifan tangxifan commented Apr 13, 2022

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)

Currently, OpenFPGA has the following limitations:

  • --fix_pins {file} is an important feature of VPR which accepts pin constraints when implementing a circuit on an FPGA
  • However, such feature is not yet under any regression tests.

What does this pull request change?

This PR improves in the following aspects:

  • Created a new tests under the basic regression test to validate the --fix_pins option

Which part of the code base require a change

  • VPR
  • Tileable routing architecture generator
  • OpenFPGA libraries
  • FPGA-Verilog
  • FPGA-Bitstream
  • FPGA-SDC
  • FPGA-SPICE
  • Flow scripts
  • Architecture library
  • Cell library
  • 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.

@tangxifan tangxifan merged commit 1bd17cc into master Apr 13, 2022
@tangxifan tangxifan deleted the fix_pins branch April 13, 2022 09:17
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