Skip to content

Commit 47ffce1

Browse files
committed
[llvm][DebugInfo][ObjC] Fix argument oreder of setter/getter to DIObjDIObjCProperty constructor
Depends on #165373 This caused the `DW_AT_APPLE_property_(setter|getter)` to be inverted when compiling from LLVM IR.
1 parent 4e922b3 commit 47ffce1

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

llvm/lib/AsmParser/LLParser.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6341,8 +6341,8 @@ bool LLParser::parseDIObjCProperty(MDNode *&Result, bool IsDistinct) {
63416341
#undef VISIT_MD_FIELDS
63426342

63436343
Result = GET_OR_DISTINCT(DIObjCProperty,
6344-
(Context, name.Val, file.Val, line.Val, setter.Val,
6345-
getter.Val, attributes.Val, type.Val));
6344+
(Context, name.Val, file.Val, line.Val, getter.Val,
6345+
setter.Val, attributes.Val, type.Val));
63466346
return false;
63476347
}
63486348

llvm/test/DebugInfo/Generic/objc-property.ll

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,21 +15,22 @@
1515
; CHECK-SAME: DW_APPLE_PROPERTY_assign, DW_APPLE_PROPERTY_readwrite,
1616
; CHECK-SAME: DW_APPLE_PROPERTY_atomic, DW_APPLE_PROPERTY_unsafe_unretained
1717
;
18-
; FIXME: this should have a DW_AT_APPLE_property_getter tag
1918
; CHECK: DW_TAG_APPLE_property
2019
; CHECK: DW_AT_APPLE_property_name ("customGetterProp")
21-
; CHECK: DW_AT_APPLE_property_setter ("customGetter")
20+
; CHECK: DW_AT_APPLE_property_getter ("customGetter")
2221
; CHECK: DW_AT_APPLE_property_attribute
2322
; CHECK-SAME: DW_APPLE_PROPERTY_getter, DW_APPLE_PROPERTY_assign, DW_APPLE_PROPERTY_readwrite,
2423
; CHECK-SAME: DW_APPLE_PROPERTY_atomic, DW_APPLE_PROPERTY_unsafe_unretained
2524
;
25+
; FIXME: this should have a DW_AT_APPLE_property_setter tag
2626
; CHECK: DW_TAG_APPLE_property
2727
; CHECK: DW_AT_APPLE_property_name ("customSetterProp")
2828
; CHECK: DW_AT_APPLE_property_setter ("customSetter:")
2929
; CHECK: DW_AT_APPLE_property_attribute
3030
; CHECK-SAME: DW_APPLE_PROPERTY_assign, DW_APPLE_PROPERTY_readwrite,
3131
; CHECK-SAME: DW_APPLE_PROPERTY_setter, DW_APPLE_PROPERTY_atomic, DW_APPLE_PROPERTY_unsafe_unretained
3232
;
33+
; FIXME: the DW_AT_APPLE_property_(setter|getter) attributes are inverted
3334
; CHECK: DW_TAG_APPLE_property
3435
; CHECK: DW_AT_APPLE_property_name ("customAccessorsProp")
3536
; CHECK: DW_AT_APPLE_property_getter ("customGetter")

0 commit comments

Comments
 (0)