Skip to content

Commit 36ca8d1

Browse files
committed
revert problematic functional changes in setOrderedRegisterNumbers
1 parent 17c206a commit 36ca8d1

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed

numexpr/necompiler.py

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
import sys
1616
import threading
1717
from typing import (TYPE_CHECKING, Any, ClassVar, Final, Generator, Iterable,
18-
Iterator, TypeAlias)
18+
Iterator, Sequence, TypeAlias)
1919

2020
if TYPE_CHECKING:
2121
from typing_extensions import Unpack
@@ -494,16 +494,14 @@ def optimizeTemporariesAllocation(ast: ASTNode) -> None:
494494
n.reg = reg
495495

496496

497-
def setOrderedRegisterNumbers(order: Iterable[ASTNode], start: int) -> int:
497+
def setOrderedRegisterNumbers(order: Sequence[ASTNode], start: int) -> int:
498498
"""
499499
Given an order of nodes, assign register numbers.
500500
"""
501-
i = -1
502-
for i, node in enumerate(order, start=start):
503-
if node.reg:
504-
node.reg.n = i
505-
total = i + 1
506-
return total
501+
for i, node in enumerate(order):
502+
assert node.reg is not None
503+
node.reg.n = start + i
504+
return start + len(order)
507505

508506

509507
def setRegisterNumbersForTemporaries(ast: ASTNode, start: int) -> tuple[int, str]:

0 commit comments

Comments
 (0)