fix comments

Author: makslevental

mlir/lib/Bindings/Python/IRCore.cpp

@@ -2786,32 +2786,12 @@ class PyOpAttributeMap {
   PyOperationRef operation;
 };
 
-// copied/borrow from
-// https://github.com/python/pythoncapi-compat/blob/b541b98df1e3e5aabb5def27422a75c876f5a88a/pythoncapi_compat.h#L222
-// bpo-40421 added PyFrame_GetLasti() to Python 3.11.0b1
-#if PY_VERSION_HEX < 0x030b00b1 && !defined(PYPY_VERSION)
-int PyFrame_GetLasti(PyFrameObject *frame) {
-#if PY_VERSION_HEX >= 0x030a00a7
-  // bpo-27129: Since Python 3.10.0a7, f_lasti is an instruction offset,
-  // not a bytes offset anymore. Python uses 16-bit "wordcode" (2 bytes)
-  // instructions.
-  if (frame->f_lasti < 0) {
-    return -1;
-  }
-  return frame->f_lasti * 2;
-#else
-  return frame->f_lasti;
-#endif
-}
-#endif
-
 constexpr size_t kMaxFrames = 512;
 
 MlirLocation tracebackToLocation(MlirContext ctx) {
   size_t framesLimit =
       PyGlobals::get().getTracebackLoc().locTracebackFramesLimit();
-  // We use a thread_local here mostly to avoid requiring a large amount of
-  // space.
+  // Use a thread_local here to avoid requiring a large amount of space.
   thread_local std::array<MlirLocation, kMaxFrames> frames;
   size_t count = 0;
 

mlir/lib/Bindings/Python/IRModule.cpp

@@ -13,7 +13,7 @@
 
 #include "Globals.h"
 #include "NanobindUtils.h"
-#include "mlir-c/Bindings/Python/Interop.h" // This is expected after nanobind.
+#include "mlir-c/Bindings/Python/Interop.h"
 #include "mlir-c/Support.h"
 #include "mlir/Bindings/Python/Nanobind.h"
 

mlir/test/lit.cfg.py

@@ -375,3 +375,6 @@ def have_host_jit_feature_support(feature_name):
 
 if config.arm_emulator_executable:
     config.available_features.add("arm-emulator")
+
+if sys.version_info >= (3, 11):
+    config.available_features.add("python-ge-311")

mlir/test/python/ir/auto_location.py

@@ -1,5 +1,5 @@
 # RUN: %PYTHON %s | FileCheck %s
-
+# REQUIRES: python-ge-311
 import gc
 from contextlib import contextmanager
 

mlir/test/python/lit.local.cfg

@@ -2,7 +2,3 @@ config.environment["ASAN_OPTIONS"] = "detect_leaks=0"
 if not config.enable_bindings_python:
     config.unsupported = True
 config.excludes.add("python_test_ops.td")
-
-import sys
-if sys.version_info >= (3, 11):
-    config.available_features.add("python-ge-311")