Name an object/class the same as the variable name when created via the factory [lexical guidelines]

[antmarzam]

Reason: makes debugging easier. Example:
m_object = object_type::type_id::create("m_object");

[rswarbrick]

I'm a fan of the idea and a quick grep shows that this is already the convention in OpenTitan. The other options are mostly either using "%m" when this is an interface that might get instantiated several times or including an array index when we're assigning to an array entry.

It should definitely be a guideline rather than a restriction, though. For example, see dv_base_mubi_cov.sv (the first example I looked at when grepping).

Looking closely, some of the other "violations" look like the variable name needs a tweak :-)

[rswarbrick]

We should probably explicitly give an example for array items (called something like "m_my_array[123]")