Return nullptr on Region::getContainingOperation()/getContainingFunction() instea...

    This avoids crashing when trying to dump an operation nested in a region that isn't yet attached to an operation, which is quite useful when debugging.

    This alone won't be enough to print an unlink Operation, it'll display `<<UNLINKED INSTRUCTION>>`.

--

PiperOrigin-RevId: 249496388

diff --git a/mlir/lib/IR/Block.cpp b/mlir/lib/IR/Block.cpp
index a440ca9..2ed8fd0 100644 (file)
--- a/mlir/lib/IR/Block.cpp
+++ b/mlir/lib/IR/Block.cpp
@@ -289,12 +289,10 @@ Region *Region::getContainingRegion() {
 }
 
 Operation *Region::getContainingOp() {
-  assert(!container.isNull() && "no container");
   return container.dyn_cast<Operation *>();
 }
 
 Function *Region::getContainingFunction() {
-  assert(!container.isNull() && "no container");
   return container.dyn_cast<Function *>();
 }