[all-commits] [llvm/llvm-project] 313b95: [lldb][test] Add test for parsing Objective-C synt...
Michael Buch via All-commits
all-commits at lists.llvm.org
Mon Oct 27 03:01:44 PDT 2025
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 313b95f3a8d7efaf8970484c169f1a106bc12f68
https://github.com/llvm/llvm-project/commit/313b95f3a8d7efaf8970484c169f1a106bc12f68
Author: Michael Buch <michaelbuch12 at gmail.com>
Date: 2025-10-27 (Mon, 27 Oct 2025)
Changed paths:
A lldb/test/API/lang/objc/synthesized-property-accessor/Makefile
A lldb/test/API/lang/objc/synthesized-property-accessor/TestSynthesizedPropertyAccessor.py
A lldb/test/API/lang/objc/synthesized-property-accessor/main.m
Log Message:
-----------
[lldb][test] Add test for parsing Objective-C synthesized properties
Prior to https://github.com/llvm/llvm-project/pull/164998, recent LLDB
versions would fail to parse synthesized property setters correctly. The
only way this failure would manifest is an error to the console:
```
error: main.o [0x00000000000000cd]: invalid Objective-C method DW_TAG_subprogram (DW_TAG_subprogram), please file a bug and attach the file at the start of this error message
```
There weren't any Objective-C tests that failed when the original regression (https://github.com/llvm/llvm-project/pull/100355) landed. This patch adds a test that explicitly checks that the type of the setter is sensible.
This test fails without https://github.com/llvm/llvm-project/pull/164998
and passes with it.
I decided not to check for the absence of the console error because that kind of test would be fragile to the removal of (or any changes to) the error message.
To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications
More information about the All-commits
mailing list