[PATCH] D86905: Flush bitcode incrementally for LTO output

[David Stuttard via Phabricator via llvm-commits]

dstuttard added a comment.

See comment about a VC++ warning being generated.



================
Comment at: llvm/include/llvm/Bitstream/BitstreamWriter.h:158
+    // Copy data to update into Bytes from the file FS and the buffer Out.
+    char Bytes[8];
+    size_t BytesNum = StartBit ? 8 : 4;
----------------
Microsoft Visual C++ is warning that this needs to be char Bytes[9]

Can this be updated (at least to silence the warning, we use warnings as errors in our builds)?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86905