Changeset 3e135c8 for src/Virtual/Tables.cpp

Timestamp:

Jul 12, 2024, 11:45:24 AM (6 weeks ago)

Author:

Andrew Beach <ajbeach@…>

Branches:

master

Children:

76b507d

Parents:

9c447e2

Message:

Not sure why we copy in the return values all the time, but we do. This prevents an unused error from appearing when the library is being compiled (shows up about 4 times right now). Switching some passes around might have also solved the problem, but I think we should be generating normalized code instead of mixing in the normalization and code generation.

File:

• src/Virtual/Tables.cpp (modified) (3 diffs)

src/Virtual/Tables.cpp

@@ -122 +122 @@
                         ) );
                 }
-                //ast::Expr * expr = buildInitExpr(...);
-                //inits.push_back( new ast::SingleInit( location, expr ) )
         }
 
@@ -162 +160 @@
         assert( vtableType );
         assert( exceptType );
+
+        // If this is called after Fix Return Statements (currently it is in
+        // Implement Concurrent Keywords) then this must be marked as unused
+        // to avoid warnings.
+        ast::ObjectDecl * frontRet = new ast::ObjectDecl( location,
+                "_retvalue",
+                new ast::ReferenceType( vtableType )
+        );
+        frontRet->attributes.emplace_back( new ast::Attribute( "unused" ) );
+
         return new ast::FunctionDecl(
                 location,
@@ -170 +178 @@
                         new ast::PointerType( exceptType )
                 ) },
-                { new ast::ObjectDecl(
-                        location,
-                        "_retvalue",
-                        new ast::ReferenceType( vtableType )
-                ) },
+                { frontRet },
                 nullptr,
                 ast::Storage::Classes(),