[Mlir-commits] [mlir] [MLIR] Remove unnecessary include from MathToEmitC.h to fix build issue (PR #125466)

llvmlistbot at llvm.org llvmlistbot at llvm.org
Sun Feb 2 23:45:35 PST 2025


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-mlir

Author: Tomer Solomon (recursion-man)

<details>
<summary>Changes</summary>

This removes the unnecessary inclusion of mlir/Dialect/EmitC/IR/EmitC.h from MathToEmitC.h, which caused a build failure due to a missing EmitCEnums.h.inc. The include was not needed, and removing it resolves the issue without requiring additional dependencies.

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


1 Files Affected:

- (modified) mlir/include/mlir/Conversion/MathToEmitC/MathToEmitC.h (-1) 


``````````diff
diff --git a/mlir/include/mlir/Conversion/MathToEmitC/MathToEmitC.h b/mlir/include/mlir/Conversion/MathToEmitC/MathToEmitC.h
index 0fc33bf790be4e..c61773026ca53d 100644
--- a/mlir/include/mlir/Conversion/MathToEmitC/MathToEmitC.h
+++ b/mlir/include/mlir/Conversion/MathToEmitC/MathToEmitC.h
@@ -8,7 +8,6 @@
 
 #ifndef MLIR_CONVERSION_MATHTOEMITC_MATHTOEMITC_H
 #define MLIR_CONVERSION_MATHTOEMITC_MATHTOEMITC_H
-#include "mlir/Dialect/EmitC/IR/EmitC.h"
 namespace mlir {
 class RewritePatternSet;
 namespace emitc {

``````````

</details>


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


More information about the Mlir-commits mailing list