[PATCH] D59396: [MsgPack] Attempt to fix failure on windows expensive checks bot

Tim Renouf via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 16 05:18:33 PDT 2019


tpr marked 2 inline comments as done.
tpr added inline comments.


================
Comment at: lib/BinaryFormat/MsgPackDocumentYAML.cpp:64
   }
-  return S;
+  return OS.str();
 }
----------------
arsenm wrote:
> Why is this necessary?
str() flushes the raw_string_ostream before returning the string.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59396





More information about the llvm-commits mailing list