diff options
author | James Y Knight <jyknight@google.com> | 2019-10-17 15:27:04 +0000 |
---|---|---|
committer | James Y Knight <jyknight@google.com> | 2019-10-17 15:27:04 +0000 |
commit | ccc4d83cda16bea1d9dfd0967dc7d2cfb24b8e75 (patch) | |
tree | 7730dd616da7b04af72b36b95cd7c7e4c9311ad0 /lldb/packages/Python/lldbsuite/test/sample_test/TestSampleTest.py | |
parent | 1c982af0599781bdb049f898a2d512656c807485 (diff) | |
download | bcm5719-llvm-ccc4d83cda16bea1d9dfd0967dc7d2cfb24b8e75.tar.gz bcm5719-llvm-ccc4d83cda16bea1d9dfd0967dc7d2cfb24b8e75.zip |
[ObjC] Diagnose implicit type coercion from ObjC 'Class' to object
pointer types.
For example, in Objective-C mode, the initialization of 'x' in:
```
@implementation MyType
+ (void)someClassMethod {
MyType *x = self;
}
@end
```
is correctly diagnosed with an incompatible-pointer-types warning, but
in Objective-C++ mode, it is not diagnosed at all -- even though
incompatible pointer conversions generally become an error in C++.
This patch fixes that oversight, allowing implicit conversions
involving Class only to/from unqualified-id, and between qualified and
unqualified Class, where the protocols are compatible.
Note that this does change some behaviors in Objective-C, as well, as
shown by the modified tests.
Of particular note is that assignment from from 'Class<MyProtocol>' to
'id<MyProtocol>' now warns. (Despite appearances, those are not
compatible types. 'Class<MyProtocol>' is not expected to have instance
methods defined by 'MyProtocol', while 'id<MyProtocol>' is.)
Differential Revision: https://reviews.llvm.org/D67983
llvm-svn: 375125
Diffstat (limited to 'lldb/packages/Python/lldbsuite/test/sample_test/TestSampleTest.py')
0 files changed, 0 insertions, 0 deletions