Set _LLVM_VERSION as default value for Target.llvm_version

Author: danigm

Tools/jit/_llvm.py

@@ -96,25 +96,25 @@ async def _find_tool(tool: str, llvm_version: str, *, echo: bool = False) -> str
 
 
 async def maybe_run(
-    tool: str, args: typing.Iterable[str], echo: bool = False, llvm_version: str = ""
+    tool: str,
+    args: typing.Iterable[str],
+    echo: bool = False,
+    llvm_version: str = _LLVM_VERSION,
 ) -> str | None:
     """Run an LLVM tool if it can be found. Otherwise, return None."""
 
-    if not llvm_version:
-        llvm_version = _LLVM_VERSION
-
     path = await _find_tool(tool, llvm_version, echo=echo)
     return path and await _run(path, args, echo=echo)
 
 
 async def run(
-    tool: str, args: typing.Iterable[str], echo: bool = False, llvm_version: str = ""
+    tool: str,
+    args: typing.Iterable[str],
+    echo: bool = False,
+    llvm_version: str = _LLVM_VERSION,
 ) -> str:
     """Run an LLVM tool if it can be found. Otherwise, raise RuntimeError."""
 
-    if not llvm_version:
-        llvm_version = _LLVM_VERSION
-
     output = await maybe_run(tool, args, echo=echo, llvm_version=llvm_version)
     if output is None:
         raise RuntimeError(f"Can't find {tool}-{llvm_version}!")

Tools/jit/_targets.py

@@ -50,7 +50,7 @@ class _Target(typing.Generic[_S, _R]):
     debug: bool = False
     verbose: bool = False
     cflags: str = ""
-    llvm_version: str = ""
+    llvm_version: str = _llvm._LLVM_VERSION
     known_symbols: dict[str, int] = dataclasses.field(default_factory=dict)
     pyconfig_dir: pathlib.Path = pathlib.Path.cwd().resolve()