[PATCH] D107421: [yaml2obj][XCOFF] Customize the string table.

[Xing GUO via Phabricator via llvm-commits]

Higuoxing added inline comments.


================
Comment at: llvm/lib/MC/StringTableBuilder.cpp:80
   if (K == WinCOFF)
-    support::endian::write32le(Buf, Size);
+    support::endian::write32le(Buf, Length ? Length : Size);
   else if (K == XCOFF)
----------------
Higuoxing wrote:
> Personally, I don't like this approach. I would do the overwriting job in the client side. Besides, this change is dangerous, what will happen if `Length > Size` ?
Sorry, I made a mistake here. I realize that you're modifying the first 4-byte of the buffer. But I still think we should do it in the client side.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107421