[pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

Author: pre-commit-ci[bot]

mypyc/analysis/dataflow.py

@@ -152,14 +152,12 @@ def cleanup_cfg(blocks: list[BasicBlock]) -> None:
         cfg = get_cfg(blocks)
         orig_blocks = blocks.copy()
         blocks.clear()
-        
+
         for i, block in enumerate(orig_blocks):
             # Keep the block if it's the entry block or has any predecessors
             # OR if it is a branch target (i.e., appears in any terminator.targets())
             is_branch_target = any(
-                block is tgt
-                for b in orig_blocks
-                for tgt in b.terminator.targets()
+                block is tgt for b in orig_blocks for tgt in b.terminator.targets()
             )
             if i == 0 or cfg.pred[block] or is_branch_target:
                 blocks.append(block)

mypyc/test-data/irbuild-set.test

@@ -791,4 +791,3 @@ L4:
     r9 = CPy_NoErrOccurred()
 L5:
     return 1
-

mypyc/transform/ir_transform.py

@@ -102,13 +102,18 @@ def transform_blocks(self, blocks: list[BasicBlock]) -> None:
         for block in block_map.values():
             for op in block.ops:
                 if isinstance(op, Branch):
-                    #raise ValueError(op.targets())
+                    # raise ValueError(op.targets())
                     branch_targets.update(op.targets())
 
         # Remove blocks that became empty (only Unreachable), are not branch targets, and were not empty before
         empties = set()
         for old_block, new_block in block_map.items():
-            if is_empty_block(new_block) and old_block not in originally_empty and old_block not in branch_targets and new_block not in branch_targets:
+            if (
+                is_empty_block(new_block)
+                and old_block not in originally_empty
+                and old_block not in branch_targets
+                and new_block not in branch_targets
+            ):
                 empties.add(new_block)
         self.builder.blocks = [block for block in self.builder.blocks if block not in empties]
 

mypyc/transform/redundant_box_elimination.py

@@ -1,7 +1,21 @@
 from typing import Dict
 
 from mypyc.ir.func_ir import FuncIR
-from mypyc.ir.ops import Assign, AssignMulti, Box, Branch, Call, CallC, ComparisonOp, DecRef, IncRef, KeepAlive, Return, SetMem, Unbox, Value, Op
+from mypyc.ir.ops import (
+    Assign,
+    AssignMulti,
+    Box,
+    Call,
+    CallC,
+    ComparisonOp,
+    DecRef,
+    IncRef,
+    KeepAlive,
+    Op,
+    Return,
+    Unbox,
+    Value,
+)
 from mypyc.irbuild.ll_builder import LowLevelIRBuilder
 from mypyc.options import CompilerOptions
 from mypyc.transform.ir_transform import IRTransform
@@ -14,6 +28,7 @@ def do_box_unbox_elimination(fn: FuncIR, options: CompilerOptions) -> None:
     transform.transform_blocks(fn.blocks)
     fn.blocks = builder.blocks
 
+
 def build_use_map(fn: FuncIR) -> Dict[Value, list[Op]]:
     # Map each Value to a list of ops that use it
     use_map: Dict[Value, list[Op]] = {}
@@ -29,6 +44,7 @@ def build_use_map(fn: FuncIR) -> Dict[Value, list[Op]]:
 
 _supported_ops = (Assign, AssignMulti, Call, CallC, ComparisonOp, Return)
 
+
 class BoxUnboxEliminationTransform(IRTransform):
     def __init__(self, builder: LowLevelIRBuilder, use_map: Dict[Value, list[Op]]):
         super().__init__(builder)
@@ -39,7 +55,11 @@ def visit_box(self, op: Box) -> Value | None:
         if len(users) == 0:
             return None
         # Check for Unbox->Box
-        if isinstance(op.src, Unbox) and op.type == op.src.src.type and all(isinstance(user, _supported_ops) for user in users):
+        if (
+            isinstance(op.src, Unbox)
+            and op.type == op.src.src.type
+            and all(isinstance(user, _supported_ops) for user in users)
+        ):
             unbox = op.src
             for user in users:
                 user.set_sources([unbox.src if src is op else src for src in user.sources()])
@@ -51,7 +71,11 @@ def visit_unbox(self, op: Unbox) -> Value | None:
         if len(users) == 0:
             return None
         # Check for Box->Unbox
-        if isinstance(op.src, Box) and op.type == op.src.src.type and all(isinstance(user, _supported_ops) for user in users):
+        if (
+            isinstance(op.src, Box)
+            and op.type == op.src.src.type
+            and all(isinstance(user, _supported_ops) for user in users)
+        ):
             box = op.src
             for user in users:
                 user.set_sources([box.src if src is op else src for src in user.sources()])