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

[Esme Yi via Phabricator via llvm-commits]

Esme added inline comments.


================
Comment at: llvm/lib/MC/XCOFFObjectWriter.cpp:314
   size_t auxiliaryHeaderSize() const {
-    assert(!needsAuxiliaryHeader() &&
-           "Auxiliary header support not implemented.");
-    return 0;
+    return HasVisibility && !is64Bit() ? XCOFF::AuxFileHeaderSizeShort : 0;
   }
----------------
DiggerLin wrote:
> add some comment here :
> // TODO: not support 64-bit auxiliary header.
As the definition in aouthdr.h, 64-bit object files will not have the auxiliary header. So this should not be a TODO?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128148