[PATCH] D142853: [CGSCC] Add pass which counts the max number of times we visit a function
Arthur Eubanks via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jan 30 12:01:26 PST 2023
aeubanks added inline comments.
================
Comment at: llvm/test/Other/count-visits.ll:7
+
+; ONE: 1 count-visits - Max number of times we visited a function
+; TWO: 2 count-visits - Max number of times we visited a function
----------------
lenary wrote:
> We're seeing test failures for this on non-asserts builds, which correspond to the message in `llvm/lib/Support/Statistic.cpp:250` about needing asserts to be enabled for statistics to work.
sorry, should be fixed in 022d0c6e300300d64a505e2998f066ef81a22784
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142853/new/
https://reviews.llvm.org/D142853
More information about the llvm-commits
mailing list