Skip to content

Commit f17e5b9

Browse files
committed
[CodeCompletion] Provide argument label completions inside #externalMacro
rdar://108163809 rdar://108681394
1 parent fe74c86 commit f17e5b9

File tree

2 files changed

+22
-1
lines changed

2 files changed

+22
-1
lines changed

lib/Sema/TypeCheckMacros.cpp

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -193,6 +193,13 @@ MacroDefinition MacroDefinitionRequest::evaluate(
193193
free(replacements);
194194
};
195195

196+
if (checkResult < 0 && ctx.CompletionCallback) {
197+
// If the macro failed to check and we are in code completion mode, pretend
198+
// it's an arbitrary macro. This allows us to get call argument completions
199+
// inside `#externalMacro`.
200+
checkResult = BridgedMacroDefinitionKind::BridgedExpandedMacro;
201+
}
202+
196203
if (checkResult < 0)
197204
return MacroDefinition::forInvalid();
198205

test/IDE/complete_macro_declaration.swift

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,5 +3,19 @@
33

44
let globalVar = 1
55
macro expect(file: Int = #^DEFAULT_ARG^#) = #externalMacro(module: "MyModule", type: "MyMacro")
6-
76
// DEFAULT_ARG: Decl[GlobalVar]/CurrModule/TypeRelation[Convertible]: globalVar[#Int#]; name=globalVar
7+
8+
@freestanding(expression)
9+
macro externalMacro() = ##^EXTERNAL_MACRO^#externalMacro
10+
// EXTERNAL_MACRO: Decl[Macro]/OtherModule[Swift]/IsSystem: externalMacro({#module: String#}, {#type: String#})[#T#]; name=externalMacro(module:type:)
11+
12+
@freestanding(expression)
13+
macro externalMacroWithTrailing() = ##^EXTERNAL_MACRO_WITH_TRAILING?check=EXTERNAL_MACRO^#externalMacro
14+
15+
@freestanding(expression)
16+
macro externalMacroCallPattern() = #externalMacro(#^EXTERNAL_MACRO_CALL_PATTERN^#)
17+
// EXTERNAL_MACRO_CALL_PATTERN: Pattern/None/Flair[ArgLabels]/TypeRelation[Convertible]: ['(']{#module: String#}, {#type: String#}[')'][#Void#]; name=module:type:
18+
19+
@freestanding(expression)
20+
macro externalMacroCallPattern() = #externalMacro(module: "MyModule", #^EXTERNAL_MACRO_TYPE_ARG_LABEL^#)
21+
// EXTERNAL_MACRO_TYPE_ARG_LABEL: Pattern/Local/Flair[ArgLabels]: {#type: String#}[#String#]; name=type:

0 commit comments

Comments
 (0)