Trim the attribute name when grabbing it from a syntax tree

DougGregor · DougGregor · commit 9950ea06ef73 · 2023-02-15T12:46:59.000-08:00
diff --git a/lib/ASTGen/Sources/ASTGen/Macros.swift b/lib/ASTGen/Sources/ASTGen/Macros.swift
@@ -345,7 +345,7 @@ func expandAttachedMacro(
   // Create an expansion context
   let context = sourceManager.createMacroExpansionContext()
 
-  let macroName = customAttrNode.attributeName.description
+  let macroName = customAttrNode.attributeName.trimmedDescription
   var evaluatedSyntaxStr: String
   do {
     switch (macro, macroRole) {
diff --git a/test/Macros/macro_expand_throwing.swift b/test/Macros/macro_expand_throwing.swift
@@ -14,7 +14,7 @@
 
 func testThrownError() {
   let name = "hello"
-  #myWarning(name) // expected-error{{#myWarning macro requires a string literal (from macro 'myWarning')}}
+  #myWarning (name) // expected-error{{#myWarning macro requires a string literal (from macro 'myWarning')}}
 
   #myWarning("experimental features ahead") // expected-warning{{experimental features ahead}}
 }

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@`
`14`	`14`
`15`	`15`	`func testThrownError() {`
`16`	`16`	`let name = "hello"`
`17`		`- #myWarning(name) // expected-error{{#myWarning macro requires a string literal (from macro 'myWarning')}}`
	`17`	`+ #myWarning (name) // expected-error{{#myWarning macro requires a string literal (from macro 'myWarning')}}`
`18`	`18`
`19`	`19`	`#myWarning("experimental features ahead") // expected-warning{{experimental features ahead}}`
`20`	`20`	`}`