354 duplicate functions should be a problem

src/ast.rs

@@ -216,6 +216,7 @@ pub struct Implementation {
     pub pou_type: PouType,
     pub statements: Vec<AstStatement>,
     pub location: SourceRange,
+    pub name_location: SourceRange,
     pub overriding: bool,
     pub generic: bool,
     pub access: Option<AccessModifier>,

src/codegen/generators/data_type_generator.rs

@@ -62,7 +62,7 @@ pub fn generate_data_types<'ink>(
     let types = generator
         .index
         .get_types()
-        .iter()
+        .elements()
         .filter(|(_, it)| !it.get_type_information().is_generic(generator.index))
         .map(|(a, b)| (a.as_str(), b))
         .collect::<Vec<(&str, &DataType)>>();

src/codegen/generators/pou_generator.rs

@@ -54,7 +54,7 @@ pub fn generate_implementation_stubs<'ink>(
 ) -> Result<LlvmTypedIndex<'ink>, Diagnostic> {
     let mut llvm_index = LlvmTypedIndex::default();
     let pou_generator = PouGenerator::new(llvm, index, annotations, types_index);
-    for (name, implementation) in index.get_implementations() {
+    for (name, implementation) in index.get_implementations().elements() {
         if !implementation.is_generic() {
             let curr_f = pou_generator.generate_implementation_stub(implementation, module)?;
             llvm_index.associate_implementation(name, curr_f)?;
@@ -76,7 +76,7 @@ pub fn generate_global_constants_for_pou_members<'ink>(
     llvm_index: &LlvmTypedIndex<'ink>,
 ) -> Result<LlvmTypedIndex<'ink>, Diagnostic> {
     let mut local_llvm_index = LlvmTypedIndex::default();
-    for (_, implementation) in index.get_implementations() {
+    for implementation in index.get_implementations().values() {
         let type_name = implementation.get_type_name();
         let pou_members = index.get_container_members(type_name);
         let variables = pou_members

src/codegen/generators/variable_generator.rs

@@ -36,11 +36,17 @@ pub fn generate_global_variables<'ctx, 'b>(
     }
 
     //all declared global variables
-    let globals = global_index.get_globals().iter().map(to_k_v);
+    let globals = global_index.get_globals().elements().map(to_k_v);
     //all initializers
-    let initializers = global_index.get_global_initializers().iter().map(to_k_v);
+    let initializers = global_index
+        .get_global_initializers()
+        .elements()
+        .map(to_k_v);
     //all enum-elements
-    let enums = global_index.get_global_qualified_enums().iter().map(to_k_v);
+    let enums = global_index
+        .get_global_qualified_enums()
+        .elements()
+        .map(to_k_v);
     //all program instances
     let programs = global_index
         .get_pous()

src/codegen/tests/code_gen_tests.rs

@@ -2810,7 +2810,12 @@ fn variable_with_same_name_as_data_type() {
     insta::assert_snapshot!(result);
 }
 
+/// THIS TEST IS MISLEADING!!!
+/// THERE IS A CONFLICT BETWEEN TIME.TIME (the VAR)
+/// AND TIME.TIME (the return variable) WHICH
+/// CANNOT BE HANDLED PROPERLY. VALIDATION SHOULD FAIL!
 #[test]
+#[ignore = "worked in the past, should fail!"]
 fn variable_with_same_name_as_function() {
     let result = codegen(
         "