Skip to content

Commit 9318570

Browse files
authored
Revert "[clang-tidy] Add MLIR check for old op builder usage. (llvm#149148)" (llvm#3299)
2 parents 1d53195 + 5845e1a commit 9318570

File tree

8 files changed

+0
-267
lines changed

8 files changed

+0
-267
lines changed

clang-tools-extra/clang-tidy/llvm/CMakeLists.txt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,11 @@ add_clang_library(clangTidyLLVMModule STATIC
1111
PreferRegisterOverUnsignedCheck.cpp
1212
PreferStaticOverAnonymousNamespaceCheck.cpp
1313
TwineLocalCheck.cpp
14-
UseNewMLIROpBuilderCheck.cpp
1514

1615
LINK_LIBS
1716
clangTidy
1817
clangTidyReadabilityModule
1918
clangTidyUtils
20-
clangTransformer
2119

2220
DEPENDS
2321
omp_gen

clang-tools-extra/clang-tidy/llvm/LLVMTidyModule.cpp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@
1818
#include "PreferRegisterOverUnsignedCheck.h"
1919
#include "PreferStaticOverAnonymousNamespaceCheck.h"
2020
#include "TwineLocalCheck.h"
21-
#include "UseNewMLIROpBuilderCheck.h"
2221

2322
namespace clang::tidy {
2423
namespace llvm_check {
@@ -41,8 +40,6 @@ class LLVMModule : public ClangTidyModule {
4140
CheckFactories.registerCheck<readability::QualifiedAutoCheck>(
4241
"llvm-qualified-auto");
4342
CheckFactories.registerCheck<TwineLocalCheck>("llvm-twine-local");
44-
CheckFactories.registerCheck<UseNewMlirOpBuilderCheck>(
45-
"llvm-use-new-mlir-op-builder");
4643
}
4744

4845
ClangTidyOptions getModuleOptions() override {

clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.cpp

Lines changed: 0 additions & 133 deletions
This file was deleted.

clang-tools-extra/clang-tidy/llvm/UseNewMLIROpBuilderCheck.h

Lines changed: 0 additions & 29 deletions
This file was deleted.

clang-tools-extra/docs/ReleaseNotes.rst

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -100,12 +100,6 @@ Improvements to clang-tidy
100100
New checks
101101
^^^^^^^^^^
102102

103-
- New :doc:`llvm-mlir-op-builder
104-
<clang-tidy/checks/llvm/use-new-mlir-op-builder>` check.
105-
106-
Checks for uses of MLIR's old/to be deprecated ``OpBuilder::create<T>`` form
107-
and suggests using ``T::create`` instead.
108-
109103
New check aliases
110104
^^^^^^^^^^^^^^^^^
111105

clang-tools-extra/docs/clang-tidy/checks/list.rst

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -247,7 +247,6 @@ Clang-Tidy Checks
247247
:doc:`linuxkernel-must-check-errs <linuxkernel/must-check-errs>`,
248248
:doc:`llvm-header-guard <llvm/header-guard>`,
249249
:doc:`llvm-include-order <llvm/include-order>`, "Yes"
250-
:doc:`llvm-use-new-mlir-op-builder <llvm/use-new-mlir-op-builder>`, "Yes"
251250
:doc:`llvm-namespace-comment <llvm/namespace-comment>`,
252251
:doc:`llvm-prefer-isa-or-dyn-cast-in-conditionals <llvm/prefer-isa-or-dyn-cast-in-conditionals>`, "Yes"
253252
:doc:`llvm-prefer-register-over-unsigned <llvm/prefer-register-over-unsigned>`, "Yes"

clang-tools-extra/docs/clang-tidy/checks/llvm/use-new-mlir-op-builder.rst

Lines changed: 0 additions & 21 deletions
This file was deleted.

clang-tools-extra/test/clang-tidy/checkers/llvm/use-new-mlir-op-builder.cpp

Lines changed: 0 additions & 72 deletions
This file was deleted.

0 commit comments

Comments
 (0)