[PATCH] D83539: [analyzer][tests] Introduce analyzer benchmarking framework

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 13 16:41:59 PDT 2020


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

Box-and-whisker diagrams 👍



================
Comment at: clang/utils/analyzer/SATestBenchmark.py:18
+
+import pandas as pd
+from typing import List, Tuple
----------------
@Szelethus ???


================
Comment at: clang/utils/analyzer/SATestBenchmark.py:139-140
+    import matplotlib
+    import seaborn as sns
+    from matplotlib import pyplot as plt
+
----------------
Why shorten? It's not that we're saving a lot of typing, and i think it's easier to read when package names are recognizable.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83539





More information about the cfe-commits mailing list