Skip to content

Add flag for rtl target to files without target - #186

Merged
micprog merged 1 commit into
masterfrom
assume_rtl
Dec 11, 2025
Merged

micprog merged 1 commit into
masterfrom
assume_rtl

Conversation

@micprog

@micprog micprog commented Oct 20, 2024

Copy link
Copy Markdown
Member

No description provided.

@phsauter phsauter left a comment

Copy link
Copy Markdown
Contributor

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

micprog commented Dec 4, 2025

Copy link
Copy Markdown
Member Author

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.

@fischeti fischeti left a comment

Copy link
Copy Markdown
Contributor

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.

3 participants