[PATCH] D136950: [XCOFF] change the decoding of External symbol's function auxiliary entry in XCOFF32 for llvm-readobj

Hubert Tong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 2 15:22:10 PDT 2022


hubert.reinterpretcast accepted this revision.
hubert.reinterpretcast added a comment.
This revision is now accepted and ready to land.

LGTM with comment.



================
Comment at: llvm/test/tools/llvm-readobj/XCOFF/symbols.test:141
+    Section:            .text 
+    Type:               0x20
+    StorageClass:       C_WEAKEXT
----------------
One of the changes made in this patch is that the `DT_FCN` bit is not checked.


================
Comment at: llvm/test/tools/llvm-readobj/XCOFF/symbols.test:339
+# SYMBOL32-NEXT:     Section: .text
+# SYMBOL32-NEXT:     Type: 0x20
+# SYMBOL32-NEXT:     StorageClass: C_WEAKEXT (0x6F)
----------------



Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D136950/new/

https://reviews.llvm.org/D136950



More information about the llvm-commits mailing list