[PATCH] D89891: [llvm-ar][Object] Fix detection of need for 64-bit archive symbol tables

[George Rimar via Phabricator via llvm-commits]

grimar added inline comments.


================
Comment at: llvm/lib/Object/ArchiveWriter.cpp:610
+    raw_svector_ostream SymtabHeader(SymtabHeaderBuf);
+    writeSymbolTableHeader(SymtabHeader, Kind, Deterministic, SymtabSize);
+
----------------
grimar wrote:
> I was a bit confused when saw `writeSymbolTableHeader` here.
> `Kind` can be changed below:
> 
> ```
>     if (LastOffset >= (1ULL << Sym64Threshold)) {
>       if (Kind == object::Archive::K_DARWIN)
>         Kind = object::Archive::K_DARWIN64;
>       else
>         Kind = object::Archive::K_GNU64;
>     }
> ```
> 
> I think would be more natural to call `writeSymbolTableHeader` after last possible modification of `Kind`, i.e. after this code block. Is it possible?
This wasn't addressed?

aside: could you please mask addressed comments as "Done"? So that reviewers can see the status.


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

https://reviews.llvm.org/D89891