Skip to content

Commit 79074a5

Browse files
committed
flow: exports_files for scripts/synth.tcl
Lets bazel-orfs reference `@orfs//flow:scripts/synth.tcl` as a direct Label, so its `_synth_tcl` attr default can point at this script instead of vendoring its own copy at the bazel-orfs repo root. bazel-orfs has carried a forked synth.tcl since parallel partition synthesis landed (e9b84aa orfs: ship parallel synthesis scripts and wrapper Makefile, bazel-orfs). The fork has drifted on a couple of points (missing the `-noabc` on `synth -run fine:`, lost `SYNTH_MINIMUM_KEEP_SIZE` branch, etc.) and the drift has caused real WNS regressions when yosys was bumped past 0.62. Exporting this label upstream is the first step toward retiring the bazel-orfs copy and letting it inherit fixes from this script directly. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
1 parent 74b5f96 commit 79074a5

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

flow/BUILD

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,15 @@ exports_files(
1818
visibility = ["//visibility:public"],
1919
)
2020

21+
# Expose synth.tcl as an addressable source label so bazel-orfs can
22+
# point its `_synth_tcl` attr default at `@orfs//flow:scripts/synth.tcl`
23+
# instead of vendoring its own (drifting) copy at the bazel-orfs repo
24+
# root. See bazel-orfs `private/rules.bzl`.
25+
exports_files(
26+
["scripts/synth.tcl"],
27+
visibility = ["//visibility:public"],
28+
)
29+
2130
# files shared between scripts/synth.sh and scripts/flow.sh steps
2231
MAKEFILE_SHARED = [
2332
"scripts/variables.json",

0 commit comments

Comments
 (0)