Skip to content

Commit 076397f

Browse files
authored
Revert "[Clang] [NFC] Introduce a helper for emitting compatibility diagnostics" (#134036)
Reverts #132348 Some tests are failing and I still need to figure out what is going on here.
1 parent a2ca2f3 commit 076397f

31 files changed

+93
-356
lines changed

clang/include/clang/Basic/CMakeLists.txt

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,6 @@ macro(clang_diag_gen component)
88
-gen-clang-diags-enums -clang-component=${component}
99
SOURCE Diagnostic.td
1010
TARGET ClangDiagnostic${component}Enums)
11-
12-
clang_tablegen(Diagnostic${component}CompatIDs.inc
13-
-gen-clang-diags-compat-ids -clang-component=${component}
14-
SOURCE Diagnostic.td
15-
TARGET ClangDiagnostic${component}CompatIDs)
1611
endmacro(clang_diag_gen)
1712

1813
clang_diag_gen(Analysis)
@@ -36,11 +31,6 @@ clang_tablegen(DiagnosticIndexName.inc -gen-clang-diags-index-name
3631
SOURCE Diagnostic.td
3732
TARGET ClangDiagnosticIndexName)
3833

39-
clang_tablegen(DiagnosticAllCompatIDs.inc
40-
-gen-clang-diags-compat-ids
41-
SOURCE Diagnostic.td
42-
TARGET ClangDiagnosticAllCompatIDs)
43-
4434
clang_tablegen(AttrList.inc -gen-clang-attr-list
4535
-I ${CMAKE_CURRENT_SOURCE_DIR}/../../
4636
SOURCE Attr.td

clang/include/clang/Basic/Diagnostic.td

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -155,19 +155,6 @@ class DefaultWarnNoWerror {
155155
}
156156
class DefaultRemark { Severity DefaultSeverity = SEV_Remark; }
157157

158-
class CompatWarningId<string name, int std, string diag, string diag_pre> {
159-
string Component = ?;
160-
string Name = name;
161-
string Diag = diag;
162-
string DiagPre = diag_pre;
163-
int Std = std;
164-
165-
// This is unused, but Tablegen will complain if it's missing because we define
166-
// the compatibility ids in the same place as the other diagnostics (which means
167-
// that we'll be inside a 'let CategoryName = "" in { ... }' block).
168-
string CategoryName = ?;
169-
}
170-
171158
// C++ compatibility warnings.
172159
multiclass CXXCompat<
173160
string message,
@@ -191,11 +178,6 @@ multiclass CXXCompat<
191178
"CXX98Compat",
192179
"CXXPre"#std_ver#"Compat"))>,
193180
DefaultIgnore;
194-
195-
def : CompatWarningId<
196-
NAME, std_ver,
197-
"compat_cxx"#std_ver#"_"#NAME,
198-
"compat_pre_cxx"#std_ver#"_"#NAME>;
199181
}
200182

201183
// These generate pairs of C++ compatibility warnings of the form:

clang/include/clang/Basic/DiagnosticAST.h

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -36,18 +36,6 @@ enum {
3636
#undef DIAG_ENUM_ITEM
3737
#undef DIAG_ENUM
3838
} // end namespace diag
39-
40-
namespace diag_compat {
41-
#define DIAG_COMPAT_IDS_BEGIN() enum {
42-
#define DIAG_COMPAT_IDS_END() \
43-
} \
44-
;
45-
#define DIAG_COMPAT_ID(IDX, NAME, ...) NAME = IDX,
46-
#include "clang/Basic/DiagnosticASTCompatIDs.inc"
47-
#undef DIAG_COMPAT_ID
48-
#undef DIAG_COMPAT_IDS_BEGIN
49-
#undef DIAG_COMPAT_IDS_END
50-
} // end namespace diag_compat
5139
} // end namespace clang
5240

5341
#endif // LLVM_CLANG_BASIC_DIAGNOSTICAST_H

clang/include/clang/Basic/DiagnosticAnalysis.h

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -35,18 +35,6 @@ enum {
3535
#undef DIAG_ENUM_ITEM
3636
#undef DIAG_ENUM
3737
} // end namespace diag
38-
39-
namespace diag_compat {
40-
#define DIAG_COMPAT_IDS_BEGIN() enum {
41-
#define DIAG_COMPAT_IDS_END() \
42-
} \
43-
;
44-
#define DIAG_COMPAT_ID(IDX, NAME, ...) NAME = IDX,
45-
#include "clang/Basic/DiagnosticAnalysisCompatIDs.inc"
46-
#undef DIAG_COMPAT_ID
47-
#undef DIAG_COMPAT_IDS_BEGIN
48-
#undef DIAG_COMPAT_IDS_END
49-
} // end namespace diag_compat
5038
} // end namespace clang
5139

5240
#endif // LLVM_CLANG_BASIC_DIAGNOSTICANALYSIS_H

clang/include/clang/Basic/DiagnosticComment.h

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -36,18 +36,6 @@ enum {
3636
#undef DIAG_ENUM_ITEM
3737
#undef DIAG_ENUM
3838
} // end namespace diag
39-
40-
namespace diag_compat {
41-
#define DIAG_COMPAT_IDS_BEGIN() enum {
42-
#define DIAG_COMPAT_IDS_END() \
43-
} \
44-
;
45-
#define DIAG_COMPAT_ID(IDX, NAME, ...) NAME = IDX,
46-
#include "clang/Basic/DiagnosticCommentCompatIDs.inc"
47-
#undef DIAG_COMPAT_ID
48-
#undef DIAG_COMPAT_IDS_BEGIN
49-
#undef DIAG_COMPAT_IDS_END
50-
} // end namespace diag_compat
5139
} // end namespace clang
5240

