[llvm] [llvm-objdump][macho] Add support for ObjC relative method lists (PR #84250)

via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 13 13:32:04 PDT 2024


================
@@ -3673,6 +3683,12 @@ struct method_list32_t {
   /* struct method32_t first;  These structures follow inline */
 };
 
+struct method_list_delta_t {
+  uint32_t entsize;
+  uint32_t count;
+  /* struct method_delta_t first;  These structures follow inline */
+};
+
----------------
alx32 wrote:

Yes, but now we're alternatively passing `method_list64_t` / `method_list32_t` to `print_method_list_delta_t` - so we have to specialize it, or hard case to `method_list_delta_t` which is OK since they're same structure layout, but can be confusing. I think the single line at the top offers best separation / layout (and making sure nothing is printed until we confirm delta list). 
Or, do you see a better solution for passing the `method_list64_t` / `method_list32_t` to `print_method_list_delta_t` ?

https://github.com/llvm/llvm-project/pull/84250


More information about the llvm-commits mailing list