Simplify _GUARD_CALLABLE_TYPE_1

tomasr8 · brandtbucher · web-flow · commit 9e1bb988ecbf · 2025-04-11T22:44:38.000+02:00
Co-authored-by: Brandt Bucher &lt;brandtbucher@gmail.com&gt;
diff --git a/Python/optimizer_bytecodes.c b/Python/optimizer_bytecodes.c
@@ -992,12 +992,8 @@ dummy_func(void) {
     }
 
     op(_GUARD_CALLABLE_TYPE_1, (callable, unused, unused2 -- callable, unused, unused2)) {
-        if (sym_is_const(ctx, callable)) {
-            PyObject *value = sym_get_const(ctx, callable);
-            assert(value != NULL);
-            if (value == (PyObject *)&PyType_Type) {
-                REPLACE_OP(this_instr, _NOP, 0, 0);
-            }
+        if (sym_get_const(ctx, callable) == (PyObject *)&PyType_Type) {
+            REPLACE_OP(this_instr, _NOP, 0, 0);
         }
         sym_set_const(callable, (PyObject *)&PyType_Type);
     }

Original file line number	Diff line number	Diff line change
`@@ -992,12 +992,8 @@ dummy_func(void) {`
`992`	`992`	`}`
`993`	`993`
`994`	`994`	`op(_GUARD_CALLABLE_TYPE_1, (callable, unused, unused2 -- callable, unused, unused2)) {`
`995`		`- if (sym_is_const(ctx, callable)) {`
`996`		`- PyObject *value = sym_get_const(ctx, callable);`
`997`		`- assert(value != NULL);`
`998`		`- if (value == (PyObject *)&PyType_Type) {`
`999`		`- REPLACE_OP(this_instr, _NOP, 0, 0);`
`1000`		`- }`
	`995`	`+ if (sym_get_const(ctx, callable) == (PyObject *)&PyType_Type) {`
	`996`	`+ REPLACE_OP(this_instr, _NOP, 0, 0);`
`1001`	`997`	`}`
`1002`	`998`	`sym_set_const(callable, (PyObject *)&PyType_Type);`
`1003`	`999`	`}`