-
Notifications
You must be signed in to change notification settings - Fork 11.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[mlir][NFC] Add rewrite header to fix standalone header compile #99370
Conversation
This uses `MlirRewriterBase` from from `mlir-c/Rewrite.h` without including it.
@llvm/pr-subscribers-mlir Author: Jordan Rupprecht (rupprecht) ChangesThis uses Full diff: https://github.com/llvm/llvm-project/pull/99370.diff 1 Files Affected:
diff --git a/mlir/include/mlir/CAPI/Rewrite.h b/mlir/include/mlir/CAPI/Rewrite.h
index f0bb9337e49ea..1038c0a575cf2 100644
--- a/mlir/include/mlir/CAPI/Rewrite.h
+++ b/mlir/include/mlir/CAPI/Rewrite.h
@@ -15,6 +15,7 @@
#ifndef MLIR_CAPI_REWRITE_H
#define MLIR_CAPI_REWRITE_H
+#include "mlir-c/Rewrite.h"
#include "mlir/CAPI/Wrap.h"
#include "mlir/IR/PatternMatch.h"
|
Landed now to address a build breakage, happy to followup w/ post-commit review |
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/85/builds/705 Here is the relevant piece of the build log for the reference:
|
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/94/builds/553 Here is the relevant piece of the build log for the reference:
|
…#99370) This uses `MlirRewriterBase` from from `mlir-c/Rewrite.h` without including it.
This uses `MlirRewriterBase` from from `mlir-c/Rewrite.h` without including it.
This uses
MlirRewriterBase
from frommlir-c/Rewrite.h
without including it.