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

[Esme Yi via Phabricator via llvm-commits]

Esme added inline comments.


================
Comment at: llvm/lib/ObjectYAML/XCOFFEmitter.cpp:396-397
   // Write the string table.
-  if (Strings.getSize() > 4)
-    Strings.write(W.OS);
+  if (StrTbl.getSize() > 4 && !Obj.StrTbl.Suppressed)
+    StrTbl.write(W.OS);
   return true;
----------------
Higuoxing wrote:
> I believe You're are able to pad the string table to a specified length without modifying the interface of `StringTableBuilder`. Please let me know if it doesn't work :)
> 
If so, we have to rewrite the length value in the string table, otherwise these padding zeros  will not be treated as string table contents.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D107421