[PATCH] D147812: [InstrProf] Use BalancedPartitioning to order temporal profiling trace data

Ellis Hoag via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 6 18:08:23 PDT 2023


ellis added inline comments.


================
Comment at: llvm/unittests/Support/CMakeLists.txt:2
 set(LLVM_LINK_COMPONENTS
+  ProfileData
   Support
----------------
thakis wrote:
> This increases the number of files needed to compile SupportTests by 200% (from ~200 to ~600). Is there a way to prevent this dep with some mocking or similar?
Thanks for flagging! In https://reviews.llvm.org/D152325 I've moved one of theses tests to `unittests/ProfileData` so we can remove the dependency.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D147812



More information about the llvm-commits mailing list