[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