Skip to content

Commit 140f6fd

Browse files
authored
Fix llvm::SmallVector error on x86 MSVC (#1932)
extraTypeStrs is declared as `::llvm::SmallVector<std::string>` x86 (32-bit) MSVC builds give an error assigning `SmallVector<std::string, 1>` to extraTypeStrs We can directly set extraTypeStrs without the temporary
1 parent ff95d92 commit 140f6fd

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

mlir/lib/Dialect/Rock/Pipelines/Pipelines.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -204,8 +204,7 @@ void rock::buildKernelPipeline(OpPassManager &pm,
204204
funcPm.addPass(rock::createRockSugarToLoopsPass());
205205
funcPm.addPass(rock::createRockCleanMathPass());
206206
math::MathExtendToSupportedTypesOptions extendToLLVMTypesOptions;
207-
SmallVector<std::string, 1> supportedFloats = {"f16"};
208-
extendToLLVMTypesOptions.extraTypeStrs = supportedFloats;
207+
extendToLLVMTypesOptions.extraTypeStrs = {"f16"};
209208
extendToLLVMTypesOptions.targetTypeStr = "f32";
210209
funcPm.addPass(
211210
math::createMathExtendToSupportedTypes(extendToLLVMTypesOptions));

0 commit comments

Comments
 (0)