Skip to content

Conversation

@micprog
Copy link
Member

@micprog micprog commented Oct 20, 2024

No description provided.

Copy link
Contributor

@phsauter phsauter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What exactly is the intended way this interacts with bender script --no-default-target?

@micprog
Copy link
Member Author

micprog commented Dec 4, 2025

What exactly is the intended way this interacts with bender script --no-default-target?

The --no-default-target removes the default targets for different script formats (e.g., simulation for vsim). The --assume-rtl target makes sure every source file from the manifests has a target, if any file does not it receives the rtl target. The arguments should not interact, as one applies to the applied targets (similar to adding a -t <TARGET> flag), one applies to the sourcefile list.

Copy link
Contributor

@fischeti fischeti left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@micprog micprog merged commit b353752 into master Dec 11, 2025
6 checks passed
@micprog micprog deleted the assume_rtl branch December 11, 2025 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants