[Mlir-commits] [mlir] [mlir][transform] Overhaul `RegionBranchOpInterface` implementations. (PR #111408)

Ingo Müller llvmlistbot at llvm.org
Thu Oct 10 08:17:16 PDT 2024


================
@@ -104,16 +104,8 @@ transform::AlternativesOp::getEntrySuccessorOperands(RegionBranchPoint point) {
 
 void transform::AlternativesOp::getSuccessorRegions(
     RegionBranchPoint point, SmallVectorImpl<RegionSuccessor> &regions) {
-  for (Region &alternative : llvm::drop_begin(
----------------
ingomueller-net wrote:

> If we a region has an argument, but the `RegionBranchOpInterface` says that no arguments are forwarded to that region: what does that mean?

Are you sure that that can even exist? I think I ran into failed type check that seem to check exactly that while working on the last commit...

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


More information about the Mlir-commits mailing list