Skip to content

[AP] Added Partial Placement Object #2742

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

Conversation

AlexandreSinger
Copy link
Contributor

Added the partial placement object. This object is used during the Analytical Placement flow to store the positions of the APBlocks. This placement is not gaurenteed to be legal; however, it does gaurentee that the blocks are placed on the device and the fixed blocks are placed in valid locations.

@github-actions github-actions bot added VPR VPR FPGA Placement & Routing Tool lang-cpp C/C++ code labels Sep 23, 2024
@AlexandreSinger AlexandreSinger force-pushed the feature-ap-partial-placement-upstreaming branch from fe9b2e5 to 9ffe1bf Compare September 26, 2024 13:08
Copy link
Contributor

@vaughnbetz vaughnbetz 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; a few suggested changes.

@AlexandreSinger AlexandreSinger force-pushed the feature-ap-partial-placement-upstreaming branch from 9ffe1bf to d33f62a Compare September 26, 2024 15:22
Added the partial placement object. This object is used during the
Analytical Placement flow to store the positions of the APBlocks. This
placement is not gaurenteed to be legal; however, it does gaurentee that
the blocks are placed on the device and the fixed blocks are placed in
valid locations.
@AlexandreSinger AlexandreSinger force-pushed the feature-ap-partial-placement-upstreaming branch from d33f62a to 2b6a935 Compare September 26, 2024 18:40
Copy link
Contributor

@vaughnbetz vaughnbetz 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.

@vaughnbetz vaughnbetz merged commit 40797ae into verilog-to-routing:master Sep 27, 2024
36 of 53 checks passed
@AlexandreSinger AlexandreSinger deleted the feature-ap-partial-placement-upstreaming branch November 27, 2024 19:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang-cpp C/C++ code VPR VPR FPGA Placement & Routing Tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants