fix review idea

Signed-off-by: Manjusaka <me@manjusaka.me>

Author: Zheaoli

Lib/test/test_capi/test_opt.py

@@ -2255,7 +2255,7 @@ def testfunc(n):
         self.assertEqual(res, TIER2_THRESHOLD)
         uops = get_opnames(ex)
         self.assertIn("_CALL_LIST_APPEND", uops)
-        self.assertIn("_POP_TOP", uops)
+        self.assertIn("_POP_TOP_NOP", uops)
 
     def test_call_isinstance_is_true(self):
         def testfunc(n):

Python/bytecodes.c

@@ -7,7 +7,6 @@
 // See Tools/cases_generator/README.md for more information.
 
 #include "Python.h"
-#include "opcode_ids.h"
 #include "pycore_abstract.h"      // _PyIndex_Check()
 #include "pycore_audit.h"         // _PySys_Audit()
 #include "pycore_backoff.h"
@@ -4373,8 +4372,8 @@ dummy_func(
             if (err) {
                 ERROR_NO_POP();
             }
-            s = self;
             c = callable;
+            s = self;
             INPUTS_DEAD();
         #if TIER_ONE
             // Skip the following POP_TOP. This is done here in tier one, and

Python/executor_cases.c.h

@@ -1005,6 +1005,12 @@ dummy_func(void) {
         sym_set_const(flag, Py_True);
     }
 
+    op(_CALL_LIST_APPEND, (callable, self, arg -- c, s)) {
+        (void)(callable);
+        c = arg;
+        s = self;
+    }
+
     op(_GUARD_IS_FALSE_POP, (flag -- )) {
         if (sym_is_const(ctx, flag)) {
             PyObject *value = sym_get_const(ctx, flag);