[flang-commits] [flang] [flang][cuda][NFC] Make pattern names homogenous (PR #114156)
via flang-commits
flang-commits at lists.llvm.org
Tue Oct 29 17:04:27 PDT 2024
llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT-->
@llvm/pr-subscribers-flang-fir-hlfir
Author: Valentin Clement (バレンタイン クレメン) (clementval)
<details>
<summary>Changes</summary>
Dialect name is uppercase. Make all the patterns prefix homogenous.
---
Full diff: https://github.com/llvm/llvm-project/pull/114156.diff
1 Files Affected:
- (modified) flang/lib/Optimizer/Transforms/CUFOpConversion.cpp (+11-11)
``````````diff
diff --git a/flang/lib/Optimizer/Transforms/CUFOpConversion.cpp b/flang/lib/Optimizer/Transforms/CUFOpConversion.cpp
index fe125db7b4061e..f1f3a95b220df5 100644
--- a/flang/lib/Optimizer/Transforms/CUFOpConversion.cpp
+++ b/flang/lib/Optimizer/Transforms/CUFOpConversion.cpp
@@ -173,7 +173,7 @@ static mlir::LogicalResult convertOpToCall(OpTy op,
return mlir::success();
}
-struct CufAllocateOpConversion
+struct CUFAllocateOpConversion
: public mlir::OpRewritePattern<cuf::AllocateOp> {
using OpRewritePattern::OpRewritePattern;
@@ -216,7 +216,7 @@ struct CufAllocateOpConversion
}
};
-struct CufDeallocateOpConversion
+struct CUFDeallocateOpConversion
: public mlir::OpRewritePattern<cuf::DeallocateOp> {
using OpRewritePattern::OpRewritePattern;
@@ -284,10 +284,10 @@ static int computeWidth(mlir::Location loc, mlir::Type type,
return width;
}
-struct CufAllocOpConversion : public mlir::OpRewritePattern<cuf::AllocOp> {
+struct CUFAllocOpConversion : public mlir::OpRewritePattern<cuf::AllocOp> {
using OpRewritePattern::OpRewritePattern;
- CufAllocOpConversion(mlir::MLIRContext *context, mlir::DataLayout *dl,
+ CUFAllocOpConversion(mlir::MLIRContext *context, mlir::DataLayout *dl,
const fir::LLVMTypeConverter *typeConverter)
: OpRewritePattern(context), dl{dl}, typeConverter{typeConverter} {}
@@ -380,7 +380,7 @@ struct CufAllocOpConversion : public mlir::OpRewritePattern<cuf::AllocOp> {
const fir::LLVMTypeConverter *typeConverter;
};
-struct CufFreeOpConversion : public mlir::OpRewritePattern<cuf::FreeOp> {
+struct CUFFreeOpConversion : public mlir::OpRewritePattern<cuf::FreeOp> {
using OpRewritePattern::OpRewritePattern;
mlir::LogicalResult
@@ -429,11 +429,11 @@ struct CufFreeOpConversion : public mlir::OpRewritePattern<cuf::FreeOp> {
}
};
-struct CufDataTransferOpConversion
+struct CUFDataTransferOpConversion
: public mlir::OpRewritePattern<cuf::DataTransferOp> {
using OpRewritePattern::OpRewritePattern;
- CufDataTransferOpConversion(mlir::MLIRContext *context,
+ CUFDataTransferOpConversion(mlir::MLIRContext *context,
const mlir::SymbolTable &symtab)
: OpRewritePattern(context), symtab{symtab} {}
@@ -718,9 +718,9 @@ class CUFOpConversion : public fir::impl::CUFOpConversionBase<CUFOpConversion> {
void cuf::populateCUFToFIRConversionPatterns(
const fir::LLVMTypeConverter &converter, mlir::DataLayout &dl,
const mlir::SymbolTable &symtab, mlir::RewritePatternSet &patterns) {
- patterns.insert<CufAllocOpConversion>(patterns.getContext(), &dl, &converter);
- patterns.insert<CufAllocateOpConversion, CufDeallocateOpConversion,
- CufFreeOpConversion>(patterns.getContext());
- patterns.insert<CufDataTransferOpConversion, CUFLaunchOpConversion>(
+ patterns.insert<CUFAllocOpConversion>(patterns.getContext(), &dl, &converter);
+ patterns.insert<CUFAllocateOpConversion, CUFDeallocateOpConversion,
+ CUFFreeOpConversion>(patterns.getContext());
+ patterns.insert<CUFDataTransferOpConversion, CUFLaunchOpConversion>(
patterns.getContext(), symtab);
}
``````````
</details>
https://github.com/llvm/llvm-project/pull/114156
More information about the flang-commits
mailing list