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

Support classes and methods #377

Closed
veripoolbot opened this issue Aug 10, 2011 · 5 comments
Closed

Support classes and methods #377

veripoolbot opened this issue Aug 10, 2011 · 5 comments

Comments

@veripoolbot
Copy link
Contributor

@veripoolbot veripoolbot commented Aug 10, 2011


Author Name: Wilson Snyder (@wsnyder)
Original Redmine Issue: 377 from https://www.veripool.org


Tracking bug for SystemVerilog. Probably a man-month+ task.

@veripoolbot
Copy link
Contributor Author

@veripoolbot veripoolbot commented Feb 6, 2016


Original Redmine Comment
Author Name: Alex Solomatnikov
Original Date: 2016-02-06T23:16:36Z


Does it make sense to enable classes given that interfaces are already supported and interfaces have similar functionality like tasks and functions?

@trosenkranz
Copy link
Contributor

@trosenkranz trosenkranz commented Jan 15, 2020

Hi Wilson,

you mentioned in #2094 that there exists a branch with an early implementation of classes.

Is there plan to support classes?

First support is in a branch, but it will be a few months until usable, and likely a year before everything related to classes works.

I looked for that branch but didn't find it - neither in this repo nor in verilator-dev by toddstrader. Please post a link so that I can look at and learn from it. I want to particpate because classes are greatly missed.

@wsnyder
Copy link
Member

@wsnyder wsnyder commented Jan 16, 2020

I'm about to create issue-377-class branch with changes to date. See README_class there. Welcome help and there's a few things listed there, but might be hard to get near mergable unless you have a big chunk of time you can commit.

Note to others - this is not ready for any production, so please don't try it and file "this doesn't work" issues ;)

wsnyder added a commit that referenced this issue Jan 16, 2020
@toddstrader
Copy link
Member

@toddstrader toddstrader commented Apr 1, 2020

Minor point, but we'll need a class type comparison test when both this and #2183 are ready.

wsnyder added a commit that referenced this issue Apr 5, 2020
@wsnyder
Copy link
Member

@wsnyder wsnyder commented Apr 5, 2020

Master now contains basic class support, see the manual for restrictions.

Closing this issue out, and opening new issues on the various improvements needed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.