[Mlir-commits] [mlir] [mlir][python] automatic location inference (PR #151246)

[Jacques Pienaar]

================
@@ -197,3 +197,69 @@ PyGlobals::lookupOperationClass(llvm::StringRef operationName) {
   // Not found and loading did not yield a registration.
   return std::nullopt;
 }
+
+bool PyGlobals::TracebackLoc::locTracebacksEnabled() const {
+  return locTracebackEnabled_;
----------------
jpienaar wrote:

If you are locking for writing, don't you also need to lock for reading? (not that I could see these being set in multiple threads in practice vs at some init time, don't know if its bad to just have these as atomics and getting back whichever one for a given thread is "fine")

https://github.com/llvm/llvm-project/pull/151246