Skip to content

Commit

Permalink
[mlir] NFC: Fix layering check / parse headers violations (llvm#110117)
Browse files Browse the repository at this point in the history
Those tools check strict dependency and standalone headers in Google,
but some internal build optimizations caused some violations not to be
detected. This change adds a missing dependency, and includes some types
that are needed for template instantiation.
  • Loading branch information
googlewalt authored Sep 26, 2024
1 parent 139688a commit 1eecc13
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
#include "mlir/Dialect/Linalg/IR/Linalg.h"
#include "mlir/Dialect/Transform/IR/TransformAttrs.h"
#include "mlir/Dialect/Transform/IR/TransformDialect.h"
#include "mlir/Dialect/Transform/IR/TransformTypes.h"
#include "mlir/Dialect/Transform/Interfaces/TransformInterfaces.h"
#include "mlir/Dialect/Utils/StructuredOpsUtils.h"
#include "mlir/IR/OpImplementation.h"
Expand Down Expand Up @@ -42,10 +43,6 @@ class UnPackOp;
} // namespace tensor

namespace transform {
class AnyOpType;
class AnyValueType;
class OperationType;
class TransformHandleTypeInterface;
// Types needed for builders.
struct TileSizesSpec {};
struct NumThreadsSpec {};
Expand Down
1 change: 1 addition & 0 deletions utils/bazel/llvm-project-overlay/mlir/BUILD.bazel
Original file line number Diff line number Diff line change
Expand Up @@ -12371,6 +12371,7 @@ cc_library(
hdrs = glob(["include/mlir/Dialect/Transform/IR/*.h"]),
deps = [
":Analysis",
":BytecodeOpInterface",
":CallOpInterfaces",
":CastInterfaces",
":ControlFlowInterfaces",
Expand Down

0 comments on commit 1eecc13

Please sign in to comment.