sv-tests icon indicating copy to clipboard operation
sv-tests copied to clipboard

Import XilinxUnisimLibrary as a test suite?

Open mithro opened this issue 4 years ago • 5 comments

Does it make sense to import the https://github.com/SymbiFlow/XilinxUnisimLibrary library as a third_party test suite?

mithro avatar Jun 30 '20 20:06 mithro

@hzeller @tgorochowik -- Thoughts? Is the UnisimLibrary pure verilog?

mithro avatar Jun 30 '20 20:06 mithro

@alainmarcel has it in the Surelog testuite in the meantime ( https://github.com/alainmarcel/Surelog/issues/568 ).

Would be worthwhile to include in sv-tests

hzeller avatar Jul 10 '20 00:07 hzeller

Assigning to Karol for now, please delegate to whoever might be free. Having it in sv-tests will help us see how the tools deal with it (.. and/or how changes in the Unisims files change that)

Here is how they are run in Surelog (they are there in a batchfile, but it is easy to see each individual line is one invocation) https://github.com/alainmarcel/Surelog/blob/master/third_party/tests/XilinxUnisimLibrary/batch.txt

Since we need unique names per test, maybe we just compose it from the filename with some prefix in the generate script. So file RX_BITSLICE.v would become test unisims-RX_BITSLICE maybe.

GitHub
System Verilog 2017 Pre-processor, Parser, UHDM Compiler. Provides IEEE Design/TB VPI and Python AST API. - alainmarcel/Surelog

hzeller avatar Jul 13 '20 22:07 hzeller

@alaindargelas - The upstream sim models has been updated with the missing verilog/src/glbl.v file which should make it possible to get further along in the parsing.

mithro avatar Jul 22 '20 17:07 mithro

Updated, also fixed my grammar for the 2 syntax errors => 0 Syntax errors. All files are parsed 100%.

On Wednesday, July 22, 2020, 10:54:40 AM PDT, Tim Ansell <[email protected]> wrote:  

@alaindargelas - The upstream sim models has been updated with the missing verilog/src/glbl.v file which should make it possible to get further along in the parsing.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

alaindargelas avatar Jul 22 '20 21:07 alaindargelas