Changeset d672350 for src/Parser/parser.yy

Timestamp:

Mar 21, 2022, 1:44:06 PM (4 years ago)

Author:

Thierry Delisle <tdelisle@…>

Branches:

ADT, ast-experimental, enum, master, pthread-emulation, qualifiedEnum

Children:

a76202d

Parents:

ef3c383 (diff), dbe2533 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.

Message:

Merge branch 'master' of plg.uwaterloo.ca:software/cfa/cfa-cc

File:

• src/Parser/parser.yy (modified) (11 diffs)

src/Parser/parser.yy

@@ -10 +10 @@
 // Created On       : Sat Sep  1 20:22:55 2001
 // Last Modified By : Peter A. Buhr
-// Last Modified On : Fri Feb 11 14:26:15 2022
-// Update Count     : 5174
+// Last Modified On : Mon Mar 14 16:35:29 2022
+// Update Count     : 5276
 //
 
@@ -610 +610 @@
         // | RESUME '(' comma_expression ')' compound_statement
         //      { SemanticError( yylloc, "Resume expression is currently unimplemented." ); $$ = nullptr; }
+        | IDENTIFIER IDENTIFIER                                                         // syntax error
+                {
+                        SemanticError( yylloc, ::toString( "Adjacent identifiers are not meaningful in an expression. "
+                                                                                           "Possible problem is identifier \"", *$1.str,
+                                                                                           "\" is a misspelled typename or an incorrectly specified type name, "
+                                                                                           "e.g., missing generic parameter or missing struct/union/enum before typename." ) );
+                        $$ = nullptr;
+                }
+        | IDENTIFIER direct_type                                                        // syntax error
+                {
+                        SemanticError( yylloc, ::toString( "Identifier \"", *$1.str, "\" cannot appear before a type. "
+                                                                                           "Possible problem is misspelled storage or CV qualifier." ) );
+                        $$ = nullptr;
+                }
         ;
 
@@ -638 +652 @@
                         // Historic, transitional: Disallow commas in subscripts.
                         // Switching to this behaviour may help check if a C compatibilty case uses comma-exprs in subscripts.
-                // { SemanticError( yylloc, "New array subscript is currently unimplemented." ); $$ = nullptr; }
                         // Current: Commas in subscripts make tuples.
                 { $$ = new ExpressionNode( build_binary_val( OperKinds::Index, $1, new ExpressionNode( build_tuple( (ExpressionNode *)($3->set_last( $5 ) ) )) ) ); }
@@ -647 +660 @@
                 // equivalent to the old x[i,j].
                 { $$ = new ExpressionNode( build_binary_val( OperKinds::Index, $1, $3 ) ); }
+        | constant '[' assignment_expression ']'                        // 3[a], 'a'[a], 3.5[a]
+                { $$ = new ExpressionNode( build_binary_val( OperKinds::Index, $1, $3 ) ); }
+        | string_literal '[' assignment_expression ']'          // "abc"[3], 3["abc"]
+                { $$ = new ExpressionNode( build_binary_val( OperKinds::Index, new ExpressionNode( $1 ), $3 ) ); }
         | postfix_expression '{' argument_expression_list_opt '}' // CFA, constructor call
                 {
@@ -1052 +1069 @@
         identifier_or_type_name ':' attribute_list_opt statement
                 { $$ = $4->add_label( $1, $3 ); }
+        | identifier_or_type_name ':' attribute_list_opt error // syntax error
+                {
+                        SemanticError( yylloc, ::toString( "Label \"", *$1.str, "\" must be associated with a statement, "
+                                                                                           "where a declaration, case, or default is not a statement. "
+                                                                                           "Move the label or terminate with a semi-colon." ) );
+                        $$ = nullptr;
+                }
         ;
 
@@ -1086 +1110 @@
         | statement_list_nodecl statement
                 { assert( $1 ); $1->set_last( $2 ); $$ = $1; }
+        | statement_list_nodecl error                                           // syntax error
+                { SemanticError( yylloc, "Declarations only allowed at the start of the switch body, i.e., after the '{'." ); $$ = nullptr; }
         ;
 
@@ -1093 +1119 @@
         | MUTEX '(' ')' comma_expression ';'
                 { $$ = new StatementNode( build_mutex( nullptr, new StatementNode( build_expr( $4 ) ) ) ); }
-                // { SemanticError( yylloc, "Mutex expression is currently unimplemented." ); $$ = nullptr; }
         ;
 
@@ -1113 +1138 @@
                         $$ = $7 ? new StatementNode( build_compound( (StatementNode *)((new StatementNode( $7 ))->set_last( sw )) ) ) : sw;
                 }
+        | SWITCH '(' comma_expression ')' '{' error '}'         // CFA, syntax error
+                { SemanticError( yylloc, "Only declarations can appear before the list of case clauses." ); $$ = nullptr; }
         | CHOOSE '(' comma_expression ')' case_clause           // CFA
                 { $$ = new StatementNode( build_switch( false, $3, $5 ) ); }
@@ -1120 +1147 @@
                         $$ = $7 ? new StatementNode( build_compound( (StatementNode *)((new StatementNode( $7 ))->set_last( sw )) ) ) : sw;
                 }
+        | CHOOSE '(' comma_expression ')' '{' error '}'         // CFA, syntax error
+                { SemanticError( yylloc, "Only declarations can appear before the list of case clauses." ); $$ = nullptr; }
         ;
 
@@ -1158 +1187 @@
 
 case_label:                                                                                             // CFA
-        CASE case_value_list ':'                                        { $$ = $2; }
+        CASE error                                                                                      // syntax error
+                { SemanticError( yylloc, "Missing case list after case." ); $$ = nullptr; }
+        | CASE case_value_list ':'                                      { $$ = $2; }
+        | CASE case_value_list error                                            // syntax error
+                { SemanticError( yylloc, "Missing colon after case list." ); $$ = nullptr; }
         | DEFAULT ':'                                                           { $$ = new StatementNode( build_default() ); }
                 // A semantic check is required to ensure only one default clause per switch/choose statement.
-        ;
-
-//label_list_opt:
-//      // empty
-//      | identifier_or_type_name ':'
-//      | label_list_opt identifier_or_type_name ':'
-//      ;
+        | DEFAULT error                                                                         //  syntax error
+                { SemanticError( yylloc, "Missing colon after default." ); $$ = nullptr; }
+        ;
 
 case_label_list:                                                                                // CFA
@@ -1403 +1432 @@
         | when_clause_opt ELSE statement
                 { $$ = build_waitfor_timeout( nullptr, maybe_build_compound( $3 ), $1 ); }
-                // "else" must be conditional after timeout or timeout is never triggered (i.e., it is meaningless)
-        | when_clause_opt timeout statement WOR ELSE statement
+        // "else" must be conditional after timeout or timeout is never triggered (i.e., it is meaningless)
+        | when_clause_opt timeout statement WOR ELSE statement // syntax error
                 { SemanticError( yylloc, "else clause must be conditional after timeout or timeout never triggered." ); $$ = nullptr; }
         | when_clause_opt timeout statement WOR when_clause ELSE statement