matthias-springer wrote: After clarifying the semantics of `RegionBranchOpInterface` #174978, I believe it is safe to implement the interface on `scf.forall`, as along as `scf.forall.in_parallel` is also implemented on the terminator. Details in the commit message.
As @joker-eph mentioned [above](https://github.com/llvm/llvm-project/pull/174221#issuecomment-3717776189), the main issue seems to be with the special merge logic of the op. This commit drops that part from the interface implementation. https://github.com/llvm/llvm-project/pull/174221 _______________________________________________ llvm-branch-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
