Replace | dict operator with .update()

This improves compatibility with Python 3.8 Internal-tag: [#53526] Signed-off-by: Krzysztof Obłonczek <[email protected]>
antmicro · Jan 9, 2024 · e4b4a62 · e4b4a62
1 parent 5e40dfd
commit e4b4a62
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/fpga_topwrap/elaboratable_wrapper.py b/fpga_topwrap/elaboratable_wrapper.py
@@ -43,10 +43,14 @@ def get_ports_hier(self) -> SignalMapping:
         """Maps elaboratable's Signature to a nested dictionary of WrapperPorts.
         See _gather_signature_ports for more details.
         """
-        return self._gather_signature_ports(self.elaboratable.signature) | {
-            "clk": self.clk,
-            "rst": self.rst,
-        }
+        ports = self._gather_signature_ports(self.elaboratable.signature)
+        ports.update(
+            {
+                "clk": self.clk,
+                "rst": self.rst,
+            }
+        )
+        return ports
 
     @lru_cache(maxsize=None)
     def _cached_wrapper(

diff --git a/tests/tests_build/test_elaboratable_wrapper.py b/tests/tests_build/test_elaboratable_wrapper.py
@@ -299,7 +299,8 @@ def test_get_ports_hier(
         clock_domain_signals: SignalMapping,
     ) -> None:
         hier_ports = elaboratable_wrapper.get_ports_hier()
-        assert signal_mapping_eq(hier_ports, nested_signature_mapping | clock_domain_signals)
+        nested_signature_mapping.update(clock_domain_signals)
+        assert signal_mapping_eq(hier_ports, nested_signature_mapping)
 
     def test_get_ports(
         self,