Skip to content

Comments

verific: add -set_vhdl_default_library_path option#5584

Open
dhvll wants to merge 4 commits intoYosysHQ:mainfrom
dhvll:verific-set_vhdl_default_library_path
Open

verific: add -set_vhdl_default_library_path option#5584
dhvll wants to merge 4 commits intoYosysHQ:mainfrom
dhvll:verific-set_vhdl_default_library_path

Conversation

@dhvll
Copy link

@dhvll dhvll commented Jan 3, 2026

This PR adds a verific -set_vhdl_default_library_path option that allows users to specify default VHDL library search paths for the Verific frontend.

Silimate’s Yosys fork includes an option to configure Verific’s default VHDL library path from the command line.

Upstreaming this option enables better support for VHDL workflows without affecting existing behavior for users who do not rely on VHDL.

The option consumes the remaining command-line arguments as library paths and configures them via vhdl_file::SetDefaultLibraryPath(). Since the option consumes positional arguments, option parsing intentionally terminates after this flag. The behavior is guarded under VERIFIC_SYSTEMVERILOG_SUPPORT and VERIFIC_VHDL_SUPPORT.

@dhvll dhvll changed the title verific: add -ignore_module option verific: add -set_vhdl_default_library_path option Jan 3, 2026
Copy link
Member

@mmicko mmicko left a comment

Choose a reason for hiding this comment

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

There is no sense for this to be under VERIFIC_SYSTEMVERILOG_SUPPORT check block, it only requires VHDL so it should only be able to be used/compiled if you have Verific with VHDL support.

@dhvll dhvll requested a review from mmicko January 5, 2026 18:02
@dhvll
Copy link
Author

dhvll commented Jan 5, 2026

Are these changes correct ?

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.

2 participants