[llvm] [CodeLayout] Size-aware machine block placement (PR #109711)

Ellis Hoag via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 24 15:10:15 PDT 2024


================
@@ -3589,13 +3607,15 @@ void MachineBlockPlacement::applyExtTsp() {
     CurrentBlockOrder.push_back(&MBB);
   }
 
-  auto BlockSizes = std::vector<uint64_t>(F->size());
-  auto BlockCounts = std::vector<uint64_t>(F->size());
+  std::vector<uint64_t> BlockCounts(F->size());
+  std::vector<uint64_t> BlockSizes(F->size());
   std::vector<codelayout::EdgeCount> JumpCounts;
----------------
ellishg wrote:

It looks like these aren't `SmallVector` because `calcExtTspScore()` has `std::vector` arguments. I think those should be changed to `ArrayRef` if possible.

https://github.com/llvm/llvm-project/pull/109711


More information about the llvm-commits mailing list