[llvm] [MachineSink] Add option for aggressive loop sinking (PR #117247)

Sirish Pande via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 2 13:41:21 PST 2024


================
@@ -784,7 +805,18 @@ bool MachineSinking::runOnMachineFunction(MachineFunction &MF) {
       // of a def-use chain, if there is any.
       // TODO: Sort the candidates using a cost-model.
       unsigned i = 0;
+
----------------
srpande wrote:

This is sinking 50 instructions. My suggestions would be to update FindCycleSinkCandidates and sink only those  candidates which have low latency. There is already precedence to look at target. Ideally we don't want sink high latency instructions into the loops.

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


More information about the llvm-commits mailing list