[Mlir-commits] [mlir] [mlir][transforms] Process RegionBranchOp with empty region (PR #123895)

[Longsheng Mou]

CoTinker wrote:

Yes, the main difference lies in using `(Region &region : regionBranchOp->getRegions())` versus `(Block &block : region)`. Additionally, if we use `getSuccessorRegions`, we would need to pass another `RegionBranchPoint` argument, which complicates access to the region. Therefore, I believe simply verifying that the region is not empty should be sufficient. What do you think?

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