[PATCH] D47468: [YAML] Quote multiline string scalars

Sam McCall via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 30 02:13:47 PDT 2018


sammccall accepted this revision.
sammccall added a comment.
This revision is now accepted and ready to land.

Thanks, LG!
comment nit to prove i read it ;)



================
Comment at: include/llvm/Support/YAMLTraits.h:546
+      continue;
+    // LF (0xA), CR (0xD) require single quotes, at least.
     case 0xA:
----------------
this echoes the code a bit. what about:
LF/CR may delimit values and so require at least single quotes.


Repository:
  rL LLVM

https://reviews.llvm.org/D47468





More information about the llvm-commits mailing list