[Mlir-commits] [mlir] [mlir][spirv] Add support for SwitchOp (PR #168713)

[Igor Wodiany]

IgWod-IMG wrote:

> I wonder if we should also add spirv.mlir.switch with an isolated-from-above region instead of branching to labels directly.

Isn't that what `spirv.mlir.selection` is already for (see the target test)? Switch is (sort of) a generalization of a conditional branch and it also uses `OpSelectionMerge`. In fact, the doc already account for `spirv.Switch`: https://mlir.llvm.org/docs/Dialects/SPIR-V/#selection it just wasn't implemented. Unless you have something else in mind. I'm opened to suggestions :)

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