xls icon indicating copy to clipboard operation
xls copied to clipboard

bazelrc: enable llvm toolchain

Open proppy opened this issue 2 years ago • 1 comments

From https://github.com/hdl/conda-eda/blob/master/hls/xls/use-llvm-toolchain.patch

FIxes #558 #141

@cdleary is it fine to enable by default?

proppy avatar Sep 28 '22 22:09 proppy

Should be fine to enable by default given that it works in CI.

Unfortunately since we changed Bazel build flags it'll need to rebuild everything which is gonna take a while for us to observe the result, may time out the first go around, but running it locally too and if that looks ok we'll put it in and see how it goes.

cdleary avatar Sep 29 '22 00:09 cdleary

Unfortunately since we changed Bazel build flags it'll need to rebuild everything which is gonna take a while for us to observe the result

That shouldn't be a concern anymore with the dedicated runners.

proppy avatar Sep 14 '23 18:09 proppy

Superseeded by https://github.com/google/xls/commit/72b254fddf8f356420fc0c9831dd7c461d41f37d.

proppy avatar Oct 09 '23 23:10 proppy