Wilson Snyder comments

Results 1229 comments of


                                            Wilson Snyder

Fix internal error on bad class name

Great, glad to get a pull on this, thanks for looking at it.

Fix internal error on bad class name

Fixed with (#5629)

Argument list too long

Not surprising. Fix is to build the list in the Makefile using files, which will need many tries to get right. Or Verilator can emit the concatenated file directly. However,...

Argument list too long

Closing for now as unlikely to get attention, though if this starts popping up more a pull would be accepted. Recommendation is to use `--output-groups`, which might become default once...

Support Force/Release via VPI

I agree this makes sense. A pull would be appreciated, as it's unlikely to get addressed soon otherwise.

SIDEEFFECT warning false positives on coverage

Will need a test case, as you probably already know ;)

SIDEEFFECT warning false positives on coverage

Please reopen if get test case.

SIDEEFFECT warning false positives on coverage

I agree that AstCoverInc is not pure, but you'll get an impure warning on this code if you do nearly anything else in the function. Also note that making it...

SIDEEFFECT warning false positives on coverage

Unfortunately there's a lot of cleanup needed to prevent SIDEEFFECT, which is why the warning exists rather than it just being fixed.

Test with iteration over hierarchy using VPI

Fixed via #5314 in version 5.034.