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

[Jakub Kuderski]

================
@@ -95,6 +95,21 @@ static LogicalResult checkAndUpdateCapabilityRequirements(
   return success();
 }
 
+static SetVector<spirv::Capability>
+addAllImpliedCapabilities(SetVector<spirv::Capability> &caps) {
----------------
kuhar wrote:

What is the difference between the return value and `caps`? Can we stick with just a return value and make `caps` a `const` reference?

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