[PATCH] D116784: [ProfileData] Read and symbolize raw memprof profiles.

Nico Weber via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 3 15:40:45 PST 2022


thakis added inline comments.


================
Comment at: llvm/lib/ProfileData/CMakeLists.txt:25
   Object
+  Symbolize
   DebugInfoDWARF
----------------
This dependency is a problem. At the moment, Symbolize adds a dep to libcurl, and clang transitiviely depends on llvm/lib/ProfileData. So this makes clang depend on libcurl (even in statically linked builds), which is something we want to avoid. See also the discussion on D113717 (in particular, https://reviews.llvm.org/D113717#3295350).

Since this likely has no super quick fix, would you mind reverting this until that dependency can be prevented somehow?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116784



More information about the llvm-commits mailing list