Skip to content

Commit e2b32a4

Browse files
authored
[ci] Release circt-{opt,synth,test} in firrtl-bin (#8928)
Add additional, useful tools as part of the `firrtl-bin` release package. This will avoid users having to get one of the "full" installations (and also deal with the fact that these installations are not truly static). This has a cost of bloating the `firrtl-bin` tarball. However, that seems acceptable given that users may not have easy alternatives for how to get these tools. Signed-off-by: Schuyler Eldridge <schuyler.eldridge@sifive.com>
1 parent 8ca2e97 commit e2b32a4

File tree

1 file changed

+15
-2
lines changed

1 file changed

+15
-2
lines changed

.github/bin/uploadReleaseArtifacts.sh

Lines changed: 15 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -131,6 +131,19 @@ mapfile -t OPT_OS < <(echo "${OPT_OS[@]}" | tr ' ' '\n' | sort -u)
131131
# JSON snippets used to configure downstream workflows
132132
#-------------------------------------------------------------------------------
133133

134+
# For non-full installs, install the following tools.
135+
binaries=(
136+
circt-opt
137+
circt-synth
138+
circt-test
139+
firtool
140+
om-linker
141+
)
142+
binaryTargets=
143+
for bin in "${binaries[@]}"; do
144+
binaryTargets="$binaryTargets install-$bin"
145+
done
146+
134147
# Configuration for a run of the static UBTI script.
135148
configStatic=$(cat <<EOF
136149
[
@@ -139,7 +152,7 @@ configStatic=$(cat <<EOF
139152
"llvm_enable_assertions": "$OPT_ASSERTIONS",
140153
"llvm_force_enable_stats": "ON",
141154
"run_tests": $OPT_RUN_TESTS,
142-
"install_target": "install-firtool install-om-linker",
155+
"install_target": "$binaryTargets",
143156
"package_name_prefix": "firrtl-bin"
144157
}
145158
]
@@ -213,7 +226,7 @@ configNativeFirtool=$(cat <<EOF
213226
[
214227
{
215228
"name": "firtool",
216-
"install_target": "install-firtool install-om-linker",
229+
"install_target": "$binaryTargets",
217230
"package_name_prefix": "firrtl-bin",
218231
"cmake_build_type":"$OPT_CMAKE_BUILD_TYPE",
219232
"llvm_enable_assertions":"$OPT_ASSERTIONS",

0 commit comments

Comments
 (0)