[PATCH] D62949: [analyzer][tests] Add normalize_plist to replace diff_plist
Hubert Tong via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jun 6 21:53:56 PDT 2019
hubert.reinterpretcast added a comment.
In D62949#1533574 <https://reviews.llvm.org/D62949#1533574>, @NoQ wrote:
> I think we should:
>
> - Pre-normalize our expected outputs so that we didn't have to normalize them in every run-line.
Okay; I think this might be possible to do in a separate patch that does not depend on this set.
> - Treat the lack of newline in plist output as a bug and try to fix it.
A quick look at this seems to point at `clang/lib/ARCMigrate/PlistReporter.cpp` and `clang/lib/StaticAnalyzer/Core/PlistDiagnostics.cpp`.
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