[PATCH] D153039: [BOLT] A new code layout algorithm for function reordering [3b/3]

[Sergey Pupyrev via Phabricator via llvm-commits]

spupyrev added a comment.

I am adding an internal test for CDS



================
Comment at: bolt/lib/Passes/ReorderFunctions.cpp:340
+    std::vector<uint64_t> Result =
+        applyCDSLayout(FuncSizes, FuncCounts, CallCounts, CallOffsets);
+
----------------
Amir wrote:
> I assume the definition lives in llvm/Transforms and we include the required component in CMakeLists?
Yes, `ReorderAlgorithm.cpp` already uses on llvm/Transforms so it's not a new dependency.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153039