Skip to content

Commit 718ec6b

Browse files
authored
Toggle the default option to false for pre-padding convolution flag (#21579)
If a lot of BOO shapes got regression because of pre-padding convolutions, then set the option default to false. And only turn it on for certain unaligned k cases. NO NEED to merge if there's no performance regression. Signed-off-by: yzhang93 <[email protected]>
1 parent 7d595e4 commit 718ec6b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

compiler/src/iree/compiler/Codegen/LLVMGPU/KernelConfig.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ static llvm::cl::opt<bool>
129129
llvm::cl::opt<bool> clGPUPadConvolution(
130130
"iree-codegen-llvmgpu-igemm-pad-convolution",
131131
llvm::cl::desc("enable pre-padding for convolutions in igemm path"),
132-
llvm::cl::init(true));
132+
llvm::cl::init(false));
133133

134134
static llvm::cl::opt<bool>
135135
clUseDirectLoad("iree-llvmgpu-use-direct-load",

0 commit comments

Comments
 (0)