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

Jakub Kuderski llvmlistbot at llvm.org
Tue Jul 29 04:49:32 PDT 2025


================
@@ -95,6 +95,16 @@ static LogicalResult checkAndUpdateCapabilityRequirements(
   return success();
 }
 
+static SetVector<spirv::Capability>
+withImpliedCapabilities(SetVector<spirv::Capability> &caps) {
+  SetVector<spirv::Capability> allCaps(caps.begin(), caps.end());
+  for (auto cap : caps) {
----------------
kuhar wrote:

```suggestion
  for (spirv::Capability cap : caps) {
```
See https://llvm.org/docs/CodingStandards.html#use-auto-type-deduction-to-make-code-more-readable

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


More information about the Mlir-commits mailing list