Home Original page

Added code coverage support to ActiveHDL by vmfox · Pull Request #461 · VUnit/vunit

@vmfox

@vmfox 

Added code coverage support to ActiveHDL

kraigher

class ActiveHDLInterface(SimulatorInterface):
class ActiveHDLInterface(VsimSimulatorMixin, SimulatorInterface):

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is it necessary to inherit from VsimSimulatorMixin? I think that could be a misunderstanding.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It probably is a misunderstanding as it's a bit of a challenge to jump into such a big project. The assumption was that akin to Riviera Pro and Modelsim, ActiveHDL is also a vsim based simulator and hence would require this inheritance. I have just tested the performance without this inheritance and everything seems to run as it was before, so will update as a part of the next commit.

Removed redundant inheritance and updated as per tests failures.

@vmfox