[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