[Mlir-commits] [mlir] [mlir][spirv] add ExecutionModeIdOp (PR #186241)

[llvmlistbot at llvm.org]

Emimendoza wrote:

> Thanks! Looks good in general, but I left few comments to address.

Thank you for the review!

I have implemented the changes. The only issue I found is that right now we only have 3 valid execution modes for the Op in mlir. The missing 8 belong to two extensions but as you said, it is up to whoever implements those extensions in mlir to add them. 

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