[PATCH] D57168: Fix broken export table if .rdata is merged with .text.

[Reid Kleckner via Phabricator via llvm-commits]

rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

lgtm



================
Comment at: lld/COFF/DLL.cpp:475-476
   void writeTo(uint8_t *Buf) const override {
+    memset(Buf + OutputSectionOff, 0, getSize());
+
     uint8_t *P = Buf + OutputSectionOff;
----------------
I think we don't need this dead store if we modify the loop below to write zero unconditionally if !E.Noname. Maybe it's worth asserting `Config->Exports.size() == Size` to ensure there is no padding.


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

https://reviews.llvm.org/D57168