[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