[Mlir-commits] [mlir] [mlir][bufferization] OwnershipBasedBufferDeallocation fixes (PR #67418)

[Martin Erhart]

================
@@ -931,8 +935,10 @@ BufferDeallocation::handleInterface(RegionBranchTerminatorOpInterface op) {
 
 bool BufferDeallocation::isFunctionWithoutDynamicOwnership(Operation *op) {
   auto funcOp = dyn_cast<FunctionOpInterface>(op);
-  return funcOp && (!options.privateFuncDynamicOwnership ||
-                    funcOp.getVisibility() != SymbolTable::Visibility::Private);
+  return funcOp &&
+         (!options.privateFuncDynamicOwnership ||
+          funcOp.getVisibility() != SymbolTable::Visibility::Private ||
----------------
maerhart wrote:

Ah, didn't notice there are all these convenience methods for visibility handling. Thanks!

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