Misc style cleanup.
This commit is contained in:
parent
c4c190398f
commit
af84c975f1
|
|
@ -4410,24 +4410,25 @@ calls.
|
||||||
=item SPLITVAR
|
=item SPLITVAR
|
||||||
|
|
||||||
Warns that a variable with a C<split_var> metacomment was not split.
|
Warns that a variable with a C<split_var> metacomment was not split.
|
||||||
Typical reasons are
|
Some possible reasons for this are:
|
||||||
|
|
||||||
The datatype of the variable is not supported for splitting. (e.g. is a
|
* The datatype of the variable is not supported for splitting. (e.g. is a
|
||||||
real).
|
real).
|
||||||
|
|
||||||
The access pattern of the variable can not be determined
|
* The access pattern of the variable can not be determined
|
||||||
statically. (e.g. is accessed as a memory).
|
statically. (e.g. is accessed as a memory).
|
||||||
|
|
||||||
The index of the array exceeds the array size.
|
* The index of the array exceeds the array size.
|
||||||
|
|
||||||
The variable is accessed from outside using dotted reference.
|
* The variable is accessed from outside using dotted reference.
|
||||||
(e.g. top.instance0.variable0 = 1).
|
(e.g. top.instance0.variable0 = 1).
|
||||||
|
|
||||||
The variable is not declared in a module, but in a package or an interface.
|
* The variable is not declared in a module, but in a package or an
|
||||||
|
interface.
|
||||||
|
|
||||||
The variable is one of parameter, localparam, genvar, or queue.
|
* The variable is a parameter, localparam, genvar, or queue.
|
||||||
|
|
||||||
The variable is tirstate or bidirectional. (e.g. inout or ref).
|
* The variable is tirstate or bidirectional. (e.g. inout or ref).
|
||||||
|
|
||||||
=item STMTDLY
|
=item STMTDLY
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -2070,8 +2070,8 @@ private:
|
||||||
refp->varp(varp);
|
refp->varp(varp);
|
||||||
if (varp->attrSplitVar()) {
|
if (varp->attrSplitVar()) {
|
||||||
refp->v3warn(SPLITVAR, varp->prettyNameQ()
|
refp->v3warn(SPLITVAR, varp->prettyNameQ()
|
||||||
<< " has split_var metacomment but will not be split because "
|
<< " has split_var metacomment but will not be split because"
|
||||||
"it is accessed from other module via dot.\n");
|
<< " it is accessed from another module via a dot.");
|
||||||
varp->attrSplitVar(false);
|
varp->attrSplitVar(false);
|
||||||
}
|
}
|
||||||
m_ds.m_dotText = "";
|
m_ds.m_dotText = "";
|
||||||
|
|
|
||||||
|
|
@ -309,7 +309,7 @@ private:
|
||||||
UASSERT_OBJ(m_varp, nodep, "Attribute not attached to variable");
|
UASSERT_OBJ(m_varp, nodep, "Attribute not attached to variable");
|
||||||
if (!VN_IS(m_modp, Module)) {
|
if (!VN_IS(m_modp, Module)) {
|
||||||
m_varp->v3warn(SPLITVAR, m_varp->prettyNameQ() << " has split_var metacomment, "
|
m_varp->v3warn(SPLITVAR, m_varp->prettyNameQ() << " has split_var metacomment, "
|
||||||
"but will not be split because it is not declared in a module.\n");
|
"but will not be split because it is not declared in a module.");
|
||||||
} else {
|
} else {
|
||||||
m_varp->attrSplitVar(true);
|
m_varp->attrSplitVar(true);
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -11,7 +11,7 @@ logic [7:0] should_show_warning_global1 [1:0] /*verilator split_var*/ ;
|
||||||
%Warning-SPLITVAR: t/t_split_var_1_bad.v:11: 'should_show_warning_ifs1' has split_var metacomment, but will not be split because it is not declared in a module.
|
%Warning-SPLITVAR: t/t_split_var_1_bad.v:11: 'should_show_warning_ifs1' has split_var metacomment, but will not be split because it is not declared in a module.
|
||||||
logic [7:0] should_show_warning_ifs1 [1:0] /*verilator split_var*/ ;
|
logic [7:0] should_show_warning_ifs1 [1:0] /*verilator split_var*/ ;
|
||||||
^~~~~~~~~~~~~~~~~~~~~~~~
|
^~~~~~~~~~~~~~~~~~~~~~~~
|
||||||
%Warning-SPLITVAR: t/t_split_var_1_bad.v:37: 'cannot_split1' has split_var metacomment but will not be split because it is accessed from other module via dot.
|
%Warning-SPLITVAR: t/t_split_var_1_bad.v:37: 'cannot_split1' has split_var metacomment but will not be split because it is accessed from another module via a dot.
|
||||||
i_sub0.cannot_split1[0] = 0;
|
i_sub0.cannot_split1[0] = 0;
|
||||||
^~~~~~~~~~~~~
|
^~~~~~~~~~~~~
|
||||||
%Warning-SELRANGE: t/t_split_var_1_bad.v:82: Selection index out of range: 13 outside 12:10
|
%Warning-SELRANGE: t/t_split_var_1_bad.v:82: Selection index out of range: 13 outside 12:10
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue