herzka added a comment.
Currently, `A.customGetterProperty` would be turned into `[A
customGetterProperty]`, which would fail to compile because that selector isn't
declared anywhere. With my fix, it does compile because it (correctly) resolves
to `[A customGet]`, which does exist. Similarly, `a.customSetterProperty = 1`
got turned into `[a setCustomSetterProperty:1]` (another nonexistent selector),
but now becomes `[a customSet:1]`.
Here are the errors I get when I run the test without my fix:
error: 'error' diagnostics seen but not expected:
File
/Users/herzka/dev/OSS/llvm/tools/clang/test/SemaObjC/objc-class-property.m Line
45: no setter method 'setCustomSetterProperty:' for assignment to property
File
/Users/herzka/dev/OSS/llvm/tools/clang/test/SemaObjC/objc-class-property.m Line
46: no getter method for read from property
2 errors generated.
https://reviews.llvm.org/D29967
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
