feature: Add support for Xcelium #619

[shareefj]

This commit adds support for compiling the VerilatorIntegrationLibrary using Cadence Xcelium.

Related issue

#619

Description

Fixes a number of syntax issues in the VerilatorIntegrationLibrary SV sources files. See accompanying description in bug ticket.

Note that I don't have Questa to verify that there has been no regression.

Usage example

These fixes will enable the renode-dpi-examples repo to be updated with support for Xcelium. I'll release a secondary PR on that repo once done.

[PiotrZierhoffer]

Hi @shareefj , thanks for the PR.

It's very interesting. We can double check it with Questa. I assume that you circumvent Hierarchical name ('bus.AddressWidth') not allowed within a constant expression by relaxing the rules on the Xcelium side?

[shareefj]

Hi Piotr,

I did try relaxing the rules but it didn't work. I was just updating the associated ticket with a summary of where I'm at but this isn't ready to go yet. I can't run the tests due to what I think is an issue with the TCPClient code. More in the ticket shortly...

But for now, can you see if you can test Questa by setting the XCELIUM `define in the latest push. It seems Verilator can't handle that syntax so I'll raise a bug on them.

[shareefj]

@PiotrZierhoffer This is ready to go. See accompanying issue and PR in renode-dpi-examples.

[shareefj]

@PiotrZierhoffer Is this something that you think you'd like to merge or should I close this and just use my own fork?