[PATCH] D89707: [CSSPGO][llvm-profgen] Parse mmap events from perf script

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 21 07:17:55 PDT 2021


thakis added inline comments.
Herald added a subscriber: lxfind.


================
Comment at: llvm/test/tools/llvm-profgen/lit.local.cfg:6
+if not lit.util.which("llvm-profgen", config.llvm_tools_dir):
+  config.unsupported = True  
----------------
Out of interest, why is this here? llvm-profgen is part of the llvm build and should always exist, right (…assuming you add it to LLVM_TEST_DEPENDS in llvm/test/CmakeLists.txt – is there a reason to not add it there?)? No other tool test has a check like this as far as I know.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89707



More information about the llvm-commits mailing list