[Mlir-commits] [mlir] [mlir][bufferization] Cache areNonConflictingSubsets results in OneShotAnalysis (PR #189895)

[Matthias Springer]

================
@@ -232,6 +237,9 @@ class OneShotAnalysisState : public AnalysisState {
   /// Cache definitions of tensor values.
   DenseMap<Value, SetVector<Value>> cachedDefinitions;
 
+  /// Cache results of areNonConflictingSubsets checks.
----------------
matthias-springer wrote:

nit: Explain the meaning of the `bool` parameter. We are caching both conflicting and non-conflicting subsets. The absence of an entry means that we don't have it cached.

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