Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions .bazelrc
Original file line number Diff line number Diff line change
@@ -1,3 +1,13 @@
# yosys 0.64 is not yet in BCR; add the unmerged PR's fork as a fallback
# registry until bazelbuild/bazel-central-registry#8862 lands. BCR is
# listed first so all other modules resolve from the official source;
# the fork is only consulted for modules/versions BCR doesn't carry yet.
# The commit hash makes the fork reference immutable. Mirrors what
# bazel-orfs/.bazelrc does upstream — ORFS root has its own .bazelrc so
# the upstream lines don't propagate automatically.
common --registry=https://bcr.bazel.build/
common --registry=https://raw.githubusercontent.com/oharboe/bazel-central-registry/0586b398db6edd245da97cbec29e26c5e2a808d7/

build --incompatible_strict_action_env
build --cxxopt "-std=c++20" --host_cxxopt "-std=c++20"

Expand Down
11 changes: 1 addition & 10 deletions MODULE.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -36,23 +36,14 @@ git_override(
bazel_dep(name = "bazel-orfs", dev_dependency = True)
bazel_dep(name = "bazel-orfs-verilog", dev_dependency = True)

BAZEL_ORFS_COMMIT = "3a5ddd7eb48c363717e65903ae4573d528327fd3"
BAZEL_ORFS_COMMIT = "6ebadeb4be5c9ada103081c9a5e668c014126616"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

The pull request description indicates that 7 tests are failing with a "metric-miss" due to synthesis output shifts introduced by Yosys 0.64. These baseline updates (e.g., to rules-base.json) should be included in this pull request to ensure that the CI remains green and the repository stays in a consistent state.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

The 2 "fail-to-build" (PDN-0233) errors mentioned in the summary represent a regression in the build process. Synthesis changes from the Yosys 0.64 bump can impact downstream stages like floorplanning and PDN. These failures should be addressed to ensure that all designs in the test suite continue to build successfully.


BAZEL_ORFS_REMOTE = "https://github.com/The-OpenROAD-Project/bazel-orfs.git"

# To bump version, run: bazelisk run @bazel-orfs//:bump
#
# `patches =` keeps small bazel-orfs fixes vendored in this repo while
# we iterate, instead of round-tripping every change through a
# bazel-orfs PR + pin bump. When a patch lands upstream, drop the
# entry here and bump BAZEL_ORFS_COMMIT.
git_override(
module_name = "bazel-orfs",
commit = BAZEL_ORFS_COMMIT,
patch_strip = 1,
patches = [
"//patches/bazel-orfs:0001-render_gds-monkey-patch-PDK_CONFIGS-not-gdsii_use_custom_config.patch",
],
remote = BAZEL_ORFS_REMOTE,
)

Expand Down

This file was deleted.

8 changes: 0 additions & 8 deletions patches/bazel-orfs/BUILD.bazel

This file was deleted.