Skip to content

Conversation

@Michael137
Copy link
Member

We already have the same test (just different variable names) in property-basic.m.

We already have the same test (just different variable names) in `property-basic.m`.
@llvmbot llvmbot added the clang Clang issues not falling into any other category label Oct 27, 2025
@llvmbot
Copy link
Member

llvmbot commented Oct 27, 2025

@llvm/pr-subscribers-clang

Author: Michael Buch (Michael137)

Changes

We already have the same test (just different variable names) in property-basic.m.


Full diff: https://github.com/llvm/llvm-project/pull/165298.diff

1 Files Affected:

  • (removed) clang/test/DebugInfo/ObjC/property-2.m (-18)
diff --git a/clang/test/DebugInfo/ObjC/property-2.m b/clang/test/DebugInfo/ObjC/property-2.m
deleted file mode 100644
index f15213131ccc0..0000000000000
--- a/clang/test/DebugInfo/ObjC/property-2.m
+++ /dev/null
@@ -1,18 +0,0 @@
-// FIXME: Check IR rather than asm, then triple is not needed.
-// RUN: %clang_cc1 -triple %itanium_abi_triple -S -debug-info-kind=limited -x objective-c < %s | grep DW_AT_name
-@interface Foo {
-  int i;
-}
-@property int i;
-@end
-
-@implementation Foo
-@synthesize i;
-@end
-
-int bar(Foo *f) {
-  int i = 1;
-  f.i = 2;
-  i = f.i;
-  return i;
-}

@Michael137 Michael137 merged commit e44dce3 into llvm:main Oct 27, 2025
9 of 11 checks passed
@Michael137 Michael137 deleted the clang/debuginfo-objc-property-test-3 branch October 27, 2025 19:40
dvbuka pushed a commit to dvbuka/llvm-project that referenced this pull request Oct 27, 2025
…lvm#165298)

We already have the same test (just different variable names) in
`property-basic.m`.
Lukacma pushed a commit to Lukacma/llvm-project that referenced this pull request Oct 29, 2025
…lvm#165298)

We already have the same test (just different variable names) in
`property-basic.m`.
aokblast pushed a commit to aokblast/llvm-project that referenced this pull request Oct 30, 2025
…lvm#165298)

We already have the same test (just different variable names) in
`property-basic.m`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang Clang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants