[PATCH] D86905: Flush bitcode incrementally for LTO output

stephan.yichao.zhao via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 18 09:44:07 PDT 2020


stephan.yichao.zhao marked an inline comment as done.
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;
----------------
stephan.yichao.zhao wrote:
> 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++? 
https://github.com/llvm/llvm-project/commit/cab6f5b2ab814a4be3fd71aacdbe10298f512833


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86905



More information about the llvm-commits mailing list