diff --git a/mlir/lib/Dialect/SPIRV/Transforms/SPIRVConversion.cpp b/mlir/lib/Dialect/SPIRV/Transforms/SPIRVConversion.cpp --- a/mlir/lib/Dialect/SPIRV/Transforms/SPIRVConversion.cpp +++ b/mlir/lib/Dialect/SPIRV/Transforms/SPIRVConversion.cpp @@ -42,14 +42,15 @@ if (targetEnv.allows(ors)) continue; - SmallVector extStrings; - for (spirv::Extension ext : ors) - extStrings.push_back(spirv::stringifyExtension(ext)); - - LLVM_DEBUG(llvm::dbgs() - << label << " illegal: requires at least one extension in [" - << llvm::join(extStrings, ", ") - << "] but none allowed in target environment\n"); + LLVM_DEBUG({ + SmallVector extStrings; + for (spirv::Extension ext : ors) + extStrings.push_back(spirv::stringifyExtension(ext)); + + llvm::dbgs() << label << " illegal: requires at least one extension in [" + << llvm::join(extStrings, ", ") + << "] but none allowed in target environment\n"; + }); return failure(); } return success(); @@ -69,14 +70,15 @@ if (targetEnv.allows(ors)) continue; - SmallVector capStrings; - for (spirv::Capability cap : ors) - capStrings.push_back(spirv::stringifyCapability(cap)); + LLVM_DEBUG({ + SmallVector capStrings; + for (spirv::Capability cap : ors) + capStrings.push_back(spirv::stringifyCapability(cap)); - LLVM_DEBUG(llvm::dbgs() - << label << " illegal: requires at least one capability in [" - << llvm::join(capStrings, ", ") - << "] but none allowed in target environment\n"); + llvm::dbgs() << label << " illegal: requires at least one capability in [" + << llvm::join(capStrings, ", ") + << "] but none allowed in target environment\n"; + }); return failure(); } return success();