[PATCH] D52036: [Analyzer] Make plist tests less fragile

2018-09-13 Thread Artem Dergachev via Phabricator via cfe-commits
NoQ added a comment. What do you think about the following approach: 1. Define a `lit` substitution `%diff_plist` that resolves to `diff -u -w -I "/" -I ".:" -I "version" -`. 2. Update these flags in your downstream fork. It's kinda cool because it'll allow us to test exactly what we want in th

[PATCH] D52036: [Analyzer] Make plist tests less fragile

2018-09-13 Thread Mikhail Maltsev via Phabricator via cfe-commits
miyuki created this revision. miyuki added reviewers: NoQ, dcoughlin, george.karpenkov. Herald added subscribers: Szelethus, mikhail.ramalho, a.sidorin, szepet, eraman, xazax.hun. A recent change https://reviews.llvm.org/D50545 started using diff to compare plist output against reference output.