hdl.ast: bring new naming rules to IOPort

Author: tpwrules

amaranth/hdl/_ast.py

@@ -2824,8 +2824,7 @@ class IOPort(IOValue):
     def __init__(self, width, *, name=None, attrs=None, metadata=None, src_loc_at=0):
         super().__init__(src_loc_at=src_loc_at)
 
-        if name is not None and not isinstance(name, str):
-            raise TypeError(f"Name must be a string, not {name!r}")
+        validate_name(name, "Name", none_ok=True)
         self.name = name or tracer.get_var_name(depth=2 + src_loc_at)
 
         self._width = operator.index(width)

tests/test_hdl_ast.py

@@ -1732,6 +1732,9 @@ def test_ioport(self):
         self.assertEqual(b.metadata, ("x", "y", "z"))
         self.assertEqual(b._ioports(), {b})
         self.assertRepr(b, "(io-port b)")
+        with self.assertRaisesRegex(NameError,
+                r"^Name must be a non-empty string$"):
+            IOPort(1, name="")
 
     def test_ioport_wrong(self):
         with self.assertRaisesRegex(TypeError,