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.

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

Enable place_and_route targets #975

Closed
lpawelcz opened this issue May 23, 2023 · 3 comments · Fixed by #1025
Closed

Enable place_and_route targets #975

lpawelcz opened this issue May 23, 2023 · 3 comments · Fixed by #1025
Labels
blocker Blocking design work build Related to build flow, build system, or build macros oss-backend Relates to the open source ASIC tool backend flows

Comments

@lpawelcz
Copy link
Contributor

Commit b08f89e introduced target for find_index example which uses place_and_route rule from bazel_rules_hdl. It is commented out because of dependency issue.

It would be nice to enable the usage of this rule in XLS. This would be a big step towards ASIC flow integration.

I already started working on this and I was able to fix openmp dependency issue (see hdl/bazel_rules_hdl#162) mentioned in TODO near the commented out place_and_route target.

Once I will be able to successfully run find_index_place_and_route target in XLS I will be happy to open PR with necessary fixes here.

CC @proppy
@lpawelcz lpawelcz moved this to In Progress in Antmicro XLS May 23, 2023
@cdleary cdleary added build Related to build flow, build system, or build macros oss-backend Relates to the open source ASIC tool backend flows labels May 24, 2023
@lpawelcz
Copy link
Contributor Author

Discovered and fixed another dependency issue in hdl/bazel_rules_hdl#164 (also hdl/bazel_rules_hdl#165)

@lpawelcz lpawelcz mentioned this issue May 25, 2023
Merged
@lpawelcz
Copy link
Contributor Author

lpawelcz commented May 30, 2023 •
edited
Loading

Managed to successfully run find_index_place_and_route in XLS workspace. This required a number of changes in XLS and bazel_rules_hdl:

I will open PRs to XLS once bazel_rules_hdl is ready.

@proppy
Copy link
Member

proppy commented May 30, 2023

Blocking #996

@proppy proppy added the blocker Blocking design work label May 30, 2023
This was referenced Jun 13, 2023
Closed
Merged
@github-project-automation github-project-automation bot moved this from In Progress to Done in Antmicro XLS Jun 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
blocker Blocking design work build Related to build flow, build system, or build macros oss-backend Relates to the open source ASIC tool backend flows
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Enable place_and_route targets antmicro/xls
3 participants
@proppy @cdleary @lpawelcz