Skip to content

[bazel] Fix bazel build after 00eaff3 #2. #135962

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

Merged
merged 1 commit into from
Apr 16, 2025
Merged

Conversation

bchetioui
Copy link
Member

The linter messed up the order of includes, which is necessary as is.

@llvmbot llvmbot added mlir mlir:bufferization Bufferization infrastructure labels Apr 16, 2025
@llvmbot
Copy link
Member

llvmbot commented Apr 16, 2025

@llvm/pr-subscribers-mlir-bufferization

@llvm/pr-subscribers-mlir

Author: Benjamin Chetioui (bchetioui)

Changes

The linter messed up the order of includes, which is necessary as is.


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

1 Files Affected:

  • (modified) mlir/include/mlir/Dialect/Bufferization/IR/BufferizationTypeInterfaces.h (+1-1)
diff --git a/mlir/include/mlir/Dialect/Bufferization/IR/BufferizationTypeInterfaces.h b/mlir/include/mlir/Dialect/Bufferization/IR/BufferizationTypeInterfaces.h
index 8672aa60a43c5..af2cad006e176 100644
--- a/mlir/include/mlir/Dialect/Bufferization/IR/BufferizationTypeInterfaces.h
+++ b/mlir/include/mlir/Dialect/Bufferization/IR/BufferizationTypeInterfaces.h
@@ -13,7 +13,7 @@
 // Bufferization Type Interfaces
 //===----------------------------------------------------------------------===//
 
+#include "mlir/IR/Types.h"  // NOLINT
 #include "mlir/Dialect/Bufferization/IR/BufferizationTypeInterfaces.h.inc"
-#include "mlir/IR/Types.h"
 
 #endif // MLIR_DIALECT_BUFFERIZATION_IR_BUFFERIZATIONTYPEINTERFACES_H_

Copy link

github-actions bot commented Apr 16, 2025

✅ With the latest revision this PR passed the C/C++ code formatter.

The linter messed up the order of includes, which is necessary as is.
@bchetioui bchetioui merged commit 9483aaa into llvm:main Apr 16, 2025
11 checks passed
var-const pushed a commit to ldionne/llvm-project that referenced this pull request Apr 17, 2025
The linter messed up the order of includes, which is necessary as is.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mlir:bufferization Bufferization infrastructure mlir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants