[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