[PATCH] D86905: Flush bitcode incrementally for LTO output

[stephan.yichao.zhao via Phabricator via llvm-commits]

stephan.yichao.zhao added inline comments.


================
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;
----------------
dstuttard wrote:
> 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)?
Yes. I will be fixing it.
What is the name of this warning from Visual C++? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86905