summaryrefslogtreecommitdiffstats
path: root/llvm/lib/Support/Statistic.cpp
diff options
context:
space:
mode:
authorFrancis Visoiu Mistrih <francisvm@yahoo.com>2017-12-18 17:38:03 +0000
committerFrancis Visoiu Mistrih <francisvm@yahoo.com>2017-12-18 17:38:03 +0000
commitb213b27ee3cba7d0b7ad2a45c8cbd42e59510220 (patch)
tree887a44fd8bb29d305bc2c240e44a01467ca9cc38 /llvm/lib/Support/Statistic.cpp
parent6c0858e41413de10b7194519fc6cb9d4a1eda959 (diff)
downloadbcm5719-llvm-b213b27ee3cba7d0b7ad2a45c8cbd42e59510220.tar.gz
bcm5719-llvm-b213b27ee3cba7d0b7ad2a45c8cbd42e59510220.zip
[YAML] Add support for non-printable characters
LLVM IR function names which disable mangling start with '\01' (https://www.llvm.org/docs/LangRef.html#identifiers). When an identifier like "\01@abc@" gets dumped to MIR, it is quoted, but only with single quotes. http://www.yaml.org/spec/1.2/spec.html#id2770814: "The allowed character range explicitly excludes the C0 control block allowed), the surrogate block #xD800-#xDFFF, #xFFFE, and #xFFFF." http://www.yaml.org/spec/1.2/spec.html#id2776092: "All non-printable characters must be escaped. [...] Note that escape sequences are only interpreted in double-quoted scalars." This patch adds support for printing escaped non-printable characters between double quotes if needed. Should also fix PR31743. Differential Revision: https://reviews.llvm.org/D41290 llvm-svn: 320996
Diffstat (limited to 'llvm/lib/Support/Statistic.cpp')
-rw-r--r--llvm/lib/Support/Statistic.cpp5
1 files changed, 3 insertions, 2 deletions
diff --git a/llvm/lib/Support/Statistic.cpp b/llvm/lib/Support/Statistic.cpp
index 23718bb0e9c..544ae2d0983 100644
--- a/llvm/lib/Support/Statistic.cpp
+++ b/llvm/lib/Support/Statistic.cpp
@@ -168,9 +168,10 @@ void llvm::PrintStatisticsJSON(raw_ostream &OS) {
const char *delim = "";
for (const Statistic *Stat : Stats.Stats) {
OS << delim;
- assert(!yaml::needsQuotes(Stat->getDebugType()) &&
+ assert(yaml::needsQuotes(Stat->getDebugType()) == yaml::QuotingType::None &&
"Statistic group/type name is simple.");
- assert(!yaml::needsQuotes(Stat->getName()) && "Statistic name is simple");
+ assert(yaml::needsQuotes(Stat->getName()) == yaml::QuotingType::None &&
+ "Statistic name is simple");
OS << "\t\"" << Stat->getDebugType() << '.' << Stat->getName() << "\": "
<< Stat->getValue();
delim = ",\n";
OpenPOWER on IntegriCloud