[Mlir-commits] [mlir] [MLIR] Introduce a SelectOpInterface (PR #104751)

[Tobias Gysi]

================
@@ -104,9 +104,11 @@ getBlockPredecessorOperands(BlockArgument blockArg) {
 
 std::optional<SmallVector<Value>>
 mlir::getControlFlowPredecessors(Value value) {
-  SmallVector<Value> result;
   if (OpResult opResult = dyn_cast<OpResult>(value)) {
-    auto regionOp = dyn_cast<RegionBranchOpInterface>(opResult.getOwner());
+    if (auto selectOp = opResult.getDefiningOp<SelectOpInterface>())
----------------
gysit wrote:

Can you update the comment on `getControlFlowPredecessors` that the function also uses the new SelectOpInterface to find the control flow predecessors?

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