[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:16 PDT 2024


================
@@ -3604,24 +3624,48 @@ void MachineBlockPlacement::applyExtTsp() {
     // not see a perf improvement with the exact block sizes.
     auto NonDbgInsts =
         instructionsWithoutDebug(MBB.instr_begin(), MBB.instr_end());
-    int NumInsts = std::distance(NonDbgInsts.begin(), NonDbgInsts.end());
+    size_t NumInsts = std::distance(NonDbgInsts.begin(), NonDbgInsts.end());
----------------
ellishg wrote:

I'm not completely sure if this works, but it's worth a try.
```suggestion
    size_t NumInsts = llvm::size(NonDbgInsts);
```

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


More information about the llvm-commits mailing list