[PATCH] D80301: [yaml][clang-tidy] Fix new line YAML serialization

Nathan James via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 4 15:30:22 PDT 2020


njames93 added inline comments.


================
Comment at: llvm/lib/Support/YAMLTraits.cpp:894
+                                           std::string &Val) {
+  Val.clear();
+  size_t CurrentPos = 0;
----------------
If you want to do the same here...
```
  SmallVector<StringRef, 8> Lines;
  Scalar.split(Lines, "\n\n");
  Val = llvm::join(Lines, "\n");
  return StringRef();```


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80301





More information about the cfe-commits mailing list