[llvm] [DependenceAnalysis] Fix SIV test crash when no AddRec after propagation (PR #154980)
Ryotaro Kasuga via llvm-commits
llvm-commits at lists.llvm.org
Sun Aug 24 15:36:00 PDT 2025
https://github.com/kasuga-fj requested changes to this pull request.
> > Do not delete the assertion. The assertion error is correct. The right approach is to stop passing non-AddRecExpr values to testSIV.
>
> I'm missing the context here. Are you speaking about this code:
>
> ```
> llvm_unreachable("SIV test expected at least one AddRec");
> return false;
> ```
Yes. And as I wrote in another reply, the propagation is not relevant here. Again, the correct approach is to stop the analysis at an earlier stage.
https://github.com/llvm/llvm-project/pull/154980
More information about the llvm-commits
mailing list