[PATCH] D69112: [NFC][XCOFF][AIX] Serialize object file writing for each CsectGroup

[Digger via Phabricator via llvm-commits]

DiggerLin added inline comments.


================
Comment at: llvm/lib/MC/XCOFFObjectWriter.cpp:120
+  // initialized.
+  static constexpr int16_t UninitializedIndex = -3;
+
----------------
daltenty wrote:
> Should we specify this relative to XCOFF::N_DEBUG?
in the symbol table , the section index maybe has -2 ,-1 ,and 0. but for section, it always start from 1, I think keep original  -1 as uninitiated is OK too.


================
Comment at: llvm/lib/MC/XCOFFObjectWriter.cpp:573
         SymbolTableIndex += 2;
+        Csect.Size = Layout.getSectionAddressSize(MCSec);
+        if (!SupportLabelDef)
----------------
two Csect.Size = Layout.getSectionAddressSize(MCSec); (duplicate with line 568 ) ?


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

https://reviews.llvm.org/D69112