[PATCH] D48241: [DebugInfo] Emit ObjC methods as part of interface.

David Blaikie via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 15 15:22:56 PDT 2018


dblaikie added a comment.

Not sure I should get too involved in this discussion, knowing so little about Objective C - but if it seems sensible, could you provide some examples (just in comments/description in this code review) of what the DWARF used to look like and what it looks like after this change?

Does this address the discoverability that's being discussed in the llvm-dev thread? There were concerns there about having to search through all the instances of a type to find all of its functions - I imagine, since Objective C classes aren't closed (if I understand correctly) that would still be a concern here? If it is, what is the benefit/tradeoff of this change (if it doesn't address the discoverability/still requires the Objective C accelerator table)?


Repository:
  rC Clang

https://reviews.llvm.org/D48241





More information about the cfe-commits mailing list