[PATCH] D96611: [analyzer][tests] Fix issue comparison script

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 12 10:10:10 PST 2021


NoQ accepted this revision.
NoQ added a comment.
This revision is now accepted and ready to land.

Thx!~



================
Comment at: clang/utils/analyzer/CmpRuns.py:401-402
+
+        old = filter_issues(old, common)
+        new = filter_issues(new, common)
+        common = set()
----------------
If `old` and `new` were sets from the beginning we could have turned this into `old = old - common` etc. and it would have been kinda neat.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96611/new/

https://reviews.llvm.org/D96611



More information about the cfe-commits mailing list