[llvm] [DependenceAnalysis] Extending SIV to handle separate loops (PR #128782)
Michael Kruse via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 30 04:46:02 PDT 2025
================
@@ -264,27 +313,37 @@ namespace llvm {
bool normalize(ScalarEvolution *SE) override;
/// isPeelFirst - Returns true if peeling the first iteration from
- /// this loop will break this dependence.
- bool isPeelFirst(unsigned Level) const override;
+ /// this loop will break this dependence. If Separate is set to true,
+ /// information about a separate level is provided.
----------------
Meinersbur wrote:
There is nothing in the header file that even explains what a "separate level" actually is. Rather than having this sentence not saying anything here, consider central documentation whati t means, e.g. as class comment.
Could contain:
* Number of loops inside common loops that are "similar" (+definition of similar)
* Have different `llvm::Loop` objects but can be interpreted as a single fused loop with `Separate=true`
https://github.com/llvm/llvm-project/pull/128782
More information about the llvm-commits
mailing list