[PATCH] D79671: [MsgPack] MsgPackDocument::readFromBlob now merges

Tim Renouf via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 18 03:42:56 PDT 2020


tpr updated this revision to Diff 264574.
tpr added a comment.

  V4: Addressed review comments:
      * Separated the idea of an empty node from a nil node
      * Made default merge behavior to fail on any conflict
      * Array merging now appends


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D79671

Files:
  llvm/include/llvm/BinaryFormat/MsgPackDocument.h
  llvm/lib/BinaryFormat/MsgPackDocument.cpp
  llvm/unittests/BinaryFormat/MsgPackDocumentTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D79671.264574.patch
Type: text/x-patch
Size: 14724 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200518/d37aa5e7/attachment.bin>


More information about the llvm-commits mailing list