[PATCH] D41290: [YAML] Add support for non-printable characters

Jonas Devlieghere via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 15 07:18:18 PST 2017


JDevlieghere added a comment.

LGTM!



================
Comment at: docs/YamlIO.rst:469
       // Determine if this scalar needs quotes.
-      static bool mustQuote(StringRef) { return true; }
+      static QuotingType mustQuote(StringRef) { return QuotingType::Simple; }
     };
----------------
s/Simple/Single/


https://reviews.llvm.org/D41290





More information about the llvm-commits mailing list