Skip to content

Commit 563ddc4

Browse files
rintarorintaro
committed
[AST] Eliminate 'SYNTAX_KIND' from MagicIdentifierKinds.def
Nothing is using it.
1 parent f25104f commit 563ddc4

16 files changed

+59
-57
lines changed

‎include/swift/AST/ASTBridging.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1766,7 +1766,7 @@ BridgedMacroExpansionExpr BridgedMacroExpansionExpr_createParsed(
17661766
BridgedSourceLoc cRightAngleLoc, BridgedNullableArgumentList cArgList);
17671767

17681768
enum ENUM_EXTENSIBILITY_ATTR(open) BridgedMagicIdentifierLiteralKind : uint8_t {
1769-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) \
1769+
#define MAGIC_IDENTIFIER(NAME, STRING) \
17701770
BridgedMagicIdentifierLiteralKind##NAME,
17711771
#include "swift/AST/MagicIdentifierKinds.def"
17721772
BridgedMagicIdentifierLiteralKindNone,

‎include/swift/AST/DefaultArgumentKind.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ enum class DefaultArgumentKind : uint8_t {
4747
/// property's initializer.
4848
StoredProperty,
4949
// Magic identifier literals expanded at the call site:
50-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) NAME,
50+
#define MAGIC_IDENTIFIER(NAME, STRING) NAME,
5151
#include "swift/AST/MagicIdentifierKinds.def"
5252
/// An expression macro.
5353
ExpressionMacro

