opentitan icon indicating copy to clipboard operation
opentitan copied to clipboard

[dv] Add some more calls to uvm_object_utils

Open rswarbrick opened this issue 8 months ago • 3 comments

These missing causes a warning from Verissimo.

This is similar to PR #26600 (but there were some more hits!)

rswarbrick avatar Apr 01 '25 16:04 rswarbrick

I've just re-pushed. I didn't realise, but dv_base_reg shouldn't be registered with the factory (it's a base class with a bunch of arguments to its constructor). I've dropped that but left the rest in.

rswarbrick avatar Apr 02 '25 08:04 rswarbrick

sigh and that applies to dv_base_mem too... Try again! :-)

rswarbrick avatar Apr 02 '25 10:04 rswarbrick

Well, that's rather silly. It turns out that almost everything I was trying to add has either too many arguments in its constructor or is a "fake name" class that lives inside an interface. In either case, you get build-time errors or run-time warnings.

The only remaining item was dv_report_server! I've just redone things (yet) again. Sigh...

rswarbrick avatar Apr 02 '25 17:04 rswarbrick

Force push is just a rebase: no other change.

rswarbrick avatar Sep 05 '25 18:09 rswarbrick