edalize icon indicating copy to clipboard operation
edalize copied to clipboard

Adding slang support to edalize

Open tianrui-wei opened this issue 3 years ago • 5 comments

slang offers system verilog language services and is capable of many things. In particular, a complete Systemverilog preprocessor and linter. This commit adds slang support to edalize

tianrui-wei avatar May 08 '21 16:05 tianrui-wei

+1

mithro avatar May 08 '21 16:05 mithro

Another thing I have in mind for this is that it adds some sort of export ip functionality, where you can specify some file sets and slang could output a single clean file instead with no includes attached. This makes it easy to integrate fusesoc into flows that requires unsupported tools.

tianrui-wei avatar May 08 '21 16:05 tianrui-wei

Thanks @tianrui-wei This is great. Just wanted to let you know that I'm a bit short on time right now and have other things I need to review first

olofk avatar May 21 '21 07:05 olofk

No problem @olofk :P Happy to wait until you can spare some time.

tianrui-wei avatar May 21 '21 07:05 tianrui-wei

Hi @olofk , I rebased my branch, did some cleanup and added some new features in the last few days, could you perhaps take another look?

tianrui-wei avatar Aug 23 '21 10:08 tianrui-wei

ping @olofk

tianrui-wei avatar Sep 20 '22 01:09 tianrui-wei

ping @olofk

tianrui-wei avatar Nov 10 '22 22:11 tianrui-wei

@olofk could you take a look at this? We would really like it to land for some openpiton stuff we're doing

Jbalkind avatar Nov 15 '22 07:11 Jbalkind

This looks find. Thank you for the work and apologies for taking so long. I also want to mention that we are moving to a new API called the flow API and it would be great to have a tool class for the new API as well, but let's start with this

olofk avatar Dec 15 '22 19:12 olofk

Thanks Olof, let me know if you need me to submit another PR for the new API

tianrui-wei avatar Dec 15 '22 21:12 tianrui-wei

Yes. That would be great. I think slang support makes a lot more sense with the new API also, because then we can use it as a frontend. Unfortunately, the flow API is not yet enabled in FuseSoC, so if you are using FuseSoC I recommend we wait until FuseSoC 2.0 is released. I'm just releasing a first release candidate for it now so hopefully shouldn't be too long.

olofk avatar Dec 18 '22 16:12 olofk