[llvm] [DA] Fix zero coeff bug in Strong SIV test with runtime assumptions (#149991) (PR #155037)
Sjoerd Meijer via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 27 05:26:41 PST 2025
================
@@ -3614,20 +3639,10 @@ DependenceInfo::depends(Instruction *Src, Instruction *Dst,
SCEVUnionPredicate(Assume, *SE));
}
- if (!Assume.empty()) {
- if (!UnderRuntimeAssumptions)
- return std::make_unique<Dependence>(Src, Dst,
- SCEVUnionPredicate(Assume, *SE));
- // Add non-redundant assumptions.
- unsigned N = Assumptions.size();
- for (const SCEVPredicate *P : Assume) {
- bool Implied = false;
- for (unsigned I = 0; I != N && !Implied; I++)
- if (Assumptions[I]->implies(P, *SE))
- Implied = true;
- if (!Implied)
- Assumptions.push_back(P);
- }
+ if (!Assume.empty() && !UnderRuntimeAssumptions) {
+ // Runtime assumptions needed but not allowed.
----------------
sjoerdmeijer wrote:
This is a real nitpick, but I would put the comment above the if anyway, which then becomes a single-statement if so you can drop the curly brackets.
https://github.com/llvm/llvm-project/pull/155037
More information about the llvm-commits
mailing list