[flang-commits] [flang] [flang][cuda][NFC] Remove unused variable (PR #121533)

via flang-commits flang-commits at lists.llvm.org
Thu Jan 2 17:20:15 PST 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-flang-fir-hlfir

Author: Valentin Clement (バレンタイン クレメン) (clementval)

<details>
<summary>Changes</summary>

Failed buildbot after https://github.com/llvm/llvm-project/pull/121524

---
Full diff: https://github.com/llvm/llvm-project/pull/121533.diff


1 Files Affected:

- (modified) flang/lib/Optimizer/Transforms/CUFOpConversion.cpp (+3-10) 


``````````diff
diff --git a/flang/lib/Optimizer/Transforms/CUFOpConversion.cpp b/flang/lib/Optimizer/Transforms/CUFOpConversion.cpp
index f08f9e412b8857..8c525fc6daff5e 100644
--- a/flang/lib/Optimizer/Transforms/CUFOpConversion.cpp
+++ b/flang/lib/Optimizer/Transforms/CUFOpConversion.cpp
@@ -792,10 +792,6 @@ struct CUFSyncDescriptorOpConversion
     : public mlir::OpRewritePattern<cuf::SyncDescriptorOp> {
   using OpRewritePattern::OpRewritePattern;
 
-  CUFSyncDescriptorOpConversion(mlir::MLIRContext *context,
-                                const mlir::SymbolTable &symTab)
-      : OpRewritePattern(context), symTab{symTab} {}
-
   mlir::LogicalResult
   matchAndRewrite(cuf::SyncDescriptorOp op,
                   mlir::PatternRewriter &rewriter) const override {
@@ -822,9 +818,6 @@ struct CUFSyncDescriptorOpConversion
     op.erase();
     return mlir::success();
   }
-
-private:
-  const mlir::SymbolTable &symTab;
 };
 
 class CUFOpConversion : public fir::impl::CUFOpConversionBase<CUFOpConversion> {
@@ -887,11 +880,11 @@ void cuf::populateCUFToFIRConversionPatterns(
     const mlir::SymbolTable &symtab, mlir::RewritePatternSet &patterns) {
   patterns.insert<CUFAllocOpConversion>(patterns.getContext(), &dl, &converter);
   patterns.insert<CUFAllocateOpConversion, CUFDeallocateOpConversion,
-                  CUFFreeOpConversion>(patterns.getContext());
+                  CUFFreeOpConversion, CUFSyncDescriptorOpConversion>(
+      patterns.getContext());
   patterns.insert<CUFDataTransferOpConversion>(patterns.getContext(), symtab,
                                                &dl, &converter);
-  patterns.insert<CUFLaunchOpConversion, CUFSyncDescriptorOpConversion>(
-      patterns.getContext(), symtab);
+  patterns.insert<CUFLaunchOpConversion>(patterns.getContext(), symtab);
 }
 
 void cuf::populateFIRCUFConversionPatterns(const mlir::SymbolTable &symtab,

``````````

</details>


https://github.com/llvm/llvm-project/pull/121533


More information about the flang-commits mailing list