Skip to content

Commit db53cdf

Browse files
committed
Addressing review feedbacks
1 parent 4610b01 commit db53cdf

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

mlir/include/mlir/Conversion/Passes.td

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1340,7 +1340,7 @@ def ConvertVectorToArmSMEPass : Pass<"convert-vector-to-arm-sme"> {
13401340
def ConvertVectorToAMDGPUPass : Pass<"convert-vector-to-amdgpu"> {
13411341
let summary = "Lower the operations from the vector dialect into the AMDGPU "
13421342
"dialect";
1343-
let dependentDialects = ["amdgpu::AMDGPUDialect", "vector::VectorDialect"];
1343+
let dependentDialects = [ "vector::VectorDialect"];
13441344
}
13451345

13461346
//===----------------------------------------------------------------------===//

mlir/lib/Conversion/VectorToAMDGPU/VectorToAMDGPU.cpp

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,8 @@ static LogicalResult transferPreconditions(
9797
return success();
9898
}
9999

100+
namespace {
101+
100102
struct TransferReadLowering final : OpRewritePattern<vector::TransferReadOp> {
101103
using OpRewritePattern::OpRewritePattern;
102104

@@ -130,13 +132,15 @@ struct TransferReadLowering final : OpRewritePattern<vector::TransferReadOp> {
130132
}
131133
};
132134

135+
} // namespace
136+
133137
void mlir::populateVectorToAMDGPUConversionPatterns(
134138
RewritePatternSet &patterns) {
135139
patterns.add<TransferReadLowering>(patterns.getContext());
136140
}
137141

138142
struct ConvertVectorToAMDGPUPass final
139-
: public impl::ConvertVectorToAMDGPUPassBase<ConvertVectorToAMDGPUPass> {
143+
: impl::ConvertVectorToAMDGPUPassBase<ConvertVectorToAMDGPUPass> {
140144
void runOnOperation() override {
141145
RewritePatternSet patterns(&getContext());
142146
populateVectorToAMDGPUConversionPatterns(patterns);

0 commit comments

Comments
 (0)