After reading the SV spec, using non-standard predict() instead of expect()

diff --git a/README b/README
index 2a1dde4352cecc4c5447f080c7c7847cd7291cb2..3f1591bb275e7dbe6e54df44a667f124e2a2a946 100644 (file)
--- a/README
+++ b/README
@@ -384,8 +384,8 @@ from SystemVerilog:
   form. In module context: "assert property (<expression>);" and within an
   always block: "assert(<expression>);". It is transformed to a $assert cell.
 
-- The "assume" and "expect" statements from SystemVerilog are also
-  supported. The same limitations as with the "assert" statement apply.
+- The "assume" statements from SystemVerilog are also supported. The same
+  limitations as with the "assert" statement apply.
 
 - The keywords "always_comb", "always_ff" and "always_latch", "logic" and
   "bit" are supported.

diff --git a/examples/smtbmc/demo1.v b/examples/smtbmc/demo1.v
index 59e4978251e049b1923e1e954149efe28af1b350..2e628b7da554121b3dc5834e17aebd04ec1e4cf9 100644 (file)
--- a/examples/smtbmc/demo1.v
+++ b/examples/smtbmc/demo1.v
@@ -8,7 +8,7 @@ module demo1(input clk, input addtwo, output iseven);
                cnt = (iseven ? cnt == 10 : cnt == 11) ? 0 : next_cnt;
        
        assert property (cnt != 15);
-       // initial expect ((iseven && addtwo) || cnt == 9);
+       // initial predict ((iseven && addtwo) || cnt == 9);
 endmodule
 
 module inc(input addtwo, output iseven, input [3:0] a, output [3:0] y);

diff --git a/frontends/ast/ast.cc b/frontends/ast/ast.cc
index c298d5a9880141269c9b237b3680b0ba17a9d9cc..82b4edef16aa6a19ed37a3a64eeb4860b4a21fd1 100644 (file)
--- a/frontends/ast/ast.cc
+++ b/frontends/ast/ast.cc
@@ -82,7 +82,7 @@ std::string AST::type2str(AstNodeType type)
        X(AST_PREFIX)
        X(AST_ASSERT)
        X(AST_ASSUME)
-       X(AST_EXPECT)
+       X(AST_PREDICT)
        X(AST_FCALL)
        X(AST_TO_BITS)
        X(AST_TO_SIGNED)

diff --git a/frontends/ast/ast.h b/frontends/ast/ast.h
index 5c2c51b8e2fd0c24a7b2e73d56262e425c0d4ab9..5310bcadb50bccf7849076b1021459e4fc3a9caf 100644 (file)
--- a/frontends/ast/ast.h
+++ b/frontends/ast/ast.h
@@ -65,7 +65,7 @@ namespace AST
                AST_PREFIX,
                AST_ASSERT,
                AST_ASSUME,
-               AST_EXPECT,
+               AST_PREDICT,
 
                AST_FCALL,
                AST_TO_BITS,

diff --git a/frontends/ast/genrtlil.cc b/frontends/ast/genrtlil.cc
index 31367b87e70a0eaaf6c29d60ded23f8cfde57247..2fb95ff5a17d266a9f8d03cf07377930ad35b594 100644 (file)
--- a/frontends/ast/genrtlil.cc
+++ b/frontends/ast/genrtlil.cc
@@ -1296,11 +1296,11 @@ RTLIL::SigSpec AstNode::genRTLIL(int width_hint, bool sign_hint)
        // generate $assert cells
        case AST_ASSERT:
        case AST_ASSUME:
