Changeset e116db3

Timestamp:

Aug 15, 2022, 5:29:20 PM (16 months ago)

Author:

Andrew Beach <ajbeach@…>

Branches:

ADT, ast-experimental, master, pthread-emulation

Children:

17c6edeb, 3ce3fb9, 4fee301

Parents:

d93ea1d

Message:

Combined some sub-passes in Tuple Expansion, two less tree traversals now have to be preformed.

File:

• src/Tuples/TupleExpansionNew.cpp (modified) (6 diffs)

src/Tuples/TupleExpansionNew.cpp

@@ -10 +10 @@
 // Created On       : Mon Aug 23 15:36:09 2021
 // Last Modified By : Andrew Beach
-// Last Modified On : Mon Jul 29 14:06:00 2022
-// Update Count     : 2
+// Last Modified On : Mon Aug 15 17:00:00 2022
+// Update Count     : 3
 //
 
@@ -100 +100 @@
 }
 
-struct TupleAssignExpander {
+/// Replaces Tuple Assign & Index Expressions, and Tuple Types.
+struct TupleMainExpander :
+                public ast::WithGuards,
+                public ast::WithVisitorRef<TupleMainExpander>,
+                public ast::WithDeclsToAdd<> {
         ast::Expr const * postvisit( ast::TupleAssignExpr const * expr ) {
                 // Just move the env on the new top level expression.
@@ -106 +110 @@
                         &ast::TupleAssignExpr::env, expr->env.get() );
         }
-};
-
-struct TupleTypeReplacer :
-                public ast::WithGuards,
-                public ast::WithVisitorRef<TupleTypeReplacer>,
-                public ast::WithDeclsToAdd<> {
+
         void previsit( ast::ParseNode const * node ) {
                 GuardValue( location ) = &node->location;
@@ -185 +184 @@
                 return newType;
         }
-private:
-        ScopedMap< int, ast::StructDecl const * > typeMap;
-        CodeLocation const * location = nullptr;
-};
-
-struct TupleIndexExpander {
+
         ast::Expr const * postvisit( ast::TupleIndexExpr const * expr ) {
                 CodeLocation const & location = expr->location;
@@ -221 +215 @@
                 return memberExpr;
         }
+private:
+        ScopedMap< int, ast::StructDecl const * > typeMap;
+        CodeLocation const * location = nullptr;
 };
 
@@ -275 +272 @@
 
 void expandTuples( ast::TranslationUnit & translationUnit ) {
-        // These may not have to be seperate passes.
-        ast::Pass<TupleAssignExpander>::run( translationUnit );
-        ast::Pass<TupleTypeReplacer>::run( translationUnit );
-        ast::Pass<TupleIndexExpander>::run( translationUnit );
+        // These can't just be combined simply (there might be a way with work).
+        ast::Pass<TupleMainExpander>::run( translationUnit );
         ast::Pass<TupleExprExpander>::run( translationUnit );
 }