Skip to content

Commit e11d150

Browse files
committed
[NFC] [ASTGen] Make BridgedDecl.attrs property
…subsuming BridgedDecl.setAttrs(_:) as its setter.
1 parent 7fcdfae commit e11d150

File tree

3 files changed

+24
-17
lines changed

3 files changed

+24
-17
lines changed

include/swift/AST/ASTBridging.h

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -863,7 +863,10 @@ BridgedSILGenNameAttr BridgedSILGenNameAttr_createParsed(
863863
// MARK: Decls
864864
//===----------------------------------------------------------------------===//
865865

866-
SWIFT_NAME("BridgedDecl.setAttrs(self:_:)")
866+
SWIFT_NAME("getter:BridgedDecl.attrs(self:)")
867+
BridgedDeclAttributes BridgedDecl_getAttrs(BridgedDecl decl);
868+
869+
SWIFT_NAME("setter:BridgedDecl.attrs(self:newValue:)")
867870
void BridgedDecl_setAttrs(BridgedDecl decl, BridgedDeclAttributes attrs);
868871

869872
enum ENUM_EXTENSIBILITY_ATTR(closed) BridgedStaticSpelling {

lib/AST/Bridging/DeclBridging.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -115,6 +115,10 @@ static AccessorKind unbridged(BridgedAccessorKind kind) {
115115
return static_cast<AccessorKind>(kind);
116116
}
117117

118+
BridgedDeclAttributes BridgedDecl_getAttrs(BridgedDecl decl) {
119+
return decl.unbridged()->getAttrs();
120+
}
121+
118122
void BridgedDecl_setAttrs(BridgedDecl decl, BridgedDeclAttributes attrs) {
119123
decl.unbridged()->getAttrs() = attrs.unbridged();
120124
}

lib/ASTGen/Sources/ASTGen/Decls.swift

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ extension ASTGenVisitor {
105105
underlyingType: self.generate(type: node.initializer.value),
106106
genericWhereClause: self.generate(genericWhereClause: node.genericWhereClause)
107107
)
108-
decl.asDecl.setAttrs(attrs.attributes)
108+
decl.asDecl.attrs = attrs.attributes
109109
return decl
110110
}
111111

@@ -129,7 +129,7 @@ extension ASTGenVisitor {
129129
end: node.memberBlock.rightBrace
130130
)
131131
)
132-
decl.asDecl.setAttrs(attrs.attributes)
132+
decl.asDecl.attrs = attrs.attributes
133133

134134
self.withDeclContext(decl.asDeclContext) {
135135
decl.setParsedMembers(self.generate(memberBlockItemList: node.memberBlock.members).lazy.bridgedArray(in: self))
@@ -158,7 +158,7 @@ extension ASTGenVisitor {
158158
end: node.memberBlock.rightBrace
159159
)
160160
)
161-
decl.asDecl.setAttrs(attrs.attributes)
161+
decl.asDecl.attrs = attrs.attributes
162162

163163
self.withDeclContext(decl.asDeclContext) {
164164
decl.setParsedMembers(self.generate(memberBlockItemList: node.memberBlock.members).lazy.bridgedArray(in: self))
@@ -188,7 +188,7 @@ extension ASTGenVisitor {
188188
),
189189
isActor: false
190190
)
191-
decl.asDecl.setAttrs(attrs.attributes)
191+
decl.asDecl.attrs = attrs.attributes
192192

193193
self.withDeclContext(decl.asDeclContext) {
194194
decl.setParsedMembers(self.generate(memberBlockItemList: node.memberBlock.members).lazy.bridgedArray(in: self))
@@ -218,7 +218,7 @@ extension ASTGenVisitor {
218218
),
219219
isActor: true
220220
)
221-
decl.asDecl.setAttrs(attrs.attributes)
221+
decl.asDecl.attrs = attrs.attributes
222222

223223
self.withDeclContext(decl.asDeclContext) {
224224
decl.setParsedMembers(self.generate(memberBlockItemList: node.memberBlock.members).lazy.bridgedArray(in: self))
@@ -250,7 +250,7 @@ extension ASTGenVisitor {
250250
end: node.memberBlock.rightBrace
251251
)
252252
)
253-
decl.asDecl.setAttrs(attrs.attributes)
253+
decl.asDecl.attrs = attrs.attributes
254254

