From 99c3a8db491b4d76f35d54ec01f46562f14d261c Mon Sep 17 00:00:00 2001
From: Frederik Hennig <frederik.hennig@fau.de>
Date: Wed, 3 Apr 2024 17:08:40 +0200
Subject: [PATCH] remove two superfluous code lines

---
 src/pystencils/backend/transformations/canonicalize_symbols.py  | 1 -
 .../backend/transformations/hoist_loop_invariant_decls.py       | 2 --
 2 files changed, 3 deletions(-)

diff --git a/src/pystencils/backend/transformations/canonicalize_symbols.py b/src/pystencils/backend/transformations/canonicalize_symbols.py
index fdbba3dd3..6fe922f28 100644
--- a/src/pystencils/backend/transformations/canonicalize_symbols.py
+++ b/src/pystencils/backend/transformations/canonicalize_symbols.py
@@ -29,7 +29,6 @@ class CanonContext:
             self.live_symbols_map[symb] = symb
             return symb
         else:
-            self.encountered_symbols.add(symb)
             for i in count():
                 replacement_name = f"{symb.name}__{i}"
                 if self._ctx.find_symbol(replacement_name) is None:
diff --git a/src/pystencils/backend/transformations/hoist_loop_invariant_decls.py b/src/pystencils/backend/transformations/hoist_loop_invariant_decls.py
index 723e4472a..592003815 100644
--- a/src/pystencils/backend/transformations/hoist_loop_invariant_decls.py
+++ b/src/pystencils/backend/transformations/hoist_loop_invariant_decls.py
@@ -73,8 +73,6 @@ class HoistLoopInvariantDeclarations:
     but makes no such assumption about instances of `CFunction`.
     """
 
-    #   TODO: Write unit tests
-
     def __init__(self, ctx: KernelCreationContext):
         self._ctx = ctx
 
-- 
GitLab