[Mlir-commits] [mlir] [mlir][spirv] Extend capabilities and extensions requirements checking (PR #68031)

Jakub Kuderski llvmlistbot at llvm.org
Mon Oct 9 18:16:54 PDT 2023


================
@@ -43,9 +43,13 @@ using namespace mlir;
 template <typename LabelT>
 static LogicalResult checkExtensionRequirements(
     LabelT label, const spirv::TargetEnv &targetEnv,
-    const spirv::SPIRVType::ExtensionArrayRefVector &candidates) {
+    const spirv::SPIRVType::ExtensionArrayRefVector &candidates,
----------------
kuhar wrote:

Instead of adding this new parameter, could we pass in only a subset of candidates to this function? IE prune it on the caller side.

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


More information about the Mlir-commits mailing list