[PATCH] D29892: ar: add llvm-dlltool support

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added inline comments.


================
Comment at: lib/Object/COFFImportFile.cpp:559-560
+  // Copied here so we can still use writeStringTable
+  memcpy(Alloc.Allocate<char>(Buffer.size()), Buffer.data(), Buffer.size());
+  return {MemoryBufferRef(StringRef(Buf, Buffer.size()), DLLName)};
+}
----------------
This must not be correct. You are memcpy'ing to some memory region and returning something different.


Repository:
  rL LLVM

https://reviews.llvm.org/D29892