Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add "reflection" adjacent feature (#5002) #5003

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

ct-clmsn
Copy link
Contributor

@ct-clmsn ct-clmsn commented Mar 19, 2024

Implements reflection adjacent feature. code generator creates accessor methods and creates a look up table to get access to I/O ports. this will allow verilator to integrate with different simulation systems. see issue #5002

@ct-clmsn
Copy link
Contributor Author

ct-clmsn commented Mar 19, 2024

@wsnyder looks like std::variant is not supported in the C++ code generated by verilator. I'd suggest using a 3rd party variant implementation mpark's variant it's fairly popular and has good performance.

@wsnyder wsnyder changed the title add "reflection" adjacent feature Add "reflection" adjacent feature (#5002) Mar 19, 2024
@wsnyder
Copy link
Member

wsnyder commented Mar 23, 2024

I'm not willing to add third-party dependencies for this feature, sorry. Please do without variant. Also I still have concerns about the support cost of this code.

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.

None yet

2 participants