[llvm] [RISCV][NFC] Don't set UnrollAndJamInnerLoopThreshold in getUnrollingPreferences (PR #118572)

via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 3 17:05:57 PST 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-backend-risc-v

Author: Michael Maitland (michaelmaitland)

<details>
<summary>Changes</summary>

This has no effect since its the default value used in llvm::gatherUnrollingPreferences.

---
Full diff: https://github.com/llvm/llvm-project/pull/118572.diff


1 Files Affected:

- (modified) llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp (-1) 


``````````diff
diff --git a/llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp b/llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
index aed476db1956fa..4e9d79d8dfd2a8 100644
--- a/llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
+++ b/llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
@@ -2242,7 +2242,6 @@ void RISCVTTIImpl::getUnrollingPreferences(Loop *L, ScalarEvolution &SE,
   UP.Runtime = true;
   UP.UnrollRemainder = true;
   UP.UnrollAndJam = true;
-  UP.UnrollAndJamInnerLoopThreshold = 60;
 
   // Force unrolling small loops can be very useful because of the branch
   // taken cost of the backedge.

``````````

</details>


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


More information about the llvm-commits mailing list