Skip to content

[mlir][xegpu] Remove unused custom pass declaration#149278

Merged
adam-smnk merged 1 commit into
llvm:mainfrom
adam-smnk:vector-to-xegpu-unused-declaration
Jul 18, 2025
Merged

[mlir][xegpu] Remove unused custom pass declaration#149278
adam-smnk merged 1 commit into
llvm:mainfrom
adam-smnk:vector-to-xegpu-unused-declaration

Conversation

@adam-smnk

@adam-smnk adam-smnk commented Jul 17, 2025

Copy link
Copy Markdown
Member

Removes unused declaration for pass creation.
Only the create function auto-generated from tablegen should be used.

Removes unused declaration for pass creation.
Only the tablegen auto-generated one should be used.
@llvmbot

llvmbot commented Jul 17, 2025

Copy link
Copy Markdown
Member

@llvm/pr-subscribers-mlir

Author: Adam Siemieniuk (adam-smnk)

Changes

Removes unused declaration for pass creation.
Only the tablegen auto-generated one should be used.


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

1 Files Affected:

  • (modified) mlir/include/mlir/Conversion/VectorToXeGPU/VectorToXeGPU.h (-3)
diff --git a/mlir/include/mlir/Conversion/VectorToXeGPU/VectorToXeGPU.h b/mlir/include/mlir/Conversion/VectorToXeGPU/VectorToXeGPU.h
index ac4915901fdec..ff99d7ce96daf 100644
--- a/mlir/include/mlir/Conversion/VectorToXeGPU/VectorToXeGPU.h
+++ b/mlir/include/mlir/Conversion/VectorToXeGPU/VectorToXeGPU.h
@@ -21,9 +21,6 @@ class RewritePatternSet;
 /// Collect a set of patterns to convert from the vector to XeGPU ops.
 void populateVectorToXeGPUConversionPatterns(RewritePatternSet &patterns);
 
-/// Create a pass to convert ops from vector to XeGPU.
-std::unique_ptr<Pass> createConvertVectorToXeGPUPass();
-
 } // namespace mlir
 
 #endif // MLIR_CONVERSION_VECTORTOXEGPU_VECTORTOXEGPU_H

@llvmbot

llvmbot commented Jul 17, 2025

Copy link
Copy Markdown
Member

@llvm/pr-subscribers-mlir-gpu

Author: Adam Siemieniuk (adam-smnk)

Changes

Removes unused declaration for pass creation.
Only the tablegen auto-generated one should be used.


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

1 Files Affected:

  • (modified) mlir/include/mlir/Conversion/VectorToXeGPU/VectorToXeGPU.h (-3)
diff --git a/mlir/include/mlir/Conversion/VectorToXeGPU/VectorToXeGPU.h b/mlir/include/mlir/Conversion/VectorToXeGPU/VectorToXeGPU.h
index ac4915901fdec..ff99d7ce96daf 100644
--- a/mlir/include/mlir/Conversion/VectorToXeGPU/VectorToXeGPU.h
+++ b/mlir/include/mlir/Conversion/VectorToXeGPU/VectorToXeGPU.h
@@ -21,9 +21,6 @@ class RewritePatternSet;
 /// Collect a set of patterns to convert from the vector to XeGPU ops.
 void populateVectorToXeGPUConversionPatterns(RewritePatternSet &patterns);
 
-/// Create a pass to convert ops from vector to XeGPU.
-std::unique_ptr<Pass> createConvertVectorToXeGPUPass();
-
 } // namespace mlir
 
 #endif // MLIR_CONVERSION_VECTORTOXEGPU_VECTORTOXEGPU_H

@adam-smnk adam-smnk requested a review from chencha3 July 17, 2025 10:02

@chencha3 chencha3 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@adam-smnk adam-smnk merged commit e73cb43 into llvm:main Jul 18, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants