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

[Sergey Pupyrev via Phabricator via llvm-commits]

spupyrev added inline comments.


================
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};
----------------
rahmanl wrote:
> I might be missing something. Isn't this supposed to be the same size as `Counts`?
I agree. Does it make sense to add an assertion to catch such problems?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159527