-       case AST_EXPECT:
+       case AST_PREDICT:
                {
                        const char *celltype = "$assert";
                        if (type == AST_ASSUME) celltype = "$assume";
-                       if (type == AST_EXPECT) celltype = "$expect";
+                       if (type == AST_PREDICT) celltype = "$predict";
 
                        log_assert(children.size() == 2);
 

diff --git a/frontends/ast/simplify.cc b/frontends/ast/simplify.cc
index cf84a399c51627229fd84d5e5eb72c097def6e8e..18a752e06f8955776ec610aace4c91bfbaaee93c 100644 (file)
--- a/frontends/ast/simplify.cc
+++ b/frontends/ast/simplify.cc
@@ -1348,7 +1348,7 @@ bool AstNode::simplify(bool const_fold, bool at_zero, bool in_lvalue, int stage,
        }
 skip_dynamic_range_lvalue_expansion:;
 
-       if (stage > 1 && (type == AST_ASSERT || type == AST_ASSUME || type == AST_EXPECT) && current_block != NULL)
+       if (stage > 1 && (type == AST_ASSERT || type == AST_ASSUME || type == AST_PREDICT) && current_block != NULL)
        {
                std::stringstream sstr;
                sstr << "$formal$" << filename << ":" << linenum << "$" << (autoidx++);
@@ -1405,7 +1405,7 @@ skip_dynamic_range_lvalue_expansion:;
                goto apply_newNode;
        }
 
-       if (stage > 1 && (type == AST_ASSERT || type == AST_ASSUME || type == AST_EXPECT) && children.size() == 1)
+       if (stage > 1 && (type == AST_ASSERT || type == AST_ASSUME || type == AST_PREDICT) && children.size() == 1)
        {
                children.push_back(mkconst_int(1, false, 1));
                did_something = true;

diff --git a/frontends/verilog/verilog_lexer.l b/frontends/verilog/verilog_lexer.l
index c9a59d66521b2e70c328b3b33de85eb300b6eeb6..aafdbbf03091d0dc32e0cda877bd6c45208f978c 100644 (file)
--- a/frontends/verilog/verilog_lexer.l
+++ b/frontends/verilog/verilog_lexer.l
@@ -63,6 +63,10 @@ YOSYS_NAMESPACE_END
        frontend_verilog_yylval.string = new std::string(std::string("\\") + yytext); \
        return TOK_ID;
 
+#define NON_KEYWORD() \
+       frontend_verilog_yylval.string = new std::string(std::string("\\") + yytext); \
+       return TOK_ID;
+
 #define YY_INPUT(buf,result,max_size) \
        result = readsome(*VERILOG_FRONTEND::lexin, buf, max_size)
 
@@ -173,7 +177,7 @@ YOSYS_NAMESPACE_END
 
 "assert"   { if (formal_mode) return TOK_ASSERT; SV_KEYWORD(TOK_ASSERT); }
 "assume"   { if (formal_mode) return TOK_ASSUME; SV_KEYWORD(TOK_ASSUME); }
-"expect"   { if (formal_mode) return TOK_EXPECT; SV_KEYWORD(TOK_EXPECT); }
+"predict"   { if (formal_mode) return TOK_PREDICT; NON_KEYWORD(); }
 "property" { if (formal_mode) return TOK_PROPERTY; SV_KEYWORD(TOK_PROPERTY); }
 "logic"    { SV_KEYWORD(TOK_REG); }
 "bit"      { SV_KEYWORD(TOK_REG); }

diff --git a/frontends/verilog/verilog_parser.y b/frontends/verilog/verilog_parser.y
index bfb4990b2d09ce219c83ebd31e54e2f46975b22c..10de3a19fc3f53704a2f334b35712726258454a8 100644 (file)
--- a/frontends/verilog/verilog_parser.y
+++ b/frontends/verilog/verilog_parser.y
@@ -113,7 +113,7 @@ static void free_attr(std::map<std::string, AstNode*> *al)
 %token TOK_SYNOPSYS_FULL_CASE TOK_SYNOPSYS_PARALLEL_CASE
 %token TOK_SUPPLY0 TOK_SUPPLY1 TOK_TO_SIGNED TOK_TO_UNSIGNED
 %token TOK_POS_INDEXED TOK_NEG_INDEXED TOK_ASSERT TOK_ASSUME
-%token TOK_EXPECT TOK_PROPERTY
+%token TOK_PREDICT TOK_PROPERTY
 
 %type <ast> range range_or_multirange  non_opt_range non_opt_multirange range_or_signed_int
 %type <ast> wire_type expr basic_expr concat_list rvalue lvalue lvalue_concat_list
@@ -967,8 +967,8 @@ assert:
        TOK_ASSUME '(' expr ')' ';' {
                ast_stack.back()->children.push_back(new AstNode(AST_ASSUME, $3));
        } |
-       TOK_EXPECT '(' expr ')' ';' {
-               ast_stack.back()->children.push_back(new AstNode(AST_EXPECT, $3));
+       TOK_PREDICT '(' expr ')' ';' {
+               ast_stack.back()->children.push_back(new AstNode(AST_PREDICT, $3));
        };
 
 assert_property:
@@ -978,8 +978,8 @@ assert_property:
        TOK_ASSUME TOK_PROPERTY '(' expr ')' ';' {
                ast_stack.back()->children.push_back(new AstNode(AST_ASSUME, $4));
        } |
-       TOK_EXPECT TOK_PROPERTY '(' expr ')' ';' {
-               ast_stack.back()->children.push_back(new AstNode(AST_EXPECT, $4));
+       TOK_PREDICT TOK_PROPERTY '(' expr ')' ';' {
+               ast_stack.back()->children.push_back(new AstNode(AST_PREDICT, $4));
        };
 
 simple_behavioral_stmt:

diff --git a/kernel/celltypes.h b/kernel/celltypes.h
index 78403fcd3e7f45170a2f836c8ea30932a46a2c48..c3f05de579901002555f59b35a805779fc45191b 100644 (file)
--- a/kernel/celltypes.h
+++ b/kernel/celltypes.h
@@ -116,7 +116,7 @@ struct CellTypes
 
                setup_type("$assert", {A, EN}, pool<RTLIL::IdString>(), true);
                setup_type("$assume", {A, EN}, pool<RTLIL::IdString>(), true);
-               setup_type("$expect", {A, EN}, pool<RTLIL::IdString>(), true);
+               setup_type("$predict", {A, EN}, pool<RTLIL::IdString>(), true);
                setup_type("$equiv", {A, B}, {Y}, true);
        }
 

diff --git a/kernel/rtlil.cc b/kernel/rtlil.cc
index 644a83a76f9bad8bed20020430a442c581b85bd6..cf3c80604fff6475288b1365b59b0c6f8282b59f 100644 (file)
--- a/kernel/rtlil.cc
+++ b/kernel/rtlil.cc
@@ -1017,7 +1017,7 @@ namespace {
                                return;
                        }
 
-                       if (cell->type.in("$assert", "$assume", "$expect")) {
+                       if (cell->type.in("$assert", "$assume", "$predict")) {
                                port("\\A", 1);
                                port("\\EN", 1);
                                check_expected();
@@ -1798,7 +1798,7 @@ RTLIL::Cell* RTLIL::Module::addAssume(RTLIL::IdString name, RTLIL::SigSpec sig_a
 
 RTLIL::Cell* RTLIL::Module::addExpect(RTLIL::IdString name, RTLIL::SigSpec sig_a, RTLIL::SigSpec sig_en)
 {
-       RTLIL::Cell *cell = addCell(name, "$expect");
+       RTLIL::Cell *cell = addCell(name, "$predict");
        cell->setPort("\\A", sig_a);
        cell->setPort("\\EN", sig_en);
        return cell;

diff --git a/kernel/satgen.h b/kernel/satgen.h
index da892c710e6eb181db6add42732a009c0ff45d93..22b11fe26c80c7f198a662495eebcbf84adfdb18 100644 (file)
--- a/kernel/satgen.h
+++ b/kernel/satgen.h
@@ -1347,7 +1347,7 @@ struct SatGen
                        return true;
                }
 
-               if (cell->type == "$expect")
+               if (cell->type == "$predict")
                {
                        std::string pf = prefix + (timestep == -1 ? "" : stringf("@%d:", timestep));
                        expects_a[pf].append((*sigmap)(cell->getPort("\\A")));

diff --git a/manual/CHAPTER_CellLib.tex b/manual/CHAPTER_CellLib.tex
index 759a5bb6a00e997935f0ecfbeefc0489d0835687..b1e000aa24a8d25f163cf1a01f4d725a80cf4cb7 100644 (file)
--- a/manual/CHAPTER_CellLib.tex
+++ b/manual/CHAPTER_CellLib.tex
@@ -421,7 +421,7 @@ pass. The combinatorial logic cells can be mapped to physical cells from a Liber
 using the {\tt abc} pass.
 
 \begin{fixme}
-Add information about {\tt \$assert}, {\tt \$assume}, {\tt \$expect}, and {\tt \$equiv} cells.
+Add information about {\tt \$assert}, {\tt \$assume}, {\tt \$predict}, and {\tt \$equiv} cells.
 \end{fixme}
 
 \begin{fixme}

diff --git a/passes/hierarchy/hierarchy.cc b/passes/hierarchy/hierarchy.cc
index 14d67884e654f8eefd103c958f4e0016a50001c9..92fcb7d40a855cec0b056109ade5ddb3d39fc7bb 100644 (file)
--- a/passes/hierarchy/hierarchy.cc
+++ b/passes/hierarchy/hierarchy.cc
@@ -313,7 +313,7 @@ bool set_keep_assert(std::map<RTLIL::Module*, bool> &cache, RTLIL::Module *mod)
        if (cache.count(mod) == 0)
                for (auto c : mod->cells()) {
                        RTLIL::Module *m = mod->design->module(c->type);
-                       if ((m != nullptr && set_keep_assert(cache, m)) || c->type.in("$assert", "$assume", "$expect"))
+                       if ((m != nullptr && set_keep_assert(cache, m)) || c->type.in("$assert", "$assume", "$predict"))
                                return cache[mod] = true;
                }
        return cache[mod];

diff --git a/passes/opt/opt_clean.cc b/passes/opt/opt_clean.cc
index d905a30bfc1bffdd40d378d8dcfbd561673d7875..1546ec3fc69d2bf6d5c766b5c365f036189c35bc 100644 (file)
--- a/passes/opt/opt_clean.cc
+++ b/passes/opt/opt_clean.cc
@@ -64,7 +64,7 @@ struct keep_cache_t
 
        bool query(Cell *cell)
        {
-               if (cell->type.in("$memwr", "$meminit", "$assert", "$assume", "$expect"))
+               if (cell->type.in("$memwr", "$meminit", "$assert", "$assume", "$predict"))
                        return true;
 
                if (cell->has_keep_attr())

diff --git a/passes/tests/test_cell.cc b/passes/tests/test_cell.cc
index 1bff02a2138200853ce134ef94d9e470f9e8b157..abd56d10bede315f32874604b599b7aad432c4b5 100644 (file)
--- a/passes/tests/test_cell.cc
+++ b/passes/tests/test_cell.cc
@@ -731,7 +731,7 @@ struct TestCellPass : public Pass {
                // cell_types["$concat"] = "A";
                // cell_types["$assert"] = "A";
                // cell_types["$assume"] = "A";
-               // cell_types["$expect"] = "A";
+               // cell_types["$predict"] = "A";
 
                cell_types["$lut"] = "*";
                cell_types["$sop"] = "*";

diff --git a/techlibs/common/simlib.v b/techlibs/common/simlib.v
index ea36e29226b2a6d4018053ddccc9d6a19f66cc0c..38687489ae819fb1217bdc075543b68ff3c4d016 100644 (file)
--- a/techlibs/common/simlib.v
+++ b/techlibs/common/simlib.v
@@ -1305,18 +1305,10 @@ endmodule
 
 // --------------------------------------------------------
 
-module \$expect (A, EN);
+module \$predict (A, EN);
 
 input A, EN;
 
-`ifndef SIMLIB_NOCHECKS
-always @* begin
-       if (A === 1'b1 && EN === 1'b1) begin
-               $display("Expectation %m passed.");
-       end
-end
-`endif
-
 endmodule
 
 // --------------------------------------------------------