early return in get_expression

vyperlang · Jan 30, 2025 · 5fb3aa3 · 5fb3aa3
1 parent 7a59827
commit 5fb3aa3
Showing 1 changed file with 3 additions and 6 deletions.
diff --git a/vyper/venom/analysis/available_expression.py b/vyper/venom/analysis/available_expression.py
@@ -208,12 +208,6 @@ def get_same(self, expr: _Expression) -> _Expression | None:
 
         return None
 
-    def exist(self, expr: _Expression) -> bool:
-        if expr.opcode not in self.buckets:
-            return False
-        bucket = self.buckets[expr.opcode]
-        return expr in bucket
-
     def copy(self) -> "_AvailableExpression":
         res = _AvailableExpression()
         for key, val in self.buckets.items():
@@ -351,9 +345,12 @@ def get_expression(
         return self._get_expression(inst, available_exprs)
 
     def _get_expression(self, inst: IRInstruction, available_exprs: _AvailableExpression):
+        if inst.opcode in IMMUTABLE_ENV_QUERIES:
+            return _Expression(inst, inst.opcode, [], self.ignore_msize)
         operands: list[IROperand | _Expression] = self._get_operands(inst, available_exprs)
         expr = _Expression(inst, inst.opcode, operands, self.ignore_msize)
 
+
         same_expr = available_exprs.get_same(expr)
         if same_expr is not None:
             self.inst_to_expr[inst] = same_expr