-
Notifications
You must be signed in to change notification settings - Fork 66
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
SystemVerilog import package - doesn't seem to do anything? #149
Comments
There was a missing conversion, it should be fixed now. |
Wow that was quick! |
That looks to work well thanks, I can now extract the packages that are used. I have nested packages so this looks like the last hurdle to full file list generation :) |
I guess contribution are welcome. Basically : 1cd4910 commit could be a useful source of inspiration. |
Thanks for the pointers, I'll see what I can do
…On Wed, 27 Jan 2021, 18:59 Thomasb81, ***@***.***> wrote:
I guess contribution are welcome.
Basically :
implement antlr visitePackage_declaration method and other visit* method
relevant for your application in cpp.
Fill structure with data you want to collect and make it available into
python world.
Implement unit test could help to preserve feature over tool change.
1cd4910
<1cd4910>
commit could be a useful source of inspiration.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#149 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AH3AOH2IFKX7L45Y3KZZ53TS4BPBNANCNFSM4WTLVFDQ>
.
|
Is there a list of what is currently implemented vs not?
On Wed, 27 Jan 2021, 19:03 William Chandler, <willumchandler@gmail.com>
wrote:
… Thanks for the pointers, I'll see what I can do
On Wed, 27 Jan 2021, 18:59 Thomasb81, ***@***.***> wrote:
> I guess contribution are welcome.
>
> Basically :
> implement antlr visitePackage_declaration method and other visit* method
> relevant for your application in cpp.
> Fill structure with data you want to collect and make it available into
> python world.
> Implement unit test could help to preserve feature over tool change.
>
> 1cd4910
> <1cd4910>
> commit could be a useful source of inspiration.
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <#149 (comment)>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AH3AOH2IFKX7L45Y3KZZ53TS4BPBNANCNFSM4WTLVFDQ>
> .
>
|
I don't think so. |
All things which are not converted to python yet are marked with
The problem is that the verilog/vhdl/systemverilog contains ridiculous amount of deprecated things. Most of the things were implemented on demand. Adding new things is simple, but we prefer to simplify HDL syntax obscurities so we do not need to deal with them later. If you do not like it you can use antlr grammars or c++ directly, but remember that there is a reason for this conversion and that is to remove a bloat and obscurities and you will have to deal with this obscurities if you use "base SV parser". If you can help it would be nice. |
Hi,
I'm trying to use hdlConvertor to discover all the files I need in my file list. (If someone has already done this please let me know :) )
So far I'm checking the error messages for missing includes directories and I'm looking for hdlAst._structural.HdlCompInst to get modules which are needed. Then I'm exploring the hierarchy and checking each module file found. This seems to work well.
The one thing that is missing is where there is an 'import something_pkg::*; this doesn't appear in the parsed object at all.
Then on lines where it is used there is just a double colon operator.
At no point does it realize that a file is missing.
Is this expected behaviour?
Thanks
The text was updated successfully, but these errors were encountered: