[PATCH] D42659: [COFF] Clarify comment. NFC

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 29 13:47:05 PST 2018


mstorsjo added inline comments.


================
Comment at: COFF/Writer.cpp:577
+  // PE/COFF images are limited to 8 byte section names. Longer names can be
+  // supported by writing a non-standard string table, but this string table is
+  // not mapped at runtime and the long names will therefore be inaccessible.
----------------
Not sure about non-standard though, since iirc dumpbin at least shows them correctly - so the extension is at least acknowledged by MS that much.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D42659





More information about the llvm-commits mailing list