[PATCH] D62949: [analyzer][tests] Add normalize_plist to replace diff_plist
Hubert Tong via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jun 7 19:41:06 PDT 2019
hubert.reinterpretcast added a comment.
I've tested the pre-normalization and it looks like I can commit it tomorrow. I noticed that the following three files appear to be unreferenced:
clang/test/Analysis/Inputs/expected-plists/cstring-plist.c.plist
clang/test/Analysis/Inputs/expected-plists/plist-stats-output.c.plist
clang/test/Analysis/Inputs/expected-plists/yaccignore.c.plist
I'll commit the deletion of these tomorrow as well.
Modifying the tools to produce a newline at the end of the output also seems to work okay with the tests. I've posted the change for that in D63041 <https://reviews.llvm.org/D63041>.
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D62949/new/
https://reviews.llvm.org/D62949
More information about the cfe-commits
mailing list