[PATCH] D41263: Fix size computation in WindowsResourceCOFFWriter::performFileLayout()

[Peter Collingbourne via Phabricator via llvm-commits]

pcc added inline comments.


================
Comment at: llvm/lib/Object/WindowsResource.cpp:379
+  FileSize += 3 * COFF::Symbol16Size +          // @feat.00, .rsrc$1, .rsrc$2
+    2 * sizeof(coff_aux_section_definition) +   // first and second aux section
+    Data.size() * COFF::Symbol16Size +          // 1 symbol per resource
----------------
Is this correct? I think the size of an auxiliary symbol is the same as the size of a regular symbol, so this looks like this would produce the same result as before.


https://reviews.llvm.org/D41263