[llvm] [NFC][X86] Pass ConstantInt for step size in createLoop (PR #181577)

via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 15 14:00:10 PST 2026


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-backend-x86

Author: Aiden Grossman (boomanaiden154)

<details>
<summary>Changes</summary>

createLoop is always called with a constant step size. Propagate the
type information so that it can be used later when deriving profile
information when possible.


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


1 Files Affected:

- (modified) llvm/lib/Target/X86/X86LowerAMXIntrinsics.cpp (+2-2) 


``````````diff
diff --git a/llvm/lib/Target/X86/X86LowerAMXIntrinsics.cpp b/llvm/lib/Target/X86/X86LowerAMXIntrinsics.cpp
index 662aec2c15241..4912e3d7dd6df 100644
--- a/llvm/lib/Target/X86/X86LowerAMXIntrinsics.cpp
+++ b/llvm/lib/Target/X86/X86LowerAMXIntrinsics.cpp
@@ -71,7 +71,7 @@ class X86LowerAMXIntrinsics {
   DomTreeUpdater &DTU;
   LoopInfo *LI;
   BasicBlock *createLoop(BasicBlock *Preheader, BasicBlock *Exit, Value *Bound,
-                         Value *Step, StringRef Name, IRBuilderBase &B,
+                         ConstantInt *Step, StringRef Name, IRBuilderBase &B,
                          Loop *L);
   template <bool IsTileLoad>
   Value *createTileLoadStoreLoops(BasicBlock *Start, BasicBlock *End,
@@ -103,7 +103,7 @@ class X86LowerAMXIntrinsics {
 
 BasicBlock *X86LowerAMXIntrinsics::createLoop(BasicBlock *Preheader,
                                               BasicBlock *Exit, Value *Bound,
-                                              Value *Step, StringRef Name,
+                                              ConstantInt *Step, StringRef Name,
                                               IRBuilderBase &B, Loop *L) {
   LLVMContext &Ctx = Preheader->getContext();
   BasicBlock *Header =

``````````

</details>


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


More information about the llvm-commits mailing list