[Mlir-commits] [mlir] [mlir] Use non thread-local allocator for DistinctAttr when threading is disabled (PR #128566)

[Tobias Gysi]

================
@@ -596,6 +596,7 @@ void MLIRContext::disableMultithreading(bool disable) {
   // Update the threading mode for each of the uniquers.
   impl->affineUniquer.disableMultithreading(disable);
   impl->attributeUniquer.disableMultithreading(disable);
+  impl->distinctAttributeAllocator.disableThreadLocalStorage(disable);
----------------
gysit wrote:

Shouldn't this be disable multi-threading for consistency? I believe the result is probably the same but it seems more intuitive to use the other function here.

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