5341
#endif // LLVM_CLANG_BASIC_DIAGNOSTICCOMMENT_H

clang/include/clang/Basic/DiagnosticCrossTU.h

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -36,18 +36,6 @@ enum {
3636
#undef DIAG_ENUM_ITEM
3737
#undef DIAG_ENUM
3838
} // end namespace diag
39-
40-
namespace diag_compat {
41-
#define DIAG_COMPAT_IDS_BEGIN() enum {
42-
#define DIAG_COMPAT_IDS_END() \
43-
} \
44-
;
45-
#define DIAG_COMPAT_ID(IDX, NAME, ...) NAME = IDX,
46-
#include "clang/Basic/DiagnosticCrossTUCompatIDs.inc"
47-
#undef DIAG_COMPAT_ID
48-
#undef DIAG_COMPAT_IDS_BEGIN
49-
#undef DIAG_COMPAT_IDS_END
50-
} // end namespace diag_compat
5139
} // end namespace clang
5240

5341
#endif // LLVM_CLANG_BASIC_DIAGNOSTICCROSSTU_H

clang/include/clang/Basic/DiagnosticDriver.h

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -36,18 +36,6 @@ enum {
3636
#undef DIAG_ENUM_ITEM
3737
#undef DIAG_ENUM
3838
} // end namespace diag
39-
40-
namespace diag_compat {
41-
#define DIAG_COMPAT_IDS_BEGIN() enum {
42-
#define DIAG_COMPAT_IDS_END() \
43-
} \
44-
;
45-
#define DIAG_COMPAT_ID(IDX, NAME, ...) NAME = IDX,
46-
#include "clang/Basic/DiagnosticDriverCompatIDs.inc"
47-
#undef DIAG_COMPAT_ID
48-
#undef DIAG_COMPAT_IDS_BEGIN
49-
#undef DIAG_COMPAT_IDS_END
50-
} // end namespace diag_compat
5139
} // end namespace clang
5240

5341
#endif // LLVM_CLANG_BASIC_DIAGNOSTICDRIVER_H

clang/include/clang/Basic/DiagnosticFrontend.h

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -36,18 +36,6 @@ enum {
3636
#undef DIAG_ENUM_ITEM
3737
#undef DIAG_ENUM
3838
} // end namespace diag
39-
40-
namespace diag_compat {
41-
#define DIAG_COMPAT_IDS_BEGIN() enum {
42-
#define DIAG_COMPAT_IDS_END() \
43-
} \
44-
;
45-
#define DIAG_COMPAT_ID(IDX, NAME, ...) NAME = IDX,
46-
#include "clang/Basic/DiagnosticFrontendCompatIDs.inc"
47-
#undef DIAG_COMPAT_ID
48-
#undef DIAG_COMPAT_IDS_BEGIN
49-
#undef DIAG_COMPAT_IDS_END
50-
} // end namespace diag_compat
5139
} // end namespace clang
5240

5341
#endif // LLVM_CLANG_BASIC_DIAGNOSTICFRONTEND_H

clang/include/clang/Basic/DiagnosticIDs.h

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@
2525
namespace clang {
2626
class DiagnosticsEngine;
2727
class DiagnosticBuilder;
28-
class LangOptions;
2928
class SourceLocation;
3029

3130
// Import the diagnostic enums themselves.
@@ -105,18 +104,6 @@ namespace clang {
105104
};
106105
}
107106

108-
namespace diag_compat {
109-
#define DIAG_COMPAT_IDS_BEGIN() enum {
110-
#define DIAG_COMPAT_IDS_END() \
111-
} \
112-
;
113-
#define DIAG_COMPAT_ID(IDX, NAME, ...) NAME = IDX,
114-
#include "clang/Basic/DiagnosticCommonCompatIDs.inc"
115-
#undef DIAG_COMPAT_ID
116-
#undef DIAG_COMPAT_IDS_BEGIN
117-
#undef DIAG_COMPAT_IDS_END
118-
} // end namespace diag_compat
119-
120107
class DiagnosticMapping {
121108
LLVM_PREFERRED_TYPE(diag::Severity)
122109
unsigned Severity : 3;
@@ -477,11 +464,6 @@ class DiagnosticIDs : public RefCountedBase<DiagnosticIDs> {
477464
/// given group name.
478465
static StringRef getNearestOption(diag::Flavor Flavor, StringRef Group);
479466

480-
/// Get the appropriate diagnostic Id to use for issuing a compatibility
481-
/// diagnostic. For use by the various DiagCompat() helpers.
482-
static unsigned getCXXCompatDiagId(const LangOptions &LangOpts,
483-
unsigned CompatDiagId);
484-
485467
private:
486468
/// Classify the specified diagnostic ID into a Level, consumable by
487469
/// the DiagnosticClient.

clang/include/clang/Basic/DiagnosticInstallAPI.h

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -35,17 +35,5 @@ enum {
3535
#undef DIAG_ENUM_ITEM
3636
#undef DIAG_ENUM
3737
} // namespace diag
38-
39-
namespace diag_compat {
40-
#define DIAG_COMPAT_IDS_BEGIN() enum {
41-
#define DIAG_COMPAT_IDS_END() \
42-
} \
43-
;
44-
#define DIAG_COMPAT_ID(IDX, NAME, ...) NAME = IDX,
45-
#include "clang/Basic/DiagnosticInstallAPICompatIDs.inc"
46-
#undef DIAG_COMPAT_ID
47-
#undef DIAG_COMPAT_IDS_BEGIN
48-
#undef DIAG_COMPAT_IDS_END
49-
} // end namespace diag_compat
5038
} // namespace clang
5139
#endif // LLVM_CLANG_BASIC_DIAGNOSTICINSTALLAPI_H

0 commit comments

Comments
 (0)