[llvm] [DependenceAnalysis] Extending SIV to handle fusable loops (PR #128782)

via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 19 07:59:16 PDT 2025


================
@@ -756,6 +793,35 @@ static bool isLoadOrStore(const Instruction *I) {
   return false;
 }
 
+// Returns true if two loops are the same or they have the same tripcount and
+// depth
+bool DependenceInfo::areLoopsSimilar(const Loop *SrcLoop,
----------------
amehsan wrote:

Thanks. You still need to change your commit message. Write your commit message so that someone who doesn't know anything about your patch understands it. (e.g. Don't use acronyms without first introducing them in the commit messag).

Also change "Same Iteration" to "Same Bounds" or "Same Iteration Space". Make sure the acronym is introduced clearly in the comments of the code as well.  

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


More information about the llvm-commits mailing list