[PATCH] D159527: [CodeLayout] Add unittest for cache-directed sort

[Rahman Lavaee via Phabricator via llvm-commits]

rahmanl added a comment.

Another example would be where the algorithm stops merging functions (with call edges) because the temporal locality degrades too much, but this can be done in a separate patch.



================
Comment at: llvm/unittests/Transforms/Utils/CodeLayoutTest.cpp:14
+  {
+    const std::vector<uint64_t> Sizes(3, 9);
+    const uint64_t Counts[3] = {140, 40, 140};
----------------
I might be missing something. Isn't this supposed to be the same size as `Counts`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159527