diff options
author | Sam Gross <colesbury@gmail.com> | 2025-01-23 09:26:25 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2025-01-23 09:26:25 (GMT) |
commit | a10f99375e7912df863cf101a38e9703cfcd72f1 (patch) | |
tree | 7909c7896fe256427c1149d2416fb7912ad3ba48 /Tools/cases_generator/generators_common.py | |
parent | d7d066c3ab6842117f9e0fb1c9dde4bce00fa1e3 (diff) | |
download | cpython-a10f99375e7912df863cf101a38e9703cfcd72f1.zip cpython-a10f99375e7912df863cf101a38e9703cfcd72f1.tar.gz cpython-a10f99375e7912df863cf101a38e9703cfcd72f1.tar.bz2 |
Revert "GH-128914: Remove conditional stack effects from `bytecodes.c` and the code generators (GH-128918)" (GH-129202)
The commit introduced a ~2.5-3% regression in the free threading build.
This reverts commit ab61d3f4303d14a413bc9ae6557c730ffdf7579e.
Diffstat (limited to 'Tools/cases_generator/generators_common.py')
-rw-r--r-- | Tools/cases_generator/generators_common.py | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/Tools/cases_generator/generators_common.py b/Tools/cases_generator/generators_common.py index 9edf3d4..f1f166a 100644 --- a/Tools/cases_generator/generators_common.py +++ b/Tools/cases_generator/generators_common.py @@ -247,7 +247,7 @@ class Emitter: if var.name == "null": continue close = "PyStackRef_CLOSE" - if "null" in var.name: + if "null" in var.name or var.condition and var.condition != "1": close = "PyStackRef_XCLOSE" if var.size: if var.size == "1": @@ -256,6 +256,9 @@ class Emitter: self.out.emit(f"for (int _i = {var.size}; --_i >= 0;) {{\n") self.out.emit(f"{close}({var.name}[_i]);\n") self.out.emit("}\n") + elif var.condition: + if var.condition != "0": + self.out.emit(f"{close}({var.name});\n") else: self.out.emit(f"{close}({var.name});\n") for input in storage.inputs: @@ -683,6 +686,8 @@ def cflags(p: Properties) -> str: flags.append("HAS_PURE_FLAG") if p.no_save_ip: flags.append("HAS_NO_SAVE_IP_FLAG") + if p.oparg_and_1: + flags.append("HAS_OPARG_AND_1_FLAG") if flags: return " | ".join(flags) else: |