Fix memory destructors not called for frame variables

Author: massimofioravanti

lib/dialect/src/LowerActionPass.cpp

@@ -5,7 +5,7 @@ Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.
 You may obtain a copy of the License at
 
-   http://www.apache.org/licenses/LICENSE-2.0
+	 http://www.apache.org/licenses/LICENSE-2.0
 
 Unless required by applicable law or agreed to in writing, software
 distributed under the License is distributed on an "AS IS" BASIS,
@@ -132,6 +132,7 @@ namespace mlir::rlc
 					builtinOperatorName<mlir::rlc::AssignOp>(),
 					mlir::ValueRange({ refToMember, decl }));
 			assert(call != nullptr);
+			builder.emitCall(decl, true, "drop", mlir::ValueRange({ decl }), false);
 		}
 	}
 

lib/dialect/src/SymbolTable.cpp

@@ -5,7 +5,7 @@ Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.
 You may obtain a copy of the License at
 
-   http://www.apache.org/licenses/LICENSE-2.0
+	 http://www.apache.org/licenses/LICENSE-2.0
 
 Unless required by applicable law or agreed to in writing, software
 distributed under the License is distributed on an "AS IS" BASIS,
@@ -583,8 +583,8 @@ mlir::Operation* mlir::rlc::ModuleBuilder::emitCall(
 		return maybeCast;
 
 	auto argTypes = arguments.getType();
-	auto overload = resolveFunctionCall(
-			emitLog ? callSite : nullptr, isMemberCall, name, arguments);
+	auto overload =
+			resolveFunctionCall(callSite, isMemberCall, name, arguments, emitLog);
 
 	if (overload == nullptr)
 		return nullptr;

tool/rlc/test/vector_in_action.rl

@@ -0,0 +1,11 @@
+# RUN: rlc %s -o %t -i %stdlib --sanitize -g
+# RUN: %t
+
+import collections.vector
+
+act has_member() -> HM:
+    frm member: Vector<Int>  
+
+fun main() -> Int:
+    let hm = has_member()
+    return 0