run pyslang from list of files

[baruchgu]

Hi
I have know to run slang from a command line:
slang --ignore-unknown-modules --error-limit 800 --ast-json rxc_top.json --ast-json-scope rxc_top -top rxc_top -f ./rtl.incdirs -f ./rtl.hdlfiles

I am looking for a way to run it with the same arguments by pyslang package's functions
I try:
filelist='-f ./rtl.incdirs -f ./rtl.hdlfiles'
tree = pyslang.SyntaxTree.fromFiles(filelist)

it fails. What is a correct way to run slang from a python scope?
== Thanks

[MikePopoloski]

You can use the Driver class to handle argument parsing for you. See the example: https://github.com/MikePopoloski/pyslang/blob/master/examples/driver.py

[baruchgu]

Great.
How can I get a handler to ast-json structure?

[MikePopoloski]

I don't know what this means exactly, but what have you tried so far and why is it not working?

[baruchgu]

driver example works. I'd like to get a file-handler to generated json file in memory without writing it to the disk.
so it be loaded by json.load()

[MikePopoloski]

It looks like the ASTSerializer class isn't exposed to Python bindings right now. It wouldn't be hard to add, but if you're just going to immediately parse it in-memory then it probably makes more sense to just visit the AST objects directly and skip the round trip through JSON for no reason?

[baruchgu]

Agree. But I already have a procedure that searches thru a Json tree. If the Json structure can not be read, I should redesign the search procedure to AST object

[MikePopoloski]

If you want, I'd accept a PR that adds bindings for the ASTSerializer. You can see how it's used in the C++ code, for example in slang_main.cpp

[baruchgu]

I guess you ask that I provide my proposal of this fix. Correct? I am not enough C++ professional for this task. I generally work in Python. תודה מראש. ברוך... בתאריך יום ב׳, 8 ביולי 2024, 18:09, מאת Michael Popoloski ‏< ***@***.***>:

…

If you want, I'd accept a PR that adds bindings for the ASTSerializer. You can see how it's used in the C++ code, for example in slang_main.cpp — Reply to this email directly, view it on GitHub <#1051 (reply in thread)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AQCC2CS5WIDVDAMM6YK4LSLZLKTTRAVCNFSM6AAAAABKQVSDHSVHI2DSMVQWIX3LMV43SRDJONRXK43TNFXW4Q3PNVWWK3TUHM4TSOBYG4ZTI> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[MikePopoloski]

Yes, otherwise you would need to wait for me to get around to it. It is not the highest priority issue.

[baruchgu]

Mike. I work on your driver.py example. len(driver.syntaxTrees) is 891 as the number of verilog files it gets. What is missing if I'd like to get the root to the whole elaborated AST tree? Like I have in json file.

…

-- תודה מראש, ברוך

On Tue, 9 Jul 2024 at 00:52, Michael Popoloski ***@***.***> wrote: Yes, otherwise you would need to wait for me to get around to it. It is not the highest priority issue. — Reply to this email directly, view it on GitHub <#1051 (reply in thread)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AQCC2CR4SR5IMQ6JVB6TE2TZLMCYNAVCNFSM6AAAAABKQVSDHSVHI2DSMVQWIX3LMV43SRDJONRXK43TNFXW4Q3PNVWWK3TUHM4TSOJRHE4DO> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[MikePopoloski]

You can look at some of the other examples and test cases in the pyslang project for how to get the root of the AST in Python.