forked from OSchip/llvm-project
acf4fd3039
A 'readonly' Objective-C property declared in the primary class can effectively be shadowed by a 'readwrite' property declared within an extension of that class, so long as the types and attributes of the two property declarations are compatible. Previously, this functionality was implemented by back-patching the original 'readonly' property to make it 'readwrite', destroying source information and causing some hideously redundant, incorrect code. Simplify the implementation to express how this should actually be modeled: as a separate property declaration in the extension that shadows (via the name lookup rules) the declaration in the primary class. While here, correct some broken Fix-Its, eliminate a pile of redundant code, clean up the ARC migrator's handling of properties declared in extensions, and fix debug info's naming of methods that come from categories. A wonderous side effect of doing this write is that it eliminates the "AddedObjCPropertyInClassExtension" method from the AST mutation listener, which in turn eliminates the last place where we rewrite entire declarations in a chained PCH file or a module file. This change (which fixes rdar://problem/18475765) will allow us to eliminate the rewritten-decls logic from the serialization library, and fixes a crash (rdar://problem/23247794) illustrated by the test/PCH/chain-categories.m example. llvm-svn: 251874 |
||
|---|---|---|
| .. | ||
| atomic-property.m | ||
| auto-fixit.m | ||
| auto-isa-fixit.m | ||
| bridge-cast-in-arc.mm | ||
| bridge-in-non-arc.m | ||
| dereference-addressof.c | ||
| fixit-autoreleasepool.m | ||
| fixit-c90.c | ||
| fixit-class-method-messaging.m | ||
| fixit-cxx0x.cpp | ||
| fixit-cxx1y-compat.cpp | ||
| fixit-cxx11-attributes.cpp | ||
| fixit-cxx11-compat.cpp | ||
| fixit-eof-space.c | ||
| fixit-errors-1.c | ||
| fixit-errors.c | ||
| fixit-function-call.cpp | ||
| fixit-include.c | ||
| fixit-include.h | ||
| fixit-interface-as-param.m | ||
| fixit-large-file.cpp | ||
| fixit-missing-method-return-type.m | ||
| fixit-missing-self-in-block.m | ||
| fixit-multiple-selector-warnings.m | ||
| fixit-newline-style.c | ||
| fixit-nsstring-compare.m | ||
| fixit-nullability-declspec.cpp | ||
| fixit-objc-arc.m | ||
| fixit-objc-bridge-related-attr.m | ||
| fixit-objc-bridge-related-property.m | ||
| fixit-objc-bridge-related.m | ||
| fixit-objc-message-comma-separator.m | ||
| fixit-objc-message.m | ||
| fixit-objc.m | ||
| fixit-pmem.cpp | ||
| fixit-recompile.c | ||
| fixit-recursive-block.c | ||
| fixit-static-object-decl.m | ||
| fixit-suffix.c | ||
| fixit-unicode-with-utf8-output.c | ||
| fixit-unicode.c | ||
| fixit-uninit.c | ||
| fixit-unrecoverable.c | ||
| fixit-unrecoverable.cpp | ||
| fixit-vexing-parse-cxx0x.cpp | ||
| fixit-vexing-parse.cpp | ||
| fixit.c | ||
| fixit.cpp | ||
| format-darwin.m | ||
| format-no-fixit.m | ||
| format.m | ||
| format.mm | ||
| messages.cpp | ||
| multiarg-selector-fixit.m | ||
| no-diagnostics-fixit-info.c | ||
| no-fixit.cpp | ||
| no-macro-fixit.c | ||
| no-typo.c | ||
| objc-literals.m | ||
| property-access-fixit.m | ||
| selector-fixit.m | ||
| typo-crash.cpp | ||
| typo-crash.m | ||
| typo-location-bugs.cpp | ||
| typo-using.cpp | ||
| typo.c | ||
| typo.cpp | ||
| typo.m | ||