Changeset ab94c37 for src

Timestamp:

Feb 5, 2025, 5:06:45 PM (8 weeks ago)

Author:

Andrew Beach <ajbeach@…>

Branches:

master

Children:

301e9f7

Parents:

d3d54b3

Message:

Change in actor ?|? function generation. It looks accessive (and in a sense it is) but this makes it constent with the general return pattern that makes sure the assignment is called. This removes a lot of warnings, but nodebug builds of the actor still fail because of another warning.

File:

• src/Concurrency/Actors.cpp (modified) (1 diff)

src/Concurrency/Actors.cpp ¶

@@ -401 +401 @@
                         ));
 
-                        // Generates: return receiver;
-                        sendBody->push_back( new ReturnStmt( decl->location, new NameExpr( decl->location, "receiver" ) ) );
+                        // Generates: &receiver_ret = &receiver; return receiver_ret;
+                        // Note: This is the general pattern used as a "return convention".
+                        sendBody->push_back( new ExprStmt( decl->location,
+                                ast::UntypedExpr::createAssign( decl->location,
+                                        new AddressExpr( new NameExpr( decl->location, "receiver_ret" ) ),
+                                        new AddressExpr( new NameExpr( decl->location, "receiver" ) )
+                                )
+                        ) );
+                        sendBody->push_back( new ReturnStmt( decl->location,
+                                new NameExpr( decl->location, "receiver_ret" ) ) );
 
                         // put it all together into the complete function decl from above