255255
self.withDeclContext(decl.asDeclContext) {
256256
decl.setParsedMembers(self.generate(memberBlockItemList: node.memberBlock.members).lazy.bridgedArray(in: self))
@@ -275,7 +275,7 @@ extension ASTGenVisitor {
275275
defaultType: self.generate(type: node.initializer?.value),
276276
genericWhereClause: self.generate(genericWhereClause: node.genericWhereClause)
277277
)
278-
decl.asDecl.setAttrs(attrs.attributes)
278+
decl.asDecl.attrs = attrs.attributes
279279
return decl
280280
}
281281
}
@@ -297,7 +297,7 @@ extension ASTGenVisitor {
297297
end: node.memberBlock.rightBrace
298298
)
299299
)
300-
decl.asDecl.setAttrs(attrs.attributes)
300+
decl.asDecl.attrs = attrs.attributes
301301

302302
self.withDeclContext(decl.asDeclContext) {
303303
decl.setParsedMembers(self.generate(memberBlockItemList: node.memberBlock.members).lazy.bridgedArray(in: self))
@@ -334,7 +334,7 @@ extension ASTGenVisitor {
334334
guard let elemDecl = self.generate(enumCaseElement: elem) else {
335335
return nil
336336
}
337-
elemDecl.asDecl.setAttrs(attrs.attributes)
337+
elemDecl.asDecl.attrs = attrs.attributes
338338
return elemDecl
339339
})
340340
return .createParsed(
@@ -564,7 +564,7 @@ extension ASTGenVisitor {
564564
arrowLoc: self.generateSourceLoc(node.returnClause.arrow),
565565
returnType: self.generate(type: node.returnClause.type)
566566
)
567-
subscriptDecl.asDecl.setAttrs(attrs.attributes)
567+
subscriptDecl.asDecl.attrs = attrs.attributes
568568

569569
if let accessors = node.accessorBlock {
570570
let storage = subscriptDecl.asAbstractStorageDecl
@@ -599,7 +599,7 @@ extension ASTGenVisitor {
599599
returnType: self.generate(type: node.signature.returnClause?.type),
600600
genericWhereClause: self.generate(genericWhereClause: node.genericWhereClause)
601601
)
602-
decl.asDecl.setAttrs(attrs.attributes)
602+
decl.asDecl.attrs = attrs.attributes
603603

604604
if let body = node.body {
605605
self.withDeclContext(decl.asDeclContext) {
@@ -626,7 +626,7 @@ extension ASTGenVisitor {
626626
thrownType: self.generate(type: node.signature.effectSpecifiers?.thrownError),
627627
genericWhereClause: self.generate(genericWhereClause: node.genericWhereClause)
628628
)
629-
decl.asDecl.setAttrs(attrs.attributes)
629+
decl.asDecl.attrs = attrs.attributes
630630

631631
if let body = node.body {
632632
self.withDeclContext(decl.asDeclContext) {
@@ -645,7 +645,7 @@ extension ASTGenVisitor {
645645
declContext: self.declContext,
646646
deinitKeywordLoc: self.generateSourceLoc(node.deinitKeyword)
647647
)
648-
decl.asDecl.setAttrs(attrs.attributes)
648+
decl.asDecl.attrs = attrs.attributes
649649

650650
if let body = node.body {
651651
self.withDeclContext(decl.asDeclContext) {
@@ -697,7 +697,7 @@ extension ASTGenVisitor {
697697
rightAngleLoc: info.rightAngleLoc,
698698
args: info.arguments
699699
)
700-
decl.asDecl.setAttrs(attrs.attributes)
700+
decl.asDecl.attrs = attrs.attributes
701701

702702
return decl
703703
}
@@ -854,7 +854,7 @@ extension ASTGenVisitor {
854854
lowerThanNames: self.generate(precedenceGroupNameList: body.lowerThanRelation?.precedenceGroups),
855855
rightBraceLoc: self.generateSourceLoc(node.rightBrace)
856856
)
857-
decl.asDecl.setAttrs(attrs.attributes)
857+
decl.asDecl.attrs = attrs.attributes
858858
return decl
859859
}
860860
}
@@ -901,7 +901,7 @@ extension ASTGenVisitor {
901901
self.generateLocatedIdentifier($0.name)
902902
}.bridgedArray(in: self)
903903
)
904-
decl.asDecl.setAttrs(attrs.attributes)
904+
decl.asDecl.attrs = attrs.attributes
905905
return decl
906906
}
907907
}

0 commit comments

Comments
 (0)