[PATCH] D97726: [sanitizers] [windows] Use InternalMmapVector instead of silencing -Wframe-larger-than

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 9 09:22:57 PST 2021


mstorsjo added a comment.

Thanks! That narrowed it down nicely.



================
Comment at: compiler-rt/lib/sanitizer_common/sanitizer_win.cpp:598
     return 0;
   if (!ReadFromFile(fd, &buf[0], sizeof(buf), &bytes_read) ||
       bytes_read != sizeof(buf))
----------------
Ah, this `sizeof(buf)` needs to be `buf.size()` or something like that.


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

https://reviews.llvm.org/D97726



More information about the llvm-commits mailing list