Skip to content

Commit

Permalink
Removed $aconst cell type
Browse files Browse the repository at this point in the history
  • Loading branch information
cliffordwolf committed Aug 30, 2016
1 parent a8124c1 commit 6f41e52
Show file tree
Hide file tree
Showing 7 changed files with 8 additions and 21 deletions.
6 changes: 3 additions & 3 deletions frontends/ast/genrtlil.cc
Original file line number Diff line number Diff line change
Expand Up @@ -754,7 +754,7 @@ void AstNode::detectSignWidthWorker(int &width_hint, bool &sign_hint, bool *foun
break;

case AST_FCALL:
if (str == "\\$anyconst" || str == "\\$aconst") {
if (str == "\\$anyconst") {
if (GetSize(children) == 1) {
while (children[0]->simplify(true, false, false, 1, -1, false, true) == true) { }
if (children[0]->type != AST_CONSTANT)
Expand Down Expand Up @@ -1447,9 +1447,9 @@ RTLIL::SigSpec AstNode::genRTLIL(int width_hint, bool sign_hint)
} break;

case AST_FCALL: {
if (str == "\\$anyconst" || str == "\\$aconst")
if (str == "\\$anyconst")
{
string myid = stringf("%s$%d", RTLIL::unescape_id(str).c_str(), autoidx++);
string myid = stringf("%s$%d", str.c_str() + 1, autoidx++);
int width = width_hint;

if (GetSize(children) > 1)
Expand Down
4 changes: 2 additions & 2 deletions frontends/ast/simplify.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1672,8 +1672,8 @@ skip_dynamic_range_lvalue_expansion:;
goto apply_newNode;
}

// $anyconst and $aconst are mapped in AstNode::genRTLIL()
if (str == "\\$anyconst" || str == "\\$aconst")
// $anyconst is mapped in AstNode::genRTLIL()
if (str == "\\$anyconst")
return false;

if (str == "\\$clog2")
Expand Down
2 changes: 1 addition & 1 deletion frontends/verilog/verilog_parser.y
Original file line number Diff line number Diff line change
Expand Up @@ -1229,7 +1229,7 @@ rvalue:
$$ = new AstNode(AST_IDENTIFIER, $2);
$$->str = *$1;
delete $1;
if ($2 == nullptr && formal_mode && ($$->str == "\\$initstate" || $$->str == "\\$anyconst" || $$->str == "\\$aconst"))
if ($2 == nullptr && formal_mode && ($$->str == "\\$initstate" || $$->str == "\\$anyconst"))
$$->type = AST_FCALL;
} |
hierarchical_id non_opt_multirange {
Expand Down
1 change: 0 additions & 1 deletion kernel/celltypes.h
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,6 @@ struct CellTypes
setup_type("$assume", {A, EN}, pool<RTLIL::IdString>(), true);
setup_type("$initstate", pool<RTLIL::IdString>(), {Y}, true);
setup_type("$anyconst", pool<RTLIL::IdString>(), {Y}, true);
setup_type("$aconst", pool<RTLIL::IdString>(), {Y}, true);
setup_type("$equiv", {A, B}, {Y}, true);
}

Expand Down
2 changes: 1 addition & 1 deletion kernel/rtlil.cc
Original file line number Diff line number Diff line change
Expand Up @@ -1030,7 +1030,7 @@ namespace {
return;
}

if (cell->type.in("$aconst", "$anyconst")) {
if (cell->type == "$anyconst") {
port("\\Y", param("\\WIDTH"));
check_expected();
return;
Expand Down
2 changes: 1 addition & 1 deletion manual/CHAPTER_CellLib.tex
Original file line number Diff line number Diff line change
Expand Up @@ -421,7 +421,7 @@ \section{Gates}
using the {\tt abc} pass.

\begin{fixme}
Add information about {\tt \$assert}, {\tt \$assume}, {\tt \$equiv}, {\tt \$initstate}, {\tt \$aconst}, and {\tt \$anyconst} cells.
Add information about {\tt \$assert}, {\tt \$assume}, {\tt \$equiv}, {\tt \$initstate}, and {\tt \$anyconst} cells.
\end{fixme}

\begin{fixme}
Expand Down
12 changes: 0 additions & 12 deletions techlibs/common/simlib.v
Original file line number Diff line number Diff line change
Expand Up @@ -1322,18 +1322,6 @@ endmodule

// --------------------------------------------------------

module \$aconst (Y);

parameter WIDTH = 0;

output [WIDTH-1:0] Y;

assign Y = 'bx;

endmodule

// --------------------------------------------------------

module \$anyconst (Y);

parameter WIDTH = 0;
Expand Down

0 comments on commit 6f41e52

Please sign in to comment.