Eliminate 'operand' local variable

gvanrossum · gvanrossum · commit a96ac7fc4bc7 · 2023-11-01T08:04:54.000-07:00
diff --git a/Python/ceval.c b/Python/ceval.c
@@ -983,15 +983,19 @@ _PyEval_EvalFrameDefault(PyThreadState *tstate, _PyInterpreterFrame *frame, int
 
     OPT_STAT_INC(traces_executed);
     _PyUOpInstruction *next_uop = self->trace;
-    uint64_t operand;
+#ifdef Py_DEBUG
+    uint64_t operand;  // Used by several DPRINTF() calls
+#endif
 #ifdef Py_STATS
     uint64_t trace_uop_execution_counter = 0;
 #endif
 
     for (;;) {
         opcode = next_uop->opcode;
         oparg = next_uop->oparg;
+#ifdef Py_DEBUG
         operand = next_uop->operand;
+#endif
         DPRINTF(3,
                 "%4d: uop %s, oparg %d, operand %" PRIu64 ", stack_level %d\n",
                 (int)(next_uop - self->trace),
diff --git a/Python/executor_cases.c.h b/Python/executor_cases.c.h
diff --git a/Tools/cases_generator/instructions.py b/Tools/cases_generator/instructions.py
@@ -162,7 +162,7 @@ def write_body(
                     f"{func}(&this_instr[{active.offset + 1}].cache);"
                 )
             else:
-                out.emit(f"{typ}{ceffect.name} = ({typ.strip()})operand;")
+                out.emit(f"{typ}{ceffect.name} = ({typ.strip()})next_uop[-1].operand;")
 
         # Write the body, substituting a goto for ERROR_IF() and other stuff
         assert dedent <= 0

Original file line number	Diff line number	Diff line change
`@@ -162,7 +162,7 @@ def write_body(`
`162`	`162`	`f"{func}(&this_instr[{active.offset + 1}].cache);"`
`163`	`163`	`)`
`164`	`164`	`else:`
`165`		`- out.emit(f"{typ}{ceffect.name} = ({typ.strip()})operand;")`
	`165`	`+ out.emit(f"{typ}{ceffect.name} = ({typ.strip()})next_uop[-1].operand;")`
`166`	`166`
`167`	`167`	`# Write the body, substituting a goto for ERROR_IF() and other stuff`
`168`	`168`	`assert dedent <= 0`