From aa0d1b6330a707570d177aba5b36d9a7d5a603a4 Mon Sep 17 00:00:00 2001 From: George Hotz Date: Wed, 24 Jan 2024 20:01:00 -0800 Subject: [PATCH] hotfix: don't use noqa: E702 that's just dumb --- .github/workflows/test.yml | 4 ++-- tinygrad/runtime/ops_llvm.py | 5 ++++- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 9763235884f7..efe009ca217e 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -63,8 +63,8 @@ jobs: source venv/bin/activate pip install $GITHUB_WORKSPACE python -c "from tinygrad.tensor import Tensor; print(Tensor([1,2,3,4,5]))" - - name: Repo line count <5000 lines - run: MAX_LINE_COUNT=5000 python sz.py + - name: Repo line count <6000 lines + run: MAX_LINE_COUNT=6000 python sz.py testcpuimagenet: name: CPU and ImageNet to C Tests diff --git a/tinygrad/runtime/ops_llvm.py b/tinygrad/runtime/ops_llvm.py index df9469d51c69..a5ec322dea35 100644 --- a/tinygrad/runtime/ops_llvm.py +++ b/tinygrad/runtime/ops_llvm.py @@ -26,7 +26,10 @@ def __call__(self, *bufs, vals:Tuple[int, ...]=(), wait=False): class LLVMDevice(Compiled): def __init__(self, device:str): - llvm.initialize(); llvm.initialize_native_target(); llvm.initialize_native_asmprinter(); llvm.initialize_native_asmparser() # noqa: E702 + llvm.initialize() + llvm.initialize_native_target() + llvm.initialize_native_asmprinter() + llvm.initialize_native_asmparser() self.optimizer: llvm.passmanagers.ModulePassManager = llvm.create_module_pass_manager() # this opt actually can change things. ex: opt=3 means no FMA, opt=2 means FMA self.target_machine: llvm.targets.TargetMachine = llvm.Target.from_triple(llvm.get_process_triple()).create_target_machine(opt=2)