‎include/swift/AST/Expr.h

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1085,13 +1085,15 @@ class RegexLiteralExpr : public LiteralExpr {
10851085
class MagicIdentifierLiteralExpr : public BuiltinLiteralExpr {
10861086
public:
10871087
enum Kind : unsigned {
1088-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) NAME,
1088+
#define MAGIC_IDENTIFIER(NAME, STRING) NAME,
10891089
#include "swift/AST/MagicIdentifierKinds.def"
10901090
};
10911091

10921092
static StringRef getKindString(MagicIdentifierLiteralExpr::Kind value) {
10931093
switch (value) {
1094-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) case NAME: return STRING;
1094+
#define MAGIC_IDENTIFIER(NAME, STRING) \
1095+
case NAME: \
1096+
return STRING;
10951097
#include "swift/AST/MagicIdentifierKinds.def"
10961098
}
10971099

@@ -1116,11 +1118,11 @@ class MagicIdentifierLiteralExpr : public BuiltinLiteralExpr {
11161118

11171119
bool isString() const {
11181120
switch (getKind()) {
1119-
#define MAGIC_STRING_IDENTIFIER(NAME, STRING, SYNTAX_KIND) \
1120-
case NAME: \
1121+
#define MAGIC_STRING_IDENTIFIER(NAME, STRING) \
1122+
case NAME: \
11211123
return true;
1122-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) \
1123-
case NAME: \
1124+
#define MAGIC_IDENTIFIER(NAME, STRING) \
1125+
case NAME: \
11241126
return false;
11251127
#include "swift/AST/MagicIdentifierKinds.def"
11261128
}

‎include/swift/AST/MagicIdentifierKinds.def

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -17,22 +17,22 @@
1717

1818
// Used for any magic identifier.
1919
#ifndef MAGIC_IDENTIFIER
20-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND)
20+
#define MAGIC_IDENTIFIER(NAME, STRING)
2121
#endif
2222

2323
// Used for magic identifiers which produce string literals.
2424
#ifndef MAGIC_STRING_IDENTIFIER
25-
#define MAGIC_STRING_IDENTIFIER(NAME, STRING, SYNTAX_KIND) MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND)
25+
#define MAGIC_STRING_IDENTIFIER(NAME, STRING) MAGIC_IDENTIFIER(NAME, STRING)
2626
#endif
2727

2828
// Used for magic identifiers which produce integer literals.
2929
#ifndef MAGIC_INT_IDENTIFIER
30-
#define MAGIC_INT_IDENTIFIER(NAME, STRING, SYNTAX_KIND) MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND)
30+
#define MAGIC_INT_IDENTIFIER(NAME, STRING) MAGIC_IDENTIFIER(NAME, STRING)
3131
#endif
3232

3333
// Used for magic identifiers which produce raw pointers.
3434
#ifndef MAGIC_POINTER_IDENTIFIER
35-
#define MAGIC_POINTER_IDENTIFIER(NAME, STRING, SYNTAX_KIND) MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND)
35+
#define MAGIC_POINTER_IDENTIFIER(NAME, STRING) MAGIC_IDENTIFIER(NAME, STRING)
3636
#endif
3737

3838
// Used when a given token always maps to a particular magic identifier kind.
@@ -45,27 +45,27 @@
4545
//
4646

4747
/// The \c #fileID magic identifier literal.
48-
MAGIC_STRING_IDENTIFIER(FileID, "#fileID", PoundFileIDExpr)
48+
MAGIC_STRING_IDENTIFIER(FileID, "#fileID")
4949
MAGIC_IDENTIFIER_TOKEN(FileID, pound_fileID)
5050

5151
/// The \c #file magic identifier literal, written in code where it is
5252
/// a synonym for \c #fileID (i.e. "Swift 6 mode" code).
53-
MAGIC_STRING_IDENTIFIER(FileIDSpelledAsFile, "#file", PoundFileExpr)
53+
MAGIC_STRING_IDENTIFIER(FileIDSpelledAsFile, "#file")
5454
// tok::pound_file is shared with FilePathSpelledAsFile; please write custom
5555
// code paths for it.
5656

5757
/// The \c #filePath magic identifier literal.
58-
MAGIC_STRING_IDENTIFIER(FilePath, "#filePath", PoundFilePathExpr)
58+
MAGIC_STRING_IDENTIFIER(FilePath, "#filePath")
5959
MAGIC_IDENTIFIER_TOKEN(FilePath, pound_filePath)
6060

6161
/// The \c #file magic identifier literal, written in code where it is
6262
/// a synonym for \c #filePath (i.e. Swift 5 mode code).
63-
MAGIC_STRING_IDENTIFIER(FilePathSpelledAsFile, "#file", PoundFileExpr)
63+
MAGIC_STRING_IDENTIFIER(FilePathSpelledAsFile, "#file")
6464
// tok::pound_file is shared with FileIDSpelledAsFile; please write custom
6565
// code paths for it.
6666

6767
/// The \c #function magic identifier literal.
68-
MAGIC_STRING_IDENTIFIER(Function, "#function", PoundFunctionExpr)
68+
MAGIC_STRING_IDENTIFIER(Function, "#function")
6969
MAGIC_IDENTIFIER_TOKEN(Function, pound_function)
7070

7171

@@ -75,11 +75,11 @@ MAGIC_STRING_IDENTIFIER(Function, "#function", PoundFunctionExpr)
7575
//
7676

7777
/// The \c #line magic identifier literal.
78-
MAGIC_INT_IDENTIFIER(Line, "#line", PoundLineExpr)
78+
MAGIC_INT_IDENTIFIER(Line, "#line")
7979
MAGIC_IDENTIFIER_TOKEN(Line, pound_line)
8080

8181
/// The \c #column magic identifier literal.
82-
MAGIC_INT_IDENTIFIER(Column, "#column", PoundColumnExpr)
82+
MAGIC_INT_IDENTIFIER(Column, "#column")
8383
MAGIC_IDENTIFIER_TOKEN(Column, pound_column)
8484

8585

@@ -89,7 +89,7 @@ MAGIC_INT_IDENTIFIER(Column, "#column", PoundColumnExpr)
8989
//
9090

9191
/// The \c #dsohandle magic identifier literal.
92-
MAGIC_POINTER_IDENTIFIER(DSOHandle, "#dsohandle", PoundDsohandleExpr)
92+
MAGIC_POINTER_IDENTIFIER(DSOHandle, "#dsohandle")
9393
MAGIC_IDENTIFIER_TOKEN(DSOHandle, pound_dsohandle)
9494

9595
#undef MAGIC_IDENTIFIER

‎lib/AST/ASTDumper.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -381,7 +381,7 @@ static StringRef getDumpString(ForeignErrorConvention::Kind value) {
381381
static StringRef getDumpString(DefaultArgumentKind value) {
382382
switch (value) {
383383
case DefaultArgumentKind::None: return "none";
384-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) \
384+
#define MAGIC_IDENTIFIER(NAME, STRING) \
385385
case DefaultArgumentKind::NAME: return STRING;
386386
#include "swift/AST/MagicIdentifierKinds.def"
387387
case DefaultArgumentKind::Inherited: return "inherited";

‎lib/AST/ASTPrinter.cpp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4008,8 +4008,7 @@ void PrintAST::printOneParameter(const ParamDecl *param,
40084008
Printer << " = ";
40094009

40104010
switch (param->getDefaultArgumentKind()) {
4011-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) \
4012-
case DefaultArgumentKind::NAME:
4011+
#define MAGIC_IDENTIFIER(NAME, STRING) case DefaultArgumentKind::NAME:
40134012
#include "swift/AST/MagicIdentifierKinds.def"
40144013
Printer.printKeyword(defaultArgStr, Options);
40154014
break;

‎lib/AST/Bridging/ExprBridging.cpp

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -384,7 +384,9 @@ BridgedMacroExpansionExpr BridgedMacroExpansionExpr_createParsed(
384384
BridgedMagicIdentifierLiteralKind
385385
BridgedMagicIdentifierLiteralKind_fromString(BridgedStringRef cStr) {
386386
StringRef str = cStr.unbridged();
387-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) \
387+
388+
// Note: STRING includes '#' e.g. '#fileID'.
389+
#define MAGIC_IDENTIFIER(NAME, STRING) \
388390
if (str == StringRef(STRING).drop_front()) \
389391
return BridgedMagicIdentifierLiteralKind##NAME;
390392
#include "swift/AST/MagicIdentifierKinds.def"
@@ -394,7 +396,7 @@ BridgedMagicIdentifierLiteralKind_fromString(BridgedStringRef cStr) {
394396
static std::optional<MagicIdentifierLiteralExpr::Kind>
395397
unbridge(BridgedMagicIdentifierLiteralKind cKind) {
396398
switch (cKind) {
397-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) \
399+
#define MAGIC_IDENTIFIER(NAME, STRING) \
398400
case BridgedMagicIdentifierLiteralKind##NAME: \
399401
return MagicIdentifierLiteralExpr::Kind::NAME;
400402
#include "swift/AST/MagicIdentifierKinds.def"

‎lib/AST/Decl.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8647,7 +8647,7 @@ static DefaultArgumentKind computeDefaultArgumentKind(DeclContext *dc,
86478647
return DefaultArgumentKind::Normal;
86488648

86498649
switch (magic->getKind()) {
8650-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) \
8650+
#define MAGIC_IDENTIFIER(NAME, STRING) \
86518651
case MagicIdentifierLiteralExpr::NAME: \
86528652
return DefaultArgumentKind::NAME;
86538653
#include "swift/AST/MagicIdentifierKinds.def"
@@ -8867,7 +8867,7 @@ bool ParamDecl::hasDefaultExpr() const {
88678867
case DefaultArgumentKind::StoredProperty:
88688868
return false;
88698869
case DefaultArgumentKind::Normal:
8870-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) \
8870+
#define MAGIC_IDENTIFIER(NAME, STRING) \
88718871
case DefaultArgumentKind::NAME:
88728872
#include "swift/AST/MagicIdentifierKinds.def"
88738873
case DefaultArgumentKind::ExpressionMacro:
@@ -8888,7 +8888,7 @@ bool ParamDecl::hasCallerSideDefaultExpr() const {
88888888
case DefaultArgumentKind::StoredProperty:
88898889
case DefaultArgumentKind::Normal:
88908890
return false;
8891-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) \
8891+
#define MAGIC_IDENTIFIER(NAME, STRING) \
88928892
case DefaultArgumentKind::NAME:
88938893
#include "swift/AST/MagicIdentifierKinds.def"
88948894
case DefaultArgumentKind::NilLiteral:
@@ -9190,7 +9190,7 @@ ParamDecl::getDefaultValueStringRepresentation(
91909190
return extractInlinableText(getASTContext(), init, scratch);
91919191
}
91929192
case DefaultArgumentKind::Inherited: return "super";
9193-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) \
9193+
#define MAGIC_IDENTIFIER(NAME, STRING) \
91949194
case DefaultArgumentKind::NAME: return STRING;
91959195
#include "swift/AST/MagicIdentifierKinds.def"
91969196
case DefaultArgumentKind::NilLiteral: return "nil";

‎lib/IDE/CompletionLookup.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -999,7 +999,7 @@ bool CompletionLookup::hasInterestingDefaultValue(const ParamDecl *param) {
999999
return true;
10001000

10011001
case DefaultArgumentKind::None:
1002-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) \
1002+
#define MAGIC_IDENTIFIER(NAME, STRING) \
10031003
case DefaultArgumentKind::NAME:
10041004
#include "swift/AST/MagicIdentifierKinds.def"
10051005
case DefaultArgumentKind::ExpressionMacro:

‎lib/SILGen/SILGen.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1719,7 +1719,7 @@ void SILGenModule::emitDefaultArgGenerator(SILDeclRef constant,
17191719
break;
17201720

17211721
case DefaultArgumentKind::Inherited:
1722-
#define MAGIC_IDENTIFIER(NAME, STRING, SYNTAX_KIND) \
1722+
#define MAGIC_IDENTIFIER(NAME, STRING) \
17231723
case DefaultArgumentKind::NAME:
17241724
#include "swift/AST/MagicIdentifierKinds.def"
17251725
case DefaultArgumentKind::NilLiteral:

0 commit comments

Comments
 (0)