[Mlir-commits] [mlir] [mlir][spirv] Fix UpdateVCEPass to deduce the correct set of capabilities (PR #151108)

[Igor Wodiany]

IgWod-IMG wrote:

I have merged the reverted PR, so buildbots should go back to being happy again. I guess we now can fix the issue and re-land this change? It'd be nice to get it to work with a single loop and container (if possible), but I don't mind having a less clever version that we know works. For what's worth I got this change running locally:

```diff
-  for (spirv::Capability cap : caps) {
+  for (unsigned i = 0; i < caps.size(); ++i) {
+    spirv::Capability cap = caps[i];
```

It seems to pass tests and Valgrind doesn't complain. But I'll let you take it from here. Also, I feel like this bug was sort of my fault so sorry for that :)

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