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

Jonas Devlieghere via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 18 10:24:04 PDT 2018


JDevlieghere added a comment.

In https://reviews.llvm.org/D48241#1134985, @labath wrote:

> This is not true. (Unlike the .apple_*** tables, ) .debug_names allows you to specify a different schema for every entry in the acelerator table. The schema is specifing using abbreviations in much the same way as .debug_abbrev specifies the schema for .debug_info. So you could easily have one abbreviation for regular classes, and a different one for objc interfaces. Currently, our .debug_names generator does not support these kinds of heterogeneous tables, but that's simply because I had no use for it. It could be added if necessary (though it will require some generalization/refactoring). OTOH, our consumer should already be able to handle these kinds of tables as input.


Indeed, you are correct, I was thinking about the Apple structure with the atoms and forgot about the abbreviation in each entries for DWARF. Thanks for pointing this out!


https://reviews.llvm.org/D48241





More information about the cfe-commits mailing list