[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