[PATCH] D45574: Add PPC64_GLINK dynamic tag

Alexander Richardson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 12 13:40:47 PDT 2018


arichardson added a comment.

LGTM other than adding a gap between MIPS tags and PPC tags



================
Comment at: include/llvm/BinaryFormat/DynamicTags.def:178
                                                 // map, used for debugging.
+PPC64_DYNAMIC_TAG(PPC64_GLINK, 0x70000000) // Address of 32 bytes before the
+                                           // first glink lazy resolver stub.
----------------
I would add a newline and a `// PPC specific dynamic table entries` comment


================
Comment at: include/llvm/BinaryFormat/DynamicTags.def:191
 #undef MIPS_DYNAMIC_TAG
+#undef MIPS_DYNAMIC_TAG_DEFINED
 #endif
----------------
Good catch, I forgot to add those in my initial change.


Repository:
  rL LLVM

https://reviews.llvm.org/D45574





More information about the llvm-commits mailing list