[PATCH] D128148: [XCOFF] write the aux header when the visibility is specified in XCOFF32.

Esme Yi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jun 19 19:14:29 PDT 2022


Esme created this revision.
Esme added reviewers: shchenz, DiggerLin, daltenty, jhenderson, PowerPC.
Herald added subscribers: hiraditya, nemanjai.
Herald added a project: All.
Esme requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

The n_type field in the symbol table entry has two interpretations in XCOFF32, and a single interpretation in XCOFF64.
The new interpretation is used in XCOFF32 if the value of the o_vstamp field in the auxiliary header is 2.
In XCOFF64 and the new XCOFF32 interpretation, the n_type field is used for the symbol type and visibility.

The patch writes the aux header with an o_vstamp field value of 2 when the visibility is specified in XCOFF32 to make the new XCOFF32 interpretation used.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D128148

Files:
  llvm/include/llvm/BinaryFormat/XCOFF.h
  llvm/lib/MC/XCOFFObjectWriter.cpp
  llvm/test/CodeGen/PowerPC/aix-aux-header.ll
  llvm/test/CodeGen/PowerPC/aix-xcoff-visibility.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D128148.438232.patch
Type: text/x-patch
Size: 11713 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220620/e24fe13e/attachment-0001.bin>


More information about the llvm-commits mailing list