Merge pull request #2936 from Pinata-Consulting/orfs-bump-and-example-tweak

Orfs bump and example tweak

Author: maliberty

flow/MODULE.bazel

@@ -9,7 +9,7 @@ module(
 bazel_dep(name = "bazel-orfs")
 git_override(
     module_name = "bazel-orfs",
-    commit = "4392a87877b4196502fc89dcdf898fd6efdb3fc0",
+    commit = "a33cd61ae13d2c4eb8d00e4475bfe214c828bc64",
     remote = "https://github.com/The-OpenROAD-Project/bazel-orfs.git",
 )
 
@@ -50,9 +50,9 @@ orfs.default(
     # and update "image" to point to the local image.
 
     # Official image https://hub.docker.com/r/openroad/orfs/tags
-    image = "docker.io/openroad/orfs:v3.0-2528-ga8780221",
+    image = "docker.io/openroad/orfs:v3.0-2581-g11b93cfa",
     # image = "gcr.io/ascenium/orfs-megaboom/flow-ubuntu22.04-builder:3d2c3d-2",
-    sha256 = "5093434a9d3adaaf8e9a4d62fc0df2537d43ceaf01bf2acf9e94d9d38131b7ca",
+    sha256 = "6b778fcdc3c7ba5cc5250cde588c43a63c45ffb3126ee072596a4173fe4d7101",
 )
 use_repo(orfs, "com_github_nixos_patchelf_download")
 use_repo(orfs, "docker_orfs")

flow/MODULE.bazel.lock

@@ -73,6 +73,21 @@ filegroup(
     srcs = [":{}_netlist".format(name) for name in BLACKBOXES],
 )
 
+# Canonicalize the netlists to avoid rebuilds unecessarily.
+#
+# This is more a demonstration than a practical solution.
+#
+# Other things could be done here, like get a netlist from git lfs,
+# process the netlist in some other way, with Naja, etc.
+genrule(
+    name = "netlists_canonicalized",
+    srcs = [":netlists"],
+    outs = ["netlists_canonicalized.v"],
+    cmd = """
+        cat $(locations :netlists) | grep -v -E '\\(\\* src = "|Generated by Yosys' > $@
+    """,
+)
+
 [orfs_macro(
     name = top,
     lef = "lef/{}.lef".format(top),
@@ -100,7 +115,7 @@ orfs_flow(
     macros = FAKERAMS,
     sources = {
         "SDC_FILE": [":constraint.sdc"],
-        "SYNTH_NETLIST_FILES": [":netlists"],
+        "SYNTH_NETLIST_FILES": [":netlists_canonicalized"],
     },
     verilog_files = [],
 )