[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