Conversation
4929918 to
e056889
Compare
iansseijelly
left a comment
There was a problem hiding this comment.
OK I guess this is good enough for now, although fragmentation is increasing
| .settings(commonSettings) | ||
| .settings(scalaTestSettings) | ||
| if (!useChisel7) { | ||
| hf = hf.dependsOn(midas_target_utils) |
There was a problem hiding this comment.
oh... how does this work? regardless of useChisel7 or not, wouldn't hf require midas as a dependency?
There was a problem hiding this comment.
I think this was our hacky way to make everything depend on midas-target-utils in the pre-chisel7 flow
|
@codex review |
There was a problem hiding this comment.
Codex Review: Here are some suggestions.
About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".
d8e653f to
5986b99
Compare
|
This currently breaks Boom RTL generation in chisel7, even with chisel version set to 7.0.0, and can be reproduced by running Specifically, the issue arises from after running the firtool transform, the |
|
Tracking llvm/circt#9012, we should bump the firtool after the fix. |
Related PRs / Issues:
Type of change:
Impact:
Contributor Checklist:
mainas the base branch?changelog:<topic>label?changelog:label?.conda-lock.ymlfile if you updated the conda requirements file?Please Backport?CI Help:
Add the following labels to modify the CI for a set of features.
Generally, a label added only affect subsequent changes to the PR (i.e. new commits, force pushing, closing/reopening).
See
ci:*for full list of labels:ci:fpga-deploy- Run FPGA-based E2E testingci:local-fpga-buildbitstream-deploy- Build local FPGA bitstreams for platforms that are releasedci:disable- Disable CI