From 5d7f22ffa41cd749a5bbb7d2ffd4961e9f1b1e78 Mon Sep 17 00:00:00 2001 From: David Nadlinger Date: Fri, 11 Oct 2019 13:45:03 +0100 Subject: [PATCH] compiler: Remove provision for unused four-parameter llptr_to_var() form [nfc] `var_type` was presumably intended to convert to a target type, but wasn't actually acted on in the function body (nor was it used anywhere in the codebase). --- artiq/compiler/transforms/llvm_ir_generator.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/artiq/compiler/transforms/llvm_ir_generator.py b/artiq/compiler/transforms/llvm_ir_generator.py index 3fc6d719e..0a8e95f79 100644 --- a/artiq/compiler/transforms/llvm_ir_generator.py +++ b/artiq/compiler/transforms/llvm_ir_generator.py @@ -719,9 +719,8 @@ class LLVMIRGenerator: self.llbuilder.store(lloperand, llfieldptr) return llalloc - def llptr_to_var(self, llenv, env_ty, var_name, var_type=None): - if var_name in env_ty.params and (var_type is None or - env_ty.params[var_name] == var_type): + def llptr_to_var(self, llenv, env_ty, var_name): + if var_name in env_ty.params: var_index = list(env_ty.params.keys()).index(var_name) return self.llbuilder.gep(llenv, [self.llindex(0), self.llindex(var_index)], inbounds=True)